Diff for /loncom/interface/lonmodifycourse.pm between versions 1.79.2.7 and 1.89

version 1.79.2.7, 2019/07/26 17:43:54 version 1.89, 2016/11/10 21:42:19
Line 39  use Apache::lonpickcourse; Line 39  use Apache::lonpickcourse;
 use lib '/home/httpd/lib/perl';  use lib '/home/httpd/lib/perl';
 use LONCAPA qw(:DEFAULT :match);  use LONCAPA qw(:DEFAULT :match);
   
 my $registered_cleanup;  
 my $modified_dom;  
   
 sub get_dc_settable {  sub get_dc_settable {
     my ($type,$cdom) = @_;      my ($type,$cdom) = @_;
     if ($type eq 'Community') {      if ($type eq 'Community') {
Line 75  sub catalog_settable { Line 72  sub catalog_settable {
             if ($confhash->{'categorizecomm'} ne 'comm') {              if ($confhash->{'categorizecomm'} ne 'comm') {
                 push(@settable,'categorize');                  push(@settable,'categorize');
             }              }
           } elsif ($type eq 'Placement') {
               if ($confhash->{'togglecatsplace'} ne 'place') {
                   push(@settable,'togglecats');
               }
               if ($confhash->{'categorizeplace'} ne 'place') {
                   push(@settable,'categorize');
               }
         } else {          } else {
             if ($confhash->{'togglecats'} ne 'crs') {              if ($confhash->{'togglecats'} ne 'crs') {
                 push(@settable,'togglecats');                  push(@settable,'togglecats');
Line 170  sub print_course_search_page { Line 174  sub print_course_search_page {
     $r->print(&Apache::loncommon::js_changer());      $r->print(&Apache::loncommon::js_changer());
     if ($type eq 'Community') {      if ($type eq 'Community') {
         $r->print('<h3>'.&mt('Search for a community in the [_1] domain',$domdesc).'</h3>');          $r->print('<h3>'.&mt('Search for a community in the [_1] domain',$domdesc).'</h3>');
       } elsif ($type eq 'Placement') {
           $r->print('<h3>'.&mt('Search for a placement test in the [_1] domain',$domdesc).'</h3>');
     } else {      } else {
         $r->print('<h3>'.&mt('Search for a course in the [_1] domain',$domdesc).'</h3>');          $r->print('<h3>'.&mt('Search for a course in the [_1] domain',$domdesc).'</h3>');
     }      }
Line 180  sub print_course_search_page { Line 186  sub print_course_search_page {
     my ($actiontext,$roleoption,$settingsoption);      my ($actiontext,$roleoption,$settingsoption);
     if ($type eq 'Community') {      if ($type eq 'Community') {
         $actiontext = &mt('Actions available after searching for a community:');          $actiontext = &mt('Actions available after searching for a community:');
       } elsif ($type eq 'Placement') {
           $actiontext = &mt('Actions available after searching for a placement test:')
     } else {      } else {
         $actiontext = &mt('Actions available after searching for a course:');          $actiontext = &mt('Actions available after searching for a course:');
     }      }
Line 187  sub print_course_search_page { Line 195  sub print_course_search_page {
        if ($type eq 'Community') {         if ($type eq 'Community') {
            $roleoption = &mt('Enter the community with the role of [_1]',$cctitle);             $roleoption = &mt('Enter the community with the role of [_1]',$cctitle);
            $settingsoption = &mt('View or modify community settings which only a [_1] may modify.',$dctitle);             $settingsoption = &mt('View or modify community settings which only a [_1] may modify.',$dctitle);
          } elsif ($type eq 'Placement') {
              $roleoption = &mt('Enter the placement test with the role of [_1]',$cctitle);
              $settingsoption = &mt('View or modify placement test settings which only a [_1] may modify.',$dctitle);
        } else {         } else {
            $roleoption = &mt('Enter the course with the role of [_1]',$cctitle);             $roleoption = &mt('Enter the course with the role of [_1]',$cctitle);
            $settingsoption = &mt('View or modify course settings which only a [_1] may modify.',$dctitle);             $settingsoption = &mt('View or modify course settings which only a [_1] may modify.',$dctitle);
        }         }
     } elsif (&Apache::lonnet::allowed('rar',$dom)) {      } elsif (&Apache::lonnet::allowed('rar',$dom)) {
         my ($roles_by_num,$description,$accessref,$accessinfo) = &Apache::lonnet::get_all_adhocroles($dom);          my %adhocroles = &Apache::lonnet::userenvironment($env{'user.domain'},$env{'user.name'},
         if ((ref($roles_by_num) eq 'ARRAY') && (ref($description) eq 'HASH')) {                                                           'adhocroles.'.$dom);
             if (@{$roles_by_num} > 1) {          if (keys(%adhocroles)) {
               my @adhoc = split(',',$adhocroles{'adhocroles.'.$dom});
               if (@adhoc > 1) {
                 if ($type eq 'Community') {                  if ($type eq 'Community') {
                     $roleoption = &mt('Enter the community with one of the available ad hoc roles');                      $roleoption = &mt('Enter the community with one of the available ad hoc roles: [_1].',
                                     join(', ',@adhoc));
                   } elsif ($type eq 'Placement') {
                       $roleoption = &mt('Enter the placement test with one of the available ad hoc roles: [_1].',
                                     join(', ',@adhoc));
                 } else {                  } else {
                     $roleoption = &mt('Enter the course with one of the available ad hoc roles.');                      $roleoption = &mt('Enter the course with one of the available ad hoc roles: [_1].',
                                     join(', ',@adhoc));
                 }                  }
             } else {              } else {
                 my $rolename = $description->{$roles_by_num->[0]};  
                 if ($type eq 'Community') {                  if ($type eq 'Community') {
                     $roleoption = &mt('Enter the community with the ad hoc role of: [_1]',$rolename);                      $roleoption = &mt('Enter the community with the ad hoc role of: [_1]',$adhoc[0]);
                   } elsif ($type eq 'Placement') {
                       $roleoption = &mt('Enter the placement test with the ad hoc role of: [_1]',$adhoc[0]);
                 } else {                  } else {
                     $roleoption = &mt('Enter the course with the ad hoc role of: [_1]',$rolename);                      $roleoption = &mt('Enter the course with the ad hoc role of: [_1]',$adhoc[0]);
                 }                  }
             }              }
         }          }
         if ($type eq 'Community') {          if ($type eq 'Community') {
             $settingsoption = &mt('View community settings which only a [_1] may modify.',$dctitle);              $settingsoption = &mt('View community settings which only a [_1] may modify.',$dctitle);
           } elsif ($type eq 'Placement') {
               $settingsoption = &mt('View placement test settings which only a [_1] may modify.',$dctitle);
         } else {          } else {
             $settingsoption = &mt('View course settings which only a [_1] may modify.',$dctitle);              $settingsoption = &mt('View course settings which only a [_1] may modify.',$dctitle);
         }          }
Line 224  sub print_course_search_page { Line 245  sub print_course_search_page {
 }  }
   
 sub print_course_selection_page {  sub print_course_selection_page {
     my ($r,$dom,$domdesc,$permission) = @_;      my ($r,$dom,$domdesc) = @_;
     my $type = $env{'form.type'};      my $type = $env{'form.type'};
     if (!defined($type)) {      if (!defined($type)) {
         $type = 'Course';          $type = 'Course';
     }      }
     &print_header($r,$type);      &print_header($r,$type);
   
     if ($permission->{'adhocrole'} eq 'custom') {  # Criteria for course search 
         my %lt = &Apache::lonlocal::texthash(  
             title    => 'Ad hoc role selection',  
             preamble => 'Please choose an ad hoc role in the course.',  
             cancel   => 'Click "OK" to enter the course, or "Cancel" to choose a different course.',  
         );  
         my %jslt = &Apache::lonlocal::texthash (  
             none => 'You are not eligible to use an ad hoc role for the selected course',  
             ok   => 'OK',  
             exit => 'Cancel',  
         );  
         &js_escape(\%jslt);  
         $r->print(<<"END");  
 <script type="text/javascript">  
 // <![CDATA[  
 \$(document).ready(function(){  
     \$( "#LC_adhocrole_chooser" ).dialog({ autoOpen: false });  
 });  
   
 function gochoose(cname,cdom,cdesc) {  
     document.courselist.pickedcourse.value = cdom+'_'+cname;  
     \$("#LC_choose_adhoc").empty();  
     var pickedaction = \$('input[name=phase]:checked', '#LCcoursepicker').val();  
     if (pickedaction == 'adhocrole') {  
         var http = new XMLHttpRequest();  
         var url = "/adm/pickcourse";  
         var params = "cid="+cdom+"_"+cname+"&context=adhoc";  
         http.open("POST", url, true);  
         http.setRequestHeader("Content-type", "application/x-www-form-urlencoded");  
         http.onreadystatechange = function() {  
             if (http.readyState == 4 && http.status == 200) {  
                 var data = \$.parseJSON(http.responseText);  
                 var len = data.roles.length;  
                 if (len == '' || len == null || len == 0) {  
                     alert('$jslt{none}');  
                 } else {  
                     if (len == 1) {  
                         \$( '[name="adhocrole"]' )[0].value = data.roles[0].name;  
                         document.courselist.submit();  
                     } else {  
                         var str = '';  
                         \$("#LC_choose_adhoc").empty();  
                         for (var i=0; i<data.roles.length; i++) {  
                             \$("<label><input type='radio' value='"+data.roles[i].name+"' name='LC_get_role' id='LC_get_role_"+i+"' />"+data.roles[i].desc+"</label><span>&nbsp;&nbsp;</span>")  
                             .appendTo("#LC_choose_adhoc");  
                         }  
                         \$( "#LC_adhocrole_chooser" ).toggle( true );  
                         \$( "#LC_get_role_0").prop("checked", true);  
                         \$( "#LC_adhocrole_chooser" ).dialog({ autoOpen: false });  
                         \$( "#LC_adhocrole_chooser" ).dialog("open");  
                         \$( "#LC_adhocrole_chooser" ).dialog({  
                             height: 400,  
                             width: 500,  
                             modal: true,  
                             resizable: false,  
                             buttons: [  
                                   {  
                                    text: "$jslt{'ok'}",  
                                    click: function() {  
                                             var rolename = \$('input[name=LC_get_role]:checked', '#LChelpdeskpicker').val();  
                                             \$( '[name="adhocrole"]' )[0].value = rolename;  
                                             document.courselist.submit();  
                                         }  
                                   },  
                                   {  
                                    text: "$jslt{'exit'}",  
                                    click: function() {  
                                         \$("#LC_adhocrole_chooser").dialog( "close" );  
                                        }  
                                   }  
                             ],  
                         });  
                         \$( "#LC_adhocrole_chooser" ).find( "form" ).on( "submit", function( event ) {  
                             event.preventDefault();  
                             var rolename = \$('input[name=LC_get_role]:checked', '#LChelpdeskpicker').val()  
                             \$( '[name="adhocrole"]' )[0].value = rolename;  
                             document.courselist.submit();  
                             \$("#LC_adhocrole_chooser").dialog( "close" );  
                         });  
                     }  
                 }  
             }  
         }  
         http.send(params);  
     } else {  
         document.courselist.submit();  
     }  
     return;  
 }  
 // ]]>  
 </script>  
   
 <div id="LC_adhocrole_chooser" title="$lt{'title'}" style="display:none">  
   <p>$lt{'preamble'}</p>  
   <form name="LChelpdeskadhoc" id="LChelpdeskpicker" action="">  
     <div id="LC_choose_adhoc">  
     </div>  
     <input type="hidden" name="adhocrole" id="LCadhocrole" value="" />  
     <input type="submit" tabindex="-1" style="position:absolute; top:-1000px" />  
   </form>  
   <p>$lt{'cancel'}</p>  
 </div>  
 END  
     } elsif ($permission->{'adhocrole'} eq 'coord') {  
         $r->print(<<"END");  
 <script type="text/javascript">  
 // <![CDATA[  
   
 function gochoose(cname,cdom,cdesc) {  
     document.courselist.pickedcourse.value = cdom+'_'+cname;  
     document.courselist.submit();  
     return;  
 }  
   
 // ]]>  
 </script>  
 END  
     }  
   
 # Criteria for course search  
     my ($filterlist,$filter) = &get_filters();      my ($filterlist,$filter) = &get_filters();
     my $action = '/adm/modifycourse';      my $action = '/adm/modifycourse';
     my $dctitle = &Apache::lonnet::plaintext('dc');      my $dctitle = &Apache::lonnet::plaintext('dc');
Line 415  sub print_modification_menu { Line 317  sub print_modification_menu {
                     );                      );
     }      }
     if ($type eq 'Community') {      if ($type eq 'Community') {
         if ($permission->{'setparms'} eq 'edit') {          if ($permission->{'setparms'} eq 'edit') { 
             $categorytitle = 'View/Modify Community Settings';              $categorytitle = 'View/Modify Community Settings';
             $linktext{'setparms'} = 'View/Modify community owner';              $linktext{'setparms'} = 'View/Modify community owner';
             $linktext{'catsettings'} = 'View/Modify catalog settings for community';              $linktext{'catsettings'} = 'View/Modify catalog settings for community';
Line 425  sub print_modification_menu { Line 327  sub print_modification_menu {
             $linktext{'catsettings'} = 'View catalog settings for community';              $linktext{'catsettings'} = 'View catalog settings for community';
         }          }
         $setquota_text = &mt('Total disk space allocated for storage of portfolio files in all groups in a community.');          $setquota_text = &mt('Total disk space allocated for storage of portfolio files in all groups in a community.');
         $setuploadquota_text = &mt('Disk space allocated for storage of content uploaded directly to a community via Content Editor.');          $setuploadquota_text = &mt('Disk space allocated for storage of content uploaded directly to a community via Content Editor.'); 
     } else {      } else {
         if ($permission->{'setparms'} eq 'edit') {          if ($permission->{'setparms'} eq 'edit') {
             $categorytitle = 'View/Modify Course Settings';              $categorytitle = 'View/Modify Course Settings';
Line 510  sub print_modification_menu { Line 412  sub print_modification_menu {
             {              {
                 linktext => $linktext{'viewparms'},                  linktext => $linktext{'viewparms'},
                 url => &phaseurl('viewparms'),                  url => &phaseurl('viewparms'),
                 permission => ($permission->{'viewparms'} && ($type ne 'Community')),                  permission => ($permission->{'viewparms'} && ($type ne 'Community') && ($type ne 'Placement')),
                 #help => '',                  #help => '',
                 icon => 'roles.png',                  icon => 'roles.png',
                 linktitle => ''                  linktitle => ''
Line 552  sub print_modification_menu { Line 454  sub print_modification_menu {
     } else {      } else {
         $menu_html .=  '<li>'.&mt('Course owner (permitted to assign Course Coordinator roles in the course).').'</li>'."\n".          $menu_html .=  '<li>'.&mt('Course owner (permitted to assign Course Coordinator roles in the course).').'</li>'."\n".
                        '<li>'.&mt("Institutional code and default authentication (both required for auto-enrollment of students from institutional datafeeds).").'</li>'."\n";                         '<li>'.&mt("Institutional code and default authentication (both required for auto-enrollment of students from institutional datafeeds).").'</li>'."\n";
         if (&showcredits($dom)) {          if (($type ne 'Placement') && &showcredits($dom)) {
             $menu_html .= '<li>'.&mt('Default credits earned by student on course completion.').'</li>'."\n";              $menu_html .= '<li>'.&mt('Default credits earned by student on course completion.').'</li>'."\n";
         }          }
         $menu_html .= ' <li>'.&mt('Override defaults for who configures self-enrollment for this specific course.').'</li>'."\n";          $menu_html .= ' <li>'.&mt('Override defaults for who configures self-enrollment for this specific course.').'</li>'."\n";
Line 595  sub print_modification_menu { Line 497  sub print_modification_menu {
 }  }
   
 sub print_adhocrole_selected {  sub print_adhocrole_selected {
     my ($r,$type,$permission) = @_;      my ($r,$type) = @_;
     &print_header($r,$type);      &print_header($r,$type);
     my ($cdom,$cnum) = split(/_/,$env{'form.pickedcourse'});      my ($cdom,$cnum) = split(/_/,$env{'form.pickedcourse'});
     my ($newrole,$selectrole);      my ($newrole,$selectrole);
     if ($permission->{'adhocrole'} eq 'coord') {      if (&Apache::lonnet::allowed('ccc',$cdom)) {
         if ($type eq 'Community') {          if ($type eq 'Community') {
             $newrole = "co./$cdom/$cnum";              $newrole = "co./$cdom/$cnum";
         } else {          } else {
             $newrole = "cc./$cdom/$cnum";              $newrole = "cc./$cdom/$cnum";
         }          }
         $selectrole = 1;          $selectrole = 1;
     } elsif ($permission->{'adhocrole'} eq 'custom') {      } elsif (&Apache::lonnet::allowed('rar',$cdom)) {
         my ($okroles,$description) = &Apache::lonnet::get_my_adhocroles($env{'form.pickedcourse'},1);          my %adhocroles = &Apache::lonnet::userenvironment($env{'user.domain'},$env{'user.name'},
         if (ref($okroles) eq 'ARRAY') {                                                           'adhocroles.'.$cdom);
           if (keys(%adhocroles)) {
             my $possrole = $env{'form.adhocrole'};              my $possrole = $env{'form.adhocrole'};
             if (($possrole ne '') && (grep(/^\Q$possrole\E$/,@{$okroles}))) {              if ($possrole ne '') {
                 my $confname = &Apache::lonnet::get_domainconfiguser($cdom);                  my @adhoc = split(',',$adhocroles{'adhocroles.'.$cdom});
                 $newrole = "cr/$cdom/$confname/$possrole./$cdom/$cnum";                  if (grep(/^\Q$possrole\E$/,@adhoc)) {
                 $selectrole = 1;                      my $confname = &Apache::lonnet::get_domainconfiguser($cdom);
                       $newrole = "cr/$cdom/$confname/$possrole./$cdom/$cnum";
                       $selectrole = 1;
                   }
             }              }
         }          }
     }      }
Line 789  sub print_set_anonsurvey_threshold { Line 695  sub print_set_anonsurvey_threshold {
     my $hidden_elements = &hidden_form_elements();      my $hidden_elements = &hidden_form_elements();
     my ($disabled,$submit);      my ($disabled,$submit);
     if ($readonly) {      if ($readonly) {
         $disabled = ' disabled="disabled"';          $disabled = ' disabled="disabled"'; 
     } else {      } else {
         $submit = '<input type="submit" value="'.$lt{'modi'}.'" />';          $submit = '<input type="submit" value="'.$lt{'modi'}.'" />';
     }      }
Line 883  sub domain_postsubtimeout { Line 789  sub domain_postsubtimeout {
     my ($cdom,$type,$settings) = @_;      my ($cdom,$type,$settings) = @_;
     return unless (ref($settings) eq 'HASH');       return unless (ref($settings) eq 'HASH'); 
     my $lctype = lc($type);      my $lctype = lc($type);
     unless ($type eq 'Community') {      unless (($type eq 'Community') || ($type eq 'Placement')) {
         $lctype = 'unofficial';          $lctype = 'unofficial';
         if ($settings->{'internal.coursecode'}) {          if ($settings->{'internal.coursecode'}) {
             $lctype = 'official';              $lctype = 'official';
Line 946  sub print_catsettings { Line 852  sub print_catsettings {
                       '&nbsp;<label><input name="hidefromcat" type="radio" value="yes" '.$excludeon.$disabled.' />'.&mt('Yes').'</label>&nbsp;&nbsp;&nbsp;<label><input name="hidefromcat" type="radio" value="" '.$excludeoff.$disabled.' />'.&mt('No').'</label><br /><p>');                        '&nbsp;<label><input name="hidefromcat" type="radio" value="yes" '.$excludeon.$disabled.' />'.&mt('Yes').'</label>&nbsp;&nbsp;&nbsp;<label><input name="hidefromcat" type="radio" value="" '.$excludeoff.$disabled.' />'.&mt('No').'</label><br /><p>');
             if ($type eq 'Community') {              if ($type eq 'Community') {
                 $r->print(&mt("If a community has been categorized using at least one of the categories defined for communities in the domain, it will be listed in the domain's publicly accessible Course/Community Catalog, unless excluded."));                  $r->print(&mt("If a community has been categorized using at least one of the categories defined for communities in the domain, it will be listed in the domain's publicly accessible Course/Community Catalog, unless excluded."));
               } elsif ($type eq 'Placement') {
                   $r->print(&mt("If a placement test has been categorized using at least one of the categories defined for placement tests in the domain, it will be listed in the domain's publicly accessible Course/Community Catalog, unless excluded."));
             } else {              } else {
                 $r->print(&mt("Unless excluded, a course will be listed in the domain's publicly accessible Course/Community Catalog, if at least one of the following applies").':<ul>'.                  $r->print(&mt("Unless excluded, a course will be listed in the domain's publicly accessible Course/Community Catalog, if at least one of the following applies").':<ul>'.
                           '<li>'.&mt('Auto-cataloging is enabled and the course is assigned an institutional code.').'</li>'.                            '<li>'.&mt('Auto-cataloging is enabled and the course is assigned an institutional code.').'</li>'.
Line 967  sub print_catsettings { Line 875  sub print_catsettings {
             } else {              } else {
                 $r->print(&mt('No categories defined for this domain'));                  $r->print(&mt('No categories defined for this domain'));
             }              }
             unless ($type eq 'Community') {               unless (($type eq 'Community') || ($type eq 'Placement')) { 
                 $r->print('<p>'.&mt('If auto-cataloging based on institutional code is enabled in the domain, a course will continue to be listed in the catalog of official courses, in addition to receiving a listing under any manually assigned categor(ies).').'</p>');                  $r->print('<p>'.&mt('If auto-cataloging based on institutional code is enabled in the domain, a course will continue to be listed in the catalog of official courses, in addition to receiving a listing under any manually assigned categor(ies).').'</p>');
             }              }
         }          }
Line 1066  sub print_course_modification_page { Line 974  sub print_course_modification_page {
         foreach my $cc (@local_ccs) {          foreach my $cc (@local_ccs) {
             $ownertable .= &Apache::loncommon::start_data_table_row()."\n";              $ownertable .= &Apache::loncommon::start_data_table_row()."\n";
             if ($cc eq $enrollvar{'courseowner'}) {              if ($cc eq $enrollvar{'courseowner'}) {
                   $ownertable .= '<td><input type="radio" name="courseowner" value="'.$cc.'" checked="checked"'.$disabled.' /></td>'."\n";                  $ownertable .= '<td><input type="radio" name="courseowner" value="'.$cc.'" checked="checked"'.$disabled.' /></td>'."\n";
             } else {              } else {
                 $ownertable .= '<td><input type="radio" name="courseowner" value="'.$cc.'"'.$disabled.' /></td>'."\n";                  $ownertable .= '<td><input type="radio" name="courseowner" value="'.$cc.'"'.$disabled.' /></td>'."\n";
             }              }
Line 1093  sub print_course_modification_page { Line 1001  sub print_course_modification_page {
         $r->print(&Apache::lonhtmlcommon::row_title(          $r->print(&Apache::lonhtmlcommon::row_title(
                       &Apache::loncommon::help_open_topic('Modify_Course_Instcode').                        &Apache::loncommon::help_open_topic('Modify_Course_Instcode').
                       '&nbsp;'.&mt('Course Code'))."\n".                        '&nbsp;'.&mt('Course Code'))."\n".
                   '<input type="text" size="15" name="coursecode" value="'.$enrollvar{'coursecode'}.'"'.$disabled.' />'.                    '<input type="text" size="15" name="coursecode" value="'.$enrollvar{'coursecode'}.'"'.$disabled.'/>'.
                   &Apache::lonhtmlcommon::row_closure());                    &Apache::lonhtmlcommon::row_closure());
         if (&showcredits($cdom)) {          if (($crstype eq 'Course') && (&showcredits($cdom))) {
             $r->print(&Apache::lonhtmlcommon::row_title(              $r->print(&Apache::lonhtmlcommon::row_title(
                           &Apache::loncommon::help_open_topic('Modify_Course_Credithours').                            &Apache::loncommon::help_open_topic('Modify_Course_Credithours').
                       '&nbsp;'.&mt('Credits (students)'))."\n".                        '&nbsp;'.&mt('Credits (students)'))."\n".
Line 1293  sub modify_course { Line 1201  sub modify_course {
                  'internal.pendingco-owners','internal.selfenrollmgrdc',                   'internal.pendingco-owners','internal.selfenrollmgrdc',
                  'internal.selfenrollmgrcc','internal.mysqltables');                   'internal.selfenrollmgrcc','internal.mysqltables');
     my ($selfenrollrows,$selfenrolltitles) = &Apache::lonuserutils::get_selfenroll_titles();      my ($selfenrollrows,$selfenrolltitles) = &Apache::lonuserutils::get_selfenroll_titles();
     unless ($type eq 'Community') {      unless (($type eq 'Community') || ($type eq 'Placement')) {
         push(@items,('internal.coursecode','internal.authtype','internal.autharg',          push(@items,('internal.coursecode','internal.authtype','internal.autharg',
                      'internal.sectionnums','internal.crosslistings'));                       'internal.sectionnums','internal.crosslistings'));
         if (&showcredits($cdom)) {            if (&showcredits($cdom)) {  
Line 1366  sub modify_course { Line 1274  sub modify_course {
             $newattr{'mysqltables'} = $env{'form.mysqltables'};              $newattr{'mysqltables'} = $env{'form.mysqltables'};
             $newattr{'mysqltables'} =~ s/\D+//g;              $newattr{'mysqltables'} =~ s/\D+//g;
         }          }
         if (&showcredits($cdom) && exists($env{'form.defaultcredits'})) {          if (($type ne 'Placement') && (&showcredits($cdom) && exists($env{'form.defaultcredits'}))) {
             $newattr{'defaultcredits'}=$env{'form.defaultcredits'};              $newattr{'defaultcredits'}=$env{'form.defaultcredits'};
             $newattr{'defaultcredits'} =~ s/[^\d\.]//g;              $newattr{'defaultcredits'} =~ s/[^\d\.]//g;
         }          }
Line 1417  sub modify_course { Line 1325  sub modify_course {
             my $putres = &Apache::lonnet::courseidput($cdom,\%crsinfo,$chome,'notime');              my $putres = &Apache::lonnet::courseidput($cdom,\%crsinfo,$chome,'notime');
             if ($putres eq 'ok') {              if ($putres eq 'ok') {
                 &update_coowners($cdom,$cnum,$chome,\%settings,\%newattr);                  &update_coowners($cdom,$cnum,$chome,\%settings,\%newattr);
                 if ($changed{'code'}) {  
                     &Apache::lonnet::devalidate_cache_new('instcats',$cdom);  
                     # Update cache of self-cataloging courses on institution's server(s).  
                     if (&Apache::lonnet::shared_institution($cdom)) {  
                         unless ($registered_cleanup) {  
                             my $handlers = $r->get_handlers('PerlCleanupHandler');  
                             $r->set_handlers('PerlCleanupHandler' => [\&devalidate_remote_instcats,@{$handlers}]);  
                             $registered_cleanup=1;  
                             $modified_dom = $cdom;  
                         }  
                     }  
                 }  
             }              }
         }          }
     }      }
Line 1490  sub modify_course { Line 1386  sub modify_course {
                     $nochgresponse .= '<li>'.&mt('[_1] still set to: [_2]',$longtype{$attr},$shown).'</li>';                      $nochgresponse .= '<li>'.&mt('[_1] still set to: [_2]',$longtype{$attr},$shown).'</li>';
                 }                  }
             }              }
             if (($type ne 'Community') && ($changed{'code'} || $changed{'owner'})) {              if (($type ne 'Community') && ($type ne 'Placement') && ($changed{'code'} || $changed{'owner'})) {
                 if ( $newattr{'courseowner'} eq '') {                  if ( $newattr{'courseowner'} eq '') {
             push(@warnings,&mt('There is no owner associated with this LON-CAPA course.').              push(@warnings,&mt('There is no owner associated with this LON-CAPA course.').
                                    '<br />'.&mt('If automated enrollment at your institution requires validation of course owners, automated enrollment will fail.'));                                     '<br />'.&mt('If automated enrollment at your institution requires validation of course owners, automated enrollment will fail.'));
Line 2171  function changePage(formname,newphase) { Line 2067  function changePage(formname,newphase) {
     if ($phase eq 'setparms') {      if ($phase eq 'setparms') {
  $js .= $javascript_validations;   $js .= $javascript_validations;
     } elsif ($phase eq 'courselist') {      } elsif ($phase eq 'courselist') {
         $js .= <<"ENDJS";          $js .= qq|
   
   function gochoose(cname,cdom,cdesc) {
       document.courselist.pickedcourse.value = cdom+'_'+cname;
       document.courselist.submit();
   }
   
 function hide_searching() {  function hide_searching() {
     if (document.getElementById('searching')) {      if (document.getElementById('searching')) {
         document.getElementById('searching').style.display = 'none';          document.getElementById('searching').style.display = 'none';
Line 2179  function hide_searching() { Line 2081  function hide_searching() {
     return;      return;
 }  }
   
 ENDJS  |;
     } elsif ($phase eq 'setquota') {      } elsif ($phase eq 'setquota') {
         my $invalid = &mt('The quota you entered contained invalid characters.');          my $invalid = &mt('The quota you entered contained invalid characters.');
         my $alert = &mt('You must enter a number');          my $alert = &mt('You must enter a number');
Line 2323  ENDSCRIPT Line 2225  ENDSCRIPT
     my $bread_text = "View/Modify Courses/Communities";      my $bread_text = "View/Modify Courses/Communities";
     if ($type eq 'Community') {      if ($type eq 'Community') {
         $bread_text = 'Community Settings';          $bread_text = 'Community Settings';
       } elsif ($type eq 'Placement') {
           $bread_text = 'Placement Test Settings';
     } else {      } else {
         $bread_text = 'Course Settings';          $bread_text = 'Course Settings';
     }      }
Line 2443  sub get_permission { Line 2347  sub get_permission {
             catsettings       => 'edit',              catsettings       => 'edit',
             processcat        => 'edit',              processcat        => 'edit',
             selfenroll        => 'edit',              selfenroll        => 'edit',
             adhocrole         => 'coord',  
         );          );
     } elsif (&Apache::lonnet::allowed('rar',$dom)) {      } elsif (&Apache::lonnet::allowed('rar',$dom)) {
         $allowed = 1;          $allowed = 1;
Line 2455  sub get_permission { Line 2358  sub get_permission {
             setparms      => 'view',              setparms      => 'view',
             catsettings   => 'view',              catsettings   => 'view',
             selfenroll    => 'view',              selfenroll    => 'view',
             adhocrole     => 'custom',  
         );          );
     }      }
     return ($allowed,\%permission);      return ($allowed,\%permission);
 }  }
   
 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 handler {  sub handler {
     my $r = shift;      my $r = shift;
     if ($r->header_only) {      if ($r->header_only) {
Line 2485  sub handler { Line 2371  sub handler {
         return OK;          return OK;
     }      }
   
     $registered_cleanup=0;  
     $modified_dom = '';  
   
     my $dom = $env{'request.role.domain'};      my $dom = $env{'request.role.domain'};
     my $domdesc = &Apache::lonnet::domain($dom,'description');      my $domdesc = &Apache::lonnet::domain($dom,'description');
     my ($allowed,$permission) = &get_permission($dom);      my ($allowed,$permission) = &get_permission($dom);
Line 2518  sub handler { Line 2401  sub handler {
             }              }
             if ($type eq 'Community') {              if ($type eq 'Community') {
                 $choose_text = "Choose a community";                  $choose_text = "Choose a community";
               } elsif ($type eq 'Placement') {
                   $choose_text = "Choose a placement test";
             } else {              } else {
                 $choose_text = "Choose a course";                  $choose_text = "Choose a course";
             }               } 
Line 2527  sub handler { Line 2412  sub handler {
               {href=>"javascript:changePage(document.$phase,'courselist')",                {href=>"javascript:changePage(document.$phase,'courselist')",
               text=>$choose_text});                text=>$choose_text});
             if ($phase eq 'courselist') {              if ($phase eq 'courselist') {
                 &print_course_selection_page($r,$dom,$domdesc,$permission);                  &print_course_selection_page($r,$dom,$domdesc);
             } else {              } else {
                 my ($checked,$cdesc,$coursehash) = &check_course($dom,$domdesc);                  my ($checked,$cdesc,$coursehash) = &check_course($dom,$domdesc);
                 if ($checked eq 'ok') {                  if ($checked eq 'ok') {
                     my $enter_text;                      my $enter_text;
                     if ($type eq 'Community') {                      if ($type eq 'Community') {
                         $enter_text = 'Enter community';                          $enter_text = 'Enter community';
                       } elsif ($type eq 'Placement') {
                           $enter_text = 'Enter placement test'; 
                     } else {                      } else {
                         $enter_text = 'Enter course';                          $enter_text = 'Enter course';
                     }                      }
Line 2548  sub handler { Line 2435  sub handler {
                         &Apache::lonhtmlcommon::add_breadcrumb                          &Apache::lonhtmlcommon::add_breadcrumb
                          ({href=>"javascript:changePage(document.$phase,'adhocrole')",                           ({href=>"javascript:changePage(document.$phase,'adhocrole')",
                            text=>$enter_text});                             text=>$enter_text});
                         &print_adhocrole_selected($r,$type,$permission);                          &print_adhocrole_selected($r,$type);
                     } else {                      } else {
                         &Apache::lonhtmlcommon::add_breadcrumb                          &Apache::lonhtmlcommon::add_breadcrumb
                         ({href=>"javascript:changePage(document.$phase,'menu')",                          ({href=>"javascript:changePage(document.$phase,'menu')",
Line 2556  sub handler { Line 2443  sub handler {
                         my ($cdom,$cnum) = split(/_/,$env{'form.pickedcourse'});                          my ($cdom,$cnum) = split(/_/,$env{'form.pickedcourse'});
                         my ($readonly,$linktext);                          my ($readonly,$linktext);
                         if ($permission->{$phase} eq 'view') {                          if ($permission->{$phase} eq 'view') {
                            $readonly = 1;                             $readonly = 1; 
                         }                          }
                         if (($phase eq 'setquota') && ($permission->{'setquota'})) {                          if (($phase eq 'setquota') && ($permission->{'setquota'})) {
                             if ($permission->{'setquota'} eq 'view') {                              if ($permission->{'setquota'} eq 'view') {
                                 $linktext = 'Set quota';                                  $linktext = 'Set quota'; 
                             } else {                              } else {
                                 $linktext = 'Display quota';                                  $linktext = 'Display quota';
                             }                              }
Line 2658  sub handler { Line 2545  sub handler {
                     $r->print('<span class="LC_error">');                      $r->print('<span class="LC_error">');
                     if ($type eq 'Community') {                      if ($type eq 'Community') {
                         $r->print(&mt('The community you selected is not a valid community in this domain'));                          $r->print(&mt('The community you selected is not a valid community in this domain'));
                       } elsif ($type eq 'Placement') {
                           $r->print(&mt('The course you selected is not a valid placement test in this domain'));
                     } else {                      } else {
                         $r->print(&mt('The course you selected is not a valid course in this domain'));                          $r->print(&mt('The course you selected is not a valid course in this domain'));
                     }                      }

Removed from v.1.79.2.7  
changed lines
  Added in v.1.89


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