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
|
