Diff for /loncom/interface/lonrequestcourse.pm between versions 1.95.2.4 and 1.99

version 1.95.2.4, 2019/07/26 17:57:12 version 1.99, 2016/09/05 01:46:08
Line 119  use Apache::loncoursequeueadmin; Line 119  use Apache::loncoursequeueadmin;
 use Apache::lonuserutils;  use Apache::lonuserutils;
 use LONCAPA qw(:DEFAULT :match);  use LONCAPA qw(:DEFAULT :match);
   
 my $registered_flush;  
 my $registered_instcats;  
 my $modified_dom;  
   
 sub handler {  sub handler {
     my ($r) = @_;      my ($r) = @_;
     &Apache::loncommon::content_type($r,'text/html');      &Apache::loncommon::content_type($r,'text/html');
Line 131  sub handler { Line 127  sub handler {
         return OK;          return OK;
     }      }
   
     $registered_flush = 0;  
     $registered_instcats = 0;  
     $modified_dom = '';  
   
     &Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'},      &Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'},
         ['action','showdom','cnum','state','crstype','queue','tabs']);          ['action','showdom','cnum','state','crstype','queue','tabs']);
     &Apache::lonhtmlcommon::clear_breadcrumbs();      &Apache::lonhtmlcommon::clear_breadcrumbs();
Line 768  function check_can_request(crschoice,act Line 760  function check_can_request(crschoice,act
     var unofficial = '';      var unofficial = '';
     var community = '';      var community = '';
     var textbook = '';      var textbook = '';
       var placement = '';
 END  END
     if (ref($can_request) eq 'HASH') {      if (ref($can_request) eq 'HASH') {
         foreach my $item (keys(%{$can_request})) {          foreach my $item (keys(%{$can_request})) {
Line 781  END Line 774  END
         unofficial => 'You are not permitted to request creation of an unofficial course in this domain.',          unofficial => 'You are not permitted to request creation of an unofficial course in this domain.',
         community => 'You are not permitted to request creation of a community in this domain.',          community => 'You are not permitted to request creation of a community in this domain.',
         textbook => 'You are not permitted to request creation of a textbook course in this domain',          textbook => 'You are not permitted to request creation of a textbook course in this domain',
           placement => 'You are not permitted to request creation of a placement test in this domain',
         all => 'You must choose a specific course type when making a new course request.',          all => 'You must choose a specific course type when making a new course request.',
         allt => '"All types" is not allowed.',          allt => '"All types" is not allowed.',
     );       ); 
Line 810  END Line 804  END
                         return false;                          return false;
                     }                      }
                 } else {                  } else {
                     if (actionchoice == 'new') {                      if (crschoice == 'placement') {
                         alert('$js_lt{'all'}'+'\\n'+'$js_lt{'allt'}');                          if (placement != 1) {
                         return false;                              alert("$js_lt{'placement'}");
                               return false;
                           }
                       } else {
                           if (actionchoice == 'new') {
                               alert('$js_lt{'all'}'+'\\n'+'$js_lt{'allt'}');
                               return false;
                           }
                     }                      }
                 }                  }
             }              }
Line 823  END Line 824  END
 END  END
     my ($pagetitle,$pageinfo,$domaintitle,$earlyout);      my ($pagetitle,$pageinfo,$domaintitle,$earlyout);
     if (ref($can_request) eq 'HASH') {      if (ref($can_request) eq 'HASH') {
         if (($can_request->{'official'}) || ($can_request->{'unofficial'}) || $can_request->{'textbook'}) {          if (($can_request->{'official'}) || ($can_request->{'unofficial'}) || ($can_request->{'textbook'}) || ($can_request->{'placement'})) {
             if ($can_request->{'community'}) {              if ($can_request->{'community'}) {
                 $pagetitle = 'Course/Community Requests';                  $pagetitle = 'Course/Community Requests';
                 $pageinfo = &mt('Request creation of a new course or community, or review your pending requests.');                  $pageinfo = &mt('Request creation of a new course or community, or review your pending requests.');
Line 1000  END Line 1001  END
                 $title = &mt('Pending requests for unofficial courses');                  $title = &mt('Pending requests for unofficial courses');
             } elsif ($env{'form.crstype'} eq 'textbook') {              } elsif ($env{'form.crstype'} eq 'textbook') {
                 $title = &mt('Pending requests for textbook courses');                  $title = &mt('Pending requests for textbook courses');
               } elsif ($env{'form.crstype'} eq 'textbook') {
                   $title = &mt('Pending requests for placement tests'); 
             } else {              } else {
                 $title = &mt('Pending course/community requests');                   $title = &mt('Pending course/community requests'); 
             }              }
Line 2101  sub print_personnel_menu { Line 2104  sub print_personnel_menu {
             official => 'Requestor is automatically assigned Course Coordinator role.',              official => 'Requestor is automatically assigned Course Coordinator role.',
         );          );
         $lt{'unofficial'} = $lt{'official'};          $lt{'unofficial'} = $lt{'official'};
         $lt{'textbook'} = $lt{'textbook'};          $lt{'textbook'} = $lt{'official'};
           $lt{'placement'} = $lt{'official'};
         $output .= &Apache::lonhtmlcommon::row_headline().          $output .= &Apache::lonhtmlcommon::row_headline().
                   '<h3>'.&Apache::loncommon::help_open_topic('Course_Request_Personnel').'&nbsp;'.$lt{$crstype}.' '.&mt('Include other personnel?').'</h3>';                    '<h3>'.&Apache::loncommon::help_open_topic('Course_Request_Personnel').'&nbsp;'.$lt{$crstype}.' '.&mt('Include other personnel?').'</h3>';
     }      }
Line 2529  sub print_request_logs { Line 2533  sub print_request_logs {
         if (ref($domconfig{'requestcourses'}) eq 'HASH') {          if (ref($domconfig{'requestcourses'}) eq 'HASH') {
             if (ref($domconfig{'requestcourses'}{'uniquecode'}) eq 'HASH') {              if (ref($domconfig{'requestcourses'}{'uniquecode'}) eq 'HASH') {
                 if ($curr{'crstype'} eq 'any') {                  if ($curr{'crstype'} eq 'any') {
                     my @types = qw(official unofficial community textbook);                      my @types = qw(official unofficial community textbook placement);
                     foreach my $type (@types) {                      foreach my $type (@types) {
                         if ($domconfig{'requestcourses'}{'uniquecode'}{$type}) {                          if ($domconfig{'requestcourses'}{'uniquecode'}{$type}) {
                             $showuniquecode = 1;                              $showuniquecode = 1;
Line 2697  sub reqstatus_names { Line 2701  sub reqstatus_names {
                         rejected  => 'Request rejected',                          rejected  => 'Request rejected',
                         cancelled => 'Request cancelled',                          cancelled => 'Request cancelled',
             );              );
     if (($crstype eq 'official') || ($crstype eq 'unofficial') || ($crstype eq 'textbook')) {      if (($crstype eq 'official') || ($crstype eq 'unofficial') || ($crstype eq 'textbook') || ($crstype eq 'placement')) {
         $statusnames{'created'} = &mt('Course created');          $statusnames{'created'} = &mt('Course created');
     } elsif ($crstype eq 'community') {      } elsif ($crstype eq 'community') {
         $statusnames{'created'} = &mt('Community created');          $statusnames{'created'} = &mt('Community created');
Line 3101  sub courseinfo_form { Line 3105  sub courseinfo_form {
     &js_escape(\%js_lt);      &js_escape(\%js_lt);
     $js_lt{'unofficial'} = $js_lt{'official'};      $js_lt{'unofficial'} = $js_lt{'official'};
     $js_lt{'textbook'} = $js_lt{'official'};      $js_lt{'textbook'} = $js_lt{'official'};
       $js_lt{'placement'} = $js_lt{'official'};
     my $js_validate = <<"ENDJS";      my $js_validate = <<"ENDJS";
 <script type="text/javascript">  <script type="text/javascript">
 // <![CDATA['  // <![CDATA['
Line 3180  sub clone_form { Line 3185  sub clone_form {
     my $type = 'Course';      my $type = 'Course';
     if ($crstype eq 'community') {      if ($crstype eq 'community') {
         $type = 'Community';          $type = 'Community';
       } elsif ($crstype eq 'placement') {
           $type = 'Placement'; 
     }      }
     my %lt = &clone_text();      my %lt = &clone_text();
     my $output .=       my $output .= 
Line 3669  sub print_request_outcome { Line 3676  sub print_request_outcome {
                                                         $instcode,$req_notifylist,\@instsections,\%domconfig);                                                          $instcode,$req_notifylist,\@instsections,\%domconfig);
     return ($result,$output,$customized);      return ($result,$output,$customized);
 }  }
       
 sub process_request {  sub process_request {
     my ($r,$lonhost,$dom,$cnum,$crstype,$now,$details,$instcode,$req_notifylist,$instsections,      my ($r,$lonhost,$dom,$cnum,$crstype,$now,$details,$instcode,$req_notifylist,$instsections,
         $domconfig) = @_;           $domconfig) = @_; 
Line 3687  sub process_request { Line 3694  sub process_request {
             $output = &mt('You are not permitted to request creation of communities');              $output = &mt('You are not permitted to request creation of communities');
         } elsif ($crstype eq 'textbook') {          } elsif ($crstype eq 'textbook') {
             $output = &mt('You are not permitted to request creation of textbook courses');              $output = &mt('You are not permitted to request creation of textbook courses');
           } elsif ($crstype eq 'placement') {
               $output = &mt('You are not permitted to request creation of placement tests');
         } else {          } else {
             $output = &mt('Unrecognized course type: [_1]',$crstype);              $output = &mt('Unrecognized course type: [_1]',$crstype);
         }          }
Line 3854  sub process_request { Line 3863  sub process_request {
                 }                  }
                 $output .= '</p>';                  $output .= '</p>';
                 $creationresult = 'created';                  $creationresult = 'created';
                 # Flush the course logs so reverse user roles immediately updated  
                 unless ($registered_flush) {  
                     my $handlers = $r->get_handlers('PerlCleanupHandler');  
                     $r->set_handlers('PerlCleanupHandler' => [\&Apache::lonnet::flushcourselogs,@{$handlers}]);  
                     $registered_flush=1;  
                 }  
                 if ($instcode ne '') {  
                     &Apache::lonnet::devalidate_cache_new('instcats',$dom);  
                     # Update cache of self-cataloging courses on institution's server(s).  
                     if (&Apache::lonnet::shared_institution($dom)) {  
                         unless ($registered_instcats) {  
                             my $handlers = $r->get_handlers('PerlCleanupHandler');  
                             $r->set_handlers('PerlCleanupHandler' => [\&devalidate_remote_instcats,@{$handlers}]);  
                             $registered_instcats=1;  
                             $modified_dom = $dom;  
                         }  
                     }  
                 }  
             } else {              } else {
                 $output = '<span class="LC_error">';                  $output = '<span class="LC_error">';
                 if ($crstype eq 'community') {                  if ($crstype eq 'community') {
Line 4007  sub process_request { Line 3998  sub process_request {
     }      }
 }  }
   
 sub devalidate_remote_instcats {  
     if ($modified_dom ne '') {  
         my %servers = &Apache::lonnet::internet_dom_servers($modified_dom);  
         my %thismachine;  
         map { $thismachine{$_} = 1; } &Apache::lonnet::current_machine_ids();  
         if (keys(%servers)) {  
             foreach my $server (keys(%servers)) {  
                 next if ($thismachine{$server});  
                 &Apache::lonnet::remote_devalidate_cache($server,['instcats:'.$modified_dom]);  
             }  
         }  
         $modified_dom = '';  
     }  
     return;  
 }  
   
 sub custom_formitems {  sub custom_formitems {
     my ($preprocess,$customhash) = @_;      my ($preprocess,$customhash) = @_;
     return unless ((ref($preprocess) eq 'HASH') && (ref($customhash) eq 'HASH'));        return unless ((ref($preprocess) eq 'HASH') && (ref($customhash) eq 'HASH'));  
Line 4331  sub pending_validation_form { Line 4306  sub pending_validation_form {
                         $buttontext = &mt('Create course');                          $buttontext = &mt('Create course');
                     }                      }
                 }                  }
                 my $hostname = &Apache::lonnet::hostname($lonhost);  
                 my $protocol = $Apache::lonnet::protocol{$lonhost};                  my $protocol = $Apache::lonnet::protocol{$lonhost};
                 $protocol = 'http' if ($protocol ne 'https');                  $protocol = 'http' if ($protocol ne 'https');
                 my $crscreator = $protocol.'://'.$hostname.'/cgi-bin/createpending.pl';                  my $crscreator = $protocol.'://'.&Apache::lonnet::hostname($lonhost).'/cgi-bin/createpending.pl';
                 $output .= '<input type="hidden" name="crscreator" value="'.$crscreator.'" />'."\n".                  $output .= '<input type="hidden" name="crscreator" value="'.$crscreator.'" />'."\n".
                            '<input type="hidden" name="token" value="'.$token.'" />'."\n".                             '<input type="hidden" name="token" value="'.$token.'" />'."\n".
                            '<input type="submit" name="validate" value="'.$buttontext.'" />'."\n".                             '<input type="submit" name="validate" value="'.$buttontext.'" />'."\n".
Line 4359  sub check_autolimit { Line 4333  sub check_autolimit {
             if (($crstype eq 'community') &&               if (($crstype eq 'community') && 
                 (exists($crsroles{$cnum.':'.$cdom.':co'}))) {                  (exists($crsroles{$cnum.':'.$cdom.':co'}))) {
                 $count ++;                  $count ++;
             } elsif ((($crstype eq 'official') || ($crstype eq 'unofficial') || ($crstype eq 'textbook')) &&              } elsif ((($crstype eq 'official') || ($crstype eq 'unofficial') || ($crstype eq 'textbook') || ($crstype eq 'placement')) &&
                      (exists($crsroles{$cnum.':'.$cdom.':cc'}))) {                       (exists($crsroles{$cnum.':'.$cdom.':cc'}))) {
                 $count ++;                  $count ++;
             }              }
Line 5075  sub process_textbook_request { Line 5049  sub process_textbook_request {
             } else {              } else {
                 $details->{dateshift} = '';                  $details->{dateshift} = '';
             }              }
         } elsif (($reqtype eq 'textbook') || ($reqtype eq 'template')) {  
             $details->{datemode} = 'delete';  
             $details->{dateshift} = '';  
         }          }
         if ($details->{dateshift} ne '') {          if ($details->{dateshift} ne '') {
             $details->{dateshift} =~ s/[^\d\.]+//g;              $details->{dateshift} =~ s/[^\d\.]+//g;

Removed from v.1.95.2.4  
changed lines
  Added in v.1.99


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