--- loncom/interface/lonmodifycourse.pm 2024/12/30 00:34:19 1.79.2.10 +++ loncom/interface/lonmodifycourse.pm 2016/10/05 13:59:46 1.86 @@ -1,7 +1,7 @@ # The LearningOnline Network with CAPA # handler for DC-only modifiable course settings # -# $Id: lonmodifycourse.pm,v 1.79.2.10 2024/12/30 00:34:19 raeburn Exp $ +# $Id: lonmodifycourse.pm,v 1.86 2016/10/05 13:59:46 raeburn Exp $ # # Copyright Michigan State University Board of Trustees # @@ -39,9 +39,6 @@ use Apache::lonpickcourse; use lib '/home/httpd/lib/perl'; use LONCAPA qw(:DEFAULT :match); -my $registered_cleanup; -my $modified_dom; - sub get_dc_settable { my ($type,$cdom) = @_; if ($type eq 'Community') { @@ -52,10 +49,6 @@ sub get_dc_settable { if (&showcredits($cdom)) { push(@items,'defaultcredits'); } - my %passwdconf = &Apache::lonnet::get_passwdconf($cdom); - if (($passwdconf{'crsownerchg'}) && ($type ne 'Placement')) { - push(@items,'nopasswdchg'); - } return @items; } } @@ -79,6 +72,13 @@ sub catalog_settable { if ($confhash->{'categorizecomm'} ne 'comm') { push(@settable,'categorize'); } + } elsif ($type eq 'Placement') { + if ($confhash->{'togglecatsplace'} ne 'place') { + push(@settable,'togglecats'); + } + if ($confhash->{'categorizeplace'} ne 'place') { + push(@settable,'categorize'); + } } else { if ($confhash->{'togglecats'} ne 'crs') { push(@settable,'togglecats'); @@ -101,7 +101,6 @@ sub get_enrollment_settings { @items = map { 'internal.'.$_; } (@{$internals}); push(@items,@{$accessdates}); } - push(@items,'internal.nopasswdchg'); my %settings = &Apache::lonnet::get('environment',\@items,$cdom,$cnum); my %enrollvar; $enrollvar{'autharg'} = ''; @@ -127,7 +126,7 @@ sub get_enrollment_settings { } elsif ($type eq "authtype" || $type eq "autharg" || $type eq "coursecode" || $type eq "crosslistings" || $type eq "selfenrollmgr" - || $type eq "autodropfailsafe" || $type eq 'nopasswdchg') { + || $type eq "autodropfailsafe") { $enrollvar{$type} = $settings{$item}; } elsif ($type eq 'defaultcredits') { if (&showcredits($cdom)) { @@ -175,16 +174,20 @@ sub print_course_search_page { $r->print(&Apache::loncommon::js_changer()); if ($type eq 'Community') { $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 { $r->print('<h3>'.&mt('Search for a course in the [_1] domain',$domdesc).'</h3>'); } $r->print(&Apache::loncommon::build_filters($filterlist,$type,undef,undef,$filter,$action, \$numtitles,'modifycourse',undef,undef,undef, \@codetitles,$dom)); - + my ($actiontext,$roleoption,$settingsoption); if ($type eq '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 { $actiontext = &mt('Actions available after searching for a course:'); } @@ -192,30 +195,43 @@ sub print_course_search_page { if ($type eq 'Community') { $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); + } 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 { $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); } } elsif (&Apache::lonnet::allowed('rar',$dom)) { - my ($roles_by_num,$description,$accessref,$accessinfo) = &Apache::lonnet::get_all_adhocroles($dom); - if ((ref($roles_by_num) eq 'ARRAY') && (ref($description) eq 'HASH')) { - if (@{$roles_by_num} > 1) { + my %adhocroles = &Apache::lonnet::userenvironment($env{'user.domain'},$env{'user.name'}, + 'adhocroles.'.$dom); + if (keys(%adhocroles)) { + my @adhoc = split(',',$adhocroles{'adhocroles.'.$dom}); + if (@adhoc > 1) { 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 { - $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 { - my $rolename = $description->{$roles_by_num->[0]}; 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 { - $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') { $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 { $settingsoption = &mt('View course settings which only a [_1] may modify.',$dctitle); } @@ -229,133 +245,14 @@ sub print_course_search_page { } sub print_course_selection_page { - my ($r,$dom,$domdesc,$permission) = @_; + my ($r,$dom,$domdesc) = @_; my $type = $env{'form.type'}; if (!defined($type)) { $type = 'Course'; } &print_header($r,$type); - if ($permission->{'adhocrole'} eq 'custom') { - 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> </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 +# Criteria for course search my ($filterlist,$filter) = &get_filters(); my $action = '/adm/modifycourse'; my $dctitle = &Apache::lonnet::plaintext('dc'); @@ -391,7 +288,8 @@ sub get_filters { sub print_modification_menu { my ($r,$cdesc,$domdesc,$dom,$type,$cid,$coursehash,$permission) = @_; &print_header($r,$type); - my ($ccrole,$categorytitle,$setquota_text,$setuploadquota_text,$cdom,$cnum); + my ($ccrole,$categorytitle,$setquota_text,$setuploadquota_text,$setparams_text,$cat_text, + $cdom,$cnum); if (ref($coursehash) eq 'HASH') { $cdom = $coursehash->{'domain'}; $cnum = $coursehash->{'num'}; @@ -403,59 +301,26 @@ sub print_modification_menu { } else { $ccrole = 'cc'; } - my %linktext; - if ($permission->{'setparms'} eq 'edit') { - %linktext = ( - 'setquota' => 'View/Modify quotas for group portfolio files, and for uploaded content', - 'setanon' => 'View/Modify responders threshold for anonymous survey submissions display', - 'selfenroll' => 'View/Modify Self-Enrollment configuration', - 'setpostsubmit' => 'View/Modify submit button behavior, post-submission', - ); - } else { - %linktext = ( - 'setquota' => 'View quotas for group portfolio files, and for uploaded content', - 'setanon' => 'View responders threshold for anonymous survey submissions display', - 'selfenroll' => 'View Self-Enrollment configuration', - 'setpostsubmit' => 'View submit button behavior, post-submission', - ); - } if ($type eq 'Community') { - if ($permission->{'setparms'} eq 'edit') { - $categorytitle = 'View/Modify Community Settings'; - $linktext{'setparms'} = 'View/Modify community owner'; - $linktext{'catsettings'} = 'View/Modify catalog settings for community'; - } else { - $categorytitle = 'View Community Settings'; - $linktext{'setparms'} = 'View community owner'; - $linktext{'catsettings'} = 'View catalog settings for community'; - } + $categorytitle = 'View/Modify Community Settings'; $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.'); + $setparams_text = 'View/Modify community owner'; + $cat_text = 'View/Modify catalog settings for community'; } else { - if ($permission->{'setparms'} eq 'edit') { - $categorytitle = 'View/Modify Course Settings'; - $linktext{'catsettings'} = 'View/Modify catalog settings for course'; - if (($type ne 'Placement') && (&showcredits($dom))) { - $linktext{'setparms'} = 'View/Modify course owner, institutional code, default authentication, credits, self-enrollment and table lifetime'; - } else { - $linktext{'setparms'} = 'View/Modify course owner, institutional code, default authentication, self-enrollment and table lifetime'; - } - } else { - $categorytitle = 'View Course Settings'; - $linktext{'catsettings'} = 'View catalog settings for course'; - if (($type ne 'Placement') && (&showcredits($dom))) { - $linktext{'setparms'} = 'View course owner, institutional code, default authentication, credits, self-enrollment and table lifetime'; - } else { - $linktext{'setparms'} = 'View course owner, institutional code, default authentication, self-enrollment and table lifetime'; - } - } + $categorytitle = 'View/Modify Course Settings'; $setquota_text = &mt('Total disk space allocated for storage of portfolio files in all groups in a course.'); $setuploadquota_text = &mt('Disk space allocated for storage of content uploaded directly to a course via Content Editor.'); + if (($type ne 'Placement') && (&showcredits($dom))) { + $setparams_text = 'View/Modify course owner, institutional code, default authentication, credits, self-enrollment and table lifetime'; + } else { + $setparams_text = 'View/Modify course owner, institutional code, default authentication, self-enrollment and table lifetime'; + } + $cat_text = 'View/Modify catalog settings for course'; } my $anon_text = &mt('Responder threshold required to display anonymous survey submissions.'); my $postsubmit_text = &mt('Override defaults for submit button behavior post-submission for this specific course.'); my $mysqltables_text = &mt('Override default for lifetime of "temporary" MySQL tables containing student performance data.'); - $linktext{'viewparms'} = 'Display current settings for automated enrollment'; my %domconf = &Apache::lonnet::get_dom('configuration',['coursecategories'],$dom); my @additional_params = &catalog_settable($domconf{'coursecategories'},$type); @@ -467,7 +332,7 @@ sub print_modification_menu { if (ref($managed_by_dc) eq 'ARRAY') { if (@{$managed_by_dc}) { return 1; - } + } } } return 0; @@ -481,7 +346,7 @@ sub print_modification_menu { ({ categorytitle => $categorytitle, items => [ { - linktext => $linktext{'setparms'}, + linktext => $setparams_text, url => &phaseurl('setparms'), permission => $permission->{'setparms'}, #help => '', @@ -489,7 +354,7 @@ sub print_modification_menu { linktitle => '' }, { - linktext => $linktext{'setquota'}, + linktext => 'View/Modify quotas for group portfolio files, and for uploaded content.', url => &phaseurl('setquota'), permission => $permission->{'setquota'}, #help => '', @@ -497,7 +362,7 @@ sub print_modification_menu { linktitle => '' }, { - linktext => $linktext{'setanon'}, + linktext => 'View/Modify responders threshold for anonymous survey submissions display', url => &phaseurl('setanon'), permission => $permission->{'setanon'}, #help => '', @@ -505,7 +370,7 @@ sub print_modification_menu { linktitle => '' }, { - linktext => $linktext{'catsettings'}, + linktext => $cat_text, url => &phaseurl('catsettings'), permission => (($permission->{'catsettings'}) && (@additional_params > 0)), #help => '', @@ -513,15 +378,15 @@ sub print_modification_menu { linktitle => '' }, { - linktext => $linktext{'viewparms'}, + linktext => 'Display current settings for automated enrollment', url => &phaseurl('viewparms'), - permission => ($permission->{'viewparms'} && ($type ne 'Community')), + permission => ($permission->{'viewparms'} && ($type ne 'Community') && ($type ne 'Placement')), #help => '', icon => 'roles.png', linktitle => '' }, { - linktext => $linktext{'selfenroll'}, + linktext => 'View/Modify Self-Enrollment configuration', icon => 'self_enroll.png', #help => 'Course_Self_Enrollment', url => &phaseurl('selfenroll'), @@ -529,7 +394,7 @@ sub print_modification_menu { linktitle => 'Configure user self-enrollment.', }, { - linktext => $linktext{'setpostsubmit'}, + linktext => 'View/Modify submit button behavior, post-submission', icon => 'emblem-readonly.png', #help => '', url => &phaseurl('setpostsubmit'), @@ -557,7 +422,7 @@ sub print_modification_menu { } else { $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"; - 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('Override defaults for who configures self-enrollment for this specific course.').'</li>'."\n"; @@ -568,7 +433,7 @@ sub print_modification_menu { '<li>'.$anon_text.'</li>'."\n". '<li>'.$postsubmit_text.'</li>'."\n"; my ($categories_link_start,$categories_link_end); - if ($permission->{'catsettings'} eq 'edit') { + if ($permission->{'catsettings'}) { $categories_link_start = '<a href="/adm/domainprefs?actions=coursecategories&phase=display">'; $categories_link_end = '</a>'; } @@ -600,25 +465,29 @@ sub print_modification_menu { } sub print_adhocrole_selected { - my ($r,$type,$permission) = @_; + my ($r,$type) = @_; &print_header($r,$type); my ($cdom,$cnum) = split(/_/,$env{'form.pickedcourse'}); my ($newrole,$selectrole); - if ($permission->{'adhocrole'} eq 'coord') { + if (&Apache::lonnet::allowed('ccc',$cdom)) { if ($type eq 'Community') { $newrole = "co./$cdom/$cnum"; } else { $newrole = "cc./$cdom/$cnum"; } $selectrole = 1; - } elsif ($permission->{'adhocrole'} eq 'custom') { - my ($okroles,$description) = &Apache::lonnet::get_my_adhocroles($env{'form.pickedcourse'},1); - if (ref($okroles) eq 'ARRAY') { - my $possrole = $env{'form.adhocrole'}; - if (($possrole ne '') && (grep(/^\Q$possrole\E$/,@{$okroles}))) { - my $confname = &Apache::lonnet::get_domainconfiguser($cdom); - $newrole = "cr/$cdom/$confname/$possrole./$cdom/$cnum"; - $selectrole = 1; + } elsif (&Apache::lonnet::allowed('rar',$cdom)) { + my %adhocroles = &Apache::lonnet::userenvironment($env{'user.domain'},$env{'user.name'}, + 'adhocroles.'.$cdom); + if (keys(%adhocroles)) { + my $possrole = $env{'form.adhocrole'}; + if ($possrole ne '') { + my @adhoc = split(',',$adhocroles{'adhocroles.'.$cdom}); + if (grep(/^\Q$possrole\E$/,@adhoc)) { + my $confname = &Apache::lonnet::get_domainconfiguser($cdom); + $newrole = "cr/$cdom/$confname/$possrole./$cdom/$cnum"; + $selectrole = 1; + } } } } @@ -664,34 +533,9 @@ sub print_settings_display { "<th>$lt{'dcon'}</th>\n". &Apache::loncommon::end_data_table_header_row()."\n"; foreach my $item (@items) { - my $shown = $enrollvar{$item}; - if ($item eq 'crosslistings') { - my (@xlists,@lcsecs); - foreach my $entry (split(/,/,$enrollvar{$item})) { - my ($xlist,$lc_sec) = split(/:/,$entry); - push(@xlists,$xlist); - push(@lcsecs,$lc_sec); - } - if (@xlists) { - my $crskey = $cnum.':'.$enrollvar{'coursecode'}; - my %reformatted = - &Apache::lonnet::auto_instsec_reformat($cdom,'declutter', - {$crskey => \@xlists}); - if (ref($reformatted{$crskey}) eq 'ARRAY') { - my @show; - my @xlcodes = @{$reformatted{$crskey}}; - for (my $i=0; $i<@xlcodes; $i++) { - push(@show,$xlcodes[$i].':'.$lcsecs[$i]); - } - if (@show) { - $shown = join(',',@show); - } - } - } - } $disp_table .= &Apache::loncommon::start_data_table_row()."\n". "<td><b>$longtype{$item}</b></td>\n". - "<td>$shown</td>\n"; + "<td>$enrollvar{$item}</td>\n"; if (grep(/^\Q$item\E$/,@modifiable_params)) { $disp_table .= '<td align="right">'.&mt('Yes').'</td>'."\n"; } else { @@ -733,7 +577,7 @@ sub print_settings_display { } sub print_setquota { - my ($r,$cdom,$cnum,$cdesc,$type,$readonly) = @_; + my ($r,$cdom,$cnum,$cdesc,$type) = @_; my $lctype = lc($type); my $headline = &mt("Set disk space quotas for $lctype: [_1]", '<span class="LC_nobreak">'.$cdesc.'</span>'); @@ -766,25 +610,19 @@ sub print_setquota { my $hidden_elements = &hidden_form_elements(); my $porthelpitem = &Apache::loncommon::help_open_topic('Modify_Course_Quota'); my $uploadhelpitem = &Apache::loncommon::help_open_topic('Modify_Course_Upload_Quota'); - my ($disabled,$submit); - if ($readonly) { - $disabled = ' disabled="disabled"'; - } else { - $submit = '<input type="submit" value="'.$lt{'modi'}.'" />'; - } $r->print(<<ENDDOCUMENT); <form action="/adm/modifycourse" method="post" name="setquota" onsubmit="return verify_quota();"> <h3>$headline</h3> <p><span class="LC_nobreak"> -$porthelpitem $lt{'gpqu'}: <input type="text" size="4" name="coursequota" value="$coursequota" $disabled /> MB +$porthelpitem $lt{'gpqu'}: <input type="text" size="4" name="coursequota" value="$coursequota" /> MB </span> <br /> <span class="LC_nobreak"> -$uploadhelpitem $lt{'upqu'}: <input type="text" size="4" name="uploadquota" value="$uploadquota" $disabled /> MB +$uploadhelpitem $lt{'upqu'}: <input type="text" size="4" name="uploadquota" value="$uploadquota" /> MB </span> </p> <p> -$submit +<input type="submit" value="$lt{'modi'}" /> </p> $hidden_elements <a href="javascript:changePage(document.setquota,'menu')">$lt{'back'}</a> @@ -794,7 +632,7 @@ ENDDOCUMENT } sub print_set_anonsurvey_threshold { - my ($r,$cdom,$cnum,$cdesc,$type,$readonly) = @_; + my ($r,$cdom,$cnum,$cdesc,$type) = @_; my %lt = &Apache::lonlocal::texthash( 'resp' => 'Responder threshold for anonymous survey submissions display:', 'sufa' => 'Anonymous survey submissions displayed when responders exceeds', @@ -817,19 +655,13 @@ sub print_set_anonsurvey_threshold { } &print_header($r,$type); my $hidden_elements = &hidden_form_elements(); - my ($disabled,$submit); - if ($readonly) { - $disabled = ' disabled="disabled"'; - } else { - $submit = '<input type="submit" value="'.$lt{'modi'}.'" />'; - } my $helpitem = &Apache::loncommon::help_open_topic('Modify_Anonsurvey_Threshold'); $r->print(<<ENDDOCUMENT); <form action="/adm/modifycourse" method="post" name="setanon" onsubmit="return verify_anon_threshold();"> <h3>$lt{'resp'} <span class="LC_nobreak">$cdesc</span></h3> <p> -$helpitem $lt{'sufa'}: <input type="text" size="4" name="threshold" value="$threshold" $disabled /> -$submit +$helpitem $lt{'sufa'}: <input type="text" size="4" name="threshold" value="$threshold" /> +<input type="submit" value="$lt{'modi'}" /> </p> $hidden_elements <a href="javascript:changePage(document.setanon,'menu')">$lt{'back'}</a> @@ -839,7 +671,7 @@ ENDDOCUMENT } sub print_postsubmit_config { - my ($r,$cdom,$cnum,$cdesc,$type,$readonly) = @_; + my ($r,$cdom,$cnum,$cdesc,$type) = @_; my %lt = &Apache::lonlocal::texthash ( 'conf' => 'Configure submit button behavior after student makes a submission', 'disa' => 'Disable submit button/keypress following student submission', @@ -880,27 +712,21 @@ sub print_postsubmit_config { } &print_header($r,$type); my $hidden_elements = &hidden_form_elements(); - my ($disabled,$submit); - if ($readonly) { - $disabled = ' disabled="disabled"'; - } else { - $submit = '<input type="submit" value="'.$lt{'modi'}.'" />'; - } my $helpitem = &Apache::loncommon::help_open_topic('Modify_Postsubmit_Config'); $r->print(<<ENDDOCUMENT); <form action="/adm/modifycourse" method="post" name="setpostsubmit" onsubmit="return verify_postsubmit();"> <h3>$lt{'conf'} <span class="LC_nobreak">($cdesc)</span></h3> <p> $helpitem $lt{'disa'}: -<label><input type="radio" name="postsubmit" $checkedon onclick="togglePostsubmit('studentsubmission');" value="1" $disabled /> +<label><input type="radio" name="postsubmit" $checkedon onclick="togglePostsubmit('studentsubmission');" value="1" /> $lt{'yes'}</label> -<label><input type="radio" name="postsubmit" $checkedoff onclick="togglePostsubmit('studentsubmission');" value="0" $disabled /> +<label><input type="radio" name="postsubmit" $checkedoff onclick="togglePostsubmit('studentsubmission');" value="0" /> $lt{'no'}</label> <div id="studentsubmission" style="display: $display"> -$lt{'nums'} <input type="text" name="postsubtimeout" value="$postsubtimeout" $disabled /><br /> +$lt{'nums'} <input type="text" name="postsubtimeout" value="$postsubtimeout" /><br /> $zero</div> <br /> -$submit +<input type="submit" value="$lt{'modi'}" /> </p> $hidden_elements <a href="javascript:changePage(document.setpostsubmit,'menu')">$lt{'back'}</a> @@ -913,7 +739,7 @@ sub domain_postsubtimeout { my ($cdom,$type,$settings) = @_; return unless (ref($settings) eq 'HASH'); my $lctype = lc($type); - unless ($type eq 'Community') { + unless (($type eq 'Community') || ($type eq 'Placement')) { $lctype = 'unofficial'; if ($settings->{'internal.coursecode'}) { $lctype = 'official'; @@ -937,7 +763,7 @@ sub domain_postsubtimeout { } sub print_catsettings { - my ($r,$cdom,$cnum,$cdesc,$type,$readonly) = @_; + my ($r,$cdom,$cnum,$cdesc,$type) = @_; &print_header($r,$type); my %lt = &Apache::lonlocal::texthash( 'back' => 'Pick another action', @@ -958,10 +784,6 @@ sub print_catsettings { my %domconf = &Apache::lonnet::get_dom('configuration',['coursecategories'],$cdom); my @cat_params = &catalog_settable($domconf{'coursecategories'},$type); if (@cat_params > 0) { - my $disabled; - if ($readonly) { - $disabled = ' disabled="disabled"'; - } my %currsettings = &Apache::lonnet::get('environment',['hidefromcat','categories'],$cdom,$cnum); if (grep(/^togglecats$/,@cat_params)) { @@ -973,9 +795,11 @@ sub print_catsettings { } $r->print('<br /><h4>'.$lt{'visi'}.'</h4>'. $lt{'exclude'}. - ' <label><input name="hidefromcat" type="radio" value="yes" '.$excludeon.$disabled.' />'.&mt('Yes').'</label> <label><input name="hidefromcat" type="radio" value="" '.$excludeoff.$disabled.' />'.&mt('No').'</label><br /><p>'); + ' <label><input name="hidefromcat" type="radio" value="yes" '.$excludeon.' />'.&mt('Yes').'</label> <label><input name="hidefromcat" type="radio" value="" '.$excludeoff.' />'.&mt('No').'</label><br /><p>'); 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.")); + } 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 { $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>'. @@ -990,21 +814,19 @@ sub print_catsettings { if (ref($cathash) eq 'HASH') { $r->print($lt{'assi'}.'<br /><br />'. &Apache::loncommon::assign_categories_table($cathash, - $currsettings{'categories'},$type,$disabled)); + $currsettings{'categories'},$type)); } else { $r->print(&mt('No categories defined for this domain')); } } else { $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>'); } } - unless ($readonly) { - $r->print('<p><input type="button" name="chgcatsettings" value="'. - &mt('Save').'" onclick="javascript:changePage(document.catsettings,'."'processcat'".');" /></p>'); - } + $r->print('<p><input type="button" name="chgcatsettings" value="'. + &mt('Save').'" onclick="javascript:changePage(document.catsettings,'."'processcat'".');" /></p>'); } else { $r->print('<span class="LC_warning">'); if ($type eq 'Community') { @@ -1021,7 +843,7 @@ sub print_catsettings { } sub print_course_modification_page { - my ($r,$cdom,$cnum,$cdesc,$crstype,$readonly) = @_; + my ($r,$cdom,$cnum,$cdesc,$crstype) = @_; my %lt=&Apache::lonlocal::texthash( 'actv' => "Active", 'inac' => "Inactive", @@ -1035,28 +857,25 @@ sub print_course_modification_page { 'domd' => 'Domain default', 'whom' => 'Who configures', ); - my ($ownertable,$ccrole,$javascript_validations,$authenitems,$ccname,$disabled); + my ($ownertable,$ccrole,$javascript_validations,$authenitems,$ccname); my %enrollvar = &get_enrollment_settings($cdom,$cnum); my %settings = &Apache::lonnet::get('environment',['internal.coursecode','internal.textbook', 'internal.selfenrollmgrdc','internal.selfenrollmgrcc', - 'internal.mysqltables'],$cdom,$cnum); + 'internal.mysqltables'], + $cdom,$cnum); my $type = &Apache::lonuserutils::get_extended_type($cdom,$cnum,$crstype,\%settings); my @specific_managebydc = split(/,/,$settings{'internal.selfenrollmgrdc'}); my @specific_managebycc = split(/,/,$settings{'internal.selfenrollmgrcc'}); my %domdefaults = &Apache::lonnet::get_domain_defaults($cdom); - my %passwdconf = &Apache::lonnet::get_passwdconf($cdom); my @default_managebydc = split(/,/,$domdefaults{$type.'selfenrolladmdc'}); if ($crstype eq 'Community') { $ccrole = 'co'; $lt{'nocc'} = &mt('There is currently no owner set for this community.'); } else { $ccrole ='cc'; - ($javascript_validations,$authenitems) = &gather_authenitems($cdom,\%enrollvar,$readonly); + ($javascript_validations,$authenitems) = &gather_authenitems($cdom,\%enrollvar); } $ccname = &Apache::lonnet::plaintext($ccrole,$crstype); - if ($readonly) { - $disabled = ' disabled="disabled"'; - } my %roleshash = &Apache::lonnet::get_my_roles($cnum,$cdom,'','',[$ccrole]); my (@local_ccs,%cc_status,%pname); foreach my $item (keys(%roleshash)) { @@ -1097,9 +916,9 @@ sub print_course_modification_page { foreach my $cc (@local_ccs) { $ownertable .= &Apache::loncommon::start_data_table_row()."\n"; 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" /></td>'."\n"; } else { - $ownertable .= '<td><input type="radio" name="courseowner" value="'.$cc.'"'.$disabled.' /></td>'."\n"; + $ownertable .= '<td><input type="radio" name="courseowner" value="'.$cc.'" /></td>'."\n"; } $ownertable .= '<td>'.$pname{$cc}.'</td>'."\n". @@ -1119,19 +938,18 @@ sub print_course_modification_page { if ($crstype eq 'Community') { $r->print(&Apache::lonhtmlcommon::row_title( &Apache::loncommon::help_open_topic('Modify_Community_Owner'). - ' '.&mt('Community Owner'))."\n". - $ownertable."\n".&Apache::lonhtmlcommon::row_closure()); + ' '.&mt('Community Owner'))."\n"); } else { $r->print(&Apache::lonhtmlcommon::row_title( &Apache::loncommon::help_open_topic('Modify_Course_Instcode'). ' '.&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'}.'" />'. &Apache::lonhtmlcommon::row_closure()); - if (&showcredits($cdom)) { + if (($crstype eq 'Course') && (&showcredits($cdom))) { $r->print(&Apache::lonhtmlcommon::row_title( &Apache::loncommon::help_open_topic('Modify_Course_Credithours'). - ' '.&mt('Credits (students)'))."\n". - '<input type="text" size="3" name="defaultcredits" value="'.$enrollvar{'defaultcredits'}.'"'.$disabled.' />'. + ' '.&mt('Credits (students)'))."\n". + '<input type="text" size="3" name="defaultcredits" value="'.$enrollvar{'defaultcredits'}.'" />'. &Apache::lonhtmlcommon::row_closure()); } $r->print(&Apache::lonhtmlcommon::row_title( @@ -1140,21 +958,8 @@ sub print_course_modification_page { $authenitems."\n". &Apache::lonhtmlcommon::row_closure(). &Apache::lonhtmlcommon::row_title( - &Apache::loncommon::help_open_topic('Modify_Course_Owner'). - ' '.&mt('Course Owner'))."\n". - $ownertable."\n".&Apache::lonhtmlcommon::row_closure()); - if (($passwdconf{'crsownerchg'}) && ($type ne 'Placement')) { - my $checked; - if ($enrollvar{'nopasswdchg'}) { - $checked = ' checked="checked"'; - } - $r->print(&Apache::lonhtmlcommon::row_title( - &Apache::loncommon::help_open_topic('Modify_Course_Chgpasswd'). - ' '.&mt('Changing passwords (internal)'))."\n". - '<label><input type="checkbox" value="1" name="nopasswdchg"'.$checked.$disabled.' />'. - &mt('Disable changing password for users with student role by course owner').'<label>'."\n". - &Apache::lonhtmlcommon::row_closure()); - } + &Apache::loncommon::help_open_topic('Modify_Course_Owner'). + ' '.&mt('Course Owner'))."\n"); } my ($cctitle,$rolename,$currmanages,$ccchecked,$dcchecked,$defaultchecked); my ($selfenrollrows,$selfenrolltitles) = &Apache::lonuserutils::get_selfenroll_titles(); @@ -1164,7 +969,8 @@ sub print_course_modification_page { $cctitle = &mt('Course personnel'); } - $r->print(&Apache::lonhtmlcommon::row_title( + $r->print($ownertable."\n".&Apache::lonhtmlcommon::row_closure(). + &Apache::lonhtmlcommon::row_title( &Apache::loncommon::help_open_topic('Modify_Course_Selfenrolladmin'). ' '.&mt('Self-enrollment configuration')). &Apache::loncommon::start_data_table()."\n". @@ -1197,7 +1003,7 @@ sub print_course_modification_page { foreach my $option ('','0','1') { $r->print('<span class="LC_nobreak"><label>'. '<input type="radio" name="selfenrollmgr_'.$item.'" '. - 'value="'.$option.'"'.$checked{$option}.$disabled.' />'. + 'value="'.$option.'"'.$checked{$option}.' />'. $optionname{$option}.'</label></span><br />'); } $r->print('</td>'."\n". @@ -1208,24 +1014,21 @@ sub print_course_modification_page { &Apache::lonhtmlcommon::row_title( &Apache::loncommon::help_open_topic('Modify_Course_Table_Lifetime'). ' '.&mt('"Temporary" Tables Lifetime (s)'))."\n". - '<input type="text" size="10" name="mysqltables" value="'.$settings{'internal.mysqltables'}.'"'.$disabled.' />'. + '<input type="text" size="10" name="mysqltables" value="'.$settings{'internal.mysqltables'}.'" />'. &Apache::lonhtmlcommon::row_closure(1). - &Apache::lonhtmlcommon::end_pick_box().'</p><p>'.$hidden_elements); - unless ($readonly) { - $r->print('<input type="button" onclick="javascript:changePage(this.form,'."'processparms'".');'); - if ($crstype eq 'Community') { - $r->print('this.form.submit();"'); - } else { - $r->print('javascript:verify_message(this.form);"'); - } - $r->print(' value="'.$lt{'gobt'}.'" />'); + &Apache::lonhtmlcommon::end_pick_box().'</p><p>'.$hidden_elements. + '<input type="button" onclick="javascript:changePage(this.form,'."'processparms'".');'); + if ($crstype eq 'Community') { + $r->print('this.form.submit();"'); + } else { + $r->print('javascript:verify_message(this.form);"'); } - $r->print('</p></form>'); + $r->print(' value="'.$lt{'gobt'}.'" /></p></form>'); return; } sub print_selfenrollconfig { - my ($r,$type,$cdesc,$coursehash,$readonly) = @_; + my ($r,$type,$cdesc,$coursehash) = @_; return unless(ref($coursehash) eq 'HASH'); my $cnum = $coursehash->{'num'}; my $cdom = $coursehash->{'domain'}; @@ -1235,7 +1038,7 @@ sub print_selfenrollconfig { '<span class="LC_nobreak">'.$cdesc.'</span>').'</h3>'."\n"); &Apache::loncreateuser::print_selfenroll_menu($r,'domain',$env{'form.pickedcourse'}, $cdom,$cnum,\%currsettings, - &hidden_form_elements(),$readonly); + &hidden_form_elements()); return; } @@ -1291,7 +1094,7 @@ sub modifiable_only_title { } sub gather_authenitems { - my ($cdom,$enrollvar,$readonly) = @_; + my ($cdom,$enrollvar) = @_; my ($krbdef,$krbdefdom)=&Apache::loncommon::get_kerberos_defaults($cdom); my $curr_authtype = ''; my $curr_authfield = ''; @@ -1315,8 +1118,7 @@ sub gather_authenitems { kerb_def_auth => $krbdef, mode => 'modifycourse', curr_authtype => $curr_authtype, - curr_autharg => $enrollvar->{'autharg'}, - readonly => $readonly, + curr_autharg => $enrollvar->{'autharg'} ); my (%authform,$authenitems); $authform{'krb'} = &Apache::loncommon::authform_kerberos(%param); @@ -1337,16 +1139,12 @@ sub modify_course { 'internal.pendingco-owners','internal.selfenrollmgrdc', 'internal.selfenrollmgrcc','internal.mysqltables'); 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', 'internal.sectionnums','internal.crosslistings')); if (&showcredits($cdom)) { push(@items,'internal.defaultcredits'); } - my %passwdconf = &Apache::lonnet::get_passwdconf($cdom); - if ($passwdconf{'crsownerchg'}) { - push(@items,'internal.nopasswdchg'); - } } my %settings = &Apache::lonnet::get('environment',\@items,$cdom,$cnum); my $description = $settings{'description'}; @@ -1362,7 +1160,6 @@ sub modify_course { } else { %changed = ( code => 0, owner => 0, - passwd => 0, ); $ccrole = 'cc'; unless ($settings{'internal.sectionnums'} eq '') { @@ -1415,21 +1212,9 @@ sub modify_course { $newattr{'mysqltables'} = $env{'form.mysqltables'}; $newattr{'mysqltables'} =~ s/\D+//g; } - if ($type ne 'Placement') { - if (&showcredits($cdom) && exists($env{'form.defaultcredits'})) { - $newattr{'defaultcredits'}=$env{'form.defaultcredits'}; - $newattr{'defaultcredits'} =~ s/[^\d\.]//g; - } - if (grep(/^nopasswdchg$/,@modifiable_params)) { - if ($env{'form.nopasswdchg'}) { - $newattr{'nopasswdchg'} = 1; - unless ($currattr{'nopasswdchg'}) { - $changed{'passwd'} = 1; - } - } elsif ($currattr{'nopasswdchg'}) { - $changed{'passwd'} = 1; - } - } + if (($type ne 'Placement') && (&showcredits($cdom) && exists($env{'form.defaultcredits'}))) { + $newattr{'defaultcredits'}=$env{'form.defaultcredits'}; + $newattr{'defaultcredits'} =~ s/[^\d\.]//g; } } @@ -1464,7 +1249,7 @@ sub modify_course { } } - if ($changed{'owner'} || $changed{'code'} || $changed{'passwd'}) { + if ($changed{'owner'} || $changed{'code'}) { my %crsinfo = &Apache::lonnet::courseiddump($cdom,'.',1,'.','.',$cnum, undef,undef,'.'); if (ref($crsinfo{$env{'form.pickedcourse'}}) eq 'HASH') { @@ -1474,29 +1259,10 @@ sub modify_course { if ($changed{'owner'}) { $crsinfo{$env{'form.pickedcourse'}}{'owner'} = $env{'form.courseowner'}; } - if ($changed{'passwd'}) { - if ($env{'form.nopasswdchg'}) { - $crsinfo{$env{'form.pickedcourse'}}{'nopasswdchg'} = 1; - } else { - delete($crsinfo{'nopasswdchg'}); - } - } my $chome = &Apache::lonnet::homeserver($cnum,$cdom); my $putres = &Apache::lonnet::courseidput($cdom,\%crsinfo,$chome,'notime'); - if (($putres eq 'ok') && (($changed{'owner'} || $changed{'code'}))) { + if ($putres eq 'ok') { &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; - } - } - } } } } @@ -1542,12 +1308,6 @@ sub modify_course { $shown = &mt('None'); } elsif (($attr eq 'mysqltables') && ($shown eq '')) { $shown = &mt('domain default'); - } elsif ($attr eq 'nopasswdchg') { - if ($shown) { - $shown = &mt('Yes'); - } else { - $shown = &mt('No'); - } } $chgresponse .= '<li>'.&mt('[_1] now set to: [_2]',$longtype{$attr},$shown).'</li>'; } else { @@ -1560,17 +1320,11 @@ sub modify_course { $shown = &mt('None'); } elsif (($attr eq 'mysqltables') && ($shown eq '')) { $shown = &mt('domain default'); - } elsif ($attr eq 'nopasswdchg') { - if ($shown) { - $shown = &mt('Yes'); - } else { - $shown = &mt('No'); - } } $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 '') { 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.')); @@ -2251,7 +2005,13 @@ function changePage(formname,newphase) { if ($phase eq 'setparms') { $js .= $javascript_validations; } 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() { if (document.getElementById('searching')) { document.getElementById('searching').style.display = 'none'; @@ -2259,7 +2019,7 @@ function hide_searching() { return; } -ENDJS +|; } elsif ($phase eq 'setquota') { my $invalid = &mt('The quota you entered contained invalid characters.'); my $alert = &mt('You must enter a number'); @@ -2403,6 +2163,8 @@ ENDSCRIPT my $bread_text = "View/Modify Courses/Communities"; if ($type eq 'Community') { $bread_text = 'Community Settings'; + } elsif ($type eq 'Placement') { + $bread_text = 'Placement Test Settings'; } else { $bread_text = 'Course Settings'; } @@ -2480,7 +2242,6 @@ sub course_settings_descrip { 'selfenrollmgrdc' => "Course-specific self-enrollment configuration by Domain Coordinator", 'selfenrollmgrcc' => "Course-specific self-enrollment configuration by Course personnel", 'mysqltables' => '"Temporary" student performance tables lifetime (seconds)', - 'nopasswdchg' => 'Disable changing password for users with student role by course owner', ); } return %longtype; @@ -2493,8 +2254,7 @@ sub hidden_form_elements { 'locarg','krbarg','krbver','counter','hidefromcat','usecategory', 'threshold','postsubmit','postsubtimeout','defaultcredits','uploadquota', 'selfenrollmgrdc','selfenrollmgrcc','action','state','currsec_st', - 'sections','newsec','mysqltables','nopasswdchg'], - ['^selfenrollmgr_','^selfenroll_'])."\n". + 'sections','newsec','mysqltables'],['^selfenrollmgr_','^selfenroll_'])."\n". '<input type="hidden" name="prevphase" value="'.$env{'form.phase'}.'" />'; return $hidden_elements; } @@ -2510,62 +2270,31 @@ sub showcredits { sub get_permission { my ($dom) = @_; my ($allowed,%permission); - my %passwdconf = &Apache::lonnet::get_passwdconf($dom); if (&Apache::lonnet::allowed('ccc',$dom)) { $allowed = 1; %permission = ( - setquota => 'edit', - processquota => 'edit', - setanon => 'edit', - processthreshold => 'edit', - setpostsubmit => 'edit', - processpostsubmit => 'edit', - viewparms => 'view', - setparms => 'edit', - processparms => 'edit', - catsettings => 'edit', - processcat => 'edit', - selfenroll => 'edit', - adhocrole => 'coord', + setquota => 1, + processquota => 1, + setanon => 1, + processthreshold => 1, + setpostsubmit => 1, + processpostsubmit => 1, + viewparms => 1, + setparms => 1, + processparms => 1, + catsettings => 1, + processcat => 1, + selfenroll => 1, ); - if ($passwdconf{'crsownerchg'}) { - $permission{passwdchg} = 'edit'; - } } elsif (&Apache::lonnet::allowed('rar',$dom)) { $allowed = 1; %permission = ( - setquota => 'view', - viewparms => 'view', - setanon => 'view', - setpostsubmit => 'view', - setparms => 'view', - catsettings => 'view', - selfenroll => 'view', - adhocrole => 'custom', + viewparms => 1, ); - if ($passwdconf{'crsownerchg'}) { - $permission{passwdchg} = 'view'; - } } 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 OK; -} - sub handler { my $r = shift; if ($r->header_only) { @@ -2574,9 +2303,6 @@ sub handler { return OK; } - $registered_cleanup=0; - $modified_dom = ''; - my $dom = $env{'request.role.domain'}; my $domdesc = &Apache::lonnet::domain($dom,'description'); my ($allowed,$permission) = &get_permission($dom); @@ -2607,6 +2333,8 @@ sub handler { } if ($type eq 'Community') { $choose_text = "Choose a community"; + } elsif ($type eq 'Placement') { + $choose_text = "Choose a placement test"; } else { $choose_text = "Choose a course"; } @@ -2616,13 +2344,15 @@ sub handler { {href=>"javascript:changePage(document.$phase,'courselist')", text=>$choose_text}); if ($phase eq 'courselist') { - &print_course_selection_page($r,$dom,$domdesc,$permission); + &print_course_selection_page($r,$dom,$domdesc); } else { my ($checked,$cdesc,$coursehash) = &check_course($dom,$domdesc); if ($checked eq 'ok') { my $enter_text; if ($type eq 'Community') { $enter_text = 'Enter community'; + } elsif ($type eq 'Placement') { + $enter_text = 'Enter placement test'; } else { $enter_text = 'Enter course'; } @@ -2637,26 +2367,17 @@ sub handler { &Apache::lonhtmlcommon::add_breadcrumb ({href=>"javascript:changePage(document.$phase,'adhocrole')", text=>$enter_text}); - &print_adhocrole_selected($r,$type,$permission); + &print_adhocrole_selected($r,$type); } else { &Apache::lonhtmlcommon::add_breadcrumb ({href=>"javascript:changePage(document.$phase,'menu')", text=>"Pick action"}); my ($cdom,$cnum) = split(/_/,$env{'form.pickedcourse'}); - my ($readonly,$linktext); - if ($permission->{$phase} eq 'view') { - $readonly = 1; - } if (($phase eq 'setquota') && ($permission->{'setquota'})) { - if ($permission->{'setquota'} eq 'view') { - $linktext = 'Set quota'; - } else { - $linktext = 'Display quota'; - } &Apache::lonhtmlcommon::add_breadcrumb ({href=>"javascript:changePage(document.$phase,'$phase')", - text=>$linktext}); - &print_setquota($r,$cdom,$cnum,$cdesc,$type,$readonly); + text=>"Set quota",}); + &print_setquota($r,$cdom,$cnum,$cdesc,$type); } elsif (($phase eq 'processquota') && ($permission->{'processquota'})) { &Apache::lonhtmlcommon::add_breadcrumb ({href=>"javascript:changePage(document.$phase,'setquota')", @@ -2669,7 +2390,7 @@ sub handler { &Apache::lonhtmlcommon::add_breadcrumb ({href=>"javascript:changePage(document.$phase,'$phase')", text=>"Threshold for anonymous submissions display"}); - &print_set_anonsurvey_threshold($r,$cdom,$cnum,$cdesc,$type,$readonly); + &print_set_anonsurvey_threshold($r,$cdom,$cnum,$cdesc,$type); } elsif (($phase eq 'processthreshold') && ($permission->{'processthreshold'})) { &Apache::lonhtmlcommon::add_breadcrumb ({href=>"javascript:changePage(document.$phase,'setanon')", @@ -2679,15 +2400,10 @@ sub handler { text=>"Result"}); &modify_anonsurvey_threshold($r,$cdom,$cnum,$cdesc,$domdesc,$type); } elsif (($phase eq 'setpostsubmit') && ($permission->{'setpostsubmit'})) { - if ($permission->{'setpostsubmit'} eq 'view') { - $linktext = 'Submit button behavior post-submission'; - } else { - $linktext = 'Configure submit button behavior post-submission'; - } &Apache::lonhtmlcommon::add_breadcrumb ({href=>"javascript:changePage(document.$phase,'$phase')", - text=>$linktext}); - &print_postsubmit_config($r,$cdom,$cnum,$cdesc,$type,$readonly); + text=>"Configure submit button behavior post-submission"}); + &print_postsubmit_config($r,$cdom,$cnum,$cdesc,$type); } elsif (($phase eq 'processpostsubmit') && ($permission->{'processpostsubmit'})) { &Apache::lonhtmlcommon::add_breadcrumb ({href=>"javascript:changePage(document.$phase,'$phase')", @@ -2699,15 +2415,10 @@ sub handler { text=>"Display settings"}); &print_settings_display($r,$cdom,$cnum,$cdesc,$type,$permission); } elsif (($phase eq 'setparms') && ($permission->{'setparms'})) { - if ($permission->{'setparms'} eq 'view') { - $linktext = 'Display settings'; - } else { - $linktext = 'Change settings'; - } &Apache::lonhtmlcommon::add_breadcrumb ({href=>"javascript:changePage(document.$phase,'$phase')", - text=>$linktext}); - &print_course_modification_page($r,$cdom,$cnum,$cdesc,$type,$readonly); + text=>"Change settings"}); + &print_course_modification_page($r,$cdom,$cnum,$cdesc,$type); } elsif (($phase eq 'processparms') && ($permission->{'processparms'})) { &Apache::lonhtmlcommon::add_breadcrumb ({href=>"javascript:changePage(document.$phase,'setparms')", @@ -2720,7 +2431,7 @@ sub handler { &Apache::lonhtmlcommon::add_breadcrumb ({href=>"javascript:changePage(document.$phase,'$phase')", text=>"Catalog settings"}); - &print_catsettings($r,$cdom,$cnum,$cdesc,$type,$readonly); + &print_catsettings($r,$cdom,$cnum,$cdesc,$type); } elsif (($phase eq 'processcat') && ($permission->{'processcat'})) { &Apache::lonhtmlcommon::add_breadcrumb ({href=>"javascript:changePage(document.$phase,'catsettings')", @@ -2734,7 +2445,7 @@ sub handler { ({href => "javascript:changePage(document.$phase,'$phase')", text => "Self-enrollment settings"}); if (!exists($env{'form.state'})) { - &print_selfenrollconfig($r,$type,$cdesc,$coursehash,$readonly); + &print_selfenrollconfig($r,$type,$cdesc,$coursehash); } elsif ($env{'form.state'} eq 'done') { &Apache::lonhtmlcommon::add_breadcrumb ({href=>"javascript:changePage(document.$phase,'$phase')", @@ -2747,6 +2458,8 @@ sub handler { $r->print('<span class="LC_error">'); if ($type eq 'Community') { $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 { $r->print(&mt('The course you selected is not a valid course in this domain')); }