--- loncom/interface/lonhelper.pm 2005/02/17 08:29:42 1.96 +++ loncom/interface/lonhelper.pm 2006/05/05 10:59:51 1.139 @@ -1,7 +1,7 @@ # The LearningOnline Network with CAPA # .helper XML handler to implement the LON-CAPA helper # -# $Id: lonhelper.pm,v 1.96 2005/02/17 08:29:42 albertel Exp $ +# $Id: lonhelper.pm,v 1.139 2006/05/05 10:59:51 foxr Exp $ # # Copyright Michigan State University Board of Trustees # @@ -25,10 +25,6 @@ # # http://www.lon-capa.org/ # -# (Page Handler -# -# (.helper handler -# =pod @@ -186,6 +182,8 @@ use Apache::Constants qw(:common); use Apache::File; use Apache::lonxml; use Apache::lonlocal; +use Apache::lonnet; + # Register all the tags with the helper, so the helper can # push and pop them @@ -255,7 +253,7 @@ sub real_handler { my $r = shift; my $uri = shift; if (!defined($uri)) { $uri = $r->uri(); } - $ENV{'request.uri'} = $uri; + $env{'request.uri'} = $uri; my $filename = '/home/httpd/html' . $uri; my $fh = Apache::File->new($filename); my $file; @@ -263,7 +261,7 @@ sub real_handler { # Send header, don't cache this page - if ($ENV{'browser.mathml'}) { + if ($env{'browser.mathml'}) { &Apache::loncommon::content_type($r,'text/xml'); } else { &Apache::loncommon::content_type($r,'text/html'); @@ -278,7 +276,7 @@ sub real_handler { my $allowed = $helper->allowedCheck(); if (!$allowed) { - $ENV{'user.error.msg'} = $ENV{'request.uri'}.':'.$helper->{REQUIRED_PRIV}. + $env{'user.error.msg'} = $env{'request.uri'}.':'.$helper->{REQUIRED_PRIV}. ":0:0:Permission denied to access this helper."; return HTTP_NOT_ACCEPTABLE; } @@ -363,6 +361,7 @@ use HTML::Entities(); use Apache::loncommon; use Apache::File; use Apache::lonlocal; +use Apache::lonnet; sub new { my $proto = shift; @@ -374,16 +373,16 @@ sub new { # If there is a state from the previous form, use that. If there is no # state, use the start state parameter. - if (defined $ENV{"form.CURRENT_STATE"}) + if (defined $env{"form.CURRENT_STATE"}) { - $self->{STATE} = $ENV{"form.CURRENT_STATE"}; + $self->{STATE} = $env{"form.CURRENT_STATE"}; } else { $self->{STATE} = "START"; } - $self->{TOKEN} = $ENV{'form.TOKEN'}; + $self->{TOKEN} = $env{'form.TOKEN'}; # If a token was passed, we load that in. Otherwise, we need to create a # new storage file # Tried to use standard Tie'd hashes, but you can't seem to take a @@ -416,16 +415,16 @@ sub new { return undef; } # Must create the storage - $self->{TOKEN} = md5_hex($ENV{'user.name'} . $ENV{'user.domain'} . + $self->{TOKEN} = md5_hex($env{'user.name'} . $env{'user.domain'} . time() . rand()); $self->{FILENAME} = $Apache::lonnet::tmpdir . md5_hex($self->{TOKEN}); } # OK, we now have our persistent storage. - if (defined $ENV{"form.RETURN_PAGE"}) + if (defined $env{"form.RETURN_PAGE"}) { - $self->{RETURN_PAGE} = $ENV{"form.RETURN_PAGE"}; + $self->{RETURN_PAGE} = $env{"form.RETURN_PAGE"}; } else { @@ -486,11 +485,11 @@ sub declareVar { } my $envname = 'form.' . $var . '.forminput'; - if (defined($ENV{$envname})) { - if (ref($ENV{$envname})) { - $self->{VARS}->{$var} = join('|||', @{$ENV{$envname}}); + if (defined($env{$envname})) { + if (ref($env{$envname})) { + $self->{VARS}->{$var} = join('|||', @{$env{$envname}}); } else { - $self->{VARS}->{$var} = $ENV{$envname}; + $self->{VARS}->{$var} = $env{$envname}; } } } @@ -502,7 +501,7 @@ sub allowedCheck { return 1; } - return Apache::lonnet::allowed($self->{REQUIRED_PRIV}, $ENV{'request.course.id'}); + return Apache::lonnet::allowed($self->{REQUIRED_PRIV}, $env{'request.course.id'}); } sub changeState { @@ -524,7 +523,7 @@ sub process { # Phase 1: Post processing for state of previous screen (which is actually # the "current state" in terms of the helper variables), if it wasn't the # beginning state. - if ($self->{STATE} ne "START" || $ENV{"form.SUBMIT"} eq &mt("Next ->")) { + if ($self->{STATE} ne "START" || $env{"form.SUBMIT"} eq &mt("Next ->")) { my $prevState = $self->{STATES}{$self->{STATE}}; $prevState->postprocess(); } @@ -575,22 +574,20 @@ sub display { } # Phase 4: Display. - my $html=&Apache::lonxml::xmlbegin(); my $stateTitle=&mt($state->title()); - my $helperTitle = &mt($self->{TITLE}); - my $bodytag = &Apache::loncommon::bodytag($helperTitle,'',''); + my $browser_searcher_js = + '<script type="text/javascript">'."\n". + &Apache::loncommon::browser_and_searcher_javascript(). + "\n".'</script>'; + + $result .= &Apache::loncommon::start_page($self->{TITLE}, + $browser_searcher_js); + my $previous = HTML::Entities::encode(&mt("<- Previous"), '<>&"'); my $next = HTML::Entities::encode(&mt("Next ->"), '<>&"'); # FIXME: This should be parameterized, not concatenated - Jeremy - my $loncapaHelper = &mt("LON-CAPA Helper:"); - $result .= <<HEADER; -$html - <head> - <title>$loncapaHelper: $helperTitle</title> - </head> - $bodytag -HEADER + if (!$state->overrideForm()) { $result.="<form name='helpform' method='POST'>"; } $result .= <<HEADER; <table border="0" width='100%'><tr><td> @@ -653,10 +650,9 @@ HEADER </tr> </table> </form> - </body> -</html> FOOTER + $result .= &Apache::loncommon::end_page(); # Handle writing out the vars to the file my $file = Apache::File->new('>'.$self->{FILENAME}); print $file $self->_varsInFile(); @@ -1074,6 +1070,16 @@ will be the state transistioned to if th the choice is not multichoice. This will override the nextstate passed to the parent C<choices> tag. +<choice> may optionally contain a 'relatedvalue' attribute, which +if present will cause a text entry to appear to the right of the +selection. The value of the relatedvalue attribute is a variable +into which the text entry will be stored e.g.: +<choice computer='numberprovided" relatedvalue="num">Type the number in:</choice> + +<choice> may contain a relatededefault atribute which, if the +relatedvalue attribute is present will be the initial value of the input +box. + =back To create the choices programmatically, either wrap the choices in @@ -1115,6 +1121,7 @@ no strict; @ISA = ("Apache::lonhelper::element"); use strict; use Apache::lonlocal; +use Apache::lonnet; BEGIN { &Apache::lonhelper::register('Apache::lonhelper::choices', @@ -1163,10 +1170,12 @@ sub start_choice { my $computer = $token->[2]{'computer'}; my $human = &mt(&Apache::lonxml::get_all_text('/choice', $parser)); - my $nextstate = $token->[2]{'nextstate'}; - my $evalFlag = $token->[2]{'eval'}; + my $nextstate = $token->[2]{'nextstate'}; + my $evalFlag = $token->[2]{'eval'}; + my $relatedVar = $token->[2]{'relatedvalue'}; + my $relatedDefault = $token->[2]{'relateddefault'}; push @{$paramHash->{CHOICES}}, [&mtn($human), $computer, $nextstate, - $evalFlag]; + $evalFlag, $relatedVar, $relatedDefault]; return ''; } @@ -1189,7 +1198,8 @@ sub render { if ($self->{'multichoice'}) { $result .= <<SCRIPT; -<script> +<script type="text/javascript"> +// <!-- function checkall(value, checkName) { for (i=0; i<document.forms.helpform.elements.length; i++) { ele = document.forms.helpform.elements[i]; @@ -1198,6 +1208,7 @@ sub render { } } } +// --> </script> SCRIPT } @@ -1266,22 +1277,28 @@ BUTTONS my $id = &new_id(); $result .= "<tr>\n<td width='20'> </td>\n"; $result .= "<td valign='top'><input type='$type' name='$var.forminput'" - . "' value='" . + . " value='" . HTML::Entities::encode($choice->[1],"<>&\"'") . "'"; if ($checkedChoices{$choice->[1]}) { - $result .= " checked "; + $result .= " checked='checked' "; } - $result .= qq{id="$id"}; + $result .= qq{id="id$id"}; my $choiceLabel = $choice->[0]; - if ($choice->[4]) { # if we need to evaluate this choice + if ($choice->[3]) { # if we need to evaluate this choice $choiceLabel = "sub { my $helper = shift; my $state = shift;" . $choiceLabel . "}"; $choiceLabel = eval($choiceLabel); $choiceLabel = &$choiceLabel($helper, $self); } - $result .= "/></td><td> ".qq{<label for="$id">}. - $choiceLabel. "</label></td></tr>\n"; + $result .= "/></td><td> ".qq{<label for="id$id">}. + $choiceLabel. "</label></td>"; + if ($choice->[4]) { + $result .='<td><input type="text" size="5" name="' + .$choice->[4].'.forminput" value="' + .$choice->[5].'" /></td>'; + } + $result .= "</tr>\n"; } $result .= "</table>\n\n\n"; $result .= $buttons; @@ -1293,7 +1310,7 @@ BUTTONS # given, switch to it sub postprocess { my $self = shift; - my $chosenValue = $ENV{'form.' . $self->{'variable'} . '.forminput'}; + my $chosenValue = $env{'form.' . $self->{'variable'} . '.forminput'}; if (!defined($chosenValue) && !$self->{'allowempty'}) { $self->{ERROR_MSG} = @@ -1315,6 +1332,10 @@ sub postprocess { $helper->changeState($choice->[2]); } } + if ($choice->[4]) { + my $varname = $choice->[4]; + $helper->{'VARS'}->{$varname} = $env{'form.'."$varname.forminput"}; + } } return 1; } @@ -1347,6 +1368,7 @@ no strict; @ISA = ("Apache::lonhelper::element"); use strict; use Apache::lonlocal; +use Apache::lonnet; BEGIN { &Apache::lonhelper::register('Apache::lonhelper::dropdown', @@ -1427,7 +1449,7 @@ sub render { HTML::Entities::encode($choice->[1],"<>&\"'") . "'"; if ($checkedChoices{$choice->[1]}) { - $result .= " selected"; + $result .= " selected='selected' "; } my $choiceLabel = $choice->[0]; if ($choice->[4]) { # if we need to evaluate this choice @@ -1436,7 +1458,7 @@ sub render { $choiceLabel = eval($choiceLabel); $choiceLabel = &$choiceLabel($helper, $self); } - $result .= ">" . &mtn($choiceLabel) . "\n"; + $result .= ">" . &mtn($choiceLabel) . "</option>\n"; } $result .= "</select>\n"; @@ -1447,7 +1469,7 @@ sub render { # given, switch to it sub postprocess { my $self = shift; - my $chosenValue = $ENV{'form.' . $self->{'variable'} . '.forminput'}; + my $chosenValue = $env{'form.' . $self->{'variable'} . '.forminput'}; if (!defined($chosenValue) && !$self->{'allowempty'}) { $self->{ERROR_MSG} = "You must choose one or more choices to" . @@ -1506,7 +1528,7 @@ no strict; @ISA = ("Apache::lonhelper::element"); use strict; use Apache::lonlocal; # A localization nightmare - +use Apache::lonnet; use Time::localtime; BEGIN { @@ -1534,6 +1556,7 @@ sub start_date { $paramHash->{'variable'} = $token->[2]{'variable'}; $helper->declareVar($paramHash->{'variable'}); $paramHash->{'hoursminutes'} = $token->[2]{'hoursminutes'}; + $paramHash->{'anytime'} = $token->[2]{'anytime'}; } sub end_date { @@ -1552,10 +1575,45 @@ sub render { my $var = $self->{'variable'}; my $date; - + + my $time=time; + my ($anytime,$onclick); + + + # first check VARS for a valid new value from the user + # then check DEFAULT_VALUE for a valid default time value + # otherwise pick now as reasonably good time + + if (defined($helper->{VARS}{$var}) + && $helper->{VARS}{$var} > 0) { + $date = localtime($helper->{VARS}{$var}); + } elsif (defined($self->{DEFAULT_VALUE})) { + my $valueFunc = eval($self->{DEFAULT_VALUE}); + die('Error in default value code for variable ' . + $self->{'variable'} . ', Perl said: ' . $@) if $@; + $time = &$valueFunc($helper, $self); + if (lc($time) eq 'anytime') { + $anytime=1; + $date = localtime(time); + $date->min(0); + } elsif (defined($time) && $time ne 0) { + $date = localtime($time); + } else { + # leave date undefined so it'll default to now + } + } + + if (!defined($date)) { + $date = localtime(time); + $date->min(0); + } + + &Apache::lonnet::logthis("date mode "); + + if ($anytime) { + $onclick = "onclick=\"javascript:updateCheck(this.form,'${var}anytime',false)\""; + } # Default date: The current hour. - $date = localtime(); - $date->min(0); if (defined $self->{ERROR_MSG}) { $result .= '<font color="#FF0000">' . $self->{ERROR_MSG} . '</font><br /><br />'; @@ -1563,10 +1621,10 @@ sub render { # Month my $i; - $result .= "<select name='${var}month'>\n"; + $result .= "<select $onclick name='${var}month'>\n"; for ($i = 0; $i < 12; $i++) { if ($i == $date->mon) { - $result .= "<option value='$i' selected>"; + $result .= "<option value='$i' selected='selected'>"; } else { $result .= "<option value='$i'>"; } @@ -1575,10 +1633,10 @@ sub render { $result .= "</select>\n"; # Day - $result .= "<select name='${var}day'>\n"; + $result .= "<select $onclick name='${var}day'>\n"; for ($i = 1; $i < 32; $i++) { if ($i == $date->mday) { - $result .= '<option selected>'; + $result .= '<option selected="selected">'; } else { $result .= '<option>'; } @@ -1587,10 +1645,10 @@ sub render { $result .= "</select>,\n"; # Year - $result .= "<select name='${var}year'>\n"; + $result .= "<select $onclick name='${var}year'>\n"; for ($i = 2000; $i < 2030; $i++) { # update this after 64-bit dates if ($date->year + 1900 == $i) { - $result .= "<option selected>"; + $result .= "<option selected='selected'>"; } else { $result .= "<option>"; } @@ -1604,22 +1662,22 @@ sub render { my $am = &mt('a.m.'); my $pm = &mt('p.m.'); # Build hour - $result .= "<select name='${var}hour'>\n"; - $result .= "<option " . ($date->hour == 0 ? 'selected ':'') . + $result .= "<select $onclick name='${var}hour'>\n"; + $result .= "<option " . ($date->hour == 0 ? 'selected="selected" ':'') . " value='0'>" . &mt('midnight') . "</option>\n"; for ($i = 1; $i < 12; $i++) { if ($date->hour == $i) { - $result .= "<option selected value='$i'>$i $am</option>\n"; + $result .= "<option selected='selected' value='$i'>$i $am</option>\n"; } else { $result .= "<option value='$i'>$i $am</option>\n"; } } - $result .= "<option " . ($date->hour == 12 ? 'selected ':'') . + $result .= "<option " . ($date->hour == 12 ? 'selected="selected" ':'') . " value='12'>" . &mt('noon') . "</option>\n"; for ($i = 13; $i < 24; $i++) { my $printedHour = $i - 12; if ($date->hour == $i) { - $result .= "<option selected value='$i'>$printedHour $pm</option>\n"; + $result .= "<option selected='selected' value='$i'>$printedHour $pm</option>\n"; } else { $result .= "<option value='$i'>$printedHour $pm</option>\n"; } @@ -1627,14 +1685,16 @@ sub render { $result .= "</select> :\n"; - $result .= "<select name='${var}minute'>\n"; - for ($i = 0; $i < 60; $i++) { + $result .= "<select $onclick name='${var}minute'>\n"; + my $selected=0; + for my $i ((0,15,30,45,59,undef,0..59)) { my $printedMinute = $i; - if ($i < 10) { + if (defined($i) && $i < 10) { $printedMinute = "0" . $printedMinute; } - if ($date->min == $i) { - $result .= "<option selected>"; + if (!$selected && $date->min == $i) { + $result .= "<option selected='selected'>"; + $selected=1; } else { $result .= "<option>"; } @@ -1642,7 +1702,23 @@ sub render { } $result .= "</select>\n"; } - + if ($self->{'anytime'}) { + $result.=(<<CHECK); +<script type="text/javascript"> +// <!-- + function updateCheck(form,name,value) { + var checkbox=form[name]; + checkbox.checked = value; + } +// --> +</script> +CHECK + $result.=" or <label><input type='checkbox' "; + if ($anytime) { + $result.=' checked="checked" ' + } + $result.="name='${var}anytime'/>".&mt('Any time').'</label>' + } return $result; } @@ -1650,40 +1726,53 @@ sub render { sub postprocess { my $self = shift; my $var = $self->{'variable'}; - my $month = $ENV{'form.' . $var . 'month'}; - my $day = $ENV{'form.' . $var . 'day'}; - my $year = $ENV{'form.' . $var . 'year'}; - my $min = 0; - my $hour = 0; - if ($self->{'hoursminutes'}) { - $min = $ENV{'form.' . $var . 'minute'}; - $hour = $ENV{'form.' . $var . 'hour'}; - } + if ($env{'form.' . $var . 'anytime'}) { + $helper->{VARS}->{$var} = undef; + } else { + my $month = $env{'form.' . $var . 'month'}; + my $day = $env{'form.' . $var . 'day'}; + my $year = $env{'form.' . $var . 'year'}; + my $min = 0; + my $hour = 0; + if ($self->{'hoursminutes'}) { + $min = $env{'form.' . $var . 'minute'}; + $hour = $env{'form.' . $var . 'hour'}; + } + + my $chosenDate; + eval {$chosenDate = Time::Local::timelocal(0, $min, $hour, $day, $month, $year);}; + my $error = $@; + + # Check to make sure that the date was not automatically co-erced into a + # valid date, as we want to flag that as an error + # This happens for "Feb. 31", for instance, which is coerced to March 2 or + # 3, depending on if it's a leap year + my $checkDate = localtime($chosenDate); + + if ($error || $checkDate->mon != $month || $checkDate->mday != $day || + $checkDate->year + 1900 != $year) { + unless (Apache::lonlocal::current_language()== ~/^en/) { + $self->{ERROR_MSG} = &mt("Invalid date entry"); + return 0; + } + # LOCALIZATION FIXME: Needs to be parameterized + $self->{ERROR_MSG} = "Can't use " . $months[$month] . " $day, $year as a " + . "date because it doesn't exist. Please enter a valid date."; - my $chosenDate; - eval {$chosenDate = Time::Local::timelocal(0, $min, $hour, $day, $month, $year);}; - my $error = $@; - - # Check to make sure that the date was not automatically co-erced into a - # valid date, as we want to flag that as an error - # This happens for "Feb. 31", for instance, which is coerced to March 2 or - # 3, depending on if it's a leap year - my $checkDate = localtime($chosenDate); - - if ($error || $checkDate->mon != $month || $checkDate->mday != $day || - $checkDate->year + 1900 != $year) { - unless (Apache::lonlocal::current_language()== ~/^en/) { - $self->{ERROR_MSG} = &mt("Invalid date entry"); return 0; } - # LOCALIZATION FIXME: Needs to be parameterized - $self->{ERROR_MSG} = "Can't use " . $months[$month] . " $day, $year as a " - . "date because it doesn't exist. Please enter a valid date."; - - return 0; + $helper->{VARS}->{$var} = $chosenDate; } - $helper->{VARS}->{$var} = $chosenDate; + if (defined($self->{VALIDATOR})) { + my $validator = eval($self->{VALIDATOR}); + die 'Died during evaluation of validator code; Perl said: ' . $@ if $@; + my $invalid = &$validator($helper, $state, $self, $self->getValue()); + if ($invalid) { + $self->{ERROR_MSG} = $invalid; + return 0; + } + } if (defined($self->{NEXTSTATE})) { $helper->changeState($self->{NEXTSTATE}); @@ -1757,6 +1846,7 @@ than 1 part. no strict; @ISA = ("Apache::lonhelper::element"); use strict; +use Apache::lonnet; BEGIN { &Apache::lonhelper::register('Apache::lonhelper::resource', @@ -1929,7 +2019,8 @@ sub render { if ($self->{'multichoice'}) { $result = <<SCRIPT; -<script> +<script type="text/javascript"> +// <!-- function checkall(value, checkName) { for (i=0; i<document.forms.helpform.elements.length; i++) { ele = document.forms.helpform.elements[i]; @@ -1938,6 +2029,7 @@ sub render { } } } +// --> </script> SCRIPT my %lt=&Apache::lonlocal::texthash( @@ -1977,6 +2069,19 @@ BUTTONS $mapUrl = $self->{MAP_URL}; } + my %defaultSymbs; + if (defined($self->{DEFAULT_VALUE})) { + my $valueFunc = eval($self->{DEFAULT_VALUE}); + die 'Error in default value code for variable ' . + $self->{'variable'} . ', Perl said: ' . $@ if $@; + my @defaultSymbs = &$valueFunc($helper, $self); + if (!$multichoice && @defaultSymbs) { # only allowed 1 + @defaultSymbs = ($defaultSymbs[0]); + } + %defaultSymbs = map { if ($_) {($_,1) } } @defaultSymbs; + delete($defaultSymbs{''}); + } + # Create the composite function that renders the column on the nav map # have to admit any language that lets me do this can't be all bad # - Jeremy (Pythonista) ;-) @@ -2005,25 +2110,40 @@ BUTTONS return $result; } else { my $col = ""; + my $raw_name = &$valueFunc($resource); my $resource_name = - HTML::Entities::encode(&$valueFunc($resource),"<>&\"'"); + HTML::Entities::encode($raw_name,"<>&\"'"); if($option_vars) { foreach my $option_var (@$option_vars) { + my $var_value = "\|\|\|" . $helper->{VARS}->{$option_var} . + "\|\|\|"; + my $checked =""; + if($var_value =~ /\Q|||$raw_name|||\E/) { + $checked = "checked='checked'"; + } $col .= "<td align='center'><input type='checkbox' name ='$option_var". ".forminput' value='". - $resource_name . "' /> </td>"; + $resource_name . "' $checked /> </td>"; } } $col .= "<td align='center'><input type='$inputType' name='${var}.forminput' "; - if (!$checked && !$multichoice) { - $col .= "checked "; - $checked = 1; - } - if ($multichoice) { # all resources start checked; see bug 1174 - $col .= "checked "; - $checked = 1; + if (%defaultSymbs) { + my $symb=$resource->symb(); + if (exists($defaultSymbs{$symb})) { + $col .= "checked='checked' "; + $checked = 1; + } + } else { + if (!$checked && !$multichoice) { + $col .= "checked='checked' "; + $checked = 1; + } + if ($multichoice) { # all resources start checked; see bug 1174 + $col .= "checked='checked' "; + $checked = 1; + } } $col .= "value='" . $resource_name . "' /></td>"; @@ -2048,6 +2168,7 @@ BUTTONS }; $result.=(<<RADIO); <script type="text/javascript"> +// <!-- function updateRadio(form,name,value) { var radiobutton=form[name]; for (var i=0; i<radiobutton.length; i++) { @@ -2063,11 +2184,12 @@ BUTTONS var which=select.selectedIndex; hidden.value=select.options[which].value; } +// --> </script> -<input type="hidden" name="${var}_part.forminput" />; +<input type="hidden" name="${var}_part.forminput" /> RADIO - $ENV{'form.condition'} = !$self->{'toponly'}; + $env{'form.condition'} = !$self->{'toponly'}; my $cols = [$renderColFunc]; if ($self->{'addparts'}) { push(@$cols, $renderPartsFunc); } push(@$cols, Apache::lonnavmaps::resource()); @@ -2139,6 +2261,10 @@ selection. Defaults to false. If true, only active students and course personnel will be shown. Defaults to false. +=item * B<emptyallowed>: + +If true, the selection of no users is allowed. Defaults to false. + =back =cut @@ -2147,7 +2273,120 @@ no strict; @ISA = ("Apache::lonhelper::element"); use strict; use Apache::lonlocal; +use Apache::lonnet; +# +# Utility function used when rendering the <student> tag. +# This function renders a segment of course personel +# Personel are broken up by the helper into past, current and +# future...each one gets is own subpage of selection. +# This sub renders one of these pages. +# Parameters: +# $sections - Set of sections in the course (hash reference). +# $students - Students in the section. (ref to array of references +# to arrays). +# $formprefix - form path prefix for form element names +# This is used to make each form element +# so that the segments having to do with each +# set of students won't collide. +# $defaultusers - reference to a hash containng +# the set of users that should be on or off. +# Returns: +# HTML text to add to the rendering of the helper. +# +sub render_student_list { + my ($self, + $sections, $students, $formprefix, $defaultusers) = @_; + + my $multiselect = $self->{'multichoice'}; + my $result = ""; + + # If multiple selections are allowed, we have a listbox + # at the top which allows quick selections from each section + # as well as from categories of personnel. + + if ($multiselect) { + $result .= '<table><tr><td>'; + + my $size = scalar(keys %$sections); + $size += 3; # We have allstudents allpersonel nosection too. + if ($size > 5) { + $size = 5; + } + $result .= '<select multiple name="'.$formprefix + .'.chosensections" size="'.$size.'">'."\n"; + $result .= '<option name="allstudents">All Students</option>'; + $result .= '<option name="allpersonnel">All Course Personnel</option>'; + $result .= '<option name="nosection">No Section</option>'; + $result .= "\n"; + foreach my $sec (sort {lc($a) cmp lc($b)} (keys(%$sections))) { + $result .= '<option name="'.$sec.'">'.$sec.'</option>'."\n"; + } + $result .= '</td><td valign="top">'; + $result .= '<input type="button" name="'.$formprefix.'.select" value="Select" onClick=' + ."'selectSections(\"$formprefix.chosensections\")'".' /></td>'; + $result .= '<td valign="top"><input type="button" name="'.$formprefix + .'.unselect" value="Unselect" onClick='. + "'unselectSections(\"$formprefix.chosensections\")' ".' /></td></tr></table>'; + } + + # Now we list the students, but the form element type + # will depend on whether or not multiselect is true. + # True -> checkboxes. + # False -> radiobuttons. + + $result .= "<table border=\"2\">\n"; + $result .= '<tr><th></th><th align="center">Name</th>'."\n"; + $result .= ' <th align="center">Section</th>'."\n"; + $result .= ' <th align="center">Status</th>'."\n"; + $result .= ' <th align="center">Role</th>'."\n"; + $result .= ' <th align="center">Username : Domain</th></tr>'."\n"; + + my $input_type; + if ($multiselect) { + $input_type = "checkbox"; + } else { + $input_type = "radio"; + } + + my $checked = 0; + for my $student (@$students) { + $result .= '<tr><td><input type="'.$input_type.'" name="'. + $self->{'variable'}.".forminput".'"'; + my $user = $student->[0]; + + # Figure out which students are checked by default... + + if(%$defaultusers) { + if (exists ($defaultusers->{$user})) { + $result .= ' checked ="checked" '; + $checked = 1; + } + } elsif (!$self->{'multichoice'} && !$checked) { + $result .= ' checked="checked" '; + $checked = 1; # First one for radio if no default specified. + } + $result .= ' value="'. HTML::Entities::encode($user . ':' + .$student->[2] . ':' + .$student->[1] . ':' + .$student->[3] . ':' + .$student->[4], "<>&\"'") + ."\" /></td><td>\n"; + $result .= HTML::Entities::encode($student->[1], '<>&"') + . '</td><td align="center" >'."\n"; + $result .= HTML::Entities::encode($student->[2], '<>&"') + . '</td><td align="center">'."\n"; + $result .= HTML::Entities::encode($student->[3], '<>&"') + . '</td><td align="center">'."\n"; + $result .= HTML::Entities::encode($student->[4], '<>&"') + . '</td><td align="center">'."\n"; + $result .= HTML::Entities::encode($student->[0], '<>&"') + . '</td></tr>'."\n"; + } + $result .=" </table> <br /> <hr />\n"; + + return $result; +} BEGIN { &Apache::lonhelper::register('Apache::lonhelper::student', @@ -2174,6 +2413,7 @@ sub start_student { if (defined($token->[2]{'nextstate'})) { $paramHash->{NEXTSTATE} = $token->[2]{'nextstate'}; } + $paramHash->{'emptyallowed'} = $token->[2]{'emptyallowed'}; } @@ -2194,7 +2434,101 @@ sub render { if ($self->{'multichoice'}) { $result = <<SCRIPT; -<script> +<script type="text/javascript"> +// <!-- + + function findElement(name) { + var i; + var ele; + for(i =0; i < document.forms.helpform.elements.length; i++) { + ele = document.forms.helpform.elements[i]; + if(ele.name == name) { + return ele; + } + } + return null; + } + function isStudent(element) { + if(element.value.indexOf(":Student") != -1) { + return 1; + } + return 0; + } + function section(element) { + var i; + var info; + if (element.value.indexOf(':') != -1) { + info = element.value.split(':'); + return info[2]; + } else { + return ""; + } + } + + function setAllStudents(value) { + var i; + var ele; + for (i =0; i < document.forms.helpform.elements.length; i++) { + ele = document.forms.helpform.elements[i]; + if(isStudent(ele)) { + ele.checked=value; + } + } + } + function setAllCoursePersonnel(value) { + var i; + var ele; + for (i =0; i < document.forms.helpform.elements.length; i++) { + ele = document.forms.helpform.elements[i]; + if(!isStudent(ele)) { + ele.checked = value; + } + } + } + function setSection(which, value) { + var i; + var ele; + for (i =0; i < document.forms.helpform.elements.length; i++) { + ele = document.forms.helpform.elements[i]; + if (ele.value.indexOf(':') != -1) { + if (section(ele) == which) { + ele.checked = value; + } + } + } + } + + function setCheckboxes(listbox, value) { + var k; + var elem; + var what; + elem = findElement(listbox); + if (elem != null) { + for (k = 0; k < elem.length; k++) { + if (elem.options[k].selected) { + what = elem.options[k].text; + if (what == 'All Students') { + setAllStudents(value); + } else if (what == 'All Course Personnel') { + setAllCoursePersonnel(value); + } else if (what == 'No Section') { + setSection('',value); + } else { + setSection(what, value); + } + } + } + } + } + function selectSections(listbox) { + setCheckboxes(listbox, true); + + } + function unselectSections(listbox) { + setCheckboxes(listbox, false); + } + /* ----------------------------- */ + function checkall(value, checkName) { for (i=0; i<document.forms.helpform.elements.length; i++) { ele = document.forms.helpform.elements[i]; @@ -2220,6 +2554,13 @@ sub render { } } } + function checkexpired() { + for (i=0; i<document.forms.helpform.elements.length; i++) { + if (document.forms.helpform.elements[i].value.indexOf(':Expired') != -1) { + document.forms.helpform.elements[i].checked=true; + } + } + } function uncheckexpired() { for (i=0; i<document.forms.helpform.elements.length; i++) { if (document.forms.helpform.elements[i].value.indexOf(':Expired') != -1) { @@ -2227,6 +2568,44 @@ sub render { } } } + function getDesiredState() { // Return desired person state radio value. + numRadio = document.forms.helpform.personstate.length; + for (i =0; i < numRadio; i++) { + if (document.forms.helpform.personstate[i].checked) { + return document.forms.helpform.personstate[i].value; + } + } + return ""; + } + + function checksections(value) { // Check selected sections. + numSections = document.forms.helpform.chosensections.length; + desiredState = getDesiredState(); + + for (var option = 0; option < numSections; option++) { + if(document.forms.helpform.chosensections.options[option].selected) { + section = document.forms.helpform.chosensections.options[option].text; + if (section == "none") { + section =""; + } + for (i = 0; i < document.forms.helpform.elements.length; i++ ) { + if (document.forms.helpform.elements[i].value.indexOf(':') != -1) { + info = document.forms.helpform.elements[i].value.split(':'); + hisSection = info[2]; + hisState = info[4]; + if (desiredState == hisState || + desiredState == "All") { + if(hisSection == section || + section =="" ) { + document.forms.helpform.elements[i].checked = value; + } + } + } + } + } + } + } +// --> </script> SCRIPT @@ -2241,33 +2620,57 @@ SCRIPT $buttons = <<BUTTONS; <br /> <table> - <tr> - <td><input type="button" onclick="checkactive()" value="$lt{'ocs'}" /></td> - <td><input type="button" onclick="uncheckexpired()" value="$lt{'ues'}" /><br /></td> - </tr> <tr> <td><input type="button" onclick="checkall(true, '$var')" value="$lt{'sas'}" /></td> <td> <input type="button" onclick="checkall(false, '$var')" value="$lt{'uas'}" /><br /></td> </tr> - <tr> - <td><input type="button" onclick="checksec(true)" value="$lt{'sfsg'}"></td> - <td><input type="text" size="5" name="chksec"> </td> - </tr> - <tr> - <td><input type="button" onclick="checksec(false)" value="$lt{'ufsg'}"></td> - <td></td> - </tr> + </table> <br /> BUTTONS - } +# $result .= $buttons; + +} if (defined $self->{ERROR_MSG}) { $result .= '<font color="#FF0000">' . $self->{ERROR_MSG} . '</font><br /><br />'; } - my $choices = []; + my %defaultUsers; + if (defined($self->{DEFAULT_VALUE})) { + my $valueFunc = eval($self->{DEFAULT_VALUE}); + die 'Error in default value code for variable ' . + $self->{'variable'} . ', Perl said: ' . $@ if $@; + my @defaultUsers = &$valueFunc($helper, $self); + if (!$self->{'multichoice'} && @defaultUsers) { # only allowed 1 + @defaultUsers = ($defaultUsers[0]); + } + %defaultUsers = map { if ($_) {($_,1) } } @defaultUsers; + delete($defaultUsers{''}); + } + + + + # my $choices = []; + + # + # We need to parcel out the personel in to three arrays: + # $current_members[] - Contains those whose roles are currently active. + # $expired_members[] - Contains those whose roles have expired. + # $future_members[] - Contains those whose roles will become active in the + # future. + # + # Constants + my $section = &Apache::loncoursedata::CL_SECTION(); + my $fullname = &Apache::loncoursedata::CL_FULLNAME(); + my $status = &Apache::loncoursedata::CL_STATUS(); + my $start_date = &Apache::loncoursedata::CL_START(); + + my $current_members = []; + my $expired_members = []; + my $future_members = []; + # Load up the non-students, if necessary if ($self->{'coursepersonnel'}) { @@ -2284,16 +2687,12 @@ BUTTONS @people = sort { $a->[0] cmp $b->[0] } @people; for my $person (@people) { - push @$choices, [join(':', @$person), $person->[0], '', $_]; + push @$current_members, [join(':', @$person), $person->[0], '', $_]; } } } } - # Constants - my $section = Apache::loncoursedata::CL_SECTION(); - my $fullname = Apache::loncoursedata::CL_FULLNAME(); - my $status = Apache::loncoursedata::CL_STATUS(); # Load up the students my $classlist = &Apache::loncoursedata::get_classlist(); @@ -2305,68 +2704,110 @@ BUTTONS } return $classlist->{$a}->[$fullname] cmp $classlist->{$b}->[$fullname]; } @keys; + + + - # username, fullname, section, type for (@keys) { - # Filter out inactive students if we've set "activeonly" - if (!$self->{'activeonly'} || $classlist->{$_}->[$status] eq + + if ( $classlist->{$_}->[$status] eq 'Active') { - push @$choices, [$_, $classlist->{$_}->[$fullname], + push @$current_members, [$_, $classlist->{$_}->[$fullname], $classlist->{$_}->[$section], $classlist->{$_}->[$status], 'Student']; + } else { + # Need to figure out if this user is future or + # Expired... If the start date is in the future + # the user is future...else expired. + + my $now = time; + if ($classlist->{$_}->[$start_date] > $now) { + push @$future_members, [$_, $classlist->{$_}->[$fullname], + $classlist->{$_}->[$section], + "Future", "Student"]; + } else { + push @$expired_members, [$_, $classlist->{$_}->[$fullname], + $classlist->{$_}->[$section], + "Expired", "Student"]; + } + } } - my $name = $self->{'coursepersonnel'} ? &mt('Name') : &mt('Student Name'); - my $type = 'radio'; - if ($self->{'multichoice'}) { $type = 'checkbox'; } - $result .= "<table cellspacing='2' cellpadding='2' border='0'>\n"; - $result .= "<tr><td></td><td align='center'><b>$name</b></td>". - "<td align='center'><b>" . &mt('Section') . "</b></td>" . - "<td align='center'><b>".&mt('Status')."</b></td>" . - "<td align='center'><b>" . &mt("Role") . "</b></td>" . - "<td align='center'><b>".&mt('Username').":".&mt('Domain')."</b></td></tr>"; - my $checked = 0; - for my $choice (@$choices) { - $result .= "<tr><td><input type='$type' name='" . - $self->{'variable'} . '.forminput' . "'"; - - if (!$self->{'multichoice'} && !$checked) { - $result .= " checked "; - $checked = 1; - } - $result .= - " value='" . HTML::Entities::encode($choice->[0] . ':' - .$choice->[2] . ':' - .$choice->[1] . ':' - .$choice->[3], "<>&\"'") - . "' /></td><td>" - . HTML::Entities::encode($choice->[1],'<>&"') - . "</td><td align='center'>" - . HTML::Entities::encode($choice->[2],'<>&"') - . "</td>\n<td>" - . HTML::Entities::encode($choice->[3],'<>&"') - . "</td>\n<td>" - . HTML::Entities::encode($choice->[4],'<>&"') - . "</td>\n<td>" - . HTML::Entities::encode($choice->[0],'<>&"') - . "</td></tr>\n"; + # Create a list of the sections that can be used to create the section + # selection list boxes: + # + my %sections; + for my $key (@keys) { + my $section_name = $classlist->{$key}->[$section]; + if ($section_name ne "") { + $sections{$section_name} = 1; + } } - $result .= "</table>\n\n"; - $result .= $buttons; - + + if ($self->{'multichoice'}) { + + # The variable $choice_widget will have the html to make the choice + # selector. + my $size=5; + if (scalar(keys(%sections)) < 5) { + $size=scalar(keys(%sections)); + } + my $result = '<select multiple name="chosensections" size="'.$size.'">'."\n"; + foreach my $sec (sort {lc($a) cmp lc($b)} (keys(%sections))) { + $result .= "<option name=\"$sec\">$sec</option>\n"; + } + $result .= "<option>none</option></select>\n"; + + + } + + # Current personel + + $result .= $self->render_student_list(\%sections, + $current_members, + "current", + \%defaultUsers); + + + # If activeonly is not set then we can also give the expired students: + # + if (!$self->{'activeonly'} && ((scalar @$expired_members) > 0)) { + + # Past + + $result .= $self->render_student_list(\%sections, + $expired_members, + "past", + \%defaultUsers); + + # And future. + + $result .= $self->render_student_list(\%sections, + $future_members, + "future", + \%defaultUsers); + } + + + return $result; } sub postprocess { my $self = shift; - my $result = $ENV{'form.' . $self->{'variable'} . '.forminput'}; - if (!$result) { - $self->{ERROR_MSG} = - &mt('You must choose at least one student to continue.'); + my $result = $env{'form.' . $self->{'variable'} . '.forminput'}; + if (!$result && !$self->{'emptyallowed'}) { + if ($self->{'coursepersonnel'}) { + $self->{ERROR_MSG} = + &mt('You must choose at least one user to continue.'); + } else { + $self->{ERROR_MSG} = + &mt('You must choose at least one student to continue.'); + } return 0; } @@ -2433,7 +2874,7 @@ no strict; @ISA = ("Apache::lonhelper::element"); use strict; use Apache::lonlocal; - +use Apache::lonnet; use Apache::lonpubdir; # for getTitleString BEGIN { @@ -2538,7 +2979,8 @@ sub render { if ($self->{'multichoice'}) { $result = <<SCRIPT; -<script> +<script type="text/javascript"> +// <!-- function checkall(value, checkName) { for (i=0; i<document.forms.helpform.elements.length; i++) { ele = document.forms.helpform.elements[i]; @@ -2556,6 +2998,7 @@ sub render { } } } +// --> </script> SCRIPT my %lt=&Apache::lonlocal::texthash( @@ -2585,19 +3028,21 @@ BUTTONS # If the subdirectory is in local CSTR space my $metadir; if ($subdir =~ m|/home/([^/]+)/public_html/(.*)|) { - my $user = $1; - my $domain = $Apache::lonnet::perlvar{'lonDefDomain'}; + my ($user,$domain)= + &Apache::loncacc::constructaccess($subdir, + $Apache::lonnet::perlvar{'lonDefDomain'}); $metadir='/res/'.$domain.'/'.$user.'/'.$2; @fileList = &Apache::lonnet::dirlist($subdir, $domain, $user, ''); } elsif ($subdir =~ m|^~([^/]+)/(.*)$|) { $subdir='/home/'.$1.'/public_html/'.$2; - my $user = $1; - my $domain = $Apache::lonnet::perlvar{'lonDefDomain'}; + my ($user,$domain)= + &Apache::loncacc::constructaccess($subdir, + $Apache::lonnet::perlvar{'lonDefDomain'}); $metadir='/res/'.$domain.'/'.$user.'/'.$2; @fileList = &Apache::lonnet::dirlist($subdir, $domain, $user, ''); } else { # local library server resource space - @fileList = &Apache::lonnet::dirlist($subdir, $ENV{'user.domain'}, $ENV{'user.name'}, ''); + @fileList = &Apache::lonnet::dirlist($subdir, $env{'user.domain'}, $env{'user.name'}, ''); } # Sort the fileList into order @@ -2659,7 +3104,7 @@ BUTTONS . ".forminput' ".qq{id="$id"}." value='" . HTML::Entities::encode($fileName,"<>&\"'"). "'"; if (!$self->{'multichoice'} && $choices == 0) { - $result .= ' checked'; + $result .= ' checked="checked"'; } $result .= "/></td><td bgcolor='$color'>". qq{<label for="$id">}. $file . "</label></td>" . @@ -2690,9 +3135,9 @@ sub fileState { my $constructionSpaceDir = shift; my $file = shift; - my ($uname,$udom)=($ENV{'user.name'},$ENV{'user.domain'}); - if ($ENV{'request.role'}=~/^ca\./) { - (undef,$udom,$uname)=split(/\//,$ENV{'request.role'}); + my ($uname,$udom)=($env{'user.name'},$env{'user.domain'}); + if ($env{'request.role'}=~/^ca\./) { + (undef,$udom,$uname)=split(/\//,$env{'request.role'}); } my $docroot = $Apache::lonnet::perlvar{'lonDocRoot'}; my $subdirpart = $constructionSpaceDir; @@ -2717,7 +3162,7 @@ sub fileState { sub postprocess { my $self = shift; - my $result = $ENV{'form.' . $self->{'variable'} . '.forminput'}; + my $result = $env{'form.' . $self->{'variable'} . '.forminput'}; if (!$result) { $self->{ERROR_MSG} = 'You must choose at least one file '. 'to continue.'; @@ -2742,8 +3187,12 @@ package Apache::lonhelper::section; <section> allows the user to choose one or more sections from the current course. -It takes the standard attributes "variable", "multichoice", and -"nextstate", meaning what they do for most other elements. +It takes the standard attributes "variable", "multichoice", +"allowempty" and "nextstate", meaning what they do for most other +elements. + +also takes a boolean 'onlysections' whcih will restrict this to only +have sections and not include groups =cut @@ -2773,6 +3222,7 @@ sub start_section { $paramHash->{'variable'} = $token->[2]{'variable'}; $helper->declareVar($paramHash->{'variable'}); $paramHash->{'multichoice'} = $token->[2]{'multichoice'}; + $paramHash->{'allowempty'} = $token->[2]{'allowempty'}; if (defined($token->[2]{'nextstate'})) { $paramHash->{NEXTSTATE} = $token->[2]{'nextstate'}; } @@ -2792,8 +3242,16 @@ sub start_section { } for my $sectionName (sort(keys(%choices))) { - - push @{$paramHash->{CHOICES}}, [$sectionName, $sectionName]; + push @{$paramHash->{CHOICES}}, [$sectionName, $sectionName]; + } + return if ($token->[2]{'onlysections'}); + + # add in groups to the end of the list + my %curr_groups; + if (&Apache::loncommon::coursegroups(\%curr_groups)) { + foreach my $group_name (sort(keys(%curr_groups))) { + push(@{$paramHash->{CHOICES}}, [$group_name, $group_name]); + } } } @@ -2807,6 +3265,72 @@ sub end_section { } 1; +package Apache::lonhelper::group; + +=pod + +=head2 Element: groupX<group, helper element> + +<group> allows the user to choose one or more groups from the current course. + +It takes the standard attributes "variable", "multichoice", + "allowempty" and "nextstate", meaning what they do for most other + elements. + +=cut + +no strict; +@ISA = ("Apache::lonhelper::choices"); +use strict; + +BEGIN { + &Apache::lonhelper::register('Apache::lonhelper::group', + ('group')); +} + +sub new { + my $ref = Apache::lonhelper::choices->new(); + bless($ref); +} + +sub start_group { + my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_; + + if ($target ne 'helper') { + return ''; + } + + $paramHash->{CHOICES} = []; + + $paramHash->{'variable'} = $token->[2]{'variable'}; + $helper->declareVar($paramHash->{'variable'}); + $paramHash->{'multichoice'} = $token->[2]{'multichoice'}; + $paramHash->{'allowempty'} = $token->[2]{'allowempty'}; + if (defined($token->[2]{'nextstate'})) { + $paramHash->{NEXTSTATE} = $token->[2]{'nextstate'}; + } + + # Populate the CHOICES element + my %choices; + + my %curr_groups; + if (&Apache::loncommon::coursegroups(\%curr_groups)) { + foreach my $group_name (sort(keys(%curr_groups))) { + push(@{$paramHash->{CHOICES}}, [$group_name, $group_name]); + } + } +} + +sub end_group { + my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$style)=@_; + + if ($target ne 'helper') { + return ''; + } + Apache::lonhelper::group->new(); +} +1; + package Apache::lonhelper::string; =pod @@ -2870,7 +3394,7 @@ sub render { my $result = ''; if (defined $self->{ERROR_MSG}) { - $result .= '<br /><font color="#FF0000">' . $self->{ERROR_MSG} . '</font><br /><br />'; + $result .= '<p><font color="#FF0000">' . $self->{ERROR_MSG} . '</font></p>'; } $result .= '<input type="string" name="' . $self->{'variable'} . '.forminput"'; @@ -2900,7 +3424,7 @@ sub postprocess { if (defined($self->{VALIDATOR})) { my $validator = eval($self->{VALIDATOR}); - die 'Died during evaluation of evaulation code; Perl said: ' . $@ if $@; + die 'Died during evaluation of validator code; Perl said: ' . $@ if $@; my $invalid = &$validator($helper, $state, $self, $self->getValue()); if ($invalid) { $self->{ERROR_MSG} = $invalid; @@ -2924,7 +3448,7 @@ package Apache::lonhelper::general; =head2 General-purpose tag: <exec>X<exec, helper tag> The contents of the exec tag are executed as Perl code, B<not> inside a -safe space, so the full range of $ENV and such is available. The code +safe space, so the full range of $env and such is available. The code will be executed as a subroutine wrapped with the following code: "sub { my $helper = shift; my $state = shift;" and @@ -2946,6 +3470,7 @@ be able to call methods on it. =cut use Apache::lonlocal; +use Apache::lonnet; BEGIN { &Apache::lonhelper::register('Apache::lonhelper::general', @@ -3077,6 +3602,7 @@ no strict; @ISA = ("Apache::lonhelper::element"); use strict; use Apache::lonlocal; +use Apache::lonnet; BEGIN { &Apache::lonhelper::register('Apache::lonhelper::final', ('final', 'exitpage')); @@ -3172,26 +3698,29 @@ sub render { my $targetURL = ''; my $finish=&mt('Finish'); if ($self->{'restartCourse'}) { - my $actionURL = '/adm/roles'; + $actionURL = '/adm/roles'; $targetURL = '/adm/menu'; - if ($ENV{'course.'.$ENV{'request.course.id'}.'.url'}=~/^uploaded/) { + if ($env{'course.'.$env{'request.course.id'}.'.url'}=~/^uploaded/) { $targetURL = '/adm/coursedocs'; } else { $targetURL = '/adm/navmaps'; } - if ($ENV{'course.'.$ENV{'request.course.id'}.'.clonedfrom'}) { + if ($env{'course.'.$env{'request.course.id'}.'.clonedfrom'}) { $targetURL = '/adm/parmset?overview=1'; } my $finish=&mt('Finish Course Initialization'); } my $previous = HTML::Entities::encode(&mt("<- Previous"), '<>&"'); my $next = HTML::Entities::encode(&mt("Next ->"), '<>&"'); + my $target = " target='loncapaclient'"; + if (($env{'browser.interface'} eq 'textual') || + ($env{'environment.remote'} eq 'off')) { $target=''; } $result .= "<center>\n" . - "<form action='".$actionURL."' method='post' target='loncapaclient'>\n" . + "<form action='".$actionURL."' method='post' $target>\n" . "<input type='button' onclick='history.go(-1)' value='$previous' />" . "<input type='hidden' name='orgurl' value='$targetURL' />" . "<input type='hidden' name='selectrole' value='1' />\n" . - "<input type='hidden' name='" . $ENV{'request.role'} . + "<input type='hidden' name='" . $env{'request.role'} . "' value='1' />\n<input type='submit' value='" . $finish . "' />\n" . "</form></center>"; @@ -3215,6 +3744,7 @@ no strict; @ISA = ('Apache::lonhelper::element'); use strict; use Apache::lonlocal; +use Apache::lonnet; BEGIN { &Apache::lonhelper::register('Apache::lonhelper::parmwizfinal', @@ -3256,6 +3786,11 @@ sub render { 'answer_date' => "0_answerdate", 'tries' => '0_maxtries', 'weight' => '0_weight' ); + my %realParmName = ('open_date' => "opendate", + 'due_date' => "duedate", + 'answer_date' => "answerdate", + 'tries' => 'maxtries', + 'weight' => 'weight' ); my $affectedResourceId = ""; my $parm_name = $parmTypeHash{$vars->{ACTION_TYPE}}; @@ -3267,7 +3802,13 @@ sub render { # Print the granularity, depending on the action if ($vars->{GRANULARITY} eq 'whole_course') { $resourceString .= '<li>'.&mt('for <b>all resources in the course</b>').'</li>'; - $level = 9; # general course, see lonparmset.pm perldoc + if ($vars->{TARGETS} eq 'course') { + $level = 11; # general course, see lonparmset.pm perldoc + } elsif ($vars->{TARGETS} eq 'section') { + $level = 6; + } else { + $level = 3; + } $affectedResourceId = "0.0"; $symb = 'a'; $paramlevel = 'general'; @@ -3277,7 +3818,13 @@ sub render { my $title = $res->compTitle(); $symb = $res->symb(); $resourceString .= '<li>'.&mt('for the map named [_1]',"<b>$title</b>").'</li>'; - $level = 8; + if ($vars->{TARGETS} eq 'course') { + $level = 10; # general course, see lonparmset.pm perldoc + } elsif ($vars->{TARGETS} eq 'section') { + $level = 5; + } else { + $level = 2; + } $affectedResourceId = $vars->{RESOURCE_ID}; $paramlevel = 'map'; } else { @@ -3288,22 +3835,34 @@ sub render { $symb = $res->symb(); my $title = $res->compTitle(); $resourceString .= '<li>'.&mt('for the resource named [_1] part [_2]',"<b>$title</b>","<b>$part</b>").'</li>'; - $level = 7; + if ($vars->{TARGETS} eq 'course') { + $level = 7; # general course, see lonparmset.pm perldoc + } elsif ($vars->{TARGETS} eq 'section') { + $level = 4; + } else { + $level = 1; + } $affectedResourceId = $vars->{RESOURCE_ID}; $paramlevel = 'full'; } - my $result = "<form name='helpform' method='get' action='/adm/parmset#$affectedResourceId&$parm_name&$level'>\n"; + my $result = "<form name='helpform' method='POST' action='/adm/parmset#$affectedResourceId&$parm_name&$level'>\n"; + $result .= "<input type='hidden' name='action' value='settable' />\n"; + $result .= "<input type='hidden' name='dis' value='helper' />\n"; + $result .= "<input type='hidden' name='pscat' value='". + $realParmName{$vars->{ACTION_TYPE}}."' />\n"; if ($vars->{GRANULARITY} eq 'resource') { $result .= "<input type='hidden' name='symb' value='". HTML::Entities::encode($symb,"'<>&\"") . "' />\n"; - $result .= "<input type='hidden' name='pscat' value='". - HTML::Entities::encode($vars->{ACTION_TYPE},"'<>&\"") . "' />\n"; - my $part = $vars->{RESOURCE_ID_part}; - if ($part eq 'All Parts' || !$part) { $part=0; } - $result .= "<input type='hidden' name='psprt' value='". - HTML::Entities::encode($part,"'<>&\"") . "' />\n"; + } elsif ($vars->{GRANULARITY} eq 'map') { + $result .= "<input type='hidden' name='pschp' value='". + $affectedResourceId."' />\n"; } + my $part = $vars->{RESOURCE_ID_part}; + if ($part eq 'All Parts' || !$part) { $part=0; } + $result .= "<input type='hidden' name='psprt' value='". + HTML::Entities::encode($part,"'<>&\"") . "' />\n"; + $result .= '<p>'.&mt('Confirm that this information is correct, then click "Finish Helper" to complete setting the parameter.').'<ul>'; # Print the type of manipulation: @@ -3341,6 +3900,8 @@ sub render { } elsif ($vars->{ACTION_TYPE} eq 'tries') { $result .= "<input type='hidden' name='pres_value' " . "value='" . $vars->{TRIES} . "' />\n"; + $result .= "<input type='hidden' name='pres_type' " . + "value='int_pos' />\n"; } elsif ($vars->{ACTION_TYPE} eq 'weight') { $result .= "<input type='hidden' name='pres_value' " . "value='" . $vars->{WEIGHT} . "' />\n"; @@ -3354,19 +3915,19 @@ sub render { } elsif ($vars->{TARGETS} eq 'section') { my $section = $vars->{SECTION_NAME}; $result .= '<li>'.&mt('for section [_1]',"<b>$section</b>").'</li>'; - $level -= 3; - $result .= "<input type='hidden' name='csec' value='" . + $result .= "<input type='hidden' name='csec' value='" . HTML::Entities::encode($section,"'<>&\"") . "' />\n"; + } elsif ($vars->{TARGETS} eq 'group') { + my $group = $vars->{GROUP_NAME}; + $result .= '<li>'.&mt('for group [_1]',"<b>$group</b>").'</li>'; + $result .= "<input type='hidden' name='cgroup' value='" . + HTML::Entities::encode($group,"'<>&\"") . "' />\n"; } else { # FIXME: This is probably wasteful! Store the name! my $classlist = Apache::loncoursedata::get_classlist(); - my $username = $vars->{USER_NAME}; - # Chop off everything after the last colon (section) - $username = substr($username, 0, rindex($username, ':')); - my $name = $classlist->{$username}->[6]; + my ($uname,$udom)=split(':',$vars->{USER_NAME}); + my $name = $classlist->{$uname.':'.$udom}->[6]; $result .= '<li>'.&mt('for [_1]',"<b>$name</b>").'</li>'; - $level -= 6; - my ($uname, $udom) = split /:/, $vars->{USER_NAME}; $result .= "<input type='hidden' name='uname' value='". HTML::Entities::encode($uname,"'<>&\"") . "' />\n"; $result .= "<input type='hidden' name='udom' value='". @@ -3384,7 +3945,6 @@ sub render { # 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' />";