--- loncom/homework/grades.pm 2002/07/08 21:18:54 1.37 +++ loncom/homework/grades.pm 2002/07/18 21:27:57 1.39 @@ -1,7 +1,7 @@ # The LearningOnline Network with CAPA # The LON-CAPA Grading handler # -# $Id: grades.pm,v 1.37 2002/07/08 21:18:54 ng Exp $ +# $Id: grades.pm,v 1.39 2002/07/18 21:27:57 ng Exp $ # # Copyright Michigan State University Board of Trustees # @@ -40,7 +40,9 @@ use Apache::lonxml; use Apache::lonnet; 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 ); sub moreinfo { my ($request,$reason) = @_; @@ -94,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) { @@ -124,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"}; @@ -131,25 +149,32 @@ 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 StudentsResource: $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"}; @@ -284,14 +327,14 @@ sub viewstudentgrade { my ($status,$foo)=split(/_/,$score,2); $result.="\n"; } } - $result.=''; + $result.=''; return $result; } @@ -324,7 +367,7 @@ sub setstudentgrade { if ($newscore eq 'incorrect') { $newscore = 'incorrect_by_override'; } if ($newscore eq 'excused') { $newscore = 'excused'; } if ($newscore eq 'ungraded') { $newscore = 'ungraded_attempted'; } - if ($newscore eq 'partial') { $newscore = 'correct_partially_by_override'; } +# if ($newscore eq 'partial') { $newscore = 'correct_partially_by_override'; } } else { #$result.="$stuname:$part:$type:unchanged $oldscore to $newscore:
\n"; } @@ -352,287 +395,6 @@ sub setstudentgrade { sub submission { my ($request,$counter,$total) = @_; - if ($counter == 0) { - $request->print(< - function updateRadio(radioButton,formtextbox,formsel,wgt) { - var pts = formtextbox.value; - var resetbox =false; - if (isNaN(pts) || pts < 0) { - alert("A number equal or greater than 0 is expected. Entered value = "+pts); - for (var i=0; i 0 && pts < wgt) { - formsel[4].selected = true; - } - if (pts == wgt) { - formsel[0].selected = true; - } - return; - } - - function keywords(keyform) { - var keywds = keyform.value; - var nret = prompt("Keywords list, separated by a space. Add/delete to list if desired.",keywds); - if (nret==null) return; - keyform.value = nret; - return; - } - -//===================== Script to add keyword(s) ================== - function getSel() { - if (document.getSelection) txt = document.getSelection(); - else if (document.selection) txt = document.selection.createRange().text; - else return; - var cleantxt = txt.replace(new RegExp('([\\f\\n\\r\\t\\v ])+', 'g')," "); - if (cleantxt=="") { - alert("Select a word or group of words from document and then click this link."); - return; - } - var nret = prompt("Add selection to keyword list?",cleantxt); - if (nret==null) return; - var curlist = document.SCORE.keywords.value; - document.SCORE.keywords.value = curlist+" "+nret; - return; - } - -//====================== Script for composing message ============== - function msgCenter(msgform,usrctr,fullname) { - var Nmsg = msgform.savemsgN.value; - savedMsgHeader(Nmsg,usrctr,fullname); - var subject = msgform.msgsub.value; - var rtrchk = eval("document.SCORE.includemsg"+usrctr); - var msgchk = rtrchk.value; -// alert("checked=>"+msgchk); - re = /msgsub/; - var shwsel = ""; - if (re.test(msgchk)) { shwsel = "checked" } - displaySubject(subject,shwsel); - for (var i=1; i<=Nmsg; i++) { - var testpt = "savemsg"+i+","; - re = /testpt/; - shwsel = ""; - if (re.test(msgchk)) { shwsel = "checked" } - var message = eval("document.SCORE.savemsg"+i+".value"); - displaySavedMsg(i,message,shwsel); - } - newmsg = eval("document.SCORE.newmsg"+usrctr+".value"); - shwsel = ""; - re = /newmsg/; - if (re.test(msgchk)) { shwsel = "checked" } - newMsg(newmsg,shwsel); - msgTail(); - return; - } - -function savedMsgHeader(Nmsg,usrctr,fullname) { - var height = 30*Nmsg+250; - var scrollbar = "no"; - if (height > 600) { - height = 600; - scrollbar = "yes"; - } -/* if (window.pWin) - window.pWin.close(); */ - pWin = window.open('', 'MessageCenter', 'toolbar=no,location=no,scrollbars='+scrollbar+',width=600,height='+height); - pWin.document.write(""); - pWin.document.write("Message Central"); - - pWin.document.write(" -SUBJAVASCRIPT -} (my $url=$ENV{'form.url'})=~s-^http://($ENV{'SERVER_NAME'}|$ENV{'HTTP_HOST'})--; if ($ENV{'form.student'} eq '') { &moreinfo($request,'Need student login id'); return ''; } my ($uname,$udom) = &finduser($ENV{'form.student'}); @@ -644,42 +406,46 @@ SUBJAVASCRIPT # header info if ($counter == 0) { + &sub_page_js($request); $request->print('

 Submission Record

'. ' Resource: '.$url.''); - } - # option to display problem, only once else it cause problems with the form later since the problem has a form. - if ($ENV{'form.vProb'} eq 'yes' && $counter == 0) { - my $rendered=&Apache::loncommon::get_student_view($symb,$uname,$udom, - $ENV{'request.course.id'}); - my $companswer=&Apache::loncommon::get_student_answers($symb,$uname,$udom, - $ENV{'request.course.id'}); - my $result.='
- - - - + last sub only + last sub & parts info + all details + + + + + ENDTABLEST if ($ENV{'form.url'}) { $request->print(''."\n"); @@ -159,23 +184,38 @@ ENDTABLEST } $request->print(''."\n"); - my ($classlist) = &getclasslist($getsec,'0'); - 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'); + my ($classlist,$seclist,$ids,$stusec,$fullname) = &getclasslist($getsec,'0'); + + $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"); } } $request->print('
 Select  Username  Fullname  Domain  Part ID '.$_.' Status 
 '.$sname.'  '.$fullname.'  '.$sdom.'  '.$status.' 
 '.$uname.'  '.$$fullname{$student}.'  '.$udom.' 
 '.$status{$_}.' 
'); @@ -193,13 +233,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 { @@ -228,28 +268,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 { @@ -266,14 +310,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
'; - $result.='
'; - $result.='Student\'s view of the problem
'.$rendered.'
'; - $result.='Correct answer:
'.$companswer; - $result.='
'; - $result.='

'; - $request->print($result); - } + # option to display problem, only once else it cause problems with the form later + # since the problem has a form. + if ($ENV{'form.vProb'} eq 'yes') { + my $rendered=&Apache::loncommon::get_student_view($symb,$uname,$udom, + $ENV{'request.course.id'}); + my $companswer=&Apache::loncommon::get_student_answers($symb,$uname,$udom, + $ENV{'request.course.id'}); + my $result.='
'; + $result.='
'; + $result.='Student\'s view of the problem
'.$rendered.'
'; + $result.='Correct answer:
'.$companswer; + $result.='
'; + $result.='

'; + $request->print($result); + } + + # kwclr is the only variable that is guaranteed to be non blank if this subroutine has been called once. + my %keyhash = (); + if ($ENV{'form.kwclr'} eq '') { + %keyhash = &Apache::lonnet::dump('nohist_handgrade', + $ENV{'course.'.$ENV{'request.course.id'}.'.domain'}, + $ENV{'course.'.$ENV{'request.course.id'}.'.num'}); + + my $loginuser = $ENV{'user.name'}.':'.$ENV{'user.domain'}; + $ENV{'form.keywords'} = $keyhash{$symb.'_keywords'} ne '' ? $keyhash{$symb.'_keywords'} : ''; + $ENV{'form.kwclr'} = $keyhash{$loginuser.'_kwclr'} ne '' ? $keyhash{$loginuser.'_kwclr'} : 'red'; + $ENV{'form.kwsize'} = $keyhash{$loginuser.'_kwsize'} ne '' ? $keyhash{$loginuser.'_kwsize'} : '0'; + $ENV{'form.kwstyle'} = $keyhash{$loginuser.'_kwstyle'} ne '' ? $keyhash{$loginuser.'_kwstyle'} : ''; + $ENV{'form.msgsub'} = $keyhash{$symb.'_subject'} ne '' ? + $keyhash{$symb.'_subject'} : &Apache::lonnet::metadata($url,'title'); + $ENV{'form.savemsgN'} = $keyhash{$symb.'_savemsgN'} ne '' ? $keyhash{$symb.'_savemsgN'} : '0'; - # beginning of form - if ($counter == 0) { - my %keyhash = &Apache::lonnet::dump('nohist_handgrade', - $ENV{'course.'.$ENV{'request.course.id'}.'.domain'}, - $ENV{'course.'.$ENV{'request.course.id'}.'.num'}); - - my $loginuser = $ENV{'user.name'}.':'.$ENV{'user.domain'}; - # if the handgrade db has never being initialized then set some default values - if ($keyhash{$symb.'_handgrade'} eq '') { - $keyhash{$symb.'_keywords'} = ''; - $keyhash{$symb.'_subject'} = &Apache::lonnet::metadata($url,'title'); - $keyhash{$loginuser.'_kwclr'} = $keyhash{$loginuser.'_kwclr'} eq '' ? 'red' : $keyhash{$loginuser.'_kwclr'}; - $keyhash{$loginuser.'_kwsize'} = $keyhash{$loginuser.'_kwsize'} eq '' ? '0' : $keyhash{$loginuser.'_kwsize'}; - $keyhash{$loginuser.'_kwstyle'} = $keyhash{$loginuser.'_kwstyle'} eq '' ? '' : $keyhash{$loginuser.'_kwstyle'}; - $keyhash{$symb.'_savemsgN'} = '0'; } $request->print(''."\n". + ''."\n". + ''."\n". ''."\n". ''."\n". ''."\n". @@ -688,51 +454,54 @@ SUBJAVASCRIPT ''."\n". ''."\n". ''."\n". - ''."\n". - ''."\n". - ''."\n". - ''."\n". - ''."\n". - ''."\n". - ''."\n". + ''."\n". + ''."\n". + ''."\n". + ''."\n". + ''."\n". + ''."\n". ''."\n"); my ($cts,$prnmsg) = (1,''); - while ($cts <= $keyhash{$symb.'_savemsgN'}) { - $prnmsg.=''."\n"; + while ($cts <= $ENV{'form.savemsgN'}) { + $prnmsg.=''."\n"; $cts++; } $request->print($prnmsg); if ($ENV{'form.handgrade'} eq 'yes') { $request->print(<Keyword Options  -Keyword List    + Keyword Options:  +List    Paste Selection to Keyword list    -Keyword Highlight Attribute

+ CLASS="page">Paste Selection to List    +Highlight Attribute

KEYWORDS } } # Student info - $request->print(($counter == 0 ? '' : '


')); + $request->print(($counter == 0 ? '' : '
')); my $fullname = ($ENV{'form.fullname'} ne '' ? $ENV{'form.fullname'} : &get_fullname($uname,$udom)); - my $result.=''; + my $result.='
Username: '.$uname. - 'Fullname: '.$fullname. - 'Domain: '.$udom.'
'. + '
'; + $result.=''; if ($ENV{'form.handgrade'} eq 'yes') { # my $subonly = &get_last_submission($symb,$uname,$udom,$ENV{'request.course.id'}); my ($classlist) = &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.='
Fullname: '.$fullname. + '   Username: '.$uname. + '   Domain: '.$udom.'
Collaborators: '; + $result.='
Collaborators: '; foreach (@collaborators) { $result.=$_.' ('.&get_fullname($_,$udom).')    '; } @@ -744,16 +513,30 @@ KEYWORDS $result.='
'."\n"; $request->print($result); - # print student answer + my ($partlist,$handgrade) = &response_type($url); + + # print student answer if ($ENV{'form.lastSub'} eq 'lastonly') { my ($string,$timestamp)=&get_last_submission ($symb,$uname,$udom,$ENV{'request.course.id'}); - $string=&keywords_highlight($string); my $lastsubonly='
'; - $lastsubonly.=''; + $lastsubonly.='
'; $lastsubonly.=''; - $lastsubonly.=''; + 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.=''; + } + } + } + } $lastsubonly.='
Last Submission Only'. - ($timestamp eq '' ? '' : '    Date Submitted: '.$timestamp).'
'; - $lastsubonly.=$string; + ($$timestamp eq '' ? '' : '    Date Submitted: '.$$timestamp).'
'.$$string[0].'
Part ID '. + $partid.' Response ID '.$respid. + ' Submission '.&keywords_highlight($subval).'
'."\n"; $request->print($lastsubonly); } else { @@ -762,58 +545,67 @@ 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.=''."\n"; - } + # display grading options + $result=''; + + $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; + 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 %record = &Apache::lonnet::restore($symb,$ENV{'request.course.id'},$udom,$uname); + my $score = ($record{'resource.0.awarded'} eq '' ? '' : $record{'resource.0.awarded'}*$wgt); - 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/); - $status = 'nothing' if ($status eq ''); - $result.='  \n"; - $result.='Compose Message
'; + $result.=''."\n"; + $result.='
Part '.$partid.' Points'; + + 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.='
'; - $request->print($result); + $request->print(''."\n"); + $request->print('
'."\n"); # print end of form if ($counter == $total) { @@ -831,55 +623,41 @@ KEYWORDS $endform.=''; $request->print($endform); } + return ''; } sub get_last_submission { my ($symb,$username,$domain,$course)=@_; if ($symb) { - my ($string,$timestamp); - my (%returnhash)=&Apache::lonnet::restore($symb,$course,$domain,$username); - if ($returnhash{'version'}) { - my %lasthash=(); - my ($version); - for ($version=1;$version<=$returnhash{'version'};$version++) { - foreach (sort(split(/\:/,$returnhash{$version.':keys'}))) { - $lasthash{$_}=$returnhash{$version.':'.$_}; + my (@string,$timestamp); + my (%returnhash)=&Apache::lonnet::restore($symb,$course,$domain,$username); + if ($returnhash{'version'}) { + my %lasthash=(); + my ($version); + for ($version=1;$version<=$returnhash{'version'};$version++) { + foreach (sort(split(/\:/,$returnhash{$version.':keys'}))) { + $lasthash{$_}=$returnhash{$version.':'.$_}; + } } - } - foreach ((keys %lasthash)) { - if ($_ =~ /\.submission$/) {$string = $lasthash{$_}} - if ($_ =~ /timestamp/) {$timestamp=scalar(localtime($lasthash{$_}))}; - } - } - $string = $string eq '' ? 'Nothing submitted - no attempts.' : $string; - return $string,$timestamp; + foreach ((keys %lasthash)) { + if ($_ =~ /\.submission$/) {push @string, (join(':',$_,$lasthash{$_}))} + if ($_ =~ /timestamp/) {$timestamp = scalar(localtime($lasthash{$_}))}; + } + } + @string = $string[0] eq '' ? 'Nothing submitted - no attempts.' : @string; + return \@string,\$timestamp; } } sub keywords_highlight { my $string = shift; - (my $url=$ENV{'form.url'})=~s-^http://($ENV{'SERVER_NAME'}|$ENV{'HTTP_HOST'})--; - my $symb=($ENV{'form.symb'} ne '' ? $ENV{'form.symb'} : (&Apache::lonnet::symbread($url))); - my $loginuser = $ENV{'user.name'}.':'.$ENV{'user.domain'}; - - my @kwkeys = ($symb.'_keywords', - $loginuser.'_kwclr', - $loginuser.'_kwsize', - $loginuser.'_kwstyle' - ); - - my %keyhash = &Apache::lonnet::get - ('nohist_handgrade',\@kwkeys, - $ENV{'course.'.$ENV{'request.course.id'}.'.domain'}, - $ENV{'course.'.$ENV{'request.course.id'}.'.num'}); - my @keylist = split(/[,\s+]/,$keyhash{$symb.'_keywords'}); - my $size = $keyhash{$kwkeys[2]} eq '0' ? '' : 'size='.$keyhash{$kwkeys[2]}; - my $styleon = $keyhash{$kwkeys[3]} eq '' ? '' : $keyhash{$kwkeys[3]}; + my $size = $ENV{'form.kwsize'} eq '0' ? '' : 'size='.$ENV{'form.kwsize'}; + my $styleon = $ENV{'form.kwstyle'} eq '' ? '' : $ENV{'form.kwstyle'}; (my $styleoff = $styleon) =~ s/\$styleon$word$styleoff\<\/font\>/gi; + my @keylist = split(/[,\s+]/,$ENV{'form.keywords'}); + foreach (@keylist) { + $string =~ s/\b$_(\b|\.)/\$styleon$_$styleoff\<\/font\>/gi; } return $string; } @@ -894,7 +672,8 @@ sub processHandGrade { my $loginuser = $ENV{'user.name'}.':'.$ENV{'user.domain'}; my %keyhash = (); - $keyhash{$symb.'_handgrade'} = 'activated'; + $ENV{'form.keywords'} =~ s/,\s{0,}|\s+/ /g; + $ENV{'form.keywords'} =~ s/^\s+|\s+$//; $keyhash{$symb.'_keywords'} = $ENV{'form.keywords'}; $keyhash{$symb.'_subject'} = $ENV{'form.msgsub'}; $keyhash{$loginuser.'_kwclr'} = $ENV{'form.kwclr'}; @@ -913,28 +692,53 @@ sub processHandGrade { while ($ctr < $ngrade) { if ($ENV{'form.newmsg'.$ctr} ne '') { $keyhash{$symb.'_savemsg'.$idx} = $ENV{'form.newmsg'.$ctr}; + $ENV{'form.savemsg'.$idx} = $ENV{'form.newmsg'.$ctr}; $idx++; } $ctr++; } - - $keyhash{$symb.'_savemsgN'} = --$idx; + $ENV{'form.savemsgN'} = --$idx; + $keyhash{$symb.'_savemsgN'} = $ENV{'form.savemsgN'}; my $putresult = &Apache::lonnet::put ('nohist_handgrade',\%keyhash, $ENV{'course.'.$ENV{'request.course.id'}.'.domain'}, $ENV{'course.'.$ENV{'request.course.id'}.'.num'}); - my (@parts) = sort(&getpartlist($url)); + if ($ENV{'form.refresh'} eq 'on') { + my $ctr = 0; + while ($ctr < $ntstu) { + ($ENV{'form.student'},my $udom) = split(/:/,$ENV{'form.unamedom'.$ctr}); + &submission($request,$ctr,$ntstu-1); + $ctr++; + } + return ''; + } if ($button eq 'Save & Next') { my $ctr = 0; while ($ctr < $ngrade) { my ($uname,$udom) = split(/:/,$ENV{'form.unamedom'.$ctr}); - &saveHandGrade($url,$symb,$uname,$udom,$ctr,@parts); + my ($errorflg) = &saveHandGrade($request,$url,$symb,$uname,$udom,$ctr); + return '' if ($errorflg eq 'error'); + + my $includemsg = $ENV{'form.includemsg'.$ctr}; + my ($subject,$message,$msgstatus) = ('','',''); + if ($includemsg =~ /savemsg|new$ctr/) { + $subject = $ENV{'form.msgsub'} if ($includemsg =~ /^msgsub/); + my (@msgnum) = split(/,/,$includemsg); + foreach (@msgnum) { + $message.=$ENV{'form.'.$_} if ($_ =~ /savemsg|newmsg/ && $_ ne ''); + } + $message =~ s/\s+/ /g; + $msgstatus = &Apache::lonmsg::user_normal_msg ($uname,$udom,$ENV{'form.msgsub'},$message); + } if ($ENV{'form.collaborator'.$ctr}) { my (@collaborators) = split(/:/,$ENV{'form.collaborator'.$ctr}); foreach (@collaborators) { - &saveHandGrade($url,$symb,$_,$udom,$ctr,@parts); + &saveHandGrade($request,$url,$symb,$_,$udom,$ctr); + if ($message ne '') { + $msgstatus = &Apache::lonmsg::user_normal_msg ($uname,$udom,$ENV{'form.msgsub'},$message); + } } } $ctr++; @@ -944,26 +748,31 @@ sub processHandGrade { my $laststu = $ENV{'form.unamedom'.($ngrade-1)}; my ($classlist) = &getclasslist($ENV{'form.section'},'0'); -# print "first=$firststu,last=$laststu
"; my (@nextlist,@prevlist); - my ($nextflg,$prevflg,$ctr,$ctprev) = (0,0,0,0); + my ($nextflg,$ctr,$ctprev) = (0,0,0); + my ($partlist,$handgrade) = &response_type($ENV{'form.url'}); foreach my $student ( sort(@{ $$classlist{$ENV{'form.section'}} }) ) { my ($uname,$udom) = split(/:/,$student); - my ($type,$status) = &student_gradeStatus($ENV{'form.url'},$udom,$uname); - next if ($status eq 'nothing' && $ENV{'form.submitonly'} eq 'yes'); + 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 '' && $ENV{'form.submitonly'} eq 'yes'); if ($nextflg == 1 && $button =~ /Next$/) { push @nextlist,$uname if ($ctr < $ntstu); $ctr++; + last if ($ctr == $ntstu); } $nextflg = 1 if ($student eq $laststu); - $prevflg = 1 if ($student eq $firststu); - if ($prevflg == 0 && $button eq 'Previous') { + if ($button eq 'Previous') { + last if ($student eq $firststu); push @prevlist,$uname; $ctprev++; } } -# print "next student=@nextlist
"; + if ($button eq 'Previous') { if ($ctprev <= $ntstu) { @nextlist = @prevlist; @@ -976,6 +785,7 @@ sub processHandGrade { } } } + $ctr = 0; my $total = scalar(@nextlist)-1; foreach my $student (@nextlist) { @@ -983,55 +793,46 @@ sub processHandGrade { &submission($request,$ctr,$total); $ctr++; } - - return 'The End'; + if ($total < 0) { + my $the_end = '

LON-CAPA User Message


'."\n"; + $the_end.='Message: No more students for this section or class.

'."\n"; + $the_end.='Click on the button below to return to the grading menu.

'."\n"; + $the_end.=&show_grading_menu_form ($symb,$url); + $request->print($the_end); + } + return ''; } sub saveHandGrade { - my ($url,$symb,$stuname,$domain,$newflg,@parts) = @_; - my %record=&Apache::lonnet::restore($symb,$ENV{'request.course.id'},$domain,$stuname); + my ($request,$url,$symb,$stuname,$domain,$newflg) = @_; +# my %record=&Apache::lonnet::restore($symb,$ENV{'request.course.id'},$domain,$stuname); my %newrecord; - - foreach my $part (@parts) { - my ($temp,$part,$type)=split(/_/,$part); - my $oldscore=$record{"resource.$part.$type"}; - my $newscore; - if ($type eq 'awarded' && $newflg >= 0) { - my $pts = ($ENV{'form.GRADE_BOX'.$newflg} ne '' ? - $ENV{'form.GRADE_BOX'.$newflg} : $ENV{'form.RADVAL'.$newflg}); - my $wgt = $ENV{'form.WGT'.$newflg}; -# my $sel = $ENV{'form.GRADE_SEL'.$newflg}; - $newscore = $pts/$wgt if ($wgt != 0); - } - if ($type eq 'solved') { - $newscore = $ENV{'form.GRADE_SEL'.$newflg} if ($newflg >= 0); - my $update=0; - if ($newscore eq 'nothing' ) { - if ($oldscore ne '') { - $update=1; - $newscore = ''; - } - } elsif ($oldscore !~ m/^$newscore/) { - $update=1; - 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'; } - if ($newscore eq 'partial') { $newscore = 'correct_partially_by_override'; } - } - if ($update) { $newrecord{"resource.$part.$type"}=$newscore; } - } else { - if ($oldscore ne $newscore) { - $newrecord{"resource.$part.$type"}=$newscore; + foreach (split(/:/,$ENV{'form.partlist'.$newflg})) { + if ($ENV{'form.GRADE_SEL'.$newflg.'_'.$_} eq 'excused') { + $newrecord{'resource.'.$_.'.solved'} = 'excused'; + } else { + my $pts = ($ENV{'form.GRADE_BOX'.$newflg.'_'.$_} ne '' ? + $ENV{'form.GRADE_BOX'.$newflg.'_'.$_} : $ENV{'form.RADVAL'.$newflg.'_'.$_}); + if ($pts eq '') { + &userError($request,'No point was assigned for part id '.$_.' and for username '.$stuname.'.'); + return 'error'; + } + my $wgt = $ENV{'form.WGT'.$newflg.'_'.$_} eq '' ? 1 : $ENV{'form.WGT'.$newflg.'_'.$_}; + my $partial= $pts/$wgt; + $newrecord{'resource.'.$_.'.awarded'} = $partial; + if ($partial == 0) { + $newrecord{'resource.'.$_.'.solved'} = 'incorrect_by_override'; + } else { + $newrecord{'resource.'.$_.'.solved'} = 'correct_by_override'; + } } - } } + if ( scalar(keys(%newrecord)) > 0 ) { $newrecord{'resource.regrader'}="$ENV{'user.name'}:$ENV{'user.domain'}"; -# while (my ($k,$v) = each %newrecord) { -# print "k=$k:v=$v:
\n"; -# } -# print "symb=$symb,courseid=$ENV{'request.course.id'},dom=$domain,name=$stuname
"; + while (my ($k,$v) = each %newrecord) { + print "k=$k:v=$v:
\n"; + } # &Apache::lonnet::cstore(\%newrecord,$symb,$ENV{'request.course.id'},$domain,$stuname); } return ''; @@ -1060,25 +861,70 @@ sub gradingmenu { my ($request) = @_; my ($symb,$url)=&get_symb_and_url($request); if (!$symb) {return '';} - my $allkeys = &Apache::lonnet::metadata($url,'keys'); - my $handgrade = ($allkeys =~ /parameter_.*?_handgrade/ ? 'yes' : 'no'); - my ($responsetype,$foo) = split(/_/,&Apache::lonnet::metadata($url,'packages')); - my $result='

 Select a Grading Method

'; $result.=''; - $result.=''. - ''; - $result.=''. - ''; + $result.=''; + my ($partlist,$handgrade) = &response_type($url); + my ($resptype,$hdgrade)=('','no'); + for (sort keys(%$handgrade)) { + my ($responsetype,$handgrade)=split(/:/,$$handgrade{$_}); + $resptype = $responsetype; + $hdgrade = $handgrade if ($handgrade eq 'yes'); + $result.=''. + ''. + ''; + } $result.='
Resource: '.$url.'
Type: '.$responsetype.'  Handgrade: '.$handgrade.'
Resource: '.$url.'
Part id: '.$_.'Type: '.$responsetype.'Handgrade: '.$handgrade.'
'; - $result.=&view_edit_entire_class_form($symb,$url).'
'; $result.=&upcsvScores_form($symb,$url).'
'; - $result.=&viewGradeaStu_form($symb,$url,$responsetype,$handgrade).'
'; - $result.=&verifyReceipt_form($symb,$url); + $result.=&viewGradeaStu_form($symb,$url,$resptype,$hdgrade).'
'; + $result.=&verifyReceipt_form($symb,$url).'
'; + $result.=&view_classlist_form($symb,$url); + + return $result; +} + +sub view_classlist_form { + my ($symb,$url)=@_; + my $result.='
'."\n"; + $result.=''."\n"; + $result.='
'."\n"; + $result.=' View Class List
'."\n"; + $result.='
'."\n". + ''."\n". + ''."\n". + ''."\n"; + $result.=' 
'."\n"; + $result.='
'."\n"; + $result.='
'."\n"; return $result; } +sub viewclasslist { + my ($request) = shift; + my ($coursedomain,$coursenum) = split(/_/,$ENV{'request.course.id'}); + my %classlist=&Apache::lonnet::dump('classlist',$coursedomain,$coursenum); + $request->print(''); + foreach (sort keys(%classlist)) { +# my ($unam,$udom) = split(/:/,$_,2); +# my $section = &Apache::lonnet::usection($udom,$unam,$ENV{'request.course.id'}); +# my $fullname = &get_fullname ($unam,$udom); +# my @uname; +# $uname[0]=$unam; +# my %userid=&Apache::lonnet::idrget($udom,@uname); +# my $value=$classlist{$_}.':'.$userid{$unam}.':'.$section.':'.$fullname; +# $classlist{$_}=$value; + $request->print(''); + } + $request->print('
'.$_.'
 '.$classlist{$_}.'
'); +# my $putresult = &Apache::lonnet::put +# ('classlist',\%classlist, +# $ENV{'course.'.$ENV{'request.course.id'}.'.domain'}, +# $ENV{'course.'.$ENV{'request.course.id'}.'.num'}); + + return ''; +} + sub view_edit_entire_class_form { my ($symb,$url)=@_; my $result.='
'."\n"; @@ -1135,8 +981,8 @@ sub viewGradeaStu_form { ''."\n"; $result.=' Select section: '."\n"; $result.='  Display students who has: '. @@ -1189,8 +1035,8 @@ sub viewgrades { my ($cdom,$cnum) = split(/_/,$ENV{'request.course.id'}); #print "Found $cdom:$cnum
"; my ($classlist) = &getclasslist('all','0'); - my $headerclr = '"#ccffff"'; - my $cellclr = '"#ffffcc"'; + my $headerclr = '"#ddffff"'; + my $cellclr = '"#ffffdd"'; #get list of parts for this problem my (@parts) = sort(&getpartlist($url)); @@ -1205,7 +1051,7 @@ sub viewgrades { ''."\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'); } @@ -1214,7 +1060,7 @@ sub viewgrades { $result.=''; #get info for each student foreach my $student ( sort(@{ $$classlist{'all'} }) ) { - my $display=&viewstudentgrade($url,$symb,$ENV{'request.course.id'},$student,@parts); +# my $display=&viewstudentgrade($url,$symb,$ENV{'request.course.id'},$student,@parts); # print "ID=$ENV{'request.course.id'}:STU=$student:DIS=$display:
\n"; $result.=&viewstudentgrade($url,$symb,$ENV{'request.course.id'},$student,@parts); } @@ -1253,6 +1099,303 @@ sub editgrades { return $result; } +sub sub_page_js { + my $request = shift; + $request->print(< + function updateRadio(radioButton,formtextbox,formsel,scores) { + var pts = formtextbox.value; + var resetbox =false; + if (isNaN(pts) || pts < 0) { + alert("A number equal or greater than 0 is expected. Entered value = "+pts); + for (var i=0; i"+msgchk); + re = /msgsub/; + var shwsel = ""; + if (re.test(msgchk)) { shwsel = "checked" } + displaySubject(subject,shwsel); + for (var i=1; i<=Nmsg; i++) { + var testpt = "savemsg"+i+","; + re = /testpt/; + shwsel = ""; + if (re.test(msgchk)) { shwsel = "checked" } + var message = eval("document.SCORE.savemsg"+i+".value"); + displaySavedMsg(i,message,shwsel); + } + newmsg = eval("document.SCORE.newmsg"+usrctr+".value"); + shwsel = ""; + re = /newmsg/; + if (re.test(msgchk)) { shwsel = "checked" } + newMsg(newmsg,shwsel); + msgTail(); + return; + } + + function savedMsgHeader(Nmsg,usrctr,fullname) { + var height = 30*Nmsg+250; + var scrollbar = "no"; + if (height > 600) { + height = 600; + scrollbar = "yes"; + } +/* if (window.pWin) + window.pWin.close(); */ + pWin = window.open('', 'MessageCenter', 'toolbar=no,location=no,scrollbars='+scrollbar+',screenx=70,screeny=75,width=600,height='+height); + pWin.document.write(""); + pWin.document.write("Message Central"); + + pWin.document.write(" +SUBJAVASCRIPT +} + sub csvupload_javascript_reverse_associate { return(<print(&verifyreceipt($request)); } elsif ($command eq 'csvupload') { $request->print(&csvupload($request)); + } elsif ($command eq 'viewclasslist') { + $request->print(&viewclasslist($request)); } elsif ($command eq 'csvuploadmap') { $request->print(&csvuploadmap($request)); # } elsif ($command eq 'receiptInput') {
UsernameNameDomain
UsernameFullnameDomain