Re: [phpesp-dev] Slight improvement (another patch)

[James E. F. <jf...@ac...>]

I am not sure if this is desired behavior or not. Clearly
in some cases it is, but I can see several cases where it
would be a pain... Anyone else have comments on it?

I believe the idea is that when creating a survey, the
question option lines will not be cleared when you click on
the "New Question" button. (Correct me if I am wrong.)

-James

On Wed, 2 Oct 2002, Robert August Vincent II wrote:

> The following patch (to the current CVS version) will allow answers to
> questions to "carry over" to new question fields:
>
> diff -ur phpESP/admin/include/tab/questions_options.inc phpESP-patched/admin/include/tab/questions_options.inc
> --- phpESP/admin/include/tab/questions_options.inc      2002-09-06 14:39:02.000000000 -0400
> +++ phpESP-patched/admin/include/tab/questions_options.inc 2002-10-02 01:58:20.000000000 -0400
> @@ -41,11 +41,11 @@
>                 for($i=1; $i<$num_choices+1; ++$i) {
>                         if ($updated && $curr_q) {
>                                 list($choice_id, $choice_content) = mysql_fetch_row($result);
> -                       } elseif ($curr_q) {
> +                       } else {
> +                               if ($curr_q) {
>                                         $choice_id      = intval($HTTP_POST_VARS["choice_id_$i"]);
> +                               }
>                                         $choice_content = $HTTP_POST_VARS["choice_content_$i"];
> -                       } else {
> -                                       $choice_id = $choice_content = '';
>                         }
>  ?>
>         <tr>
>
>