Diff for /loncom/homework/grades.pm between versions 1.804 and 1.806

version 1.804, 2024/12/10 18:38:10 version 1.806, 2024/12/14 17:47:39
Line 1263  sub do_passback { Line 1263  sub do_passback {
     my $cdom = $env{"course.$env{'request.course.id'}.domain"};      my $cdom = $env{"course.$env{'request.course.id'}.domain"};
     my $cnum = $env{"course.$env{'request.course.id'}.num"};      my $cnum = $env{"course.$env{'request.course.id'}.num"};
     my $crstype = &Apache::loncommon::course_type();      my $crstype = &Apache::loncommon::course_type();
     my ($launcher,$appname,$setter,$linkuri,$linkprotector,$scope,$chosen);      my ($launchsymb,$appname,$setter,$linkuri,$linkprotector,$scope,$chosen);
     if ($env{'form.passback'} ne '') {      if ($env{'form.passback'} ne '') {
         $chosen = &unescape($env{'form.passback'});          $chosen = &unescape($env{'form.passback'});
         ($linkuri,$linkprotector,$scope) = split("\0",$chosen);          ($linkuri,$linkprotector,$scope) = split("\0",$chosen);
         ($launcher,$appname,$setter) = &get_passback_launcher($cdom,$cnum,$chosen);          ($launchsymb,$appname,$setter) = &get_passback_launcher($cdom,$cnum,$chosen);
     }      }
     if ($launcher ne '') {      if ($launchsymb ne '') {
         $request->print(&launcher_info_box($launcher,$appname,$setter,$linkuri,$scope));          $request->print(&launcher_info_box($launchsymb,$appname,$setter,$linkuri,$scope));
     }      }
     my $error;      my $error;
     if ($perm{'mgr'}) {      if ($perm{'mgr'}) {
         if ($launcher ne '') {          if ($launchsymb ne '') {
             my @poss_students = &Apache::loncommon::get_env_multiple('form.stuinfo');              my @poss_students = &Apache::loncommon::get_env_multiple('form.stuinfo');
             if (@poss_students) {              if (@poss_students) {
                 my %possibles;                  my %possibles;
Line 1312  sub do_passback { Line 1312  sub do_passback {
                             if ($lti_in_use->{'scoreformat'} =~ /^(decimal|ratio|percentage)$/) {                              if ($lti_in_use->{'scoreformat'} =~ /^(decimal|ratio|percentage)$/) {
                                 $scoretype = $1;                                  $scoretype = $1;
                             }                              }
                             my $pbsymb = &Apache::loncommon::symb_from_tinyurl($linkuri,$cnum,$cdom);  
                             my $pbmap;                              my $pbmap;
                             if ($pbsymb =~ /\.(page|sequence)$/) {                              if ($launchsymb =~ /\.(page|sequence)$/) {
                                 $pbmap = &Apache::lonnet::deversion((&Apache::lonnet::decode_symb($pbsymb))[2]);                                  $pbmap = &Apache::lonnet::deversion((&Apache::lonnet::decode_symb($launchsymb))[2]);
                             } else {                              } else {
                                 $pbmap = &Apache::lonnet::deversion((&Apache::lonnet::decode_symb($pbsymb))[0]);                                  $pbmap = &Apache::lonnet::deversion((&Apache::lonnet::decode_symb($launchsymb))[0]);
                             }                              }
                             $pbmap = &Apache::lonnet::clutter($pbmap);                              $pbmap = &Apache::lonnet::clutter($pbmap);
                             my $pbscope;                              my $pbscope;
Line 1332  sub do_passback { Line 1331  sub do_passback {
                             my $numstudents = scalar(keys(%tosend));                              my $numstudents = scalar(keys(%tosend));
                             my %prog_state = &Apache::lonhtmlcommon::Create_PrgWin($request,$numstudents);                              my %prog_state = &Apache::lonhtmlcommon::Create_PrgWin($request,$numstudents);
                             my $outcome = &Apache::loncommon::start_data_table().                              my $outcome = &Apache::loncommon::start_data_table().
                                          &Apache::loncommon::start_data_table_header_row();                                            &Apache::loncommon::start_data_table_header_row();
                             my $loop = 0;                              my $loop = 0;
                             while ($loop < 2) {                              while ($loop < 2) {
                                 $outcome .= '<th>'.&mt('No.').'</th>'.                                  $outcome .= '<th>'.&mt('No.').'</th>'.
                                            '<th>'.&nameUserString('header').'&nbsp;'.&mt('Section/Group').'</th>'.                                              '<th>'.&nameUserString('header').'&nbsp;'.&mt('Section/Group').'</th>'.
                                            '<th>'.&mt('Score').'</th>';                                              '<th>'.&mt('Score').'</th>';
                                  $loop++;                                  $loop++;
                             }                              }
                             $outcome .= &Apache::loncommon::end_data_table_header_row()."\n";                              $outcome .= &Apache::loncommon::end_data_table_header_row()."\n";
                             my $ctr=0;                              my $ctr=0;
Line 1365  sub do_passback { Line 1364  sub do_passback {
                                     $possible = 0;                                      $possible = 0;
                                     my $navmap = Apache::lonnavmaps::navmap->new($uname,$udom);                                      my $navmap = Apache::lonnavmaps::navmap->new($uname,$udom);
                                     if (ref($navmap)) {                                      if (ref($navmap)) {
                                         my $res = $navmap->getBySymb($pbsymb);                                          my $res = $navmap->getBySymb($launchsymb);
                                         if (ref($res)) {                                          if (ref($res)) {
                                             my $partlist = $res->parts();                                              my $partlist = $res->parts();
                                             if (ref($partlist) eq 'ARRAY') {                                              if (ref($partlist) eq 'ARRAY') {
                                                 my %record = &Apache::lonnet::restore($pbsymb,$env{'request.course.id'},$udom,$uname);                                                  my %record = &Apache::lonnet::restore($launchsymb,$env{'request.course.id'},$udom,$uname);
                                                 foreach my $part (@{$partlist}) {                                                  foreach my $part (@{$partlist}) {
                                                     next if ($record{"resource.$part.solved"} =~/^excused/);                                                      next if ($record{"resource.$part.solved"} =~/^excused/);
                                                     my $weight = &Apache::lonnet::EXT("resource.$part.weight",$pbsymb,$udom,$uname,$usec);                                                      my $weight = &Apache::lonnet::EXT("resource.$part.weight",$launchsymb,$udom,$uname,$usec);
                                                     $possible += $weight;                                                      $possible += $weight;
                                                     if (($record{'version'}) && (exists($record{"resource.$part.awarded"}))) {                                                      if (($record{'version'}) && (exists($record{"resource.$part.awarded"}))) {
                                                         my $awarded = $record{"resource.$part.awarded"};                                                          my $awarded = $record{"resource.$part.awarded"};
Line 1386  sub do_passback { Line 1385  sub do_passback {
                                     }                                      }
                                 } elsif (($pbscope eq 'map') || ($pbscope eq 'nonrec')) {                                  } elsif (($pbscope eq 'map') || ($pbscope eq 'nonrec')) {
                                     ($total,$possible) =                                      ($total,$possible) =
                                         &Apache::lonhomework::get_lti_score($uname,$udom,$pbmap,$pbscope);                                          &Apache::lonhomework::get_lti_score($uname,$udom,$usec,$pbmap,$pbscope);
                                 }                                  }
                                 if (($id ne '') && ($url ne '') && ($possible)) {                                  if (($id ne '') && ($url ne '') && ($possible)) {
                                     my ($sent,$score,$code,$result) =                                      my ($sent,$score,$code,$result) =
Line 1408  sub do_passback { Line 1407  sub do_passback {
                                                  'id' => $id,                                                   'id' => $id,
                                                  'clientip' => $pb{'clientip'},                                                   'clientip' => $pb{'clientip'},
                                                  'whodoneit' => $env{'user.name'}.':'.$env{'user.domain'},                                                   'whodoneit' => $env{'user.name'}.':'.$env{'user.domain'},
                                                 };                                              };
                                             my $value='';                                              my $value='';
                                             foreach my $key (keys(%{$store})) {                                              foreach my $key (keys(%{$store})) {
                                                 $value.=&escape($key).'='.&Apache::lonnet::freeze_escape($store->{$key}).'&';                                                  $value.=&escape($key).'='.&Apache::lonnet::freeze_escape($store->{$key}).'&';
Line 1420  sub do_passback { Line 1419  sub do_passback {
                                             if ($ctr%2 ==1) {                                              if ($ctr%2 ==1) {
                                                 $outcome .= &Apache::loncommon::start_data_table_row();                                                  $outcome .= &Apache::loncommon::start_data_table_row();
                                             }                                              }
                                             my $section = $classlist->{$student}->[&Apache::loncoursedata::CL_SECTION()];  
                                             my $group = $classlist->{$student}->[&Apache::loncoursedata::CL_GROUP()];                                              my $group = $classlist->{$student}->[&Apache::loncoursedata::CL_GROUP()];
                                             $outcome .= '<td align="right">'.$ctr.'&nbsp;</td>'.                                              $outcome .= '<td align="right">'.$ctr.'&nbsp;</td>'.
                                                        '<td>'.&nameUserString(undef,$$fullname{$student},$uname,$udom).                                                          '<td>'.&nameUserString(undef,$$fullname{$student},$uname,$udom).
                                                        '&nbsp;'.$section.($group ne '' ?'/'.$group:'').'</td>'.                                                          '&nbsp;'.$usec.($group ne '' ?'/'.$group:'').'</td>'.
                                                        '<td>'.$score.'</td>'."\n";                                                          '<td>'.$score.'</td>'."\n";
                                             if ($ctr%2 ==0) {                                              if ($ctr%2 ==0) {
                                                 $outcome .= &Apache::loncommon::end_data_table_row()."\n";                                                  $outcome .= &Apache::loncommon::end_data_table_row()."\n";
                                             }                                              }
Line 1441  sub do_passback { Line 1439  sub do_passback {
                                     }                                      }
                                     if ($no_passback) {                                      if ($no_passback) {
                                         &Apache::lonnet::log($udom,$uname,$uhome,$no_passback." score: $score; total: $total; possible: $possible");                                          &Apache::lonnet::log($udom,$uname,$uhome,$no_passback." score: $score; total: $total; possible: $possible");
                                           my $key = &Time::HiRes::time().':'.$uname.':'.$udom.':'.
                                                     "$linkuri\0$linkprotector\0$scope"; 
                                         my $ltigrade = {                                          my $ltigrade = {
                                             'ltinum'   => $ltinum,                                                           $key => {
                                             'lti'      => $lti_in_use,                                                                     'ltinum'   => $ltinum,
                                             'crsdef'   => $crsdef,                                                                     'lti'      => $lti_in_use,
                                             'cid'      => $cdom.'_'.$cnum,                                                                     'crsdef'   => $crsdef,
                                             'uname'    => $uname,                                                                     'cid'      => $cdom.'_'.$cnum,
                                             'udom'     => $udom,                                                                     'uname'    => $uname,
                                             'uhome'    => $uhome,                                                                     'udom'     => $udom,
                                             'pbid'     => $id,                                                                     'uhome'    => $uhome,
                                             'pburl'    => $url,                                                                     'pbid'     => $id,
                                             'pbtype'   => $pb{'type'},                                                                     'pburl'    => $url,
                                             'pbscope'  => $pbscope,                                                                     'pbtype'   => $pb{'type'},
                                             'pbmap'    => $pbmap,                                                                     'pbscope'  => $pbscope,
                                             'pbsymb'   => $pbsymb,                                                                     'pbmap'    => $pbmap,
                                             'format'   => $scoretype,                                                                     'pbsymb'   => $launchsymb,
                                             'scope'    => $scope,                                                                     'format'   => $scoretype,
                                             'clientip' => $pb{'clientip'},                                                                     'scope'    => $scope,
                                             'linkprot' => $linkprotector.':'.$linkuri,                                                                     'clientip' => $pb{'clientip'},
                                             'total'    => $total,                                                                     'linkprot' => $linkprotector.':'.$linkuri,
                                             'possible' => $possible,                                                                     'total'    => $total,
                                             'score'    => $score,                                                                     'possible' => $possible,
                                                                      'score'    => $score,
                                                                    },
                                         };                                          };
                                         &Apache::lonnet::put('linkprot_passback_pending',$ltigrade,$cdom,$cnum);                                          &Apache::lonnet::put('linkprot_passback_pending',$ltigrade,$cdom,$cnum);
                                     }                                      }
Line 1575  sub get_passback_launcher { Line 1577  sub get_passback_launcher {
             }              }
         }          }
     }      }
     if ($linkuri =~ m{^\Q/tiny/$cdom/\E(\w+)$}) {      my $launchsymb = &Apache::loncommon::symb_from_tinyurl($linkuri,$cnum,$cdom);
         my $key = $1;      if ($launchsymb eq '') {
         my $tinyurl;          my %passback = &Apache::lonnet::dump('nohist_linkprot_passback',$cdom,$cnum);
         my ($result,$cached)=&Apache::lonnet::is_cached_new('tiny',$cdom."\0".$key);          foreach my $poss_symb (keys(%passback)) {
         if (defined($cached)) {              if (ref($passback{$poss_symb}) eq 'HASH') {
             $tinyurl = $result;                  if (exists($passback{$poss_symb}{$chosen})) {
         } else {                      $launchsymb = $poss_symb;
             my $configuname = &Apache::lonnet::get_domainconfiguser($cdom);                      last;
             my %currtiny = &Apache::lonnet::get('tiny',[$key],$cdom,$configuname);                  }
             if ($currtiny{$key} ne '') {  
                 $tinyurl = $currtiny{$key};  
                 &Apache::lonnet::do_cache_new('tiny',$cdom."\0".$key,$currtiny{$key},600);  
             }              }
         }          }
         if ($tinyurl) {          if ($launchsymb ne '') {
             my ($crsnum,$launchsymb) = split(/\&/,$tinyurl);              return ($launchsymb,$appname,$setter);
             if ($crsnum eq $cnum) {          }
                 my %passback = &Apache::lonnet::get('nohist_linkprot_passback',[$launchsymb],$cdom,$cnum);      } else {
                 if (ref($passback{$launchsymb}) eq 'HASH') {          my %passback = &Apache::lonnet::get('nohist_linkprot_passback',[$launchsymb],$cdom,$cnum);
                     if (exists($passback{$launchsymb}{$chosen})) {          if (ref($passback{$launchsymb}) eq 'HASH') {
                         return ($launchsymb,$appname,$setter);              if (exists($passback{$launchsymb}{$chosen})) {
                     }                  return ($launchsymb,$appname,$setter);
                 }  
             }              }
         }          }
     }      }
Line 1679  sub passbacks_for_symb { Line 1677  sub passbacks_for_symb {
             }              }
             if (ref($passback{$symb}) eq 'HASH') {              if (ref($passback{$symb}) eq 'HASH') {
                 foreach my $launcher (keys(%{$passback{$symb}})) {                  foreach my $launcher (keys(%{$passback{$symb}})) {
                     $needpb{$launcher} = 1;                      $needpb{$launcher} = $symb;
                 }                  }
             }              }
         }          }
Line 1696  sub passbacks_for_symb { Line 1694  sub passbacks_for_symb {
                         }                          }
                         if (ref($passback{$mapsymb}) eq 'HASH') {                          if (ref($passback{$mapsymb}) eq 'HASH') {
                             foreach my $launcher (keys(%{$passback{$mapsymb}})) {                              foreach my $launcher (keys(%{$passback{$mapsymb}})) {
                                 $needpb{$launcher} = 1;                                  $needpb{$launcher} = $mapsymb;
                             }                              }
                         }                          }
                     }                      }
Line 1713  sub passbacks_for_symb { Line 1711  sub passbacks_for_symb {
                                 foreach my $launcher (keys(%{$passback{$key}})) {                                  foreach my $launcher (keys(%{$passback{$key}})) {
                                     my ($linkuri,$linkprotector,$scope) = split("\0",$launcher);                                      my ($linkuri,$linkprotector,$scope) = split("\0",$launcher);
                                     next unless ($scope eq 'rec');                                      next unless ($scope eq 'rec');
                                     $needpb{$launcher} = 1;                                      $needpb{$launcher} = $key;
                                 }                                  }
                             }                              }
                         }                          }
Line 1778  sub process_passbacks { Line 1776  sub process_passbacks {
                             if ($pb{'lti_in_use'}->{'scoreformat'} =~ /^(decimal|ratio|percentage)$/) {                              if ($pb{'lti_in_use'}->{'scoreformat'} =~ /^(decimal|ratio|percentage)$/) {
                                 $pb{'scoretype'} = $1;                                  $pb{'scoretype'} = $1;
                             }                              }
                             $pb{'symb'} = &Apache::loncommon::symb_from_tinyurl($pb{'linkuri'},$cnum,$cdom);                              $pb{'symb'} = $needpb->{$launcher};
                             if ($pb{'symb'} =~ /\.(page|sequence)$/) {                              if ($pb{'symb'} =~ /\.(page|sequence)$/) {
                                 $pb{'map'} = &Apache::lonnet::deversion((&Apache::lonnet::decode_symb($pb{'symb'}))[2]);                                  $pb{'map'} = &Apache::lonnet::deversion((&Apache::lonnet::decode_symb($pb{'symb'}))[2]);
                             } else {                              } else {
Line 1850  sub process_passbacks { Line 1848  sub process_passbacks {
                         $possible = $possible_by_symb{$pb{'symb'}};                          $possible = $possible_by_symb{$pb{'symb'}};
                     } elsif (($pb{'pbscope'} eq 'map') || ($pb{'pbscope'} eq 'nonrec')) {                      } elsif (($pb{'pbscope'} eq 'map') || ($pb{'pbscope'} eq 'nonrec')) {
                         ($total,$possible) =                          ($total,$possible) =
                             &Apache::lonhomework::get_lti_score($uname,$udom,$pb{'map'},$pb{'pbscope'},                              &Apache::lonhomework::get_lti_score($uname,$udom,$usec,$pb{'map'},$pb{'pbscope'},
                                                                 \%total_by_symb,\%possible_by_symb);                                                                  \%total_by_symb,\%possible_by_symb);
                     }                      }
                     if (!$possible) {                      if (!$possible) {
Line 7004  sub updateGradeByPage { Line 7002  sub updateGradeByPage {
             map { $parentmapsymbs{$_} = 1; } @recurseup;              map { $parentmapsymbs{$_} = 1; } @recurseup;
         }          }
         my %passback = &Apache::lonnet::get('nohist_linkprot_passback',\@allsymbs,$cdom,$cnum);          my %passback = &Apache::lonnet::get('nohist_linkprot_passback',\@allsymbs,$cdom,$cnum);
         my (%uniqsymbs,$use_symbs_in_map);          my (%uniqsymbs,$use_symbs_in_map,%launch_to_symb);
         if (keys(%passback)) {          if (keys(%passback)) {
             foreach my $possible (keys(%passback)) {              foreach my $possible (keys(%passback)) {
                 if (ref($passback{$possible}) eq 'HASH') {                  if (ref($passback{$possible}) eq 'HASH') {
                     if ($possible eq $mapsymb) {                      if ($possible eq $mapsymb) {
                         foreach my $launcher (keys(%{$passback{$possible}})) {                          foreach my $launcher (keys(%{$passback{$possible}})) {
                             $possmappb{$launcher} = 1;                              $possmappb{$launcher} = 1;
                               $launch_to_symb{$launcher} = $possible;
                         }                          }
                         $use_symbs_in_map = 1;                          $use_symbs_in_map = 1;
                     } elsif (exists($parentmapsymbs{$possible})) {                      } elsif (exists($parentmapsymbs{$possible})) {
Line 7019  sub updateGradeByPage { Line 7018  sub updateGradeByPage {
                             if ($scope eq 'rec') {                              if ($scope eq 'rec') {
                                 $possmappb{$launcher} = 1;                                  $possmappb{$launcher} = 1;
                                 $use_symbs_in_map = 1;                                  $use_symbs_in_map = 1;
                                   $launch_to_symb{$launcher} = $possible;
                             }                              }
                         }                          }
                     } elsif (grep(/^\Q$possible$\E$/,@updates)) {                      } elsif (grep(/^\Q$possible$\E$/,@updates)) {
                         foreach my $launcher (keys(%{$passback{$possible}})) {                          foreach my $launcher (keys(%{$passback{$possible}})) {
                             $possrespb{$launcher} = 1;                              $possrespb{$launcher} = 1;
                               $launch_to_symb{$launcher} = $possible;
                         }                          }
                         $uniqsymbs{$possible} = 1;                          $uniqsymbs{$possible} = 1;
                     }                      }
Line 7045  sub updateGradeByPage { Line 7046  sub updateGradeByPage {
             my %pbids = &Apache::lonnet::get('nohist_'.$cdom.'_'.$cnum.'_linkprot_pb',\@posslaunchers,$udom,$uname);              my %pbids = &Apache::lonnet::get('nohist_'.$cdom.'_'.$cnum.'_linkprot_pb',\@posslaunchers,$udom,$uname);
             foreach my $key (keys(%pbids)) {              foreach my $key (keys(%pbids)) {
                 if (ref($pbids{$key}) eq 'ARRAY') {                  if (ref($pbids{$key}) eq 'ARRAY') {
                     $needpb{$key} = 1;                      if ($launch_to_symb{$key}) {
                           $needpb{$key} = $launch_to_symb{$key};
                       }
                 }                  }
             }              }
             my @symbs = keys(%uniqsymbs);              my @symbs = keys(%uniqsymbs);

Removed from v.1.804  
changed lines
  Added in v.1.806


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