Diff for /loncom/interface/lonwhatsnew.pm between versions 1.9 and 1.13

version 1.9, 2005/04/11 15:52:40 version 1.13, 2005/05/23 23:06:23
Line 52  sub handler { Line 52  sub handler {
     }      }
     &Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'},['command']);      &Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'},['command']);
   
     my $command = $env{'form.command'};      my $command;
       if ($env{'form.action'} eq 'reset') {
     if ($command eq '') {          $command = 'reset';
         $command = "info";      } elsif ($env{'form.action'} eq 'update') {
           $command = 'update';
       } else {
           $command = $env{'form.command'};
     }      }
   
     &Apache::loncommon::content_type($r,'text/html');      &Apache::loncommon::content_type($r,'text/html');
Line 68  sub handler { Line 71  sub handler {
     }      }
   
     &Apache::lonhtmlcommon::clear_breadcrumbs();      &Apache::lonhtmlcommon::clear_breadcrumbs();
     if ($command eq 'config') {      if ($command eq 'chgthreshold') {
         &Apache::lonhtmlcommon::add_breadcrumb          &Apache::lonhtmlcommon::add_breadcrumb
             ({href=>'/adm/whatsnew?command=config',              ({href=>'/adm/whatsnew?command=threshold',
               text=>"Configure display"});                text=>"Change thresholds"});
         $r->print(&Apache::lonhtmlcommon::breadcrumbs          $r->print(&Apache::lonhtmlcommon::breadcrumbs
             (undef,'Course Action Items','Course_Action_Items_Config'));              (undef,'Course Action Items','Course_Action_Items_Thresholds'));
     } else {      } else {
         &Apache::lonhtmlcommon::add_breadcrumb          &Apache::lonhtmlcommon::add_breadcrumb
             ({href=>'/adm/whatsnew?command=info',              ({href=>'/adm/whatsnew',
               text=>"Display Action Items"});                text=>"Display Action Items"});
         $r->print(&Apache::lonhtmlcommon::breadcrumbs          $r->print(&Apache::lonhtmlcommon::breadcrumbs
             (undef,'Course Action Items','Course_Action_Items_Display'));              (undef,'Course Action Items','Course_Action_Items_Display'));
Line 94  sub display_main_box { Line 97  sub display_main_box {
     my ($r,$command) = @_;      my ($r,$command) = @_;
     my $domain=&Apache::loncommon::determinedomain();      my $domain=&Apache::loncommon::determinedomain();
     my $tabbg=&Apache::loncommon::designparm('coordinator.tabbg',$domain);      my $tabbg=&Apache::loncommon::designparm('coordinator.tabbg',$domain);
     my $selconfig;  
     my $selinfo;  
     if ($command eq 'config') {  
         $selinfo = 'selected="selected"';  
     } else {  
         $selconfig = 'selected="selected"';  
     }  
     my $picker = ('  
 <form>  
  <nobr>  
   <input type="submit" value="'.&mt('Change page to:').'" />  
    <select name="command">  
     <option value="info" '.$selinfo.'">'.&mt('Display Action Items').'</option>  
     <option value="" '.$selconfig.'">'.&mt('Configure Settings').'</option>  
    </select>  
  </nobr>  
 </form>');  
                                                                                  
     $r->print('<table width="100%" border="0" cellpadding="5" cellspacing="0"><tr><td width="100%">');      $r->print('<table width="100%" border="0" cellpadding="5" cellspacing="0"><tr><td width="100%">');
                                                                                  
     if ($command eq 'config') {      my %threshold_titles = (
         &display_config_box($r,$picker);                           av_attempts => 'Average number of attempts',
                            degdiff => 'Degree of difficulty',
                            numstudents => 'Total number of students with submissions',
       );
       if ($command eq 'chgthreshold') {
           &display_config_box($r,$command,$tabbg,\%threshold_titles);
     } else {      } else {
         &display_actions_box($r,$picker);          &display_actions_box($r,$command,\%threshold_titles);
     }      }
     $r->print(<<END_OF_BLOCK);      $r->print(<<END_OF_BLOCK);
   </td>    </td>
Line 155  ENDHEAD Line 145  ENDHEAD
 #-------------------------------  #-------------------------------
                                                                                                                                                                   
 sub display_actions_box() {  sub display_actions_box() {
     my ($r,$picker) = @_;      my ($r,$command,$threshold_titles) = @_;
   
     my $rowColor1 = "#ffffff";      my $rowColor1 = "#ffffff";
     my $rowColor2 = "#eeeeee";      my $rowColor2 = "#eeeeee";
Line 164  sub display_actions_box() { Line 154  sub display_actions_box() {
     my %unread = ();      my %unread = ();
     my %ungraded = ();      my %ungraded = ();
     my %bombed = ();      my %bombed = ();
       my %triggered = ();
     my @newmsgs = ();      my @newmsgs = ();
     my @critmsgs = ();      my @critmsgs = ();
     my @newdiscussions = ();      my @newdiscussions = ();
     my @tograde = ();      my @tograde = ();
     my @bombs = ();      my @bombs = ();
       my @warnings = ();
   
     my $domain=&Apache::loncommon::determinedomain();      my $domain=&Apache::loncommon::determinedomain();
     my $function;      my $function;
Line 179  sub display_actions_box() { Line 171  sub display_actions_box() {
         $function='admin';          $function='admin';
     }      }
   
       my %threshold = (
                         av_attempts => 0,
                         degdiff => 0.01,
                         numstudents => 0,
                        );
   
     my $pgbg=&Apache::loncommon::designparm($function.'.pgbg',$domain);      my $pgbg=&Apache::loncommon::designparm($function.'.pgbg',$domain);
     my $tabbg=&Apache::loncommon::designparm($function.'.tabbg',$domain);      my $tabbg=&Apache::loncommon::designparm($function.'.tabbg',$domain);
   
     &getitems(\%unread,\%ungraded,\%bombed,\@newdiscussions,\@tograde,\@bombs);  
     my ($msgcount,$critmsgcount) = &getmail(\@newmsgs,\@critmsgs);  
   
     unless ($env{'request.course.id'}) {      unless ($env{'request.course.id'}) {
         $r->print('<br /><b><center>You are accessing an invalid course</center></b><br /><br />');          $r->print('<br /><b><center>You are accessing an invalid course</center></b><br /><br />');
         return;          return;
     }      }
   
     $r->print('<b>'.$picker.'</b><br /><hr width="100%" /><table border="0" width="100%" cellpadding="2" cellspacing="4"><tr><td align="left" valign="top" width="45%">');      my $result;
       my $cdom = $env{'course.'.$env{'request.course.id'}.'.domain'};
 ## UNREAD COURSE DISCUSSION POSTS ##      my $crs = $env{'course.'.$env{'request.course.id'}.'.num'};
     $r->print(<<"END");  
               <table border="0" cellpadding="0" cellspacing="0" bgcolor="#000000" width="100%">  
                <tr><td>  
                 <table border="0" cellpadding="1" cellspacing="1" bgcolor="#000000" width="100%">  
                  <tr>  
                   <td bgcolor="$tabbg"><b>Unread course discussion posts:</b></td>  
                  </tr>  
                  <tr>  
                    <td bgcolor="#ffffff">  
                    <table cellpadding="2" cellspacing="0" border="0" width="100%">  
 END  
   
     if (@newdiscussions > 0) {      if ($command eq 'reset') {
         $r->print('<tr bgcolor="#cccccc"><td><b><small>Location</small></b></td><td><b><small>Type</small></b><td align="right"><b><small>Number of new posts</small></b></td></tr>');          $result = &process_reset($cdom,$crs);
 #        @newdiscussions = sort { &cmp_title($a,$b) } @newdiscussions;      } elsif ($command eq 'update') {
         my $rowNum = 0;          $result = &process_update($cdom,$crs,$threshold_titles);
         foreach my $ressymb (@newdiscussions) {      }
             my $forum_title = $unread{$ressymb}{'title'};      if ($result) {
             my $type = 'Resource';          $r->print($result.'<hr width="100%" />');
     my $feedurl=&Apache::lonfeedback::get_feedurl($ressymb);  
             if ($feedurl =~ /bulletinboard/) {  
                 $type = 'Bulletin Board';  
             }  
             my $unreadnum = keys(%{$unread{$ressymb}});  
             $unreadnum = $unreadnum - 2;  
             if ($unreadnum > 0) {  
                 if ($rowNum %2 == 1) {  
                     $rowColor = $rowColor1;  
                 } else {  
                     $rowColor = $rowColor2;  
                 }  
                 $r->print('<tr bgcolor="'.$rowColor.'"><td><small><a href="'.$feedurl.'?symb='.$unread{$ressymb}{symb}.'">'.$forum_title.':</a>&nbsp;</td><td><small>'.$type.'</small></td><td align="right">'.$unreadnum.'&nbsp;</td></tr>');  
                 $rowNum ++;  
             }  
         }  
     } else {  
         $r->print('<tr><td bgcolor="#ffffff"><br><center>&nbsp;<i><b><small>No unread posts in course discussions</small></b></i><br><br></td></tr>');  
     }      }
     $r->print('</table></td></tr></table></td></tr></table><br />');  
       &get_curr_thresholds(\%threshold,$cdom,$crs);
       &getitems(\%unread,\%ungraded,\%bombed,\%triggered,\@newdiscussions,\@tograde,\@bombs,\@warnings,$rowColor1,$rowColor2,\%threshold,$cdom,$crs);
       my ($msgcount,$critmsgcount) = &getmail(\@newmsgs,\@critmsgs);
   
       $r->print('<br /><table border="0" width="100%" cellpadding="2" cellspacing="4"><tr><td align="left" valign="top" width="45%">');
   
 ## UNGRADED ITEMS ##  ## UNGRADED ITEMS ##
     $r->print(<<END);      $r->print(<<END);
Line 239  END Line 210  END
             <tr><td>              <tr><td>
              <table border="0" cellpadding="1" cellspacing="1" bgcolor="#000000" width="100%">               <table border="0" cellpadding="1" cellspacing="1" bgcolor="#000000" width="100%">
               <tr>                <tr>
                <td bgcolor="$tabbg"><b>Problems requiring handgrading:</b></td></tr>                 <td bgcolor="$tabbg"><b>Problems requiring handgrading</b></td></tr>
                   <tr>                    <tr>
                    <td bgcolor="#ffffff">                     <td bgcolor="#ffffff">
                      <table cellpadding="2" cellspacing="0" border="0" width="100%">                       <table cellpadding="2" cellspacing="0" border="0" width="100%">
Line 281  END Line 252  END
 END  END
      my $bombnum = 0;       my $bombnum = 0;
      if (@bombs > 0) {       if (@bombs > 0) {
           $r->print('<tr bgcolor="#cccccc"><td><b><small>Resource</small></b></td><td align="right"><b><small>Number of errors</small></b></td></tr>');
 #        @bombs = sort { &cmp_title($a,$b) } @bombs;  #        @bombs = sort { &cmp_title($a,$b) } @bombs;
         foreach my $bomb (@bombs) {          foreach my $bomb (@bombs) {
             if ($bombnum %2 == 1) {              if ($bombnum %2 == 1) {
Line 288  END Line 260  END
             } else {              } else {
                 $rowColor = $rowColor2;                  $rowColor = $rowColor2;
             }              }
             $r->print('<tr bgcolor="'.$rowColor.'"><td>'.$bombed{$bomb}{errorlink}.'</td></tr>');              $r->print('<tr bgcolor="'.$rowColor.'"><td><small>'.$bombed{$bomb}{errorlink}.'</small></td><td align="right"><small>'.$bombed{$bomb}{errorcount}.'</small></td></tr>');
             $bombnum ++;              $bombnum ++;
         }          }
     } else {      } else {
Line 296  END Line 268  END
     }      }
     $r->print('</table></td></tr></table></td></tr></table><br />');      $r->print('</table></td></tr></table></td></tr></table><br />');
   
   # DEGDIFF AND AV. TRIES TRIGGERS
       $r->print(<<"END");
              <table border="0" cellpadding="0" cellspacing="0" bgcolor="#000000" width="100%">
               <tr>
                <td>
                  <table border="0" cellpadding="1" cellspacing="1" bgcolor="#000000" width="100%">
                  <tr>
                   <td bgcolor="$tabbg"><b>Problems with av. attempts &ge; $threshold{'av_attempts'} or deg. difficulty &ge; $threshold{'degdiff'}<br /> and total number of students with submissions &ge; $threshold{'numstudents'}</b></td>
                  </tr>
                  <tr>
                   <td bgcolor="$tabbg" align="right"><a href="/adm/whatsnew?command=chgthreshold"><b><small>Change thresholds?</small></b></a></td>
                  </tr>
                   <tr>
                   <td bgcolor="#ffffff">
                    <table width="100%" cellspacing="0" cellpadding="0" border="0">
   END
       my $warningnum = 0;
       if (@warnings > 0) {
   #        @warnings = sort { &cmp_title($a,$b) } @warnings;
           $r->print('<form name="reset_tracking" method="post">'.
                    '  <input type="hidden" name="action" value="reset"'."\n");
           $r->print('<tr bgcolor="#cccccc"><td><b><small>Resource</small></b></td><td align="right"><b><small>Part</small></b></td><td align="right"><b><small>Num. students</small></b></td><td align="right"><b><small>Av. Attempts</small></b></td><td align="right"><b><small>Deg. Diff</small></b></td><td align="right"><b><small>Last Reset</small></b></td><td align="right"><b><small>Reset Count?</small></b></td></tr>');
           foreach my $res (@warnings) {
               if ($warningnum %2 == 1) {
                   $rowColor = $rowColor1;
               } else {
                   $rowColor = $rowColor2;
               }
               my ($map,$id,$url)=&Apache::lonnet::decode_symb($res);
               my $linkurl=&Apache::lonnet::clutter($url);
               my $rowspan;
               if ($triggered{$res}{numparts} > 1) {
                   $rowspan = 'rowspan="'.$triggered{$res}{numparts}.'"';
               }
               $linkurl .= '?symb='.&Apache::lonnet::escape($res);
               $r->print('<tr bgcolor="'.$rowColor.'"><td '.$rowspan.'><a href="'.$linkurl.'"><small>'.$triggered{$res}{title}.'</small></a></td>'.$triggered{$res}{text});
               $warningnum ++;
           }
           $r->print('<tr bgcolor="#cccccc"><td colspan="7" align="right"><br /><b><small><input type="submit" name="counters" value="Reset counters to 0" /></form>'); 
       } else {
           $r->print('<tr><td bgcolor="#ffffff"><br /><center><b><i><small>No problems satisfy threshold criteria.</small></i></b></center><br /></td></tr>');
       }
       $r->print('</table></td></tr></table></td></tr></table><br />');
   
     $r->print('</td><td width="5%">&nbsp;</td><td align="left" valign="top" width-"50%">');      $r->print('</td><td width="5%">&nbsp;</td><td align="left" valign="top" width-"50%">');
   
   ## UNREAD COURSE DISCUSSION POSTS ##
       $r->print(<<"END");
                 <table border="0" cellpadding="0" cellspacing="0" bgcolor="#000000" width="100%">
                  <tr><td>
                   <table border="0" cellpadding="1" cellspacing="1" bgcolor="#000000" width="100%">
                    <tr>
                     <td bgcolor="$tabbg"><b>Unread course discussion posts</b></td>
                    </tr>
                    <tr>
                      <td bgcolor="#ffffff">
                      <table cellpadding="2" cellspacing="0" border="0" width="100%">
   END
                                                                                     
       if (@newdiscussions > 0) {
           $r->print('<tr bgcolor="#cccccc"><td><b><small>Location</small></b></td><td><b><small>Type</small></b><td align="right"><b><small>Number of new posts</small></b></td></tr>');
   #        @newdiscussions = sort { &cmp_title($a,$b) } @newdiscussions;
           my $rowNum = 0;
           foreach my $ressymb (@newdiscussions) {
               my $forum_title = $unread{$ressymb}{'title'};
               my $type = 'Resource';
               my $feedurl=&Apache::lonfeedback::get_feedurl($ressymb);
               if ($feedurl =~ /bulletinboard/) {
                   $type = 'Bulletin Board';
               }
               my $unreadnum = keys(%{$unread{$ressymb}});
               $unreadnum = $unreadnum - 2;
               if ($unreadnum > 0) {
                   if ($rowNum %2 == 1) {
                       $rowColor = $rowColor1;
                   } else {
                       $rowColor = $rowColor2;
                   }
                   $r->print('<tr bgcolor="'.$rowColor.'"><td><small><a href="'.$feedurl.'?symb='.$unread{$ressymb}{symb}.'">'.$forum_title.'</a>&nbsp;</td><td><small>'.$type.'</small></td><td align="right">'.$unreadnum.'&nbsp;</td></tr>');
                   $rowNum ++;
               }
           }
       } else {
           $r->print('<tr><td bgcolor="#ffffff"><br><center>&nbsp;<i><b><small>No unread posts in course discussions</small></b></i><br><br></td></tr>');
       }
       $r->print('</table></td></tr></table></td></tr></table><br />');
   
 ## MESSAGES ##  ## MESSAGES ##
     $r->print(<<END);      $r->print(<<END);
            <table border="0" cellpadding="0" cellspacing="0" bgcolor="#000000" width="100%">             <table border="0" cellpadding="0" cellspacing="0" bgcolor="#000000" width="100%">
Line 371  END Line 428  END
     $r->print('</td></tr></table>');      $r->print('</td></tr></table>');
 }  }
   
   #-------------------------------
   # display_config_box
   #
   # Display the threshold setting screen 
   #
   #-------------------------------
                                                                                   
   sub display_config_box() {
       my ($r,$command,$tabbg,$threshold_titles) = @_;
       my %threshold = ();
       my $rowColor1 = "#ffffff";
       my $rowColor2 = "#eeeeee";
       my $rowColor;
   
       my @thresholditems = ("av_attempts","degdiff","numstudents");
       my %threshold_titles = (
                            av_attempts => 'Average number of attempts',
                            degdiff => 'Degree of difficulty',
                            numstudents => 'Total number of students with submissions',
                            );
       &get_curr_thresholds(\%threshold);
   
       $r->print('<br /><form name="thresholdform" method="post"><table border="0" cellpadding="2" cellspacing="4"><tr><td align="left" valign="top" width="45%">
              <table border="0" cellpadding="0" cellspacing="0" bgcolor="#000000">
               <tr>
                <td>
                  <table border="0" cellpadding="1" cellspacing="1" bgcolor="#000000">
                   <tr>
                   <td bgcolor="#ffffff">
                    <table cellspacing="0" cellpadding="4" border="0">
        <tr bgcolor="'.$tabbg.'">
         <th>Threshold Name</th>
         <th>Current value</th>
         <th>Change?</th>
        </tr>');
       my $rowNum =0;
       foreach my $type (@thresholditems) {
           my $parameter = 'internal.threshold_'.$type;
   # onchange is javascript to automatically check the 'Set' button.
           my $onchange = 'onFocus="javascript:window.document.forms'.
                 "['thresholdform'].elements['".$parameter."_setparmval']".
                 '.checked=true;"';
           if ($rowNum %2 == 1) {
               $rowColor = $rowColor1;
           } else {
               $rowColor = $rowColor2;
           }
           $r->print('
        <tr bgcolor="'.$rowColor.'">
         <td>'.$threshold_titles{$type}.'</td>
         <td>'.&Apache::lonhtmlcommon::textbox($parameter.'_value',
                                               $threshold{$type},
                                               10,$onchange).'</td>
         <td>'
              .&Apache::lonhtmlcommon::checkbox($parameter.'_setparmval').
         '</td>
        </tr>');
           $rowNum ++;
       }
       $r->print('</table></td></tr></table></td></tr></table>
              <br /><input type="submit" name="threshold" value="Make changes" />
                    <input type="hidden" name="action" value="update" />
                  </form>');
   }
   
 sub getitems {  sub getitems {
     my ($unread,$ungraded,$bombed,$newdiscussions,$tograde,$bombs) = @_;      my ($unread,$ungraded,$bombed,$triggered,$newdiscussions,$tograde,$bombs,$warnings,$rowColor1,$rowColor2,$threshold,$cdom,$crs) = @_;
     my $navmap = Apache::lonnavmaps::navmap->new();      my $navmap = Apache::lonnavmaps::navmap->new();
     my @allres=$navmap->retrieveResources();      my @allres=$navmap->retrieveResources();
     my %discussiontime = &Apache::lonnet::dump('discussiontimes',      my %discussiontime = &Apache::lonnet::dump('discussiontimes',$cdom,$crs);
                $env{'course.'.$env{'request.course.id'}.'.domain'},      my %lastread = &Apache::lonnet::dump('nohist_'.$env{'request.course.id'}.
                $env{'course.'.$env{'request.course.id'}.'.num'});                  '_discuss',$env{'user.domain'},$env{'user.name'},'lastread');
     my %lastread = &Apache::lonnet::dump('nohist_'.$env{'request.course.id'}.'_discuss',$env{'user.domain'},$env{'user.name'},'lastread');  
     my %lastreadtime = ();      my %lastreadtime = ();
     my @discussions = ();      my @discussions = ();
     my ($classlist,$keylist) = &Apache::loncoursedata::get_classlist();      my ($classlist,$keylist) = &Apache::loncoursedata::get_classlist();
   
       my %resourcetracker =  &Apache::lonnet::dump('nohist_resourcetracker',
                  $cdom,$crs);
       my %res_title;
       my $warningnum = 0;
     foreach my $key (keys(%lastread)) {      foreach my $key (keys(%lastread)) {
         my $newkey = $key;          my $newkey = $key;
         $newkey =~ s/_lastread$//;          $newkey =~ s/_lastread$//;
Line 392  sub getitems { Line 517  sub getitems {
         my $result = '';          my $result = '';
         my $applies = 0;          my $applies = 0;
         my $symb = $resource->symb();          my $symb = $resource->symb();
         %{$$bombed{$symb}} = ();  #        %{$$bombed{$symb}} = ();
         %{$$ungraded{$symb}} = ();          %{$$ungraded{$symb}} = ();
           %{$$triggered{$symb}} = ();
           $$triggered{$symb}{numparts} = 0;
         my $title = $resource->compTitle();          my $title = $resource->compTitle();
           $res_title{$symb} = $title;
         my $ressymb = $resource->wrap_symb();          my $ressymb = $resource->wrap_symb();
 # Check for unread discussion postings  # Check for unread discussion postings
         if (defined($discussiontime{$ressymb})) {          if (defined($discussiontime{$ressymb})) {
Line 407  sub getitems { Line 535  sub getitems {
             if (defined($lastreadtime{$ressymb})) {              if (defined($lastreadtime{$ressymb})) {
                 $prevread = $lastreadtime{$ressymb};                  $prevread = $lastreadtime{$ressymb};
             }              }
             my %contrib = &Apache::lonnet::restore($ressymb,$env{'request.course.id'},              my %contrib = &Apache::lonnet::restore($ressymb,
             $env{'course.'.$env{'request.course.id'}.'.domain'},                               $env{'request.course.id'},$cdom,$crs);
             $env{'course.'.$env{'request.course.id'}.'.num'});  
             if ($contrib{'version'}) {              if ($contrib{'version'}) {
                 for (my $id=1;$id<=$contrib{'version'};$id++) {                  for (my $id=1;$id<=$contrib{'version'};$id++) {
                     unless (($contrib{'hidden'}=~/\.$id\./) || ($contrib{'deleted'}=~/\.$id\./)) {                      unless (($contrib{'hidden'}=~/\.$id\./) || ($contrib{'deleted'}=~/\.$id\./)) {
Line 432  sub getitems { Line 559  sub getitems {
                 my ($uname,$udom) = split(/:/,$student);                  my ($uname,$udom) = split(/:/,$student);
                 my %status=&Apache::grades::student_gradeStatus($url,$symb,$udom,$uname,$partlist);                  my %status=&Apache::grades::student_gradeStatus($url,$symb,$udom,$uname,$partlist);
                 my $submitted = 0;                  my $submitted = 0;
                 my $graded = 0;                  my $ungraded = 0;
                 foreach (keys(%status)) {                  foreach (keys(%status)) {
                     $submitted = 1 if ($status{$_} ne 'nothing');                      $submitted = 1 if ($status{$_} ne 'nothing');
                     $graded = 1 if ($status{$_} !~ /^correct/);                      $ungraded = 1 if ($status{$_} =~ /^ungraded/);
                     my ($foo,$partid,$foo1) = split(/\./,$_);                      my ($foo,$partid,$foo1) = split(/\./,$_);
                     if ($status{'resource.'.$partid.'.submitted_by'} ne '') {                      if ($status{'resource.'.$partid.'.submitted_by'} ne '') {
                         $submitted = 0;                          $submitted = 0;
                     }                      }
                 }                  }
                 next if (!$submitted || !$graded);                  next if (!$submitted || !$ungraded);
                 $ctr ++;                  $ctr ++;
             }              }
             if ($ctr) {              if ($ctr) {
Line 454  sub getitems { Line 581  sub getitems {
 # Check for bombs  # Check for bombs
         if ($resource->getErrors()) {          if ($resource->getErrors()) {
             my $errors = $resource->getErrors();              my $errors = $resource->getErrors();
               $errors =~ s/^,//;
             my @bombs = split(/,/, $errors);              my @bombs = split(/,/, $errors);
             my $errorcount = scalar(@bombs);              my $errorcount = scalar(@bombs);
             my $errorlink = '<a href="/adm/email?display='.              my $errorlink = '<a href="/adm/email?display='.
                             &Apache::lonnet::escape($$bombs[0]).'">';                              &Apache::lonnet::escape($bombs[0]).'">'.
                               $title.'</a>';
             $$bombed{$symb}{errorcount} = $errorcount;              $$bombed{$symb}{errorcount} = $errorcount;
             $$bombed{$symb}{errorlink} = $errorlink;              $$bombed{$symb}{errorlink} = $errorlink;
             push(@{$bombs}, $symb);              push(@{$bombs}, $symb);
         }          }
   # Compile maxtries and degree of difficulty for problem parts
           my @parts = @{$resource->parts()};
           my %stats;
           my %lastreset = ();
           my $warning = 0;
           my $rowColor;
           foreach my $part (@parts) {
               %{$stats{$part}} = ();
               my ($attempts,$users,$corrects,$degdiff,$av_attempts);
               if (exists($resourcetracker{$symb."\0".$part."\0attempts"})) {
                   $attempts = $resourcetracker{$symb."\0".$part."\0attempts"};
               }
               if (exists($resourcetracker{$symb."\0".$part."\0users"})) {
                   $users = $resourcetracker{$symb."\0".$part."\0users"};
               }
               if (exists($resourcetracker{$symb."\0".$part."\0correct"})) {
                   $corrects = $resourcetracker{$symb."\0".$part."\0correct"};
               }
               if ($attempts > 0) {
                   $degdiff =  1 - ($corrects/$attempts);
                   $degdiff = sprintf("%.2f",$degdiff);
               }
               if ($users > 0) {
                   $av_attempts = $attempts/$users;
               }
               if ((($degdiff ne '' && $degdiff >= $$threshold{'degdiff'}) || ($av_attempts ne '' && $av_attempts >= $$threshold{'av_attempts'})) && ($users >= $$threshold{'numstudents'})) {
                   $stats{$part}{degdiff} = $degdiff;
                   $stats{$part}{attempts} = $av_attempts;
                   $stats{$part}{users} = $users;
                   my %resethash = &Apache::lonnet::restore($symb,'nohist_resourcetracker',$cdom,$crs);
                   $lastreset{$part} = &get_counter_resets(\%resethash,$part);
                   $warning = 1;
               }
           }
           if ($warning) {
               if ($warningnum %2 == 1) {
                   $rowColor = $rowColor1;
               } else {
                   $rowColor = $rowColor2;
               }
               $$triggered{$symb}{title} = $resource->title;
               foreach my $part (@parts) {
                   if (exists($stats{$part}{users})) {
                       my $resetname = 'reset_'.&Apache::lonnet::escape($symb."\0".$part);
                       my $resettitle = 'title_'.&Apache::lonnet::escape($symb."\0".$part);
                       if ($$triggered{$symb}{numparts}) {
                           $$triggered{$symb}{text} .= '<tr bgcolor="'.$rowColor.'">'."\n";
                       }
                       if (@parts > 1) {
                           $$triggered{$symb}{text} .= '
                            <td align="right"><small>part - '.$part.'<small></td>';
                       } else {
                           $$triggered{$symb}{text} .= '
                            <td align="right"><small>single part</small></td>';
                       }
                       $$triggered{$symb}{text} .= '
                            <td align="right"><small>'.$stats{$part}{users}.'</small></td>
                            <td align="right"><small>'.$stats{$part}{attempts}.'</small></td>
                            <td align="right"><small>'.$stats{$part}{degdiff}.'</small></td>
                            <td align="right"><small>'.$lastreset{$part}.'</small></td>
                            <td align="right"><small><input type="checkbox" name="'.$resetname.'" /><input type="hidden" name="'.$resettitle.'" value="'.&Apache::lonnet::escape($$triggered{$symb}{title}).'" /></td> 
                           </tr>';
                       $$triggered{$symb}{numparts} ++;
                   }
               }
               push(@{$warnings},$symb);
               $warningnum ++;
           }
     }      }
 # Compile maxtries and degree of difficulty.  }
   
   sub get_counter_resets {
       my ($resethash,$part) = @_;
       my $lastreset = 'None';
       if ($$resethash{'version'}) {
           for (my $version=1;$version<=$$resethash{'version'};$version++) {
               if (exists($$resethash{$version.':'.$part.'prev_attempts'})) {
                    $lastreset = $$resethash{$version.':timestamp'};
               }
           }
       }
       unless ($lastreset eq 'None') {
           $lastreset = localtime($lastreset);
       }
       return $lastreset;
   }
   
   sub get_curr_thresholds {
       my ($threshold,$cdom,$crs) = @_;
       my %coursesettings = &Apache::lonnet::dump('environment',
                                        $cdom,$crs,'internal.threshold');
       if (exists($coursesettings{'internal.threshold_av_attempts'})) {
           $$threshold{'av_attempts'} = $coursesettings{'internal.threshold_av_attempts'};
       }
       if (exists($coursesettings{'internal.threshold_degdiff'})) {
           $$threshold{'degdiff'} = $coursesettings{'internal.threshold_degdiff'};
       }
       if (exists($coursesettings{'internal.threshold_numstudents'})) {
           $$threshold{'numstudents'} = $coursesettings{'internal.threshold_numstudents'};
       }
   }
   
   sub process_reset {
       my ($dom,$crs) = @_;
       my $result = '<b>Counters reset for following problems (and parts):</b><br />';
       my @agg_types = ('attempts','users','correct');
       my %agg_titles = (
                        attempts => 'Number of submissions',
                        users => 'Students with submissions',
                        correct => 'Number of correct submissions',
                        );
       my @resets = ();
       my %titles = ();
       foreach my $key (keys %env) {
           next if ($key !~ /^form\.reset_(.+)$/);
           my $title = &Apache::lonnet::unescape($env{'form.title_'.$1});
           my $reset_item = &Apache::lonnet::unescape($1);
           my %curr_aggregates = &Apache::lonnet::dump('nohist_resourcetracker',$dom,$crs,$reset_item);
           my %resethash = ();
           my %aggregates = ();
           my ($symb,$part) = split/\0/,$reset_item;
           foreach my $type (@agg_types) {
               $aggregates{$reset_item."\0".$type} = 0;
               $resethash{$part."\0".'prev_'.$type} = $curr_aggregates{$reset_item."\0".$type};
           }  
           my $putresult = &Apache::lonnet::put('nohist_resourcetracker',\%aggregates,
                             $dom,$crs);
           if ($putresult eq 'ok') {
               my $storeresult = &Apache::lonnet::cstore(\%resethash,$symb,'nohist_resourcetracker',$dom,$crs);
               $result .= $title.' -part '.$part.': ';
               my %new_aggregates = &Apache::lonnet::dump('nohist_resourcetracker',$dom,$crs,$reset_item);
               foreach my $type (@agg_types) {
                   $result .= $agg_titles{$type}.' = '.$new_aggregates{$reset_item."\0".$type}.'; ';
               }
               $result =~ s/; $//;
               $result .= '<br />';
           } else {
               $result = $title.' -part '.$part.': '.&mt('Unable to reset counters to zero due to ').$putresult.'.<br />'."\n";
           }
       }
       return $result;
   }
   
   sub process_update {
       my ($dom,$crs,$threshold_titles) = @_;
       my $setoutput = '<b>Changes to threshold(s):</b><br />';
       foreach (keys %env) {
           next if ($_!~/^form\.(.+)\_setparmval$/);
           my $name  = $1;
           my $value = $env{'form.'.$name.'_value'};
           if ($name && defined($value)) {
               my $put_result = &Apache::lonnet::put('environment',
                                                     {$name=>$value},$dom,$crs);
              
               my ($shortname) = ($name =~ /^internal\.threshold_(.+)$/); 
               if ($put_result eq 'ok') {
                   $setoutput.=&mt('Set threshold for').' <b>'.$$threshold_titles{$shortname}.'</b> '.&mt('to').' <b>'.$value.'</b>.<br />';
               } else {
                   $setoutput.=&mt('Unable to set threshold for').' <b>'.$name.'</b> '.&mt('to').
                       ' <b>'.$value.'</b> '.&mt('due to').' '.$put_result.'.<br />';
               }
           }
       }
       return $setoutput;
 }  }
   
 sub getmail {  sub getmail {
Line 471  sub getmail { Line 762  sub getmail {
 # Check for unread mail in course  # Check for unread mail in course
     my $msgcount = 0;      my $msgcount = 0;
   
     my @messages = &Apache::lonnet::getkeys('nohist_email');      my @messages = sort(&Apache::lonnet::getkeys('nohist_email'));
     foreach my $message (@messages) {      foreach my $message (@messages) {
  my $msgid=&Apache::lonnet::escape($message);   my $msgid=&Apache::lonnet::escape($message);
         my ($sendtime,$shortsubj,$fromname,$fromdom,$fromcid,$status)=          my ($sendtime,$shortsubj,$fromname,$fromdom,$status,$fromcid)=
             &Apache::lonmsg::unpackmsgid($msgid);              &Apache::lonmsg::unpackmsgid($msgid);
         if ($fromcid eq $env{'request.course.id'}) {          if (($fromcid) && ($fromcid eq $env{'request.course.id'})) {
             if (defined($sendtime) && $sendtime!~/error/) {              if (defined($sendtime) && $sendtime!~/error/) {
                 my $numsendtime = $sendtime;                  my $numsendtime = $sendtime;
                 $sendtime = &Apache::lonlocal::locallocaltime($sendtime);                  $sendtime = &Apache::lonlocal::locallocaltime($sendtime);
                 if ($status eq 'new') {                  if ($status eq 'new') {
                     $$msgcount ++;                      $msgcount ++;
                       if ($shortsubj eq '') {
                           $shortsubj = &mt('No subject');
                       }
                       $shortsubj = &Apache::lonnet::unescape($shortsubj);
                     push(@{$newmsgs}, {                      push(@{$newmsgs}, {
                         msgid    => $msgid,                          msgid    => $msgid,
                         sendtime => $sendtime,                          sendtime => $sendtime,
                         shortsub => &Apache::lonnet::unescape($shortsubj),                          shortsub => $shortsubj,
                         from     => $fromname,                          from     => $fromname,
                         fromdom  => $fromdom                          fromdom  => $fromdom
                         });                          });
Line 499  sub getmail { Line 794  sub getmail {
     my $result = '';      my $result = '';
     my $critmsgcount = 0;      my $critmsgcount = 0;
     foreach my $msgid (sort(keys(%what))) {      foreach my $msgid (sort(keys(%what))) {
         my ($sendtime,$shortsubj,$fromname,$fromdom,$fromcid,$status)=          my ($sendtime,$shortsubj,$fromname,$fromdom,$status,$fromcid)=
             &Apache::lonmsg::unpackmsgid($_);              &Apache::lonmsg::unpackmsgid($msgid);
         if ($fromcid eq  $env{'request.course.id'}) {          if (($fromcid) && ($fromcid eq  $env{'request.course.id'})) {
             if (defined($sendtime) && $sendtime!~/error/) {              if (defined($sendtime) && $sendtime!~/error/) {
                 my $numsendtime = $sendtime;                  my $numsendtime = $sendtime;
                 $sendtime = &Apache::lonlocal::locallocaltime($sendtime);                  $sendtime = &Apache::lonlocal::locallocaltime($sendtime);
                 $critmsgcount ++;                  $critmsgcount ++;
                   if ($shortsubj eq '') {
                       $shortsubj = &mt('No subject');
                   }
                   $shortsubj = &Apache::lonnet::unescape($shortsubj);
                 push(@{$critmsgs}, {                  push(@{$critmsgs}, {
                         msgid    => $msgid,                          msgid    => $msgid,
                         sendtime => $sendtime,                          sendtime => $sendtime,
                         shortsub => &Apache::lonnet::unescape($shortsubj),                          shortsub => $shortsubj,
                         from     => $fromname,                          from     => $fromname,
                         fromdom  => $fromdom                          fromdom  => $fromdom
                         });                          });

Removed from v.1.9  
changed lines
  Added in v.1.13


FreeBSD-CVSweb <freebsd-cvsweb@FreeBSD.org>