--- loncom/interface/lonhelper.pm	2003/05/05 15:17:25	1.18
+++ loncom/interface/lonhelper.pm	2003/05/15 16:14:52	1.30
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # .helper XML handler to implement the LON-CAPA helper
 #
-# $Id: lonhelper.pm,v 1.18 2003/05/05 15:17:25 bowersj2 Exp $
+# $Id: lonhelper.pm,v 1.30 2003/05/15 16:14:52 bowersj2 Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -193,10 +193,38 @@ my $substate;
 # end of the element tag is located.
 my $paramHash; 
 
+# Note from Jeremy 5-8-2003: It is *vital* that the real handler be called
+# as a subroutine from the handler, or very mysterious things might happen.
+# I don't know exactly why, but it seems that the scope where the Apache
+# server enters the perl handler is treated differently from the rest of
+# the handler. This also seems to manifest itself in the debugger as entering
+# the perl handler in seemingly random places (sometimes it starts in the
+# compiling phase, sometimes in the handler execution phase where it runs
+# the code and stepping into the "1;" the module ends with goes into the handler,
+# sometimes starting directly with the handler); I think the cause is related.
+# In the debugger, this means that breakpoints are ignored until you step into
+# a function and get out of what must be a "faked up scope" in the Apache->
+# mod_perl connection. In this code, it was manifesting itself in the existence
+# of two seperate file-scoped $helper variables, one set to the value of the
+# helper in the helper constructor, and one referenced by the handler on the
+# "$helper->process()" line. The second was therefore never set, and was still
+# undefined when I tried to call process on it.
+# By pushing the "real handler" down into the "real scope", everybody except the 
+# actual handler function directly below this comment gets the same $helper and
+# everybody is happy.
+# The upshot of all of this is that for safety when a handler is  using 
+# file-scoped variables in LON-CAPA, the handler should be pushed down one 
+# call level, as I do here, to ensure that the top-level handler function does
+# not get a different file scope from the rest of the code.
+sub handler {
+    my $r = shift;
+    return real_handler($r);
+}
+
 # For debugging purposes, one can send a second parameter into this
 # function, the 'uri' of the helper you wish to have rendered, and
 # call this from other handlers.
-sub handler {
+sub real_handler {
     my $r = shift;
     my $uri = shift;
     if (!defined($uri)) { $uri = $r->uri(); }
@@ -206,6 +234,7 @@ sub handler {
     my $file;
     read $fh, $file, 100000000;
 
+
     # Send header, don't cache this page
     if ($r->header_only) {
         if ($ENV{'browser.mathml'}) {
@@ -326,8 +355,6 @@ sub new {
 	$self->{STATE} = "START";
     }
 
-    Apache::loncommon::get_unprocessed_cgi($ENV{QUERY_STRING});
-
     $self->{TOKEN} = $ENV{'form.TOKEN'};
     # If a token was passed, we load that in. Otherwise, we need to create a 
     # new storage file
@@ -432,7 +459,11 @@ sub declareVar {
 
     my $envname = 'form.' . $var . '.forminput';
     if (defined($ENV{$envname})) {
-        $self->{VARS}->{$var} = $ENV{$envname};
+        if (ref($ENV{$envname})) {
+            $self->{VARS}->{$var} = join('|||', @{$ENV{$envname}});
+        } else {
+            $self->{VARS}->{$var} = $ENV{$envname};
+        }
     }
 }
 
@@ -516,17 +547,23 @@ sub display {
     </head>
     $bodytag
 HEADER
-    if (!$state->overrideForm()) { $result.="<form name='helpform' method='GET'>"; }
+    if (!$state->overrideForm()) { $result.="<form name='helpform' method='POST'>"; }
     $result .= <<HEADER;
         <table border="0"><tr><td>
         <h2><i>$stateTitle</i></h2>
 HEADER
 
+    $result .= "<table><tr><td rowspan='2' valign='top'>";
+
     if (!$state->overrideForm()) {
         $result .= $self->_saveVars();
     }
-    $result .= $state->render() . "<p>&nbsp;</p>";
+    $result .= $state->render();
 
+    $result .= "</td><td valign='top'>";
+
+    # Warning: Copy and pasted from below, because it's too much trouble to 
+    # turn this into a subroutine
     if (!$state->overrideForm()) {
         $result .= '<center>';
         if ($self->{STATE} ne $self->{START_STATE}) {
@@ -537,9 +574,30 @@ HEADER
             $result .= "<a href=\"$returnPage\">End Helper</a>";
         }
         else {
-            $result .= '<input name="back" type="button" ';
+            $result .= '<nobr><input name="back" type="button" ';
             $result .= 'value="&lt;- Previous" onclick="history.go(-1)" /> ';
-            $result .= '<input name="SUBMIT" type="submit" value="Next -&gt;" />';
+            $result .= '<input name="SUBMIT" type="submit" value="Next -&gt;" /></nobr>&nbsp;';
+        }
+        $result .= "</center>\n";
+    }
+
+    $result .= "</td></tr><tr><td valign='bottom'>";
+
+    # Warning: Copy and pasted from above, because it's too much trouble to 
+    # turn this into a subroutine
+    if (!$state->overrideForm()) {
+        $result .= '<center>';
+        if ($self->{STATE} ne $self->{START_STATE}) {
+            #$result .= '<input name="SUBMIT" type="submit" value="&lt;- Previous" />&nbsp;&nbsp;';
+        }
+        if ($self->{DONE}) {
+            my $returnPage = $self->{RETURN_PAGE};
+            $result .= "<a href=\"$returnPage\">End Helper</a>";
+        }
+        else {
+            $result .= '<nobr><input name="back" type="button" ';
+            $result .= 'value="&lt;- Previous" onclick="history.go(-1)" /> ';
+            $result .= '<input name="SUBMIT" type="submit" value="Next -&gt;" /></nobr>';
         }
         $result .= "</center>\n";
     }
@@ -548,6 +606,8 @@ HEADER
     #    $result .= "|$key| -> " . $self->{VARS}->{$key} . "<br />";
     #}
 
+    $result .= "</td></tr></table>";
+
     $result .= <<FOOTER;
               </td>
             </tr>
@@ -660,6 +720,7 @@ sub render {
     for my $element (@{$self->{ELEMENTS}}) {
         push @results, $element->render();
     }
+
     return join("\n", @results);
 }
 
@@ -672,29 +733,41 @@ package Apache::lonhelper::element;
 
 =head2 Element Base Class
 
-The Apache::lonhelper::element base class provides support methods for
-the elements to use, such as a multiple value processer.
+The Apache::lonhelper::element base class provides support for elements
+and defines some generally useful tags for use in elements.
 
-B<Methods>:
+B<finalcode tag>
 
-=over 4
-
-=item * process_multiple_choices(formName, varName): Process the form 
-element named "formName" and place the selected items into the helper 
-variable named varName. This is for things like checkboxes or 
-multiple-selection listboxes where the user can select more then 
-one entry. The selected entries are delimited by triple pipes in 
-the helper variables, like this:  
+Each element can contain a "finalcode" tag that, when the special FINAL
+helper state is used, will be executed, surrounded by "sub { my $helper = shift;"
+and "}". It is expected to return a string describing what it did, which 
+may be an empty string. See course initialization helper for an example. This is
+generally intended for helpers like the course initialization helper, which consist
+of several panels, each of which is performing some sort of bite-sized functionality.
+
+B<defaultvalue tag>
+
+Each element that accepts user input can contain a "defaultvalue" tag that,
+when surrounded by "sub { my $helper = shift; my $state = shift; " and "}",
+will form a subroutine that when called will provide a default value for
+the element. How this value is interpreted by the element is specific to
+the element itself, and possibly the settings the element has (such as 
+multichoice vs. single choice for <choices> tags). 
+
+This is also intended for things like the course initialization wizard, where the
+user is setting various parameters. By correctly grabbing current settings 
+and including them into the helper, it allows the user to come back to the
+helper later and re-execute it, without needing to worry about overwriting
+some setting accidentally.
 
- CHOICE_1|||CHOICE_2|||CHOICE_3
-
-=back
+Again, see the course initialization helper for examples.
 
 =cut
 
 BEGIN {
     &Apache::lonhelper::register('Apache::lonhelper::element',
-                                 ('nextstate'));
+                                 ('nextstate', 'finalcode',
+                                  'defaultvalue'));
 }
 
 # Because we use the param hash, this is often a sufficent
@@ -729,6 +802,36 @@ sub start_nextstate {
 
 sub end_nextstate { return ''; }
 
+sub start_finalcode {
+    my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_;
+
+    if ($target ne 'helper') {
+        return '';
+    }
+    
+    $paramHash->{FINAL_CODE} = &Apache::lonxml::get_all_text('/finalcode',
+                                                             $parser);
+    return '';
+}
+
+sub end_finalcode { return ''; }
+
+sub start_defaultvalue {
+    my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_;
+
+    if ($target ne 'helper') {
+        return '';
+    }
+    
+    $paramHash->{DEFAULT_VALUE} = &Apache::lonxml::get_all_text('/defaultvalue',
+                                                             $parser);
+    $paramHash->{DEFAULT_VALUE} = 'sub { my $helper = shift; my $state = shift;' .
+        $paramHash->{DEFAULT_VALUE} . '}';
+    return '';
+}
+
+sub end_defaultvalue { return ''; }
+
 sub preprocess {
     return 1;
 }
@@ -745,28 +848,6 @@ sub overrideForm {
     return 0;
 }
 
-sub process_multiple_choices {
-    my $self = shift;
-    my $formname = shift;
-    my $var = shift;
-
-    # Must extract values from data directly, as there
-    # may be more then one.
-    my @values;
-    for my $formparam (split (/&/, $ENV{QUERY_STRING})) {
-        my ($name, $value) = split(/=/, $formparam);
-        if ($name ne $formname) {
-            next;
-        }
-        $value =~ tr/+/ /;
-        $value =~ s/%([a-fA-F0-9][a-fA-F0-9])/pack("C", hex($1))/eg;
-        push @values, $value;
-    }
-    $helper->{VARS}->{$var} = join('|||', @values);
-    
-    return;
-}
-
 1;
 
 package Apache::lonhelper::message;
@@ -875,6 +956,10 @@ the result is stored in.
 <choices> takes an attribute "multichoice" which, if set to a true
 value, will allow the user to select multiple choices.
 
+<choices> takes an attribute "allowempty" which, if set to a true 
+value, will allow the user to select none of the choices without raising
+an error message.
+
 B<SUB-TAGS>
 
 <choices> can have the following subtags:
@@ -924,6 +1009,18 @@ You can mix and match methods of creatin
 "push" onto the choice list, rather then wiping it out. (You can even 
 remove choices programmatically, but that would probably be bad form.)
 
+B<defaultvalue support>
+
+Choices supports default values both in multichoice and single choice mode.
+In single choice mode, have the defaultvalue tag's function return the 
+computer value of the box you want checked. If the function returns a value
+that does not correspond to any of the choices, the default behavior of selecting
+the first choice will be preserved.
+
+For multichoice, return a string with the computer values you want checked,
+delimited by triple pipes. Note this matches how the result of the <choices>
+tag is stored in the {VARS} hash.
+
 =cut
 
 no strict;
@@ -949,9 +1046,10 @@ sub start_choices {
     }
 
     # Need to initialize the choices list, so everything can assume it exists
-    $paramHash->{'variable'} = $token->[2]{'variable'};
+    $paramHash->{'variable'} = $token->[2]{'variable'} if (!defined($paramHash->{'variable'}));
     $helper->declareVar($paramHash->{'variable'});
     $paramHash->{'multichoice'} = $token->[2]{'multichoice'};
+    $paramHash->{'allowempty'} = $token->[2]{'allowempty'};
     $paramHash->{CHOICES} = [];
     return '';
 }
@@ -1007,6 +1105,11 @@ sub render {
     }
 </script>
 SCRIPT
+    }
+
+    # Only print "select all" and "unselect all" if there are five or
+    # more choices; fewer then that and it looks silly.
+    if ($self->{'multichoice'} && scalar(@{$self->{CHOICES}}) > 4) {
         $buttons = <<BUTTONS;
 <br />
 <input type="button" onclick="checkall(true, '$var')" value="Select All" />
@@ -1023,18 +1126,52 @@ BUTTONS
     
     $result .= "<table>\n\n";
 
+    my %checkedChoices;
+    my $checkedChoicesFunc;
+
+    if (defined($self->{DEFAULT_VALUE})) {
+        $checkedChoicesFunc = eval ($self->{DEFAULT_VALUE});
+        die 'Error in default value code for variable ' . 
+            {'variable'} . ', Perl said:' . $@ if $@;
+    } else {
+        $checkedChoicesFunc = sub { return ''; };
+    }
+
+    # Process which choices should be checked.
+    if ($self->{'multichoice'}) {
+        for my $selectedChoice (split(/\|\|\|/, (&$checkedChoicesFunc($helper, $self)))) {
+            $checkedChoices{$selectedChoice} = 1;
+        }
+    } else {
+        # single choice
+        my $selectedChoice = &$checkedChoicesFunc($helper, $self);
+        
+        my $foundChoice = 0;
+        
+        # check that the choice is in the list of choices.
+        for my $choice (@{$self->{CHOICES}}) {
+            if ($choice->[1] eq $selectedChoice) {
+                $checkedChoices{$choice->[1]} = 1;
+                $foundChoice = 1;
+            }
+        }
+        
+        # If we couldn't find the choice, pick the first one 
+        if (!$foundChoice) {
+            $checkedChoices{$self->{CHOICES}->[0]->[1]} = 1;
+        }
+    }
+
     my $type = "radio";
     if ($self->{'multichoice'}) { $type = 'checkbox'; }
-    my $checked = 0;
     foreach my $choice (@{$self->{CHOICES}}) {
         $result .= "<tr>\n<td width='20'>&nbsp;</td>\n";
         $result .= "<td valign='top'><input type='$type' name='$var.forminput'"
             . "' value='" . 
             HTML::Entities::encode($choice->[1]) 
             . "'";
-        if (!$self->{'multichoice'} && !$checked) {
+        if ($checkedChoices{$choice->[1]}) {
             $result .= " checked ";
-            $checked = 1;
         }
         my $choiceLabel = $choice->[0];
         if ($choice->[4]) {  # if we need to evaluate this choice
@@ -1057,15 +1194,14 @@ sub postprocess {
     my $self = shift;
     my $chosenValue = $ENV{'form.' . $self->{'variable'} . '.forminput'};
 
-    if (!$chosenValue) {
+    if (!defined($chosenValue) && !$self->{'allowempty'}) {
         $self->{ERROR_MSG} = "You must choose one or more choices to" .
             " continue.";
         return 0;
     }
 
-    if ($self->{'multichoice'}) {
-        $self->process_multiple_choices($self->{'variable'}.'.forminput',
-                                        $self->{'variable'});
+    if (ref($chosenValue)) {
+        $helper->{VARS}->{$self->{'variable'}} = join('|||', @$chosenValue);
     }
 
     if (defined($self->{NEXTSTATE})) {
@@ -1311,7 +1447,10 @@ variable stores the results. It also tak
 which controls whether the user can select more then one resource. The 
 "toponly" attribute controls whether the resource display shows just the
 resources in that sequence, or recurses into all sub-sequences, defaulting
-to false.
+to false. The "suppressEmptySequences" attribute reflects the 
+suppressEmptySequences argument to the render routine, which will cause
+folders that have all of their contained resources filtered out to also
+be filtered out.
 
 B<SUB-TAGS>
 
@@ -1372,6 +1511,7 @@ sub start_resource {
     $paramHash->{'variable'} = $token->[2]{'variable'};
     $helper->declareVar($paramHash->{'variable'});
     $paramHash->{'multichoice'} = $token->[2]{'multichoice'};
+    $paramHash->{'suppressEmptySequences'} = $token->[2]{'suppressEmptySequences'};
     $paramHash->{'toponly'} = $token->[2]{'toponly'};
     return '';
 }
@@ -1540,6 +1680,7 @@ BUTTONS
                                        'showParts' => 0,
                                        'filterFunc' => $filterFunc,
                                        'resource_no_folder_link' => 1,
+                                       'suppressEmptySequences' => $self->{'suppressEmptySequences'},
                                        'iterator_map' => $mapUrl }
                                        );
 
@@ -1551,11 +1692,6 @@ BUTTONS
 sub postprocess {
     my $self = shift;
 
-    if ($self->{'multichoice'}) {
-        $self->process_multiple_choices($self->{'variable'}.'.forminput',
-                                        $self->{'variable'});
-    }
-
     if ($self->{'multichoice'} && !$helper->{VARS}->{$self->{'variable'}}) {
         $self->{ERROR_MSG} = 'You must choose at least one resource to continue.';
         return 0;
@@ -1689,7 +1825,7 @@ BUTTONS
             $checked = 1;
         }
         $result .=
-            " value='" . HTML::Entities::encode($_)
+            " value='" . HTML::Entities::encode($_ . ':' . $choices->{$_}->[$section])
             . "' /></td><td>"
             . HTML::Entities::encode($choices->{$_}->[$fullname])
             . "</td><td align='center'>" 
@@ -1713,10 +1849,6 @@ sub postprocess {
         return 0;
     }
 
-    if ($self->{'multichoice'}) {
-        $self->process_multiple_choices($self->{'variable'}.'.forminput',
-                                        $self->{'variable'});
-    }
     if (defined($self->{NEXTSTATE})) {
         $helper->changeState($self->{NEXTSTATE});
     }
@@ -1835,6 +1967,10 @@ sub render {
 
     my $filterFunc = $self->{FILTER_FUNC};
     my $buttons = '';
+    my $type = 'radio';
+    if ($self->{'multichoice'}) {
+        $type = 'checkbox';
+    }
 
     if ($self->{'multichoice'}) {
         $result = <<SCRIPT;
@@ -1847,14 +1983,30 @@ sub render {
             }
         }
     }
+
+    function checkallclass(value, className) {
+        for (i=0; i<document.forms.helpform.elements.length; i++) {
+            ele = document.forms.helpform.elements[i];
+            if (ele.type == "$type" && ele.onclick) {
+                document.forms.helpform.elements[i].checked=value;
+            }
+        }
+    }
 </script>
 SCRIPT
         $buttons = <<BUTTONS;
 <br /> &nbsp;
 <input type="button" onclick="checkall(true, '$var')" value="Select All Files" />
 <input type="button" onclick="checkall(false, '$var')" value="Unselect All Files" />
+BUTTONS
+
+        if ($helper->{VARS}->{'construction'}) {
+            $buttons .= <<BUTTONS;
+<input type="button" onclick="checkallclass(true, 'Published')" value="Select All Published" />
+<input type="button" onclick="checkallclass(false, 'Published')" value="Unselect All Published" />
 <br /> &nbsp;
 BUTTONS
+       }
     }
 
     # Get the list of files in this directory.
@@ -1876,15 +2028,11 @@ BUTTONS
         $result .= '<br /><font color="#FF0000">' . $self->{ERROR_MSG} . '</font><br /><br />';
     }
 
-    $result .= '<table border="0" cellpadding="1" cellspacing="1">';
+    $result .= '<table border="0" cellpadding="2" cellspacing="0">';
 
     # Keeps track if there are no choices, prints appropriate error
     # if there are none. 
     my $choices = 0;
-    my $type = 'radio';
-    if ($self->{'multichoice'}) {
-        $type = 'checkbox';
-    }
     # Print each legitimate file choice.
     for my $file (@fileList) {
         $file = (split(/&/, $file))[0];
@@ -1893,14 +2041,43 @@ BUTTONS
         }
         my $fileName = $subdir .'/'. $file;
         if (&$filterFunc($file)) {
-            $result .= '<tr><td align="right">' .
-                "<input type='$type' name='" . $var
+	    my $status;
+	    my $color;
+	    if ($helper->{VARS}->{'construction'}) {
+		($status, $color) = @{fileState($subdir, $file)};
+	    } else {
+		$status = '';
+		$color = '';
+	    }
+
+            # Netscape 4 is stupid and there's nowhere to put the
+            # information on the input tag that the file is Published,
+            # Unpublished, etc. In *real* browsers we can just say
+            # "class='Published'" and check the className attribute of
+            # the input tag, but Netscape 4 is too stupid to understand
+            # that attribute, and un-comprehended attributes are not
+            # reflected into the object model. So instead, what I do 
+            # is either have or don't have an "onclick" handler that 
+            # does nothing, give Published files the onclick handler, and
+            # have the checker scripts check for that. Stupid and clumsy,
+            # and only gives us binary "yes/no" information (at least I
+            # couldn't figure out how to reach into the event handler's
+            # actual code to retreive a value), but it works well enough
+            # here.
+        
+            my $onclick = '';
+            if ($status eq 'Published' && $helper->{VARS}->{'construction'}) {
+                $onclick = 'onclick="a=1" ';
+            }
+            $result .= '<tr><td align="right"' . " bgcolor='$color'>" .
+                "<input $onclick type='$type' name='" . $var
             . ".forminput' value='" . HTML::Entities::encode($fileName) .
                 "'";
             if (!$self->{'multichoice'} && $choices == 0) {
                 $result .= ' checked';
             }
-            $result .= "/></td><td>" . $file . "</td></tr>\n";
+            $result .= "/></td><td bgcolor='$color'>" . $file .
+                 "</td><td bgcolor='$color'>$status</td></tr>\n";
             $choices++;
         }
     }
@@ -1916,6 +2093,37 @@ BUTTONS
     return $result;
 }
 
+# Determine the state of the file: Published, unpublished, modified.
+# Return the color it should be in and a label as a two-element array
+# reference.
+# Logic lifted from lonpubdir.pm, even though I don't know that it's still
+# the most right thing to do.
+
+sub fileState {
+    my $constructionSpaceDir = shift;
+    my $file = shift;
+    
+    my $docroot = $Apache::lonnet::perlvar{'lonDocRoot'};
+    my $subdirpart = $constructionSpaceDir;
+    $subdirpart =~ s/^\/home\/$ENV{'user.name'}\/public_html//;
+    my $resdir = $docroot . '/res/' . $ENV{'user.domain'} . '/' . $ENV{'user.name'} .
+        $subdirpart;
+
+    my @constructionSpaceFileStat = stat($constructionSpaceDir . '/' . $file);
+    my @resourceSpaceFileStat = stat($resdir . '/' . $file);
+    if (!@resourceSpaceFileStat) {
+        return ['Unpublished', '#FFCCCC'];
+    }
+
+    my $constructionSpaceFileModified = $constructionSpaceFileStat[9];
+    my $resourceSpaceFileModified = $resourceSpaceFileStat[9];
+    
+    if ($constructionSpaceFileModified > $resourceSpaceFileModified) {
+        return ['Modified', '#FFFFCC'];
+    }
+    return ['Published', '#CCFFCC'];
+}
+
 sub postprocess {
     my $self = shift;
     my $result = $ENV{'form.' . $self->{'variable'} . '.forminput'};
@@ -1925,10 +2133,6 @@ sub postprocess {
         return 0;
     }
 
-    if ($self->{'multichoice'}) {
-        $self->process_multiple_choices($self->{'variable'}.'.forminput',
-                                        $self->{'variable'});
-    }
     if (defined($self->{NEXTSTATE})) {
         $helper->changeState($self->{NEXTSTATE});
     }
@@ -2145,6 +2349,101 @@ sub end_eval {
 
 1;
 
+package Apache::lonhelper::final;
+
+=pod
+
+=head2 Element: final
+
+<final> is a special element that works with helpers that use the <finalcode>
+tag. It goes through all the states and elements, executing the <finalcode>
+snippets and collecting the results. Finally, it takes the user out of the
+helper, going to a provided page.
+
+=cut
+
+no strict;
+@ISA = ("Apache::lonhelper::element");
+use strict;
+
+BEGIN {
+    &Apache::lonhelper::register('Apache::lonhelper::final',
+                                 ('final', 'exitpage'));
+}
+
+sub new {
+    my $ref = Apache::lonhelper::element->new();
+    bless($ref);
+}
+
+sub start_final { return ''; }
+
+sub end_final {
+    my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_;
+
+    if ($target ne 'helper') {
+        return '';
+    }
+
+    Apache::lonhelper::final->new();
+   
+    return '';
+}
+
+sub start_exitpage {
+    my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_;
+
+    if ($target ne 'helper') {
+        return '';
+    }
+
+    $paramHash->{EXIT_PAGE} = &Apache::lonxml::get_all_text('/exitpage',
+                                                            $parser);
+
+    return '';
+}
+
+sub end_exitpage { return ''; }
+
+sub render {
+    my $self = shift;
+
+    my @results;
+
+    # Collect all the results
+    for my $stateName (keys %{$helper->{STATES}}) {
+        my $state = $helper->{STATES}->{$stateName};
+        
+        for my $element (@{$state->{ELEMENTS}}) {
+            if (defined($element->{FINAL_CODE})) {
+                # Compile the code.
+                my $code = 'sub { my $helper = shift; ' . $element->{FINAL_CODE} .
+                    '}';
+                $code = eval($code);
+                die 'Error while executing final code for element with var ' .
+                    $element->{'variable'} . ', Perl said: ' . $@ if $@;
+
+                my $result = &$code($helper);
+                if ($result) {
+                    push @results, $result;
+                }
+            }
+        }
+    }
+
+    if (scalar(@results) == 0) {
+        return '';
+    }
+
+    my $result = "<ul>\n";
+    for my $re (@results) {
+        $result .= '    <li>' . $re . "</li>\n";
+    }
+    return $result . '</ul>';
+}
+
+1;
+
 package Apache::lonhelper::parmwizfinal;
 
 # This is the final state for the parmwizard. It is not generally useful,
@@ -2192,60 +2491,72 @@ sub render {
                         'due_date' => "0_duedate",
                         'answer_date' => "0_answerdate");
     
-    my $result = "<form name='helpform' method='get' action='/adm/parmset'>\n";
-    $result .= '<p>Confirm that this information is correct, then click &quot;Finish Wizard&quot; to complete setting the parameter.<ul>';
     my $affectedResourceId = "";
     my $parm_name = $parmTypeHash{$vars->{ACTION_TYPE}};
     my $level = "";
-    
-    # Print the type of manipulation:
-    $result .= '<li>Setting the <b>' . $dateTypeHash{$vars->{ACTION_TYPE}}
-               . "</b></li>\n";
-    if ($vars->{ACTION_TYPE} eq 'due_date' || 
-        $vars->{ACTION_TYPE} eq 'answer_date') {
-        # for due dates, we default to "date end" type entries
-        $result .= "<input type='hidden' name='recent_date_end' " .
-            "value='" . $vars->{PARM_DATE} . "' />\n";
-        $result .= "<input type='hidden' name='pres_value' " . 
-            "value='" . $vars->{PARM_DATE} . "' />\n";
-        $result .= "<input type='hidden' name='pres_type' " .
-            "value='date_end' />\n";
-    } elsif ($vars->{ACTION_TYPE} eq 'open_date') {
-        $result .= "<input type='hidden' name='recent_date_start' ".
-            "value='" . $vars->{PARM_DATE} . "' />\n";
-        $result .= "<input type='hidden' name='pres_value' " .
-            "value='" . $vars->{PARM_DATE} . "' />\n";
-        $result .= "<input type='hidden' name='pres_type' " .
-            "value='date_start' />\n";
-    } 
-    
+    my $resourceString;
+    my $symb;
+    my $paramlevel;
+
     # Print the granularity, depending on the action
     if ($vars->{GRANULARITY} eq 'whole_course') {
-        $result .= '<li>for <b>all resources in the course</b></li>';
+        $resourceString .= '<li>for <b>all resources in the course</b></li>';
         $level = 9; # general course, see lonparmset.pm perldoc
         $affectedResourceId = "0.0";
+        $symb = 'a';
+        $paramlevel = 'general';
     } elsif ($vars->{GRANULARITY} eq 'map') {
         my $navmap = Apache::lonnavmaps::navmap->new(
                            $ENV{"request.course.fn"}.".db",
                            $ENV{"request.course.fn"}."_parms.db", 0, 0);
         my $res = $navmap->getById($vars->{RESOURCE_ID});
         my $title = $res->compTitle();
+        $symb = $res->symb();
         $navmap->untieHashes();
-        $result .= "<li>for the map named <b>$title</b></li>";
+        $resourceString .= "<li>for the map named <b>$title</b></li>";
         $level = 8;
         $affectedResourceId = $vars->{RESOURCE_ID};
+        $paramlevel = 'map';
     } else {
         my $navmap = Apache::lonnavmaps::navmap->new(
                            $ENV{"request.course.fn"}.".db",
                            $ENV{"request.course.fn"}."_parms.db", 0, 0);
         my $res = $navmap->getById($vars->{RESOURCE_ID});
+        $symb = $res->symb();
         my $title = $res->compTitle();
         $navmap->untieHashes();
-        $result .= "<li>for the resource named <b>$title</b></li>";
+        $resourceString .= "<li>for the resource named <b>$title</b></li>";
         $level = 7;
         $affectedResourceId = $vars->{RESOURCE_ID};
+        $paramlevel = 'full';
     }
 
+    my $result = "<form name='helpform' method='get' action='/adm/parmset#$affectedResourceId&$parm_name&$level'>\n";
+    $result .= '<p>Confirm that this information is correct, then click &quot;Finish Wizard&quot; to complete setting the parameter.<ul>';
+    
+    # Print the type of manipulation:
+    $result .= '<li>Setting the <b>' . $dateTypeHash{$vars->{ACTION_TYPE}}
+               . "</b></li>\n";
+    if ($vars->{ACTION_TYPE} eq 'due_date' || 
+        $vars->{ACTION_TYPE} eq 'answer_date') {
+        # for due dates, we default to "date end" type entries
+        $result .= "<input type='hidden' name='recent_date_end' " .
+            "value='" . $vars->{PARM_DATE} . "' />\n";
+        $result .= "<input type='hidden' name='pres_value' " . 
+            "value='" . $vars->{PARM_DATE} . "' />\n";
+        $result .= "<input type='hidden' name='pres_type' " .
+            "value='date_end' />\n";
+    } elsif ($vars->{ACTION_TYPE} eq 'open_date') {
+        $result .= "<input type='hidden' name='recent_date_start' ".
+            "value='" . $vars->{PARM_DATE} . "' />\n";
+        $result .= "<input type='hidden' name='pres_value' " .
+            "value='" . $vars->{PARM_DATE} . "' />\n";
+        $result .= "<input type='hidden' name='pres_type' " .
+            "value='date_start' />\n";
+    } 
+
+    $result .= $resourceString;
+    
     # Print targets
     if ($vars->{TARGETS} eq 'course') {
         $result .= '<li>for <b>all students in course</b></li>';
@@ -2258,7 +2569,10 @@ sub render {
     } else {
         # FIXME: This is probably wasteful! Store the name!
         my $classlist = Apache::loncoursedata::get_classlist();
-        my $name = $classlist->{$vars->{USER_NAME}}->[6];
+        my $username = $vars->{USER_NAME};
+        # Chop off everything after the last colon (section)
+        $username = substr($username, 0, rindex($username, ':'));
+        my $name = $classlist->{$username}->[6];
         $result .= "<li>for <b>$name</b></li>";
         $level -= 6;
         my ($uname, $udom) = split /:/, $vars->{USER_NAME};
@@ -2276,6 +2590,12 @@ sub render {
     # print pres_marker
     $result .= "\n<input type='hidden' name='pres_marker'" .
         " value='$affectedResourceId&$parm_name&$level' />\n";
+    
+    # Make the table appear
+    $result .= "\n<input type='hidden' value='true' name='prevvisit' />";
+    $result .= "\n<input type='hidden' value='all' name='pschp' />";
+    $result .= "\n<input type='hidden' value='$symb' name='pssymb' />";
+    $result .= "\n<input type='hidden' value='$paramlevel' name='parmlev' />";
 
     $result .= "<br /><br /><center><input type='submit' value='Finish Helper' /></center></form>\n";