--- loncom/homework/grades.pm 2002/07/10 21:08:38 1.38 +++ loncom/homework/grades.pm 2002/07/19 20:42:18 1.40 @@ -1,7 +1,7 @@ # The LearningOnline Network with CAPA # The LON-CAPA Grading handler # -# $Id: grades.pm,v 1.38 2002/07/10 21:08:38 ng Exp $ +# $Id: grades.pm,v 1.40 2002/07/19 20:42:18 ng Exp $ # # Copyright Michigan State University Board of Trustees # @@ -42,7 +42,7 @@ use Apache::loncommon; use Apache::lonhomework; use Apache::lonmsg qw(:user_normal_msg); use Apache::Constants qw(:common); -use Time::HiRes qw( gettimeofday tv_interval ); +#use Time::HiRes qw( gettimeofday tv_interval ); sub moreinfo { my ($request,$reason) = @_; @@ -96,26 +96,23 @@ sub verifyreceipt { } sub student_gradeStatus { - my ($url,$udom,$uname) = @_; - my $symb=($ENV{'form.symb'} ne '' ? $ENV{'form.symb'} : (&Apache::lonnet::symbread($url))); - my %record= &Apache::lonnet::restore($symb,$ENV{'request.course.id'},$udom,$uname); - foreach my $part (&getpartlist($url)) { - my ($temp,$part,$type)=split(/_/,$part); - if ($type eq 'solved') { - my ($status,$foo)=split(/_/,$record{"resource.$part.$type"},2); - $status = 'partial' if ($foo =~ /^partially/); + my ($url,$udom,$uname,$partlist) = @_; + my $symb=($ENV{'form.symb'} ne '' ? $ENV{'form.symb'} : (&Apache::lonnet::symbread($url))); + my %record= &Apache::lonnet::restore($symb,$ENV{'request.course.id'},$udom,$uname); + my %partstatus = (); + foreach (@$partlist) { + my ($status,$foo)=split(/_/,$record{"resource.$_.solved"},2); $status = 'nothing' if ($status eq ''); - return $type,$status; - } - } - return ''; + $partstatus{$_} = $status; + } + return %partstatus; } sub get_fullname { - my ($sname,$sdom) = @_; + my ($uname,$udom) = @_; my %name=&Apache::lonnet::get('environment', ['lastname','generation', 'firstname','middlename'], - $sdom,$sname); + $udom,$uname); my $fullname; my ($tmp) = keys(%name); if ($tmp !~ /^(con_lost|error|no_such_host)/i) { @@ -126,6 +123,25 @@ sub get_fullname { return $fullname; } +sub response_type { + my ($url) = shift; + my $allkeys = &Apache::lonnet::metadata($url,'keys'); + my %seen = (); + my (@partlist,%handgrade); + foreach (split(/,/,&Apache::lonnet::metadata($url,'packages'))) { + if (/^\w+response_\d{1,2}.*/) { + my ($responsetype,$part) = split(/_/,$_,2); + my ($partid,$respid) = split(/_/,$part); + $handgrade{$part} = $responsetype.':'.($allkeys =~ /parameter_$part\_handgrade/ ? 'yes' : 'no'); + next if ($seen{$partid} > 0); + $seen{$partid}++; + push @partlist,$partid; + } + } + return \@partlist,\%handgrade; +} + + sub listStudents { my ($request) = shift; my $cdom =$ENV{"course.$ENV{'request.course.id'}.domain"}; @@ -133,25 +149,33 @@ sub listStudents { my $getsec =$ENV{'form.section'}; my $submitonly=$ENV{'form.submitonly'}; + my $result='

 View Submissions for a Student or a Group of Students

'; + $result.=''; + $result.=''; + my ($partlist,$handgrade) = &response_type($ENV{'form.url'}); + for (sort keys(%$handgrade)) { + my ($responsetype,$handgrade)=split(/:/,$$handgrade{$_}); + $result.=''. + ''. + ''; + } + $result.='
Resource: '.$ENV{'form.url'}.'
Part id: '.$_.'Type: '.$responsetype.'Handgrade: '.$handgrade.'
'; + $request->print($result); + $request->print(< View Submissions for a Student or a Group of Students - Resource: $ENV{'form.url'}

-
 View Options
+  View Problem: no yes
 Submissions: - last sub only - last sub & parts info - all details - - - - -\n"; + my $result="\n"; foreach my $part (@parts) { my ($temp,$part,$type)=split(/_/,$part); my $score=$record{"resource.$part.$type"}; @@ -293,7 +328,7 @@ sub viewstudentgrade { my ($status,$foo)=split(/_/,$score,2); $result.="
- - - - + handgrade only + last sub only + last sub & parts info + all details + + + + + ENDTABLEST if ($ENV{'form.url'}) { $request->print(''."\n"); @@ -161,32 +185,40 @@ ENDTABLEST } $request->print(''."\n"); - my $t0=&Time::HiRes::time(); + my ($classlist,$seclist,$ids,$stusec,$fullname) = &getclasslist($getsec,'0'); - my ($classlist) = &getclasslist($getsec,'0'); - my $t1=&Time::HiRes::time(); - print "getclasslist=",$t1-$t0,"
"; - - foreach my $student ( sort(@{ $$classlist{$getsec} }) ) { - my ($sname,$sdom) = split(/:/,$student); - my ($type,$status) = &student_gradeStatus($ENV{'form.url'},$cdom,$sname); - next if ($status eq 'nothing' && $submitonly eq 'yes'); + $result='
 Select  Username  Fullname  Domain  Grade Status 
'. + ''. + ''. + ''; + foreach (sort(@$partlist)) { + $result.=''; + } + $request->print($result.''."\n"); + + foreach my $student (sort(@{ $$classlist{$getsec} }) ) { + my ($uname,$udom) = split(/:/,$student); + my (%status) = &student_gradeStatus($ENV{'form.url'},$udom,$uname,$partlist); + my $statusflg = ''; + foreach (keys(%status)) { + $statusflg = 1 if ($status{$_} ne 'nothing'); + } + next if ($statusflg eq '' && $submitonly eq 'yes'); - my $fullname = &get_fullname($sname,$sdom); if ( $Apache::grades::viewgrades eq 'F' ) { - $request->print("\n".''. - ''."\n". - ''."\n". - ''."\n". - ''."\n"); - $request->print(''."\n"); + $result=''. + ''."\n". + ''."\n". + ''."\n". + ''."\n"; - $request->print(''); + foreach (sort keys(%status)) { + $result.=''."\n"; + } + $request->print($result.''."\n"); } } - my $t2=&Time::HiRes::time(); - print "processclasslist=",$t2-$t1,"
"; $request->print('
 Select  Username  Fullname  Domain  Part ID '.$_.' Status 
 '.$sname.'  '.$fullname.'  '.$sdom.'  '.$status.' 
 '.$uname.'  '.$$fullname{$student}.'  '.$udom.' 
 '.$status{$_}.' 
'); $request->print(''); } @@ -202,13 +234,13 @@ sub processGroup { return; } foreach (@stuchecked) { - my ($sname,$sdom,$fullname) = split(/:/); - $ENV{'form.student'} = $sname; + my ($uname,$udom,$fullname) = split(/:/); + $ENV{'form.student'} = $uname; $ENV{'form.fullname'} = $fullname; &submission($request,$ctr,$total); $ctr++; } - return 'The End'; + return ''; } sub userError { @@ -237,28 +269,32 @@ sub finduser { } sub getclasslist { - my ($getsec,$hideexpired) = @_; - my ($coursedomain,$coursenum) = split(/_/,$ENV{'request.course.id'}); - my %classlist=&Apache::lonnet::dump('classlist',$coursedomain,$coursenum); - my $now = time; - my (@holdsec,@sections); - foreach my $student (keys(%classlist)) { - my ($end,$start)=split(/:/,$classlist{$student}); - # still a student? - if (($hideexpired) && ($end) && ($end < $now)) { - next; - } - my ($unam,$udom) = split(/:/,$student,2); - my $section = &Apache::lonnet::usection($udom,$unam,$ENV{'request.course.id'}); - $section = ($section ne '-1' ? $section : 'no'); - push @holdsec,$section; - push (@{ $classlist{$getsec} }, $student) if ($getsec eq 'all' || $getsec eq $section); - } - my %seen = (); - foreach my $item (@holdsec) { - push (@sections, $item) unless $seen{$item}++; - } - return (\%classlist,\@sections); + my ($getsec,$hideexpired) = @_; + my %classlist=&Apache::lonnet::dump('classlist', + $ENV{'course.'.$ENV{'request.course.id'}.'.domain'}, + $ENV{'course.'.$ENV{'request.course.id'}.'.num'}); + my $now = time; + my (@holdsec,@sections,%allids,%stusec,%fullname); + foreach (keys(%classlist)) { + my ($end,$start,$id,$section,$fullname)=split(/:/,$classlist{$_}); + # still a student? + if (($hideexpired) && ($end) && ($end < $now)) { + next; + } + $section = ($section ne '' ? $section : 'no'); + push @holdsec,$section; + if ($getsec eq 'all' || $getsec eq $section) { + push (@{ $classlist{$getsec} }, $_); + $allids{$_}=$id; + $stusec{$_}=$section; + $fullname{$_}=$fullname; + } + } + my %seen = (); + foreach my $item (@holdsec) { + push (@sections, $item) unless $seen{$item}++; + } + return (\%classlist,\@sections,\%allids,\%stusec,\%fullname); } sub getpartlist { @@ -275,14 +311,13 @@ sub getpartlist { sub viewstudentgrade { my ($url,$symb,$courseid,$student,@parts) = @_; - my $result =''; my $cellclr = '"#ffffdd"'; my ($username,$domain) = split(/:/,$student); my $fullname = &get_fullname($username,$domain); my %record=&Apache::lonnet::restore($symb,$courseid,$domain,$username); - $result.="
$username$fullname$domain
$username$fullname$domain'."\n". + ''."\n". ''."\n". ''."\n". ''."\n". @@ -451,44 +487,60 @@ KEYWORDS # Student info $request->print(($counter == 0 ? '' : '
')); my $fullname = ($ENV{'form.fullname'} ne '' ? $ENV{'form.fullname'} : &get_fullname($uname,$udom)); - my $result.='
'. + my $result=''; + if ($$timestamp eq '') { + $lastsubonly.=''; + } else { + for my $part (sort keys(%$handgrade)) { + foreach (@$string) { + my ($partid,$respid) = /^resource\.(\d{1,2})\.(\d{1,2})\.submission/; + if ($part eq ($partid.'_'.$respid)) { + my ($ressub,$subval) = split(/:/,$_,2); + $lastsubonly.='' + if ($ENV{'form.lastSub'} eq 'lastonly' || + ($ENV{'form.lastSub'} eq 'hdgrade' && $$handgrade{$part} =~ /:yes$/)); + } + } + } + } + $lastsubonly.='
'. '
'; - $result.=''; + + $result.='
Username: '.$uname. - 'Fullname: '.$fullname. - 'Domain: '.$udom.'
'; if ($ENV{'form.handgrade'} eq 'yes') { # my $subonly = &get_last_submission($symb,$uname,$udom,$ENV{'request.course.id'}); - my ($classlist) = &getclasslist('all','0'); +# my ($classlist) = &getclasslist('all','0'); + my ($classlist,$seclist,$ids,$stusec,$fullname) = &getclasslist('all','0'); my @collaborators; # foreach ( sort(@{ $$classlist{'all'} }) ) { -# my ($sname,$sdom) = split(/:/); -# push @collaborators,$sname if (grep /\b$sname(\b|\.)/i,$subonly); +# my ($uname,$udom) = split(/:/); +# push @collaborators,$uname if (grep /\b$uname(\b|\.)/i,$subonly); # } # push @collaborators,'leede','carlandmm','freyniks'; # as a test to display collaborators. if (scalar(@collaborators) != 0) { $result.=''."\n"; $result.=''."\n"; } } - $result.='
Fullname: '.$fullname. + '   Username: '.$uname. + '   Domain: '.$udom.'
Collaborators: '; foreach (@collaborators) { - $result.=$_.' ('.&get_fullname($_,$udom).')    '; + $result.=$_.' ('.$$fullname{$_.':'.$udom}.')    '; +# $result.=$_.' ('.&get_fullname($_,$udom).')    '; } $result.='
'."\n"; - $request->print($result); + $request->print($result.'
'."\n"); - # print student answer - if ($ENV{'form.lastSub'} eq 'lastonly') { + my ($partlist,$handgrade) = &response_type($url); + + # print student answer + if ($ENV{'form.lastSub'} =~ /^(lastonly|hdgrade)$/) { my ($string,$timestamp)=&get_last_submission ($symb,$uname,$udom,$ENV{'request.course.id'}); - $string=&keywords_highlight(join('::::',@$string)); - my $lastsubonly='
'; - $lastsubonly.=''; - $lastsubonly.=''; - $lastsubonly.='
Last Submission Only'. - ($timestamp eq '' ? '' : '    Date Submitted: '.@$timestamp).'
'; - $lastsubonly.=$string; - $lastsubonly.='
'."\n"; + my $lastsubonly.='
Last Submission Only'. + ($$timestamp eq '' ? '' : '    Date Submitted: '.$$timestamp).'
'.$$string[0].'
Part ID '. + $partid.' Response ID '.$respid. + ' Submission '.&keywords_highlight($subval).'
'."\n"; $request->print($lastsubonly); } else { $request->print(&Apache::loncommon::get_previous_attempt($symb,$uname,$udom, @@ -496,104 +548,109 @@ KEYWORDS '.submission','Apache::grades::keywords_highlight')); } - my $wgt = &Apache::lonnet::EXT('resource.partid.weight',$symb,$udom,$uname); - my $wgtmsg = ($wgt > 0 ? '(problem weight)' : 'problem weight assigned by computer'); - $wgt = ($wgt > 0 ? $wgt : '1'); - my %record = &Apache::lonnet::restore($symb,$ENV{'request.course.id'},$udom,$uname); - my $score = ($record{'resource.0.awarded'} eq '' ? '' : $record{'resource.0.awarded'}*$wgt); - - # display grading options - $result=''. + $result=''."\n". + ''."\n". ''."\n"; - $result.=''."\n"; - $result.=''."\n"; - $result.=''; - $result.=''."\n"; - $result.='
Points'; - - my $ctr = 0; - $result.=''; # display radio buttons in a nice table with 10 across - while ($ctr<=$wgt) { - $result.= '\n"; - $result.=(($ctr+1)%10 == 0 ? '' : ''); - $ctr++; - } - $result.='
'.$ctr."
'; + $result.=' Compose Message
'."\n" if ($ENV{'form.handgrade'} eq 'yes'); + $request->print($result); - $result.='		 or /'.$wgt.' '.$wgtmsg.' '; + my %seen = (); + my @partlist; + my %record = &Apache::lonnet::restore($symb,$ENV{'request.course.id'},$udom,$uname); +# while (my ($k,$v) = each (%record)){ +# print "key=$k ==> value=$v
"; +# } + for (sort keys(%$handgrade)) { + my ($partid,$respid) = split(/_/); + next if ($seen{$partid} > 0); + $seen{$partid}++; + next if ($$handgrade{$_} =~ /:no$/); + push @partlist,$partid; + my $wgt = &Apache::lonnet::EXT('resource.'.$partid.'.weight',$symb,$udom,$uname); + my $wgtmsg = ($wgt > 0 ? '(problem weight)' : 'problem weight assigned by computer'); + $wgt = ($wgt > 0 ? $wgt : '1'); + my $score = ($record{'resource.'.$partid.'.awarded'} eq '' ? '' : $record{'resource.'.$partid.'.awarded'}*$wgt); + + # display grading options + $result=''; + $result.=''; + $result.=''."\n"; + $result.='
Part '.$partid.' Points'; - foreach my $part (&getpartlist($url)) { - print "parts=$part
"; - my ($temp,$part,$type)=split(/_/,$part); - if ($type eq 'solved') { - my ($status,$foo)=split(/_/,$record{"resource.$part.$type"},2); - $status = 'partial' if ($foo =~ /partially/); - $status = 'nothing' if ($status eq ''); - $result.='  \n"; - } + my $ctr = 0; + $result.=''; # display radio buttons in a nice table 10 across + while ($ctr<=$wgt) { + $result.= '\n"; + $result.=(($ctr+1)%10 == 0 ? '' : ''); + $ctr++; + } + $result.='
'.$ctr."
'; + + $result.='		 or /'.$wgt.' '.$wgtmsg.' '; + + $result.=''."  \n"; + $result.=''; + $result.='
'; + $request->print($result); } - $result.='Compose Message
'."\n"; -# $result.='
'; - $result.='
'; - $request->print($result); + $request->print(''."\n"); + $request->print(''."\n"); # print end of form if ($counter == $total) { my $endform.='
'; my $ntstu =''."\n"; + ''. + ''."\n"; my $nsel = ($ENV{'form.NTSTU'} ne '' ? $ENV{'form.NTSTU'} : '1'); - $ntstu =~ s/
'; + $ntstu =~ s/