[phpesp-dev] [ phpesp-Bugs-1253855 ] Problems with order tab

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

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

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
>Status: Closed
>Resolution: Fixed
Priority: 5
Submitted By: Nobody/Anonymous (nobody)
>Assigned to: Matthew Gregg (greggmc)
Summary: Problems with order tab

Initial Comment:
If the questions for the survey have html as part of the 
question text the list of questions in the order list can 
get really messed up because tags are opened and 
then not closed becuase the text is limited to 100 
chars. 

Fix is to add a line:
 $content=strip_tags($content);
just before the line:
 if($tid != 99) {

I have attached a file that fixes this problem as well as 
adds the functionality that was lost since v1.6, i.e the 
ability to move to any position not just up or down. The 
up or down works fine when you have a small number of 
questions BUT if you need to move 10 questions to the 
end of a 100 question survey its just plain silly.

I think this should be added to CVS. 

But to check it out just make a copy of 'order.inc' and 
copy my version over the top. My version has not been 
100% tested but I'm pretty confident that it is all good.

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

>Comment By: Matthew Gregg (greggmc)
Date: 2005-08-08 08:43

Message:
Logged In: YES 
user_id=14116

strip_tags added to CVS head.  I'll accept a patch for
question ordering that uses client side javascript(ajax) to
accomplish it and applies cleanly against CVS.

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

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