Re: [phpesp-dev] new rate question type and forked survey patch

[Luis F. <lfa...@fu...>]

Hi,
The forks are great, thanks! I needed this one a lot :-)
I have some suggestions about the implementation. Instead of making it 
as a question, you could have a separated table for skips, because there 
is nothing in the structure of a question that matters for skips. This 
would avoid the bug of having the questions tab treating skips as 
questions and generating an error. You could also associate the skip 
with a section break and only check it in this break. This way you can 
really fork a survey instead of just skiping sections, because you can 
make a jump just after the section and then another jump next section 
based on the same question. The way it's now if you have, for example, a 
section to be answered just by men and then another just for women, you 
can't jump the women's section after answering men's section because any 
skip you make based on the sex is applied before answering men's section.
[]s
Luis

angek wrote:

>Hi James,
>
>Here's a diff file created against the current cvs version of phpESP (as
>at 13/04/03).
>
>This diff file incorporates the forked survey capabilities and now works
>correctly with your patches (prev/next for private surveys).
>  
>