--- loncom/homework/inputtags.pm 2006/06/12 22:03:44 1.196
+++ loncom/homework/inputtags.pm 2006/10/17 15:11:11 1.209
@@ -1,7 +1,7 @@
# The LearningOnline Network with CAPA
# input definitons
#
-# $Id: inputtags.pm,v 1.196 2006/06/12 22:03:44 albertel Exp $
+# $Id: inputtags.pm,v 1.209 2006/10/17 15:11:11 albertel Exp $
#
# Copyright Michigan State University Board of Trustees
#
@@ -140,9 +140,9 @@ sub start_textfield {
my $resid=$Apache::inputtags::response[-1];
if ($target eq 'web') {
$Apache::lonxml::evaluate--;
+ my $partid=$Apache::inputtags::part;
+ my $oldresponse = &HTML::Entities::encode($Apache::lonhomework::history{"resource.$partid.$resid.submission"},'<>&"');
if ($Apache::inputtags::status[-1] eq 'CAN_ANSWER') {
- my $partid=$Apache::inputtags::part;
- my $oldresponse = &HTML::Entities::encode($Apache::lonhomework::history{"resource.$partid.$resid.submission"},'<>&"');
my $cols = &Apache::lonxml::get_param('cols',$parstack,$safeeval);
if ( $cols eq '') { $cols = 80; }
my $rows = &Apache::lonxml::get_param('rows',$parstack,$safeeval);
@@ -161,9 +161,13 @@ sub start_textfield {
&Apache::lonxml::get_all_text("/textfield",$parser,$style);
}
} else {
- #right or wrong don't show it
- #$result='
'.$oldresponse.'
';
- $result='';
+ #show past answer in the essayresponse case
+ if ($oldresponse =~ /\S/
+ && &Apache::londefdef::is_inside_of($tagstack,
+ 'essayresponse') ) {
+ $result='
'.
+ $oldresponse.'
';
+ }
#get rid of any startup text
&Apache::lonxml::get_all_text("/textfield",$parser,$style);
}
@@ -279,13 +283,12 @@ sub needs_exam_box {
sub start_textline {
my ($target,$token,$tagstack,$parstack,$parser,$safeeval)=@_;
my $result = "";
+ my $id = &start_input($parstack,$safeeval);
if ($target eq 'web') {
$Apache::lonxml::evaluate--;
my $partid=$Apache::inputtags::part;
my $id=$Apache::inputtags::response[-1];
- if ($Apache::inputtags::status[-1] eq 'CAN_ANSWER'
- || ($Apache::inputtags::status[-1] eq 'CANNOT_ANSWER' &&
- $Apache::lonhomework::history{"resource.$partid.solved"} !~ /^correct/ )) {
+ if (!&Apache::response::show_answer()) {
my $size = &Apache::lonxml::get_param('size',$parstack,$safeeval);
my $maxlength;
if ($size eq '') { $size=20; } else {
@@ -319,7 +322,8 @@ sub start_textline {
}
} else {
#right or wrong don't show what was last typed in.
- $result=''.$Apache::inputtags::answertxt{$id}.'';
+ my $count = scalar(@Apache::inputtags::inputlist)-1;
+ $result=''.$Apache::inputtags::answertxt{$id}[$count].'';
#$result='';
}
} elsif ($target eq 'edit') {
@@ -354,6 +358,7 @@ sub end_textline {
my ($target,$token,$tagstack,$parstack,$parser,$safeeval)=@_;
if ($target eq 'web') { $Apache::lonxml::evaluate++; }
elsif ($target eq 'edit') { return ('','no'); }
+ &end_input();
return "";
}
@@ -417,19 +422,7 @@ sub file_selector {
$result.=&mt('Submit a file: (only one file can be uploaded)').
'
';
- my $uploadedfile= &HTML::Entities::encode($Apache::lonhomework::history{"resource.$part.$id.uploadedfile"},'<>&"');
-
- if ($uploadedfile) {
- my $url=$Apache::lonhomework::history{"resource.$part.$id.uploadedurl"};
- &Apache::lonxml::extlink($url);
- &Apache::lonnet::allowuploaded('/adm/essayresponse',$url);
- my $icon=&Apache::loncommon::icon($url);
- my $curfile=''.$uploadedfile.'';
- $result.=&mt('Currently submitted: [_1]',$curfile);
- } else {
- #$result.=&mt('(Hand in a file you have prepared on your computer)');
- }
+ $result .= &show_past_file_submission($part,$id);
}
if ( $which eq 'both') {
$result.=' '.''.&mt('OR:').' ';
@@ -439,38 +432,59 @@ sub file_selector {
&mt('Select Portfolio Files').' '.
''.
' ';
- if ($Apache::lonhomework::history{"resource.$part.$id.portfiles"}=~/[^\s]/){
- my (@filelist,@bad_file_list);
- foreach my $file (split(',',&unescape($Apache::lonhomework::history{"resource.$part.$id.portfiles"}))) {
- my (undef,undef,$domain,$user)=&Apache::lonxml::whichuser();
- my $url="/uploaded/$domain/$user/portfolio$file";
- my $icon=&Apache::loncommon::icon($url);
- push(@filelist,''.$file.'');
- if (! &Apache::lonnet::stat_file($url)) {
- &Apache::lonnet::logthis("bad file is $url");
- push(@bad_file_list,''.$file.'');
- }
- }
- $result.=&mt("Portfolio files previously selected: [_1]",join(', ',@filelist));
- if (@bad_file_list) {
- $result.=' '.&mt('These file(s) don\'t exist:[_1]',join(', ',@bad_file_list));
- }
- }
+ $result .= &show_past_portfile_submission($part,$id);
+
}
$result.='';
return $result;
}
-sub checkstatus {
- my ($value,$awardref,$msgref)=@_;
- for (my $i=0;$i<=$#$awardref;$i++) {
- if ($$awardref[$i] eq $value) {
- return ($$awardref[$i],$$msgref[$i]);
- }
+sub show_past_file_submission {
+ my ($part,$id) = @_;
+ my $uploadedfile= &HTML::Entities::encode($Apache::lonhomework::history{"resource.$part.$id.uploadedfile"},'<>&"');
+
+ return if (!$uploadedfile);
+
+ my $url=$Apache::lonhomework::history{"resource.$part.$id.uploadedurl"};
+ &Apache::lonxml::extlink($url);
+ &Apache::lonnet::allowuploaded('/adm/essayresponse',$url);
+ my $icon=&Apache::loncommon::icon($url);
+ my $curfile=''.$uploadedfile.'';
+ return &mt('Currently submitted: [_1]',$curfile);
+
+}
+
+sub show_past_portfile_submission {
+ my ($part,$id) = @_;
+ if ($Apache::lonhomework::history{"resource.$part.$id.portfiles"}!~/[^\s]/){
+ return;
+ }
+ my (@file_list,@bad_file_list);
+ foreach my $file (split(/\s*,\s*/,&unescape($Apache::lonhomework::history{"resource.$part.$id.portfiles"}))) {
+ my (undef,undef,$domain,$user)=&Apache::lonnet::whichuser();
+ my $url="/uploaded/$domain/$user/portfolio$file";
+ my $icon=&Apache::loncommon::icon($url);
+ push(@file_list,''.$file.'');
+ if (! &Apache::lonnet::stat_file($url)) {
+ &Apache::lonnet::logthis("bad file is $url");
+ push(@bad_file_list,''.$file.'');
+ }
+ }
+ my $files = ''.
+ join(', ',@file_list).
+ '';
+ my $result = &mt("Portfolio files previously selected: [_1]",$files);
+ if (@bad_file_list) {
+ my $bad_files = ''.
+ join(', ',@bad_file_list).
+ '';
+ $result.=' '.&mt('These file(s) don\'t exist: [_1]',$bad_files);
}
- return(undef,undef);
+ return $result;
+
}
sub valid_award {
@@ -490,15 +504,29 @@ sub valid_award {
return 0;
}
+{
+ my @awards = ('EXTRA_ANSWER', 'MISSING_ANSWER', 'ERROR', 'NO_RESPONSE',
+ 'TOO_LONG',
+ 'UNIT_INVALID_INSTRUCTOR', 'UNIT_INVALID_STUDENT',
+ 'UNIT_IRRECONCIBLE', 'UNIT_FAIL', 'NO_UNIT',
+ 'UNIT_NOTNEEDED', 'WANTED_NUMERIC', 'BAD_FORMULA',
+ 'COMMA_FAIL', 'SIG_FAIL', 'INCORRECT', 'MISORDERED_RANK',
+ 'INVALID_FILETYPE', 'DRAFT', 'SUBMITTED', 'ASSIGNED_SCORE',
+ 'APPROX_ANS', 'EXACT_ANS');
+ my $i=0;
+ my %fwd_awards = map { ($_,$i++) } @awards;
+ my $max=scalar(@awards);
+ @awards=reverse(@awards);
+ $i=0;
+ my %rev_awards = map { ($_,$i++) } @awards;
+
sub finalizeawards {
my ($awardref,$msgref,$nameref,$reverse)=@_;
- my $result=undef;
- my $award;
- my $msg;
+ my $result;
if ($#$awardref == -1) { $result = "NO_RESPONSE"; }
if ($result eq '' ) {
my $blankcount;
- foreach $award (@$awardref) {
+ foreach my $award (@$awardref) {
if ($award eq '') {
$result='MISSING_ANSWER';
$blankcount++;
@@ -506,25 +534,31 @@ sub finalizeawards {
}
if ($blankcount == ($#$awardref + 1)) { $result = 'NO_RESPONSE'; }
}
- if (defined($result)) { return ($result,$msg); }
+ if (defined($result)) { return ($result); }
# these awards are ordered from most important error through best correct
-
- my @awards = ('EXTRA_ANSWER', 'MISSING_ANSWER', 'ERROR', 'NO_RESPONSE',
- 'TOO_LONG',
- 'UNIT_INVALID_INSTRUCTOR', 'UNIT_INVALID_STUDENT',
- 'UNIT_IRRECONCIBLE', 'UNIT_FAIL', 'NO_UNIT',
- 'UNIT_NOTNEEDED', 'WANTED_NUMERIC', 'BAD_FORMULA',
- 'COMMA_FAIL', 'SIG_FAIL', 'INCORRECT', 'MISORDERED_RANK',
- 'INVALID_FILETYPE', 'DRAFT', 'SUBMITTED', 'ASSIGNED_SCORE',
- 'APPROX_ANS', 'EXACT_ANS');
- if ($reverse) { @awards=reverse(@awards); }
- foreach my $possibleaward (@awards) {
- ($result,$msg)=&checkstatus($possibleaward,$awardref,$msgref);
- if (defined($result)) { return ($result,$msg); }
+ my $awards = (!$reverse) ? \%fwd_awards : \%rev_awards ;
+
+ my $best = $max;
+ my $j=0;
+ my $which;
+ foreach my $award (@$awardref) {
+ if ($awards->{$award} < $best) {
+ $best = $awards->{$award};
+ $which = $j;
+ }
+ $j++;
+ }
+ if (defined($which)) {
+ if (ref($nameref)) {
+ return ($$awardref[$which],$$msgref[$which],$$nameref[$which]);
+ } else {
+ return ($$awardref[$which],$$msgref[$which]);
+ }
}
return ('ERROR',undef);
}
+}
sub decideoutput {
my ($award,$awarded,$awardmsg,$solved,$previous,$target)=@_;
@@ -534,10 +568,11 @@ sub decideoutput {
my $bgcolor='orange';
my $added_computer_text=0;
my %possiblecolors =
- ( 'correct' => '#aaffaa',
- 'charged_try' => '#ffaaaa',
+ ( 'correct' => '#aaffaa',
+ 'charged_try' => '#ffaaaa',
'not_charged_try' => '#ffffaa',
- 'no_message' => '#fffff',
+ 'no_grade' => '#ffffaa',
+ 'no_message' => '#ffffff',
);
my $part = $Apache::inputtags::part;
@@ -696,7 +731,7 @@ sub decideoutput {
$button=1;
} elsif ($award eq 'SUBMITTED') {
$message = &mt("Your submission has been recorded.");
- $bgcolor=$possiblecolors{'correct'};
+ $bgcolor=$possiblecolors{'no_grade'};
$button=1;
} elsif ($award eq 'DRAFT') {
$message = &mt("A draft copy has been saved.");
@@ -718,18 +753,24 @@ sub decideoutput {
$message = &mt("Unknown message").": $award";
$button=1;
}
- my (undef,undef,$domain,$user)=&Apache::lonxml::whichuser();
+ my (undef,undef,$domain,$user)=&Apache::lonnet::whichuser();
foreach my $resid(@Apache::inputtags::response){
- &Apache::lonnet::logthis("handback is ".$Apache::lonhomework::history{"resource.$part.$resid.handback"});
if ($Apache::lonhomework::history{"resource.$part.$resid.handback"}) {
- $message.= ' Returned File';
- }
+ $message.=' ';
+ my @files = split(/\s*,\s*/,
+ $Apache::lonhomework::history{"resource.$part.$resid.handback"});
+ my $file_msg;
+ foreach my $file (@files) {
+ $file_msg.= ' '.$file.'';
+ }
+ $message .= &mt('Returned file(s): [_1]',$file_msg);
+ }
}
if (lc($Apache::lonhomework::problemstatus) eq 'no' &&
$Apache::inputtags::status[-1] ne 'SHOW_ANSWER') {
$message = &mt("Answer Submitted: Your final submission will be graded after the due date.");
- $bgcolor=$possiblecolors{'correct'};
+ $bgcolor=$possiblecolors{'no_grade'};
$button=1;
}
if ($Apache::inputtags::status[-1] eq 'SHOW_ANSWER' &&