[phpesp-dev] [ phpesp-Bugs-740889 ] Required on 'Section Text'

[SourceForge.net <no...@so...>]

Bugs item #740889, was opened at 2003-05-20 23:21
Message generated for change (Comment added) made by greggmc
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=108956&aid=740889&group_id=8956

Category: SQL
Group: v1.6
>Status: Closed
Resolution: None
Priority: 5
Submitted By: Paul Jackson (beachbum_boy)
Assigned to: Nobody/Anonymous (nobody)
Summary: Required on 'Section Text'

Initial Comment:
* PROBLEM: *
If you set the required feild to 'Y' on a 'Section Text' you 
basically can never progress beyond this page in the 
survey. So now you wait for a response from a question 
with no response capability. Obviously this is a silly 
situation that a designer shouldn't do but trust me it 
happens.

* FIX: *
In the espresponse.inc file the 16th line (in 
the 'response_check_required' function) the SQL should 
be:

$sql = "SELECT id,type_id,content FROM question ". 
survey_select_section_sql($sid,$section) ." AND 
required='Y' AND deleted='N' AND type_id < 99 ORDER 
BY position";

Notice the type_id < 99. This will include 'Page Breaks' 
as well.

* SPOOKY: * We had exactly the same problem with 
another product: 'Ultimate$urvey' at the same time 
almost to the minute! Spooky!


----------------------------------------------------------------------

Comment By: Matthew Gregg (greggmc)
Date: 2003-05-21 08:04

Message:
Logged In: YES 
user_id=14116

Fixed in CVS.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=108956&aid=740889&group_id=8956