--- loncom/homework/grades.pm 2001/02/13 18:41:00 1.3 +++ loncom/homework/grades.pm 2001/05/01 17:30:18 1.7 @@ -1,5 +1,5 @@ # The LON-CAPA Grading handler -# 2/9 Guy Albertelli +# 2/9,2/13 Guy Albertelli package Apache::grades; use strict; @@ -13,20 +13,160 @@ use Apache::Constants qw(:common); sub moreinfo { my ($request,$reason) = @_; $request->print("Unable to process request: $reason"); - $request->print('
'."\n"); - $request->print(''."\n"); - $request->print(''."\n"); - $request->print("Student:".''."
\n"); - $request->print("Domain:".''."
\n"); - $request->print(''."
\n"); - $request->print('
'); + if ( $Apache::grades::viewgrades eq 'F' ) { + $request->print('
'."\n"); + $request->print(''."\n"); + $request->print(''."\n"); + $request->print("Student:".''."
\n"); + $request->print("Domain:".''."
\n"); + $request->print(''."
\n"); + $request->print('
'); + } + return ''; } -#FIXME - needs to be much smarter +#FIXME - needs to handle multiple matches sub finduser { my ($name) = @_; - return ($name,$ENV{'user.domain'}); + my $domain = ''; + + if ( $Apache::grades::viewgrades eq 'F' ) { + #get classlist + my ($cdom,$cnum) = split(/_/,$ENV{'request.course.id'}); + my $chome=$ENV{"course.$ENV{'request.course.id'}.home"}; + #print "Found $cdom:$cnum:$chome
"; + my (%classlist) = &getclasslist($cdom,$cnum,$chome,'0'); + foreach my $student ( sort(@{ $classlist{'allids'} }) ) { + my ($posname,$posdomain) = split(/:/,$student); + if ($posname =~ $name) { $name=$posname; $domain=$posdomain; last; } + } + return ($name,$domain); + } else { + return ($ENV{'user.name'},$ENV{'user.domain'}); + } +} + +sub getclasslist { + my ($coursedomain,$coursenum,$coursehome,$hideexpired) = @_; + my $classlist=&Apache::lonnet::reply("dump:$coursedomain:$coursenum:classlist",$coursehome); + my %classlist=(); + my $now = time; + foreach my $record (split /&/, $classlist) { + my ($name,$value)=split(/=/,&Apache::lonnet::unescape($record)); + my ($end,$start)=split(/:/,$value); + # still a student? + if (($hideexpired) && ($end) && ($end < $now)) { + print "Skipping:$name:$end:$now
\n"; + next; + } + push( @{ $classlist{'allids'} }, $name); + } + return (%classlist); +} + +sub getpartlist { + my ($url) = @_; + my @parts =(); + my (@metakeys) = split(/,/,&Apache::lonnet::metadata($url,'keys')); + foreach my $key (@metakeys) { + if ( $key =~ m/stores_([0-9]+)_.*/ ) { + push(@parts,$key); + } + } + return @parts; +} + +sub viewstudentgrade { + my ($url,$symb,$courseid,$student,@parts) = @_; + my $result =''; + + my ($stuname,$domain) = split(/:/,$student); + + my %record=&Apache::lonnet::restore($symb,$courseid,$domain,$stuname, + &Apache::lonnet::homeserver($stuname,$domain)); + + $result.="$stuname$domain\n"; + foreach my $part (@parts) { + my ($temp,$part,$type)=split(/_/,$part); + #print "resource.$part.$type = ".$record{"resource.$part.$type"}."
\n"; + if ($type eq 'awarded') { + my $score=$record{"resource.$part.$type"}; + $result.="\n"; + } elsif ($type eq 'tries') { + my $score=$record{"resource.$part.$type"}; + $result.="\n" + } elsif ($type eq 'solved') { + my $score=$record{"resource.$part.$type"}; + $result.="\n"; + } + } + $result.=''; + return $result; +} +#FIXME need to look at the meatdata spec on what type of data to accept and provide an +#interface based on that, also do that to above function. +sub setstudentgrade { + my ($url,$symb,$courseid,$student,@parts) = @_; + + my $result =''; + + my ($stuname,$domain) = split(/:/,$student); + + my %record=&Apache::lonnet::restore($symb,$courseid,$domain,$stuname, + &Apache::lonnet::homeserver($stuname,$domain)); + my %newrecord; + + foreach my $part (@parts) { + my ($temp,$part,$type)=split(/_/,$part); + my $oldscore=$record{"resource.$part.$type"}; + my $newscore=$ENV{"form.GRADE.$student.$part.$type"}; + if ($type eq 'solved') { + my $update=0; + if ($newscore eq 'nothing' ) { + if ($oldscore ne '') { + $update=1; + $newscore = ''; + } + } elsif ($oldscore !~ m/^$newscore/) { + $update=1; + $result.="Updating $stuname to $newscore
\n"; + if ($newscore eq 'correct') { $newscore = 'correct_by_override'; } + if ($newscore eq 'incorrect') { $newscore = 'incorrect_by_override'; } + if ($newscore eq 'excused') { $newscore = 'excused'; } + if ($newscore eq 'ungraded') { $newscore = 'ungraded_attempted'; } + } else { + #$result.="$stuname:$part:$type:unchanged $oldscore to $newscore:
\n"; + } + if ($update) { $newrecord{"resource.$part.$type"}=$newscore; } + } else { + if ($oldscore ne $newscore) { + $newrecord{"resource.$part.$type"}=$newscore; + $result.="Updating $student"."'s status for $part.$type to $newscore
\n"; + } else { + #$result.="$stuname:$part:$type:unchanged $oldscore to $newscore:
\n"; + } + } + } + if ( scalar(keys(%newrecord)) > 0 ) { + $newrecord{"resource.regrader"}="$ENV{'user.name'}:$ENV{'user.domain'}"; + &Apache::lonnet::cstore(\%newrecord,$symb,$courseid,$domain,$stuname, + &Apache::lonnet::homeserver($stuname,$domain)); + $result.="Stored away ".scalar(keys(%newrecord))." elements.
\n"; + } + return $result; } sub submission { @@ -45,9 +185,88 @@ sub submission { return $result; } +sub viewgrades { + my ($request) = @_; + my $result=''; + + #get resource reference + my $url=$ENV{'form.url'}; + $url=~s-^http://($ENV{'SERVER_NAME'}|$ENV{'HTTP_HOST'})--; + my $symb=$ENV{'form.symb'}; + if (!$symb) { $symb=&Apache::lonnet::symbread($url); } + if ($symb eq '') { $request->print("Unable to handle ambiguous references:$url:."); return ''; } + + #get classlist + my ($cdom,$cnum) = split(/_/,$ENV{'request.course.id'}); + my $chome=$ENV{"course.$ENV{'request.course.id'}.home"}; + #print "Found $cdom:$cnum:$chome
"; + my (%classlist) = &getclasslist($cdom,$cnum,$chome,'0'); + + + #get list of parts for this problem + my (@parts) = &getpartlist($url); + + #start the form + $result = '
'."\n". + ''."\n". + ''."\n". + ''."\n". + ''."\n". + ''."\n". + ''."\n"; + foreach my $part (@parts) { + my $display=&Apache::lonnet::metadata($url,$part.'.display'); + if (!$display) { $display = &Apache::lonnet::metadata($url,$part.'.name'); } + $result.="\n"; + } + $result.=""; + #get info for each student + foreach my $student ( sort(@{ $classlist{'allids'} }) ) { + $result.=&viewstudentgrade($url,$symb,$ENV{'request.course.id'},$student,@parts); + } + $result.='
UserIdDomain$display
'; + + return $result; +} + +sub editgrades { + my ($request) = @_; + my $result=''; + + my $symb=$ENV{'form.symb'}; + if ($symb eq '') { $request->print("Unable to handle ambiguous references:$symb:$ENV{'form.url'}"); return ''; } + my $url=$ENV{'form.url'}; + #get classlist + my ($cdom,$cnum) = split(/_/,$ENV{'request.course.id'}); + my $chome=$ENV{"course.$ENV{'request.course.id'}.home"}; + #print "Found $cdom:$cnum:$chome
"; + my (%classlist) = &getclasslist($cdom,$cnum,$chome,'0'); + + #get list of parts for this problem + my (@parts) = &getpartlist($url); + + $result.='
'."\n". + ''."\n". + ''."\n". + ''."\n". + '
'."\n"; + + foreach my $student ( sort(@{ $classlist{'allids'} }) ) { + $result.=&setstudentgrade($url,$symb,$ENV{'request.course.id'},$student,@parts); + } + + $result.='
'; + return $result; +} + sub send_header { my ($request)= @_; $request->print(&Apache::lontexconvert::header()); +# $request->print(" +#"); $request->print(''); } @@ -70,15 +289,20 @@ sub handler { $request->send_http_header; return OK if $request->header_only; my $url=$ENV{'form.url'}; + my $symb=$ENV{'form.symb'}; my $command=$ENV{'form.command'}; &send_header($request); - if ($url eq '') { + if ($url eq '' && $symb eq '') { $request->print("Non-Contextual Access Unsupported:$command:$url:"); } else { - $Apache::grades::viewgrades=&Apache::lonnet::allowed('vgr',$url); + $Apache::grades::viewgrades=&Apache::lonnet::allowed('vgr',$ENV{'request.course.id'}); if ($command eq 'submission') { $request->print(&submission($request)); + } elsif ($command eq 'viewgrades') { + $request->print(&viewgrades($request)); + } elsif ($command eq 'editgrades') { + $request->print(&editgrades($request)); } else { $request->print("Unknown action:$command:"); }