Diff for /loncom/interface/coursecatalog.pm between versions 1.83 and 1.88.2.2

version 1.83, 2014/04/28 14:23:47 version 1.88.2.2, 2018/04/29 00:31:25
Line 38  use Apache::lonlocal; Line 38  use Apache::lonlocal;
 use Apache::courseclassifier;  use Apache::courseclassifier;
 use Apache::lonacc;  use Apache::lonacc;
 use LONCAPA;  use LONCAPA;
   use LONCAPA::lonauthcgi;
   
 sub handler {  sub handler {
     my ($r) = @_;      my ($r) = @_;
Line 79  sub handler { Line 80  sub handler {
     my %domconfig =      my %domconfig =
         &Apache::lonnet::get_dom('configuration',['coursecategories'],$codedom);          &Apache::lonnet::get_dom('configuration',['coursecategories'],$codedom);
     my $knownuser = &user_is_known();      my $knownuser = &user_is_known();
       my $canviewall = &canview_all($knownuser,$codedom);
       
     my ($cathash,$cattype);      my ($cathash,$cattype);
     if (ref($domconfig{'coursecategories'}) eq 'HASH') {      if (ref($domconfig{'coursecategories'}) eq 'HASH') {
         $cathash = $domconfig{'coursecategories'}{'cats'};          $cathash = $domconfig{'coursecategories'}{'cats'};
         if ($knownuser) {          if ($knownuser || $canviewall) {
             $cattype = $domconfig{'coursecategories'}{'auth'};              $cattype = $domconfig{'coursecategories'}{'auth'};
         } else {          } else {
             $cattype = $domconfig{'coursecategories'}{'unauth'};              $cattype = $domconfig{'coursecategories'}{'unauth'};
         }          }
           if ($cattype eq '') {
               $cattype = 'std';
           }
     } else {      } else {
         $cathash = {};          $cathash = {};
         $cattype = 'std';          $cattype = 'std';
Line 98  sub handler { Line 103  sub handler {
         ({href=>"/adm/coursecatalog",          ({href=>"/adm/coursecatalog",
           text=>"Course/Community Catalog"});            text=>"Course/Community Catalog"});
         $r->print(&Apache::lonhtmlcommon::breadcrumbs('Course/Community Catalog'));          $r->print(&Apache::lonhtmlcommon::breadcrumbs('Course/Community Catalog'));
         if ($knownuser) {          if ($knownuser || $canviewall) {
             $r->print('<div>'.&mt('No catalog of LON-CAPA courses/communities is provided for: [_1]',$domdesc).'</div>');              $r->print('<div>'.&mt('No catalog of LON-CAPA courses/communities is provided for: [_1]',$domdesc).'</div>');
         } else {          } else {
             if ($domconfig{'coursecategories'}{'auth'} eq 'none') {              if ($domconfig{'coursecategories'}{'auth'} eq 'none') {
Line 146  sub handler { Line 151  sub handler {
                 my %courses = &Apache::lonnet::courseiddump($codedom,'.',1,'.','.',                  my %courses = &Apache::lonnet::courseiddump($codedom,'.',1,'.','.',
                                                             $cnum,undef,undef,'.',1);                                                              $cnum,undef,undef,'.',1);
                 if (keys(%courses)) {                  if (keys(%courses)) {
                     $env{'form.coursenum'} = $cnum;                        $env{'form.coursenum'} = $cnum;
                     my %courseinfo = &build_courseinfo_hash(\%courses,$knownuser,$codedom);                      my %courseinfo = &build_courseinfo_hash(\%courses,$knownuser,$codedom,$canviewall);
                     undef($env{'form.coursenum'});                      undef($env{'form.coursenum'});
                     if (ref($courseinfo{$codedom.'_'.$cnum}) eq 'HASH') {                      if (ref($courseinfo{$codedom.'_'.$cnum}) eq 'HASH') {
                         $r->print(&Apache::lonhtmlcommon::start_pick_box());                          $r->print(&Apache::lonhtmlcommon::start_pick_box());
Line 246  function setCourseId(caller) { Line 251  function setCourseId(caller) {
   
 ENDSCRIPT  ENDSCRIPT
         $catjs .= &courselink_javascript();          $catjs .= &courselink_javascript();
         if (&user_is_dc($codedom)) {          if (&user_is_dc($codedom) || $canviewall) {
             $catjs .= <<ENDTOGGJS              $catjs .= <<ENDTOGGJS
   
 function toggleStatuses() {  function toggleStatuses() {
Line 277  ENDTOGGJS Line 282  ENDTOGGJS
         }          }
         if ($env{'form.currcat_0'} eq 'instcode::0') {          if ($env{'form.currcat_0'} eq 'instcode::0') {
             $numtitles = &instcode_course_selector($r,$codedom,$formname,$domdesc,              $numtitles = &instcode_course_selector($r,$codedom,$formname,$domdesc,
                                                    $catlinks,$catjs,\@codetitles,$cattype);                                                     $catlinks,$catjs,\@codetitles,$cattype,$canviewall);
             if ($env{'form.state'} eq 'listing') {              if ($env{'form.state'} eq 'listing') {
                 $r->print(&print_course_listing($codedom,$numtitles,undef,undef,undef,                  $r->print(&print_course_listing($codedom,$numtitles,undef,undef,undef,
                                                 \@codetitles));                                                  \@codetitles,$canviewall));
             }              }
         } else {          } else {
             my (%add_entries);              my (%add_entries);
Line 290  ENDTOGGJS Line 295  ENDTOGGJS
                 if (!$deeper) {                  if (!$deeper) {
                     $alert = &mt('Choose a category to display');                      $alert = &mt('Choose a category to display');
                 }                  }
                   &js_escape(\$alert);
                 $catjs .= <<ENDJS;                  $catjs .= <<ENDJS;
 function check_selected() {  function check_selected() {
     if (document.coursecats.$selitem.options[document.coursecats.$selitem.selectedIndex].value == "") {      if (document.coursecats.$selitem.options[document.coursecats.$selitem.selectedIndex].value == "") {
Line 304  ENDJS Line 310  ENDJS
             if ($env{'form.currcat_0'} ne '') {              if ($env{'form.currcat_0'} ne '') {
                 $r->print('<form name="'.$formname.                  $r->print('<form name="'.$formname.
                           '" method="post" action="/adm/coursecatalog">'.                            '" method="post" action="/adm/coursecatalog">'.
                           &additional_filters($codedom,$has_subcats)."\n");                            &additional_filters($codedom,$has_subcats,$canviewall)."\n");
                 $r->print('<input type="hidden" name="catalog_maxdepth" value="'.                  $r->print('<input type="hidden" name="catalog_maxdepth" value="'.
                           $deeper.'" />'."\n");                            $deeper.'" />'."\n");
                 for (my $i=0; $i<$deeper; $i++) {                  for (my $i=0; $i<$deeper; $i++) {
Line 325  ENDJS Line 331  ENDJS
                           $display_button.'" /></form><br /><br />');                            $display_button.'" /></form><br /><br />');
             }              }
             if ($env{'form.state'} eq 'listing') {              if ($env{'form.state'} eq 'listing') {
                 $r->print(&print_course_listing($codedom,undef,\@trails,\%allitems,$subcats,\@codetitles));                  $r->print(&print_course_listing($codedom,undef,\@trails,\%allitems,$subcats,\@codetitles,
                                                   $canviewall));
             }              }
         }          }
     }      }
Line 386  sub coursesearch { Line 393  sub coursesearch {
     my %lt = &Apache::lonlocal::texthash (      my %lt = &Apache::lonlocal::texthash (
                                            crlk => 'Course look-up',                                               crlk => 'Course look-up',  
                                            code => 'Code',                                             code => 'Code',
                                            ifyo => 'Enter the character code (six letters and numbers)',                                             ifyo => 'Enter the course code (six letters and numbers)',
                                            srch => 'Find course',                                             srch => 'Find course',
                                          );                                           );
     return <<"END";      return <<"END";
Line 428  END Line 435  END
 }  }
   
 sub instcode_course_selector {  sub instcode_course_selector {
     my ($r,$codedom,$formname,$domdesc,$catlinks,$catjs,$codetitles,$cattype) = @_;      my ($r,$codedom,$formname,$domdesc,$catlinks,$catjs,$codetitles,$cattype,$canviewall) = @_;
     my %coursecodes = ();      my %coursecodes = ();
     my %codes = ();      my %codes = ();
     my %cat_titles = ();      my %cat_titles = ();
Line 448  sub instcode_course_selector { Line 455  sub instcode_course_selector {
              '</script>';               '</script>';
     if ($totcodes) {      if ($totcodes) {
         if (($env{'form.state'} eq 'listing') && ($numtitles > 0)) {          if (($env{'form.state'} eq 'listing') && ($numtitles > 0)) {
             $add_entries{'onLoad'} = 'setElements();';              $add_entries{'onload'} = 'setElements();';
         }          }
         if (&user_is_dc($codedom)) {          if (&user_is_dc($codedom) || $canviewall) {
             $add_entries{'onLoad'} .= ' toggleStatuses();toggleWasActive();'              $add_entries{'onload'} .= ' toggleStatuses();toggleWasActive();'
         }          }
         &cat_header($r,$codedom,$js,\%add_entries,$catlinks,$numtitles,$cattype);          &cat_header($r,$codedom,$js,\%add_entries,$catlinks,$numtitles,$cattype);
         my $cat_maxdepth = $env{'form.catalog_maxdepth'};          my $cat_maxdepth = $env{'form.catalog_maxdepth'};
Line 459  sub instcode_course_selector { Line 466  sub instcode_course_selector {
                   '<input type="hidden" name="catalog_maxdepth" value="'.$cat_maxdepth.'" />'."\n".                    '<input type="hidden" name="catalog_maxdepth" value="'.$cat_maxdepth.'" />'."\n".
                   '<input type="hidden" name="showdom" value="'.$env{'form.showdom'}.'" />'."\n".                    '<input type="hidden" name="showdom" value="'.$env{'form.showdom'}.'" />'."\n".
                   '<input type="hidden" name="currcat_0" value="instcode::0" />'.                    '<input type="hidden" name="currcat_0" value="instcode::0" />'.
                   &additional_filters($codedom));                    &additional_filters($codedom,undef,$canviewall));
         if ($numtitles > 0) {          if ($numtitles > 0) {
             $r->print('<b>'.&mt('Choose which course(s) to list.').'</b><br />'.              $r->print('<b>'.&mt('Choose which course(s) to list.').'</b><br />'.
                       &Apache::courseclassifier::build_instcode_selectors($numtitles,                        &Apache::courseclassifier::build_instcode_selectors($numtitles,
Line 680  sub get_depth_values { Line 687  sub get_depth_values {
 }  }
   
 sub additional_filters {  sub additional_filters {
     my ($codedom,$has_subcats) = @_;      my ($codedom,$has_subcats,$canviewall) = @_;
     my $is_dc = &user_is_dc($codedom);      my $is_dc = &user_is_dc($codedom);
     my $output = '<div class="LC_left_float">';      my $output = '<div class="LC_left_float">';
     if ($is_dc) {      if ($is_dc || $canviewall) {
         $output .= '<fieldset><legend>'.&mt('Options').'</legend>';          $output .= '<fieldset><legend>'.&mt('Options').'</legend>';
     }      }
     $output .= '<table><tr><td valign="top">';      $output .= '<table><tr><td valign="top">';
Line 724  sub additional_filters { Line 731  sub additional_filters {
                '<label><input type="checkbox" name="showselfenroll" value="1" '.                 '<label><input type="checkbox" name="showselfenroll" value="1" '.
                $show_selfenroll_status.'/>'.$selfenroll_text.                 $show_selfenroll_status.'/>'.$selfenroll_text.
                '</label></span><br />';                 '</label></span><br />';
     if ($is_dc) {      if ($is_dc || $canviewall) {
         my ($titlesref,$orderref) = &get_statustitles('filters');          my ($titlesref,$orderref) = &get_statustitles('filters');
         my $showdetails_status;          my $showdetails_status;
         if ($env{'form.showdetails'}) {          if ($env{'form.showdetails'}) {
Line 735  sub additional_filters { Line 742  sub additional_filters {
             $showhidden_status = 'checked="checked" ';              $showhidden_status = 'checked="checked" ';
         }          }
         my @currstatuses = &Apache::loncommon::get_env_multiple('form.showcounts');          my @currstatuses = &Apache::loncommon::get_env_multiple('form.showcounts');
         my $dc_title = &Apache::lonnet::plaintext('dc');  
         my ($details_text,$hidden_text,$statusdisplay,$cellborder);          my ($details_text,$hidden_text,$statusdisplay,$cellborder);
         my $wasactivedisplay = 'none';          my $wasactivedisplay = 'none';
         if ($env{'form.showdetails'}) {          if ($env{'form.showdetails'}) {
Line 749  sub additional_filters { Line 755  sub additional_filters {
             $cellborder = 'border-left: 0px';              $cellborder = 'border-left: 0px';
         }          }
         if ($env{'form.currcat_0'} eq 'communities::0') {          if ($env{'form.currcat_0'} eq 'communities::0') {
             $details_text = &mt('Show full details for each community ([_1] only)',$dc_title);              $details_text = &mt('Show full details for each community (domain staff only)');
             $hidden_text = &mt('Include communities set to be hidden from catalog ([_1] only)',$dc_title);              $hidden_text = &mt('Include communities set to be hidden from catalog (domain staff only)');
         } else {          } else {
             $details_text = &mt('Show full details for each course ([_1] only)',$dc_title);              $details_text = &mt('Show full details for each course (domain staff only)');
             $hidden_text = &mt('Include courses set to be hidden from catalog ([_1] only)',$dc_title);              $hidden_text = &mt('Include courses set to be hidden from catalog (domain staff only)');
           }
           if ($is_dc) {
               $output .= '<span class="LC_nobreak">'.
                          '<label><input type="checkbox" name="showhidden" value="1" '.
                          $showhidden_status.'/>'.$hidden_text.
                          '</label></span><br />'."\n";
         }          }
         $output .= '<span class="LC_nobreak">'.          $output .= '<span class="LC_nobreak">'.
                    '<label><input type="checkbox" name="showhidden" value="1" '.  
                    $showhidden_status.'/>'.$hidden_text.  
                    '</label></span><br />'."\n".  
                    '<span class="LC_nobreak">'.  
                    '<label><input type="checkbox" name="showdetails" value="1" '.                     '<label><input type="checkbox" name="showdetails" value="1" '.
                    $showdetails_status.'onclick="toggleStatuses();" />'.                     $showdetails_status.'onclick="toggleStatuses();" />'.
                    $details_text.'</label></span></td>'."\n".                     $details_text.'</label></span></td>'."\n".
Line 865  sub user_is_dc { Line 873  sub user_is_dc {
     return;      return;
 }  }
   
   sub canview_all {
       my ($knownuser,$codedom) = @_;
       my $canviewall = 0;
       my $page = 'coursecatalog';
       if (&LONCAPA::lonauthcgi::can_view($page)) {
           $canviewall = 1;
       } elsif (&LONCAPA::lonauthcgi::check_ipbased_access($page)) {
           $canviewall= 1;
       } elsif (($knownuser) && ($codedom ne '')) {
           if (&Apache::lonnet::allowed('dcd',$codedom)) {
               $canviewall = 1;
           }
       }
       return $canviewall;
   }
   
 sub get_statustitles {  sub get_statustitles {
     my ($caller) = @_;      my ($caller) = @_;
     my @status_order = ('Active','Future','Previous');      my @status_order = ('Active','Future','Previous');
Line 958  sub search_courselist { Line 982  sub search_courselist {
 }  }
   
 sub print_course_listing {  sub print_course_listing {
     my ($domain,$numtitles,$trails,$allitems,$subcats,$codetitles) = @_;      my ($domain,$numtitles,$trails,$allitems,$subcats,$codetitles,$canviewall) = @_;
     my $output;      my $output;
     my %courses;      my %courses;
     my $knownuser = &user_is_known();      my $knownuser = &user_is_known();
       my $canviewall = &canview_all();
     my $details = $env{'form.coursenum'};      my $details = $env{'form.coursenum'};
     if (&user_is_dc($domain)) {      if (&user_is_dc($domain) || $canviewall) {
         if ($env{'form.showdetails'}) {          if ($env{'form.showdetails'}) {
             $details = 1;              $details = 1;
         }          }
Line 998  sub print_course_listing { Line 1023  sub print_course_listing {
             $output .= '</p>';              $output .= '</p>';
             return $output;              return $output;
         }          }
         if (($knownuser) && (!$env{'form.showdetails'}) && (!&user_is_dc($domain))) {          if (($knownuser) && (!$env{'form.showdetails'}) && (!&user_is_dc($domain)) && (!$canviewall)) {
             $output = '<b>'.&mt('Note for students:').'</b> '              $output = '<b>'.&mt('Note for students:').'</b> '
                      .&mt('If you are officially enrolled in a course but the course is not listed in your LON-CAPA courses, click the "Show more details" link for the specific course and check the default access dates and/or automated enrollment settings.')                       .&mt('If you are officially enrolled in a course but the course is not listed in your LON-CAPA courses, click the "Show more details" link for the specific course and check the default access dates and/or automated enrollment settings.')
                      .'<br /><br />';                       .'<br /><br />';
Line 1006  sub print_course_listing { Line 1031  sub print_course_listing {
     }      }
     my $now = time;      my $now = time;
     $output .= &construct_data_table($knownuser,$domain,\%courses,$details,undef,      $output .= &construct_data_table($knownuser,$domain,\%courses,$details,undef,
                                      $now,$trails,$allitems);                                       $now,$trails,$allitems,$canviewall);
     $output .= "\n".'<form name="linklaunch" method="post" action="">'.      $output .= "\n".'<form name="linklaunch" method="post" action="">'.
                '<input type="hidden" name="backto" value="coursecatalog" />'.                 '<input type="hidden" name="backto" value="coursecatalog" />'.
                '<input type="hidden" name="courseid" value="" />'.                 '<input type="hidden" name="courseid" value="" />'.
Line 1015  sub print_course_listing { Line 1040  sub print_course_listing {
 }  }
   
 sub construct_data_table {  sub construct_data_table {
     my ($knownuser,$domain,$courses,$details,$usersections,$now,$trails,$allitems) = @_;      my ($knownuser,$domain,$courses,$details,$usersections,$now,$trails,$allitems,$canviewall) = @_;
     my %sortname;      my %sortname;
     if (($details eq '') || ($env{'form.showdetails'})) {      if (($details eq '') || ($env{'form.showdetails'})) {
         $sortname{'Code'} = 'code';          $sortname{'Code'} = 'code';
Line 1047  sub construct_data_table { Line 1072  sub construct_data_table {
         $output .= '</th>';          $output .= '</th>';
     }      }
     my (@fields,%fieldtitles,$wasactiveon);      my (@fields,%fieldtitles,$wasactiveon);
     if ($knownuser) {      if ($knownuser || ($canviewall && $details)) {
         if ($details) {          if ($details) {
             if ($env{'form.currcat_0'} eq 'communities::0') {              if ($env{'form.currcat_0'} eq 'communities::0') {
                 $output .= '<th>'.&mt('Default Access Dates for Members').'</th>'.                  $output .= '<th>'.&mt('Default Access Dates for Members').'</th>'.
Line 1060  sub construct_data_table { Line 1085  sub construct_data_table {
             }              }
             my ($titlesref,$orderref) = &get_statustitles();              my ($titlesref,$orderref) = &get_statustitles();
             my @statuses;              my @statuses;
             if (&user_is_dc($domain)) {              if (&user_is_dc($domain) || $canviewall) {
                 @statuses = &Apache::loncommon::get_env_multiple('form.showcounts');                  @statuses = &Apache::loncommon::get_env_multiple('form.showcounts');
                 if (grep(/^Previous$/,@statuses)) {                  if (grep(/^Previous$/,@statuses)) {
                     if ($env{'form.wasactive'} eq 'date') {                       if ($env{'form.wasactive'} eq 'date') { 
Line 1107  sub construct_data_table { Line 1132  sub construct_data_table {
             $deftextbook = $domdefaults{'textbookcredits'};              $deftextbook = $domdefaults{'textbookcredits'};
         }          }
     }      }
     my %courseinfo = &build_courseinfo_hash($courses,$knownuser,$domain,$details,      my %courseinfo = &build_courseinfo_hash($courses,$knownuser,$domain,$canviewall,$details,
                                             $usersections,\@fields,\%fieldtitles,                                              $usersections,\@fields,\%fieldtitles,
                                             $wasactiveon,\%numbers,\%creditsum,                                              $wasactiveon,\%numbers,\%creditsum,
                                             $showcredits,$defofficial,$defunofficial,$deftextbook);                                              $showcredits,$defofficial,$defunofficial,$deftextbook);
Line 1141  sub construct_data_table { Line 1166  sub construct_data_table {
         foreach my $course (@{$Sortby{$item}}) {          foreach my $course (@{$Sortby{$item}}) {
             $output.=&Apache::loncommon::start_data_table_row();               $output.=&Apache::loncommon::start_data_table_row(); 
             $output.=&courseinfo_row($courseinfo{$course},$knownuser,$details,              $output.=&courseinfo_row($courseinfo{$course},$knownuser,$details,
                                      \$count,$now,$course,$trails,$allitems,\%numbers);                                       \$count,$now,$course,$trails,$allitems,\%numbers,$canviewall);
             $output.=&Apache::loncommon::end_data_table_row();              $output.=&Apache::loncommon::end_data_table_row();
         }          }
     }      }
     if (($knownuser) && ($count > 1) && $env{'form.showdetails'}) {      if (($knownuser || $canviewall) && ($count > 1) && $env{'form.showdetails'}) {
         if (&user_is_dc($domain)) {          if (&user_is_dc($domain) || $canviewall) {
             my %lt = &Apache::lonlocal::texthash (              my %lt = &Apache::lonlocal::texthash (
                                                      'Active'   => 'Total current students',                                                       'Active'   => 'Total current students',
                                                      'Future'   => 'Total future students',                                                       'Future'   => 'Total future students',
Line 1205  sub construct_data_table { Line 1230  sub construct_data_table {
 }  }
   
 sub build_courseinfo_hash {  sub build_courseinfo_hash {
     my ($courses,$knownuser,$domain,$details,$usersections,$fields,$fieldtitles,      my ($courses,$knownuser,$domain,$canviewall,$details,$usersections,$fields,$fieldtitles,
         $wasactiveon,$numbers,$creditsum,$showcredits,$defofficial,$defunofficial) = @_;          $wasactiveon,$numbers,$creditsum,$showcredits,$defofficial,$defunofficial) = @_;
     my %courseinfo;      my %courseinfo;
     my $now = time;      my $now = time;
     my $gettotals;      my $gettotals;
     if ((keys(%{$courses}) > 0) && (&user_is_dc($domain)) && ($details)) {      if ((keys(%{$courses}) > 0) && (&user_is_dc($domain) || $canviewall) && ($details)) {
         $gettotals = 1;          $gettotals = 1;
     }      }
     my (%uniquecodes,$nocodes,$defcreds);      my (%uniquecodes,$nocodes,$defcreds);
Line 1306  sub build_courseinfo_hash { Line 1331  sub build_courseinfo_hash {
             $defcreds = $courseinfo{$course}{'defaultcredits'};              $defcreds = $courseinfo{$course}{'defaultcredits'};
         }          }
         if (((defined($env{'form.coursenum'}) && ($cnum eq $env{'form.coursenum'}))) ||          if (((defined($env{'form.coursenum'}) && ($cnum eq $env{'form.coursenum'}))) ||
             ($knownuser && ($details == 1))) {              (($knownuser || $canviewall) && ($details == 1))) {
             my $milestone;              my $milestone;
             if ($wasactiveon eq 'accessend') {              if ($wasactiveon eq 'accessend') {
                 if ($coursehash{'default_enrollment_end_date'}) {                  if ($coursehash{'default_enrollment_end_date'}) {
Line 1459  sub count_students { Line 1484  sub count_students {
 }  }
   
 sub courseinfo_row {  sub courseinfo_row {
     my ($info,$knownuser,$details,$countref,$now,$course,$trails,$allitems,$numbers) = @_;      my ($info,$knownuser,$details,$countref,$now,$course,$trails,$allitems,$numbers,$canviewall) = @_;
     my ($cdom,$cnum,$title,$ownerlast,$code,$owner,$seclist,$xlist_items,      my ($cdom,$cnum,$title,$ownerlast,$code,$owner,$seclist,$xlist_items,
         $accessdates,$showsyllabus,$counts,$autoenrollment,$output,$categories);          $accessdates,$showsyllabus,$counts,$autoenrollment,$output,$categories);
     if (ref($info) eq 'HASH') {      if (ref($info) eq 'HASH') {
Line 1508  sub courseinfo_row { Line 1533  sub courseinfo_row {
     }      }
     $output .= '</font></td>'.      $output .= '</font></td>'.
                '<td>'.$ownerlast.'</td>';                 '<td>'.$ownerlast.'</td>';
     if ($knownuser) {      if (($knownuser) || ($canviewall && $details)) {
         if ($details) {          if ($details) {
             $output .=              $output .=
                 '<td>'.$accessdates.'</td>'.                  '<td>'.$accessdates.'</td>'.
Line 1582  sub identify_sections { Line 1607  sub identify_sections {
 sub get_valid_classes {  sub get_valid_classes {
     my ($seclist,$xlist_items,$crscode,$owners,$cdom,$cnum) = @_;      my ($seclist,$xlist_items,$crscode,$owners,$cdom,$cnum) = @_;
     my $response;      my $response;
     my %validations;      my (@sections,@xlists,%possclasses,%okclasses,%validations);
     @{$validations{'sections'}} = ();      @{$validations{'sections'}} = ();
     @{$validations{'xlists'}} = ();      @{$validations{'xlists'}} = ();
     my $totalitems = 0;      my $totalitems = 0;
     if ($seclist) {      if ($seclist) {
         foreach my $sec (split(/, /,$seclist)) {          @sections = split(/,\s+/,$seclist);
           map { $possclasses{$crscode.$_} = 1; } @sections;
       }
       if ($xlist_items) {
           @xlists = split(/,\s+/,$xlist_items);
           map { $possclasses{$_} = 1; } @xlists;
       }
       my %okclasses = &Apache::lonnet::auto_validate_instclasses($cdom,$cnum,$owners,
                                                                  \%possclasses);
       if (keys(%okclasses)) {
           foreach my $sec (@sections) {
             my $class = $crscode.$sec;              my $class = $crscode.$sec;
             if (&Apache::lonnet::auto_validate_class_sec($cdom,$cnum,$owners,              if ($okclasses{$crscode.$sec}) {
  $class) eq 'ok') {  
                 if (!grep(/^\Q$sec$\E/,@{$validations{'sections'}})) {                  if (!grep(/^\Q$sec$\E/,@{$validations{'sections'}})) {
                     push(@{$validations{'sections'}},$sec);                      push(@{$validations{'sections'}},$sec);
                     $totalitems ++;                      $totalitems ++;
                 }                  }
             }              }
         }          }
     }          foreach my $item (@xlists) {
     if ($xlist_items) {              if ($okclasses{$item}) {
         foreach my $item (split(/, /,$xlist_items)) {                  if (!grep(/^\Q$item\E$/,@{$validations{'xlists'}})) {
             if (&Apache::lonnet::auto_validate_class_sec($cdom,$cnum,$owners,  
  $item) eq 'ok') {  
                 if (!grep(/^\Q$item$\E/,@{$validations{'xlists'}})) {  
                     push(@{$validations{'xlists'}},$item);                      push(@{$validations{'xlists'}},$item);
                     $totalitems ++;                      $totalitems ++;
                 }                  }

Removed from v.1.83  
changed lines
  Added in v.1.88.2.2


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