--- loncom/interface/lonmodifycourse.pm	2024/08/26 10:48:59	1.79.2.9.2.7
+++ loncom/interface/lonmodifycourse.pm	2023/08/03 15:57:23	1.102
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # handler for DC-only modifiable course settings
 #
-# $Id: lonmodifycourse.pm,v 1.79.2.9.2.7 2024/08/26 10:48:59 raeburn Exp $
+# $Id: lonmodifycourse.pm,v 1.102 2023/08/03 15:57:23 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -79,6 +79,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');
@@ -97,7 +104,7 @@ sub get_enrollment_settings {
     my ($cdom,$cnum) = @_;
     my ($internals,$accessdates) = &autoenroll_keys();
     my @items;
-    if ((ref($internals) eq 'ARRAY') && (ref($accessdates) eq 'ARRAY')) {
+    if ((ref($internals) eq 'ARRAY') && (ref($accessdates) eq 'ARRAY')) { 
         @items = map { 'internal.'.$_; } (@{$internals});
         push(@items,@{$accessdates});
     }
@@ -175,6 +182,8 @@ 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>');
     }
@@ -185,6 +194,8 @@ sub print_course_search_page {
     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,6 +203,9 @@ 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);
@@ -202,6 +216,8 @@ sub print_course_search_page {
             if (@{$roles_by_num} > 1) {
                 if ($type eq 'Community') {
                     $roleoption = &mt('Enter the community with one of the available ad hoc roles');
+                } elsif ($type eq 'Placement') {
+                    $roleoption = &mt('Enter the placement test with one of the available ad hoc roles.');
                 } else {
                     $roleoption = &mt('Enter the course with one of the available ad hoc roles.');
                 }
@@ -209,6 +225,8 @@ sub print_course_search_page {
                 my $rolename = $description->{$roles_by_num->[0]};
                 if ($type eq 'Community') {
                     $roleoption = &mt('Enter the community with the ad hoc role of: [_1]',$rolename);
+                } elsif ($type eq 'Placement') {
+                    $roleoption = &mt('Enter the placement test with the ad hoc role of: [_1]',$rolename);
                 } else {
                     $roleoption = &mt('Enter the course with the ad hoc role of: [_1]',$rolename);
                 }
@@ -216,6 +234,8 @@ sub print_course_search_page {
         }
         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);
         }
@@ -413,7 +433,6 @@ sub print_modification_menu {
                       'setpostsubmit' => 'View/Modify submit button behavior, post-submission',
                       'setltiauth'    => 'View/Modify re-authentication requirement for LTI launch of deep-linked item',
                       'setexttool'    => 'View/Modify External Tools permissions',
-                      'setcrseditors' => 'View/Modify permitted course resource editors',
                     );
     } else {
         %linktext = (
@@ -423,11 +442,10 @@ sub print_modification_menu {
                       'setpostsubmit' => 'View submit button behavior, post-submission',
                       'setltiauth'    => 'View re-authentication requirement for LTI launch of deep-linked item',
                       'setexttool'    => 'View External Tools permissions',
-                      'setcrseditors' => 'View permitted course resource editors',
                     );
     }
     if ($type eq 'Community') {
-        if ($permission->{'setparms'} eq 'edit') {
+        if ($permission->{'setparms'} eq 'edit') { 
             $categorytitle = 'View/Modify Community Settings';
             $linktext{'setparms'} = 'View/Modify community owner, self-enrollment and table lifetime';
             $linktext{'catsettings'} = 'View/Modify catalog settings for community';
@@ -437,7 +455,7 @@ sub print_modification_menu {
             $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.');
-        $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 {
         if ($permission->{'setparms'} eq 'edit') {
             $categorytitle = 'View/Modify Course Settings';
@@ -463,7 +481,7 @@ sub print_modification_menu {
         $extendedtype = ucfirst(&Apache::lonuserutils::get_extended_type($cdom,$cnum,$type,\%settings));
     }
     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 $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.');
     my $ltiauth_text = &mt('Override default for requirement for re-authentication for LTI-limited launch of deep-linked item.');
     my $exttool_text = &mt('Override default permissions for external tools use for this specific course.');
@@ -527,7 +545,7 @@ sub print_modification_menu {
             {
                 linktext => $linktext{'viewparms'},
                 url => &phaseurl('viewparms'),
-                permission => ($permission->{'viewparms'} && ($type ne 'Community')),
+                permission => ($permission->{'viewparms'} && ($type ne 'Community') && ($type ne 'Placement')),
                 #help => '',
                 icon => 'roles.png',
                 linktitle => ''
@@ -564,14 +582,6 @@ sub print_modification_menu {
                 permission => $permission->{'setexttool'},
                 linktitle => '',
             },
-            {
-                linktext => $linktext{'setcrseditors'},
-                icon => 'crseditors.png',
-                #help => '',
-                url => &phaseurl('setcrseditors'),
-                permission => $permission->{'setcrseditors'},
-                linktitle => '',
-            },
         ]
         },
         );
@@ -605,9 +615,9 @@ sub print_adhocrole_selected {
         }
         $selectrole = 1;
     } elsif ($permission->{'adhocrole'} eq 'custom') {
-        my ($okroles,$description) = &Apache::lonnet::get_my_adhocroles($env{'form.pickedcourse'},1);
+        my ($okroles,$description) = &Apache::lonnet::get_my_adhocroles($env{'form.pickedcourse'},1);      
         if (ref($okroles) eq 'ARRAY') {
-            my $possrole = $env{'form.adhocrole'};
+            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";
@@ -807,7 +817,7 @@ sub print_set_anonsurvey_threshold {
     my %settings = &Apache::lonnet::get('environment',['internal.anonsurvey_threshold'],$cdom,$cnum);
     my $threshold = $settings{'internal.anonsurvey_threshold'};
     if ($threshold eq '') {
-        my %domconfig =
+        my %domconfig = 
             &Apache::lonnet::get_dom('configuration',['coursedefaults'],$cdom);
         if (ref($domconfig{'coursedefaults'}) eq 'HASH') {
             $threshold = $domconfig{'coursedefaults'}{'anonsurvey_threshold'};
@@ -822,7 +832,7 @@ sub print_set_anonsurvey_threshold {
     my $hidden_elements = &hidden_form_elements();
     my ($disabled,$submit);
     if ($readonly) {
-        $disabled = ' disabled="disabled"';
+        $disabled = ' disabled="disabled"'; 
     } else {
         $submit = '<input type="submit" value="'.$lt{'modi'}.'" />';
     }
@@ -862,11 +872,11 @@ sub print_postsubmit_config {
     if ($postsubmit eq '') {
         my %domconfig =
             &Apache::lonnet::get_dom('configuration',['coursedefaults'],$cdom);
-        $postsubmit = 1;
+        $postsubmit = 1; 
         if (ref($domconfig{'coursedefaults'}) eq 'HASH') {
             if (ref($domconfig{'coursedefaults'}{'postsubmit'}) eq 'HASH') {
                 if ($domconfig{'coursedefaults'}{'postsubmit'}{'client'} eq 'off') {
-                    $postsubmit = 0;
+                    $postsubmit = 0; 
                 }
             }
         }
@@ -900,7 +910,7 @@ sub print_postsubmit_config {
 <h4><span class="LC_nobreak">$showtype: $cdesc</span></h4>
 <form action="/adm/modifycourse" method="post" name="setpostsubmit" onsubmit="return verify_postsubmit();">
 <p>
-$helpitem $lt{'disa'}:
+$helpitem $lt{'disa'}: 
 <label><input type="radio" name="postsubmit" $checkedon onclick="togglePostsubmit('studentsubmission');" value="1" $disabled />
 $lt{'yes'}</label>&nbsp;&nbsp;
 <label><input type="radio" name="postsubmit" $checkedoff onclick="togglePostsubmit('studentsubmission');" value="0" $disabled />
@@ -923,7 +933,7 @@ ENDDOCUMENT
 
 sub domain_postsubtimeout {
     my ($cdom,$type,$settings) = @_;
-    return unless (ref($settings) eq 'HASH');
+    return unless (ref($settings) eq 'HASH'); 
     my $lctype = &get_lctype($type,$settings);
     my %domconfig =
         &Apache::lonnet::get_dom('configuration',['coursedefaults'],$cdom);
@@ -943,7 +953,7 @@ sub domain_postsubtimeout {
 sub get_lctype {
     my ($type,$settings) = @_;
     my $lctype = lc($type);
-    unless ($type eq 'Community') {
+    unless (($type eq 'Community') || ($type eq 'Placement')) {
         $lctype = 'unofficial';
         if (ref($settings) eq 'HASH') {
             if ($settings->{'internal.coursecode'}) {
@@ -983,14 +993,14 @@ sub print_catsettings {
         if ($readonly) {
             $disabled = ' disabled="disabled"';
         }
-        my %currsettings =
+        my %currsettings = 
             &Apache::lonnet::get('environment',['hidefromcat','categories'],$cdom,$cnum);
         if (grep(/^togglecats$/,@cat_params)) {
             my $excludeon = '';
             my $excludeoff = ' checked="checked" ';
             if ($currsettings{'hidefromcat'} eq 'yes') {
                 $excludeon = $excludeoff;
-                $excludeoff = '';
+                $excludeoff = ''; 
             }
             $r->print('<br /><h4>'.$lt{'visi'}.'</h4>'.
                       '<p>'.
@@ -998,6 +1008,8 @@ 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></p><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.").'</p>');
+            } 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.").'</p>');
             } 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").':</p><ul>'.
                           '<li>'.&mt('Auto-cataloging is enabled and the course is assigned an institutional code.').'</li>'.
@@ -1030,7 +1042,7 @@ sub print_catsettings {
                           '<p>'.&mt('No categories defined for this domain.'));
                 $shownsave = 1;
             }
-            if ($type eq 'Community') {
+            if (($type eq 'Community') || ($type eq 'Placement')) {
                 $r->print('</p>');
             } elsif ($shownsave) {
                 $r->print('<br />'.&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.').'</p>');
@@ -1071,7 +1083,7 @@ sub print_course_modification_page {
             'gobt' => "Save",
             'sett' => 'Setting',
             'domd' => 'Domain default',
-            'whom' => 'Who configures',
+            'whom' => 'Who configures',  
     );
     my ($ownertable,$ccrole,$javascript_validations,$authenitems,$ccname,$disabled);
     my %enrollvar = &get_enrollment_settings($cdom,$cnum);
@@ -1105,11 +1117,11 @@ sub print_course_modification_page {
             $cc_status{$uname.':'.$udom} = $lt{'actv'};
         }
     }
-    if (($enrollvar{'courseowner'} ne '') &&
+    if (($enrollvar{'courseowner'} ne '') && 
         (!grep(/^$enrollvar{'courseowner'}$/,@local_ccs))) {
         push(@local_ccs,$enrollvar{'courseowner'});
         my ($owneruname,$ownerdom) = split(/:/,$enrollvar{'courseowner'});
-        $pname{$enrollvar{'courseowner'}} =
+        $pname{$enrollvar{'courseowner'}} = 
                          &Apache::loncommon::plainname($owneruname,$ownerdom);
         my $active_cc = &Apache::loncommon::check_user_status($ownerdom,$owneruname,
                                                               $cdom,$cnum,$ccrole);
@@ -1139,7 +1151,7 @@ sub print_course_modification_page {
             } else {
                 $ownertable .= '<td><input type="radio" name="courseowner" value="'.$cc.'"'.$disabled.' /></td>'."\n";
             }
-            $ownertable .=
+            $ownertable .= 
                  '<td>'.$pname{$cc}.'</td>'."\n".
                  '<td>'.$cc.'</td>'."\n".
                  '<td>'.$cc_status{$cc}.' '.$ccname.'</td>'."\n".
@@ -1216,7 +1228,7 @@ sub print_course_modification_page {
               '<th>'.$lt{'whom'}.'</th>'.
               &Apache::loncommon::end_data_table_header_row()."\n");
     my %optionname;
-    $optionname{''} = &mt('Use domain default');
+    $optionname{''} = &mt('Use domain default'); 
     $optionname{'0'} = $dctitle;
     $optionname{'1'} = $cctitle;
     foreach my $item (@{$selfenrollrows}) {
@@ -1231,12 +1243,12 @@ sub print_course_modification_page {
             $checked{'1'} = ' checked="checked"';
         } else {
             $checked{''} = ' checked="checked"';
-        }
+        } 
         $r->print(&Apache::loncommon::start_data_table_row()."\n".
                  '<td>'.$selfenrolltitles->{$item}.'</td>'."\n".
                  '<td>'.&mt('[_1] configures',$default).'</td>'."\n".
                  '<td>');
-        foreach my $option ('','0','1') {
+        foreach my $option ('','0','1') {  
             $r->print('<span class="LC_nobreak"><label>'.
                       '<input type="radio" name="selfenrollmgr_'.$item.'" '.
                       'value="'.$option.'"'.$checked{$option}.$disabled.' />'.
@@ -1289,102 +1301,171 @@ sub print_selfenrollconfig {
     return;
 }
 
-sub print_default_overrides {
-    my ($r,$cdom,$cnum,$cdesc,$type,$readonly,$item) = @_;
-    my (%titles,$checkeddom,$checkedcrs,$divsty,$currcrsval,$crsdefault,%crschecked,
-        $helpfile,$title,$crselements,@currcrseditors);
-    %titles = &default_overrides_titles($type);
-    my ($title,$domdefdisplay,$settings,$optiontext,$options) =
-        &default_overrides_common($item,$cdom,$cnum,$type,\%titles);
+sub print_set_ltiauth {
+    my ($r,$cdom,$cnum,$cdesc,$type,$readonly) = @_;
+    my %lt = &Apache::lonlocal::texthash(
+                'requ' => 'Requirement for re-authentication for student LTI-limited launch of deep-linked item',
+                'link' => 'Link protection can be set to accept username for an enrolled student (if sent by Consumer)',
+                'logi' => 'Login needed, regardless of user information sent by LTI Consumer in (signed) parameters',
+                'used' => 'Use domain default',
+                'cour' => 'Use course-specific setting',
+                'curd' => 'Current domain default is',
+                'valu' => 'Value for this course',
+                'modi' => 'Save',
+                'back' => 'Pick another action',
+    );
+    my ($domdef,$checkeddom,$checkedcrs,$domdefdisplay,$divsty,$authok,$authno);
+    $domdef = 0;
     $checkeddom = ' checked="checked"';
+    $domdefdisplay = $lt{'logi'}; 
     $divsty = 'display:none';
-
-    if ($item eq 'ltiauth') {
-        $helpfile = 'Modify_Course_LTI_Authen';
-        $crsdefault = 0;
-    } else {
-        if ($item eq 'exttool') {
-            $helpfile = 'Modify_Course_External_Tool';
-            $crsdefault = 'both';
-        } elsif ($item eq 'crsauthor') {
-            $helpfile = 'Modify_Course_Crsauthor';
-            $crsdefault = 1;
-        } elsif ($item eq 'crseditors') {
-            $helpfile = 'Modify_Course_Resource_Editors';
-            $crsdefault = 'edit,xml';
-        }
+    $authno = ' checked="checked"';
+    my %domconfig =
+        &Apache::lonnet::get_dom('configuration',['coursedefaults'],$cdom);
+    if (ref($domconfig{'coursedefaults'}) eq 'HASH') {
+        $domdef = $domconfig{'coursedefaults'}{'ltiauth'};
     }
-    $currcrsval = $settings->{'internal.'.$item};
-    if ($currcrsval ne '') {
+    if ($domdef) {
+        $domdefdisplay = $lt{'link'};
+    }
+    my %settings = &Apache::lonnet::get('environment',['internal.ltiauth'],$cdom,$cnum);
+    my $ltiauth = $settings{'internal.ltiauth'};
+
+    if ($ltiauth ne '') {
         $checkedcrs = $checkeddom;
         $checkeddom = '';
         $divsty = 'display:inline-block';
-        if ($item eq 'crseditors') {
-            @currcrseditors = split(/,/,$currcrsval);
+        if ($ltiauth) {
+            $authok = ' checked="checked"';
         }
-        foreach my $option (@{$options}) {
-            if ($item eq 'crseditors') {
-                if (grep(/^\Q$option\E$/,@currcrseditors)) {
-                    $crschecked{$option} = ' checked="checked"';
-                }
-            } elsif ($currcrsval eq $option) {
-                $crschecked{$option} = ' checked="checked"';
-            } else {
-                $crschecked{$option} = '';
+    }
+    &print_header($r,$type);
+    my $hidden_elements = &hidden_form_elements();
+    my ($disabled,$submit);
+    if ($readonly) {
+        $disabled = ' disabled="disabled"';
+    } else {
+        $submit = '<input type="button" onclick="javascript:changePage(this.form,'."'processltiauth'".');" value="'.$lt{'modi'}.'" />';
+    }
+    my $helpitem = &Apache::loncommon::help_open_topic('Modify_Course_LTI_Authen');
+    my $showtype = &mt($type);
+    $r->print(<<ENDDOCUMENT);
+<h3>$helpitem $lt{'requ'}</h3>
+<h4><span class="LC_nobreak">$showtype: $cdesc</span></h4>
+<form action="/adm/modifycourse" method="post" name="setltiauth">
+<p><span class="LC_nobreak">$lt{'curd'}: <span style="font-style:italic">$domdefdisplay</span></span></p>
+<p><span class="LC_nobreak">
+<label><input type="radio" name="ltiauthset" value="dom" onclick="toggleLTIOptions(this.form);"$checkeddom$disabled />$lt{'used'}</label></span><br />
+<span class="LC_nobreak">
+<label><input type="radio" name="ltiauthset" value="course" onclick="toggleLTIOptions(this.form);"$checkedcrs$disabled />$lt{'cour'}</label></span></p>
+<fieldset id="crsltiauth" style="$divsty">
+<legend>$lt{'valu'}</legend>
+<span class="LC_nobreak">
+<label><input type="radio" name="ltiauth" value="0"$authno$disabled />$lt{'logi'}</label>
+</span><br />
+<span class="LC_nobreak">
+<label><input type="radio" name="ltiauth" value="1"$authok$disabled />$lt{'link'}</label>
+</span>
+</fieldset><p>
+$submit
+$hidden_elements
+</p>
+</form>
+ENDDOCUMENT
+    my @actions =
+        ('<a href="javascript:changePage(document.setltiauth,'."'menu'".')">'.
+         $lt{'back'}.'</a>');
+    $r->print('<br />'.&Apache::lonhtmlcommon::actionbox(\@actions));
+    return;
+}
+
+sub print_set_exttool {
+    my ($r,$cdom,$cnum,$cdesc,$type,$readonly) = @_;
+    my %titles = &exttool_titles($type);
+    my ($domdef,$domdefdom,$checkeddom,$checkedcrs,$domdefdisplay,$divsty);
+    $domdef = 0;
+    $domdefdom = 1;
+    $checkeddom = ' checked="checked"';
+    $divsty = 'display:none';
+    my %settings = &Apache::lonnet::get('environment',['internal.coursecode',
+                                        'internal.textbook'],$cdom,$cnum);
+    my $lctype = &get_lctype($type,\%settings);
+    my %domconfig =
+        &Apache::lonnet::get_dom('configuration',['coursedefaults'],$cdom);
+    if (ref($domconfig{'coursedefaults'}) eq 'HASH') {
+        if (ref($domconfig{'coursedefaults'}{'exttool'}) eq 'HASH') {
+            if (exists($domconfig{'coursedefaults'}{'exttool'}{$lctype})) {
+                $domdef = $domconfig{'coursedefaults'}{'exttool'}{$lctype};
+            }
+        }
+        if (ref($domconfig{'coursedefaults'}{'domexttool'}) eq 'HASH') {
+            if (exists($domconfig{'coursedefaults'}{'domexttool'}{$lctype})) {
+                $domdefdom = $domconfig{'coursedefaults'}{'domexttool'}{$lctype};
             }
         }
-    } else {
-        if ($item eq 'crseditors') {
-            my %domdefs = &Apache::lonnet::get_domain_defaults($cdom);
-            @currcrseditors = split(/,/,$domdefs{'crseditors'});
-        }
-        foreach my $option (@{$options}) {
-            if ($item eq 'crseditors') {
-                if (grep(/^\Q$option\E$/,@currcrseditors)) {
-                    $crschecked{$option} = ' checked="checked"';
-                }
-            } elsif ($crsdefault eq $option) {
-                $crschecked{$option} = ' checked="checked"';
+    }
+    if ($domdef && $domdefdom) {
+        $domdefdisplay = $titles{'both'};
+    } elsif ($domdef) {
+        $domdefdisplay = $titles{'crs'};
+    } elsif ($domdefdom) {
+        $domdefdisplay = $titles{'dom'}; 
+    } else {
+        $domdefdisplay = $titles{'none'};
+    }
+    my %settings = &Apache::lonnet::get('environment',['internal.exttool'],$cdom,$cnum);
+    my $crsexttool = $settings{'internal.exttool'};
+    my %crschecked = (
+                         both => ' checked="checked"',
+                         dom  => '',
+                         crs => '',
+                         none => '',
+                     ); 
+    if ($crsexttool ne '') {
+        $checkedcrs = $checkeddom;
+        $checkeddom = '';
+        $divsty = 'display:inline-block';
+        foreach my $option ('both','dom','crs','none') {
+            if ($crsexttool eq $option) {
+                $crschecked{$option} = ' checked="checked"'; 
             } else {
                 $crschecked{$option} = '';
             }
         }
     }
-    my ($disabled,$submit,$inputtype,$separator);
+    &print_header($r,$type);
+    my $hidden_elements = &hidden_form_elements();
+    my ($disabled,$submit);
     if ($readonly) {
         $disabled = ' disabled="disabled"';
     } else {
-        $submit = '<input type="button" onclick="javascript:changePage(this.form,'."'process${item}'".');" value="'.$titles{'modi'}.'" />';
-    }
-    if ($item eq 'crseditors') {
-        $inputtype = 'checkbox';
-        $separator = '&nbsp;'x2;
-    } else {
-        $inputtype = 'radio';
-        $separator = '<br />';
-    }
-    foreach my $option (@{$options}) {
-        $crselements .= '<span class="LC_nobreak">'.
-                        '<label><input type="'.$inputtype.'" name="'.$item.'" value="'.$option.'"'.
-                        $crschecked{$option}.$disabled.' />'.$optiontext->{$option}.'</label>'.
-                        '</span>'.$separator."\n";
+        $submit = '<input type="button" onclick="javascript:changePage(this.form,'."'processexttool'".');" value="'.$titles{'modi'}.'" />';
     }
-    &print_header($r,$type);
-    my $hidden_elements = &hidden_form_elements();
-    my $helpitem = &Apache::loncommon::help_open_topic($helpfile);
+    my $helpitem = &Apache::loncommon::help_open_topic('Modify_Course_External_Tool');
     my $showtype = &mt($type);
     $r->print(<<ENDDOCUMENT);
-<h3>$helpitem $title</h3>
+<h3>$helpitem $titles{'extt'}</h3>
 <h4><span class="LC_nobreak">$showtype: $cdesc</span></h4>
-<form action="/adm/modifycourse" method="post" name="set$item">
+<form action="/adm/modifycourse" method="post" name="setexttool">
 <p><span class="LC_nobreak">$titles{'curd'}: <span style="font-style:italic">$domdefdisplay</span></span></p>
 <p><span class="LC_nobreak">
-<label><input type="radio" name="${item}set" value="dom" onclick="toggleOptions(this.form,'set$item');"$checkeddom$disabled />$titles{'used'}</label></span><br />
+<label><input type="radio" name="exttoolset" value="dom" onclick="toggleExtToolOptions(this.form);"$checkeddom$disabled />$titles{'used'}</label></span><br />
 <span class="LC_nobreak">
-<label><input type="radio" name="${item}set" value="course" onclick="toggleOptions(this.form,'set$item');"$checkedcrs$disabled />$titles{'cour'}</label></span></p>
-<fieldset id="crs$item" style="$divsty">
+<label><input type="radio" name="exttoolset" value="course" onclick="toggleExtToolOptions(this.form);"$checkedcrs$disabled />$titles{'cour'}</label></span></p>
+<fieldset id="crsexttool" style="$divsty">
 <legend>$titles{'valu'}</legend>
-$crselements
+<span class="LC_nobreak">
+<label><input type="radio" name="exttool" value="both"$crschecked{'both'}$disabled />$titles{'both'}</label>
+</span><br />
+<span class="LC_nobreak">
+<label><input type="radio" name="exttool" value="dom"$crschecked{'dom'}$disabled />$titles{'dom'}</label>
+</span><br />
+<span class="LC_nobreak">
+<label><input type="radio" name="exttool" value="crs"$crschecked{'crs'}$disabled />$titles{'crs'}</label>
+</span><br />
+<span class="LC_nobreak">
+<label><input type="radio" name="exttool" value="none"$crschecked{'none'}$disabled />$titles{'none'}</label>
+</span>
 </fieldset><p>
 $submit
 $hidden_elements
@@ -1392,30 +1473,20 @@ $hidden_elements
 </form>
 ENDDOCUMENT
     my @actions =
-        ('<a href="javascript:changePage(document.set'.$item.','."'menu'".')">'.
+        ('<a href="javascript:changePage(document.setexttool,'."'menu'".')">'.
          $titles{'back'}.'</a>');
     $r->print('<br />'.&Apache::lonhtmlcommon::actionbox(\@actions));
     return;
 }
 
-sub default_overrides_titles {
+sub exttool_titles {
     my ($type) = @_;
     my %titles = &Apache::lonlocal::texthash(
-                'requ' => 'Requirement for re-authentication for student LTI-limited launch of deep-linked item',
-                'link' => 'Link protection can be set to accept username for an enrolled student (if sent by Consumer)',
-                'logi' => 'Login needed, regardless of user information sent by LTI Consumer in (signed) parameters',
                 'extt' => 'External Tool permissions',
                 'none' => 'Use of external tools not permitted',
                 'crs' => 'Only external tools defined in course may be used',
                 'dom' => 'Only external tools defined in domain may be used',
                 'both' => 'External tools defined/configured in either domain or course may be used',
-                'stan' => "'In-course' authoring of standard LON-CAPA problems",
-                'perc' => 'Permitted course resource editors',
-                'edit' => 'Standard editor (Edit)',
-                'xml'  => 'Text editor (EditXML)',
-                'daxe' => 'Daxe editor (Daxe)',
-                'on'   => 'In-course authoring available',
-                'off'  => 'In-course authoring unavailable',
                 'used' => 'Use domain default',
                 'cour' => 'Use course-specific setting',
                 'curd' => 'Current domain default is',
@@ -1432,102 +1503,6 @@ sub default_overrides_titles {
     return %titles;
 }
 
-sub default_overrides_common {
-    my ($item,$cdom,$cnum,$type,$titles) = @_;
-    my ($title,$domdef,$domdefdom,$domdefdisplay,%settings,%optiontext,@options);
-    if ($item eq 'ltiauth') {
-        %settings = &Apache::lonnet::get('environment',['internal.'.$item],$cdom,$cnum);
-        $title = $titles->{'requ'};
-        $domdef = 0;
-        @options = ('0','1');
-        %optiontext = (
-                        0 => $titles->{'logi'},
-                        1 => $titles->{'link'},
-                      );
-    } else {
-        %settings =
-            &Apache::lonnet::get('environment',['internal.'.$item,'internal.coursecode',
-                                                'internal.textbook'],$cdom,$cnum);
-        if ($item eq 'exttool') {
-            $title = $titles->{'extt'};
-            $domdefdom = 1;
-            $domdef = 0;
-            @options = ('both','dom','crs','none');
-            map { $optiontext{$_} = $titles->{$_}; } @options;
-        } elsif ($item eq 'crsauthor') {
-            $title = $titles->{'stan'};
-            $domdef = 1;
-            @options = ('1','0');
-            %optiontext = (
-                            1 => $titles->{'on'},
-                            0 => $titles->{'off'},
-                          );
-        } elsif ($item eq 'crseditors') {
-            $title = $titles->{'perc'};
-            $domdef = 1;
-            @options = ('edit','xml','daxe');
-            map { $optiontext{$_} = $titles->{$_}; } @options;
-        }
-    }
-    my %domconfig =
-        &Apache::lonnet::get_dom('configuration',['coursedefaults'],$cdom);
-    if (ref($domconfig{'coursedefaults'}) eq 'HASH') {
-        if (($item eq 'ltiauth') || ($item eq 'crseditors')) {
-            $domdef = $domconfig{'coursedefaults'}{$item};
-        } else {
-            my $lctype = &get_lctype($type,\%settings);
-            if (ref($domconfig{'coursedefaults'}{$item}) eq 'HASH') {
-                if (exists($domconfig{'coursedefaults'}{$item}{$lctype})) {
-                    $domdef = $domconfig{'coursedefaults'}{$item}{$lctype};
-                }
-            }
-            if ($item eq 'exttool') {
-                if (ref($domconfig{'coursedefaults'}{'domexttool'}) eq 'HASH') {
-                    if (exists($domconfig{'coursedefaults'}{'domexttool'}{$lctype})) {
-                        $domdefdom = $domconfig{'coursedefaults'}{'domexttool'}{$lctype};
-                    }
-                }
-            }
-        }
-    }
-    if ($item eq 'ltiauth') {
-        if ($domdef) {
-            $domdefdisplay = $titles->{'link'};
-        } else {
-            $domdefdisplay = $titles->{'logi'};
-        }
-    } elsif ($item eq 'exttool') {
-        if ($domdef && $domdefdom) {
-            $domdefdisplay = $titles->{'both'};
-        } elsif ($domdef) {
-            $domdefdisplay = $titles->{'crs'};
-        } elsif ($domdefdom) {
-            $domdefdisplay = $titles->{'dom'};
-        } else {
-            $domdefdisplay = $titles->{'none'};
-        }
-    } elsif ($item eq 'crsauthor') {
-        if ($domdef) {
-            $domdefdisplay = $titles->{'on'};
-        } else {
-            $domdefdisplay = $titles->{'off'};
-        }
-    } elsif ($item eq 'crseditors') {
-        if (ref($domdef) eq 'ARRAY') {
-            if (@{$domdef} == 0) {
-                $domdefdisplay = &mt('No permitted editors');
-            } elsif (@{$domdef} == 1) {
-                $domdefdisplay = $titles->{$domdef->[0]}.' ('.&mt('only').')';
-            } else {
-                $domdefdisplay = join(', ', map { $titles->{$_}; } @{$domdef});
-            }
-        } else {
-            $domdefdisplay = join(', ', map { $titles->{$_}; } ('edit','xml'));
-        }
-    }
-    return ($title,$domdefdisplay,\%settings,\%optiontext,\@options);
-}
-
 sub modify_selfenrollconfig {
     my ($r,$type,$cdesc,$coursehash) = @_;
     return unless(ref($coursehash) eq 'HASH');
@@ -1602,7 +1577,7 @@ sub gather_authenitems {
     unless ($curr_authtype eq '') {
         $curr_authfield = $curr_authtype.'arg';
     }
-    my $javascript_validations =
+    my $javascript_validations = 
         &Apache::lonuserutils::javascript_validations('modifycourse',$krbdefdom,
                                                       $curr_authtype,$curr_authfield);
     my %param = ( formname => 'document.'.$env{'form.phase'},
@@ -1633,10 +1608,10 @@ 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)) {
+        if (&showcredits($cdom)) {  
             push(@items,'internal.defaultcredits');
         }
         my %passwdconf = &Apache::lonnet::get_passwdconf($cdom);
@@ -1759,7 +1734,7 @@ sub modify_course {
         $newattr{'courseowner'}=$env{'form.courseowner'};
         unless ( $newattr{'courseowner'} eq $currattr{'courseowner'} ) {
             $changed{'owner'} = 1;
-        }
+        } 
     }
 
     if ($changed{'owner'} || $changed{'code'} || $changed{'passwd'}) {
@@ -1826,7 +1801,7 @@ sub modify_course {
         } else {
             if ($env{'course.'.$cdom.'_'.$cnum.'.description'} ne '') {
                 my %newenv;
-                map { $newenv{'course.'.$cdom.'_'.$cnum.'.internal.'.$_} = $newattr{$_}; } @changes;
+                map { $newenv{'course.'.$cdom.'_'.$cnum.'.internal.'.$_} = $newattr{$_}; } @changes;   
                 &Apache::lonnet::appenv(\%newenv);
             }
             foreach my $attr (@modifiable_params) {
@@ -1868,7 +1843,7 @@ sub modify_course {
                     $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.'));
@@ -1885,6 +1860,7 @@ sub modify_course {
 			            if ($course_check eq 'ok') {
                                         my $outcome = &Apache::lonnet::auto_new_course($cnum,$cdom,$inst_course_id,$newattr{'courseowner'},$coowners);
 			                unless ($outcome eq 'ok') {
+                               
 				            push(@warnings,&mt('If automatic enrollment is enabled for "[_1]", automated enrollment may fail for "[_2]" - section: [_3] for the following reason: "[_4]".',$description,$newattr{'coursecode'},$instsec,$outcome).'<br/>');
 			                }
 			            } else {
@@ -1948,7 +1924,7 @@ sub modify_course {
         $nochgresponse .=  "</ul><br/><br/>";
     }
     my ($warning,$numwarnings);
-    my $numwarnings = scalar(@warnings);
+    my $numwarnings = scalar(@warnings); 
     if ($numwarnings) {
         $warning = &mt('The following [quant,_1,warning was,warnings were] generated when applying your changes to automated enrollment:',$numwarnings).'<p><ul>';
         foreach my $warn (@warnings) {
@@ -2180,7 +2156,7 @@ sub modify_quota {
                     $showresult{$item} = 1;
                 }
             } else {
-                if ($item eq 'coursequota') {
+                if ($item eq 'coursequota') { 
                     $r->print(&mt('The proposed group portfolio quota contained invalid characters, so the quota is unchanged.').'<br />');
                 } else {
                     $r->print(&mt('The proposed quota for content uploaded via the Content Editor contained invalid characters, so the quota is unchanged.').'<br />');
@@ -2193,7 +2169,7 @@ sub modify_quota {
         my $putreply = &Apache::lonnet::put('environment',\%cenv,$cdom,
                                             $cnum);
         foreach my $key (sort(keys(%showresult))) {
-            if (($oldsettings{'internal.'.$key} eq '') &&
+            if (($oldsettings{'internal.'.$key} eq '') && 
                 ($env{'form.'.$key} == $default{$key})) {
                 if ($key eq 'uploadquota') {
                     if ($type eq 'Community') {
@@ -2203,7 +2179,7 @@ sub modify_quota {
                         $r->print(&mt('The disk space allocated for files uploaded to this course via the Content Editor is the default quota for this domain: [_1] MB.',
                                       $default{$key}).'<br />');
                     }
-                } else {
+                } else { 
                     if ($type eq 'Community') {
                         $r->print(&mt('The disk space allocated for group portfolio files in this community is the default quota for this domain: [_1] MB.',
                                       $default{$key}).'<br />');
@@ -2263,7 +2239,7 @@ sub modify_anonsurvey_threshold {
     my %oldsettings = &Apache::lonnet::get('environment',['internal.anonsurvey_threshold'],$cdom,$cnum);
     my %domconfig =
         &Apache::lonnet::get_dom('configuration',['coursedefaults'],$cdom);
-    my $defaultthreshold;
+    my $defaultthreshold; 
     if (ref($domconfig{'coursedefaults'}) eq 'HASH') {
         $defaultthreshold = $domconfig{'coursedefaults'}{'anonsurvey_threshold'};
         if ($defaultthreshold eq '') {
@@ -2330,14 +2306,14 @@ sub modify_postsubmit_config {
     $r->print('<h3>'.$lt{'subb'}.'</h3>'."\n".
               '<h4><span class="LC_nobreak">'.&mt($type).': '.$cdesc.'</span></h4>'."\n".
               '<form action="/adm/modifycourse" method="post" name="processpostsubmit"><p>'."\n");
-    my %oldsettings =
+    my %oldsettings = 
         &Apache::lonnet::get('environment',['internal.postsubmit','internal.postsubtimeout','internal.coursecode','internal.textbook'],$cdom,$cnum);
     my $postsubmit = $env{'form.postsubmit'};
     if ($postsubmit eq '1') {
         my $postsubtimeout = $env{'form.postsubtimeout'};
         $postsubtimeout =~ s/[^\d\.]+//g;
         if (($oldsettings{'internal.postsubmit'} eq $postsubmit) && ($oldsettings{'internal.postsubtimeout'} eq $postsubtimeout)) {
-            $r->print($lt{'unch'});
+            $r->print($lt{'unch'}); 
         } else {
             my %cenv = (
                          'internal.postsubmit' => $postsubmit,
@@ -2353,12 +2329,12 @@ sub modify_postsubmit_config {
                 } else {
                     $r->print($lt{'erro'});
                 }
-            } else {
+            } else { 
                 $cenv{'internal.postsubtimeout'} = $postsubtimeout;
                 my $putreply = &Apache::lonnet::put('environment',\%cenv,$cdom,$cnum);
                 if ($putreply eq 'ok') {
                     if ($postsubtimeout eq '0') {
-                        $r->print(&mt('Submit button will be disabled after student submission until page is reloaded.'));
+                        $r->print(&mt('Submit button will be disabled after student submission until page is reloaded.')); 
                     } else {
                         $r->print(&mt('Submit button will be disabled after student submission for [quant,_1,second].',$postsubtimeout));
                     }
@@ -2372,7 +2348,7 @@ sub modify_postsubmit_config {
             $r->print($lt{'unch'});
         } else {
             if (exists($oldsettings{'internal.postsubtimeout'})) {
-                &Apache::lonnet::del('environment',['internal.postsubtimeout'],$cdom,$cnum);
+                &Apache::lonnet::del('environment',['internal.postsubtimeout'],$cdom,$cnum);  
             }
             my %cenv = (
                          'internal.postsubmit' => $postsubmit,
@@ -2417,7 +2393,7 @@ sub modify_catsettings {
         my (%cenv,@changes,@nochanges);
         my %currsettings =
             &Apache::lonnet::get('environment',['hidefromcat','categories'],$cdom,$cnum);
-        my (@newcategories,%showitem);
+        my (@newcategories,%showitem); 
         if (grep(/^togglecats$/,@cat_params)) {
             if ($currsettings{'hidefromcat'} ne $env{'form.hidefromcat'}) {
                 push(@changes,'hidefromcat');
@@ -2463,9 +2439,9 @@ sub modify_catsettings {
                     foreach my $cat (@newcategories) {
                         if (!grep(/^\Q$cat\E$/,@currcategories)) {
                             $catchg = 1;
-                            last;
-                        }
-                    }
+                            last;                     
+                        } 
+                    } 
                 }
             } else {
                 if (@newcategories > 0) {
@@ -2533,87 +2509,182 @@ sub modify_catsettings {
     return;
 }
 
-sub modify_default_overrides {
-    my ($r,$cdom,$cnum,$cdesc,$domdesc,$type,$item) = @_;
-    my (%titles,$oldcrsval,$newcrsval,%resulttext,$itemvalue,$nochange,$change,$status,$error);
-    %titles = &default_overrides_titles($type);
-    my ($title,$domdefdisplay,$oldsettings,$optiontext,$options) =
-        &default_overrides_common($item,$cdom,$cnum,$type,\%titles);
-    $oldcrsval = $oldsettings->{'internal.'.$item};
-    if ($item eq 'ltiauth') {
-        %resulttext =
-            &Apache::lonlocal::texthash(
-                                         chg   => 'Re-authentication requirement for LTI launch of deep-linked changed',
-                                         nochg => 'Re-authentication requirement for LTI launch of deep-linked item is unchanged',
-                                       );
-    } elsif ($item eq 'exttool') {
-        %resulttext =
-            &Apache::lonlocal::texthash(
-                                         chg   => 'External Tool permissions changed',
-                                         nochg => 'External Tool permissions unchanged',
-                                       );
-    } elsif ($item eq 'crsauthor') {
-        %resulttext =
-            &Apache::lonlocal::texthash(
-                                         chg   => 'In-course authoring permissions changed',
-                                         nochg => 'In-course authoring permissions unchanged',
-                                       );
-    } elsif ($item eq 'crseditors') {
-        %resulttext =
-            &Apache::lonlocal::texthash(
-                                         chg   => 'Permitted course resource editors changed',
-                                         nochg => 'Permitted course resource editors unchanged',
-                                       );
-    }
+sub modify_ltiauth {
+    my ($r,$cdom,$cnum,$cdesc,$domdesc,$type) = @_;
+    my %lt = &Apache::lonlocal::texthash(
+                'requ' => 'Requirement for re-authentication for student LTI-limited launch of deep-linked item',
+                'link' => 'Link protection can be set to accept username for an enrolled student (if sent by Consumer)',
+                'logi' => 'Login needed, regardless of user information sent by LTI Consumer in (signed) parameters',
+                'used' => 'Use domain default',
+                'cour' => 'Use course-specific setting',
+                'modi' => 'Save',
+                'back' => 'Pick another action',
+    );
     &print_header($r,$type);
-    $r->print('<h3>'.$title.'</h3>'."\n".
+    $r->print('<h3>'.$lt{'requ'}.'</h3>'."\n".
               '<h4><span class="LC_nobreak">'.&mt($type).': '.$cdesc.'</span></h4>'."\n".
-              '<form action="/adm/modifycourse" method="post" name="process'.$item.'">'."\n");
-    if ($env{'form.'.$item.'set'} eq 'dom') {
-        if ($oldcrsval eq '') {
+              '<form action="/adm/modifycourse" method="post" name="processltiauth">'."\n");
+    my %oldsettings = &Apache::lonnet::get('environment',['internal.ltiauth'],$cdom,$cnum);
+    my $oldltiauth = $oldsettings{'internal.ltiauth'};
+    my $domdef;
+    my %domconfig =
+        &Apache::lonnet::get_dom('configuration',['coursedefaults'],$cdom);
+    if (ref($domconfig{'coursedefaults'}) eq 'HASH') {
+        $domdef = $domconfig{'coursedefaults'}{'ltiauth'};
+    }
+    my ($newltiauth,$nochange,$change,$status,$error,$ltiauth);
+    if ($env{'form.ltiauthset'} eq 'dom') {
+        if ($oldltiauth eq '') {
             $nochange = 1;
         } else {
             $change = 1;
         }
-    } elsif ($env{'form.'.$item.'set'} eq 'course') {
-        if ($item eq 'crseditors') {
-            my @neweditors;
-            my @posseditors = &Apache::loncommon::get_env_multiple('form.'.$item);
-            foreach my $editor (@posseditors) {
-                if (grep(/^\Q$editor\E$/,@{$options})) {
-                    push(@neweditors,$editor);
+    } elsif ($env{'form.ltiauthset'} eq 'course') {
+        if ($env{'form.ltiauth'} =~ /^0|1$/) { 
+            $newltiauth = $env{'form.ltiauth'};
+        }
+        if ($oldltiauth == $newltiauth) {
+            $nochange = 1;
+        } else {
+            $change = 1;
+        }
+    }
+    if ($change) {
+        if ($newltiauth ne '') {
+            my %cenv = (
+                         'internal.ltiauth' => $newltiauth,
+                       );
+            if (&Apache::lonnet::put('environment',\%cenv,$cdom,$cnum) eq 'ok') {
+                if ($env{'course.'.$cdom.'_'.$cnum.'.description'} ne '') {
+                    &Apache::lonnet::appenv(
+                       {'course.'.$cdom.'_'.$cnum.'.internal.ltiauth' => $newltiauth});
                 }
+            } else {
+                $error = 1;
             }
-            $newcrsval = join(',',@neweditors);
         } else {
-            my $posscrsval = $env{'form.'.$item};
-            if (grep(/^\Q$posscrsval\E$/,@{$options})) {
-                $newcrsval = $posscrsval;
+            if (&Apache::lonnet::del('environment',['internal.ltiauth'],$cdom,$cnum) eq 'ok') {
+                if (exists($env{'course.'.$cdom.'_'.$cnum.'.internal.ltiauth'})) {
+                    &Apache::lonnet::delenv('course.'.$cdom.'_'.$cnum.'.internal.ltiauth');
+                }
+            } else {
+                $error = 1;
+            }
+        }
+    }
+    if ($error) {
+        $nochange = 1;
+    }
+    if ($nochange) {
+        $ltiauth = $oldltiauth;
+    } else {
+        $ltiauth = $newltiauth;
+    }
+    if ($ltiauth eq '') {
+        $status = $lt{'used'}.': ';
+        if ($domdef) {
+            $status .= '<span style="font-style:italic">'.$lt{'link'}.'</span>';
+        } else {
+            $status .= '<span style="font-style:italic">'.$lt{'logi'}.'</span>';
+        }
+    } else {
+        $status = $lt{'cour'}.': ';
+        if ($ltiauth) {
+            $status .= '<span style="font-style:italic">'.$lt{'link'}.'</span>';
+        } else {
+            $status .= '<span style="font-style:italic">'.$lt{'logi'}.'</span>';
+        }
+    }
+    if ($error) {
+        $r->print('<p class="LC_warning">'.&mt('An error occurred when saving your changes').'</p>');
+    }
+    $r->print('<p>');
+    if ($nochange) {
+        $r->print(&mt('Re-authentication requirement for LTI launch of deep-linked item is unchanged'));
+    } elsif ($change) {
+        $r->print(&mt('Re-authentication requirement for LTI launch of deep-linked changed'));
+    }
+    $r->print('<br />'.$status.'</p>'.
+              &hidden_form_elements().'</form>');
+    my @actions =
+        ('<a href="javascript:changePage(document.processltiauth,'."'menu'".')">'.
+         $lt{'back'}.'</a>');
+    $r->print('<br />'.&Apache::lonhtmlcommon::actionbox(\@actions));
+    return;
+}
+
+sub modify_exttool {
+    my ($r,$cdom,$cnum,$cdesc,$domdesc,$type) = @_;
+    my %titles = &exttool_titles($type);
+    &print_header($r,$type);
+    $r->print('<h3>'.$titles{'extt'}.'</h3>'."\n".
+              '<h4><span class="LC_nobreak">'.$type.': '.$cdesc.'</span></h4>'."\n".
+              '<form action="/adm/modifycourse" method="post" name="processexttool">'."\n");
+    my %oldsettings = &Apache::lonnet::get('environment',['internal.exttool'],$cdom,$cnum);
+    my $oldcrsexttool = $oldsettings{'internal.exttool'};
+    my $domdefdom = 1;
+    my $domdef = 0;
+    my $domdefdisplay;
+    my %settings = &Apache::lonnet::get('environment',['internal.coursecode',
+                                        'internal.textbook'],$cdom,$cnum);
+    my $lctype = &get_lctype($type,\%settings);
+    my %domconfig =
+        &Apache::lonnet::get_dom('configuration',['coursedefaults'],$cdom);
+    if (ref($domconfig{'coursedefaults'}) eq 'HASH') {
+        if (ref($domconfig{'coursedefaults'}{'domexttool'}) eq 'HASH') {
+            if (exists($domconfig{'coursedefaults'}{'domexttool'}{$lctype})) {
+                $domdefdom = $domconfig{'coursedefaults'}{'domexttool'}{$lctype};
+            }
+        }
+        if (ref($domconfig{'coursedefaults'}{'exttool'}) eq 'HASH') {
+            if (exists($domconfig{'coursedefaults'}{'exttool'}{$lctype})) {
+                $domdef = $domconfig{'coursedefaults'}{'exttool'}{$lctype};
             }
         }
-        if ($oldcrsval eq $newcrsval) {
+    }
+    if ($domdef && $domdefdom) {
+        $domdefdisplay = $titles{'both'};
+    } elsif ($domdef) {
+        $domdefdisplay = $titles{'crs'};
+    } elsif ($domdefdom) {
+        $domdefdisplay = $titles{'dom'};
+    } else {
+        $domdefdisplay = $titles{'none'};
+    }
+    my ($newcrsexttool,$nochange,$change,$status,$error,$exttool);
+    if ($env{'form.exttoolset'} eq 'dom') {
+        if ($oldcrsexttool eq '') {
+            $nochange = 1;
+        } else {
+            $change = 1;
+        }
+    } elsif ($env{'form.exttoolset'} eq 'course') {
+        if ($env{'form.exttool'} =~ /^both|dom|crs|none$/) {
+            $newcrsexttool = $env{'form.exttool'};
+        }
+        if ($oldcrsexttool eq $newcrsexttool) {
             $nochange = 1;
         } else {
             $change = 1;
         }
     }
     if ($change) {
-        if ($newcrsval ne '') {
+        if ($newcrsexttool ne '') {
             my %cenv = (
-                         'internal.'.$item => $newcrsval,
+                         'internal.exttool' => $newcrsexttool,
                        );
             if (&Apache::lonnet::put('environment',\%cenv,$cdom,$cnum) eq 'ok') {
                 if ($env{'course.'.$cdom.'_'.$cnum.'.description'} ne '') {
                     &Apache::lonnet::appenv(
-                       {'course.'.$cdom.'_'.$cnum.'.internal.'.$item => $newcrsval});
+                       {'course.'.$cdom.'_'.$cnum.'.internal.exttool' => $newcrsexttool});
                 }
             } else {
                 $error = 1;
             }
         } else {
-            if (&Apache::lonnet::del('environment',['internal.'.$item],$cdom,$cnum) eq 'ok') {
-                if (exists($env{'course.'.$cdom.'_'.$cnum.'.internal.'.$item})) {
-                    &Apache::lonnet::delenv('course.'.$cdom.'_'.$cnum.'.internal.'.$item);
+            if (&Apache::lonnet::del('environment',['internal.exttool'],$cdom,$cnum) eq 'ok') {
+                if (exists($env{'course.'.$cdom.'_'.$cnum.'.internal.exttool'})) {
+                    &Apache::lonnet::delenv('course.'.$cdom.'_'.$cnum.'.internal.exttool');
                 }
             } else {
                 $error = 1;
@@ -2624,33 +2695,28 @@ sub modify_default_overrides {
         $nochange = 1;
     }
     if ($nochange) {
-        $itemvalue = $oldcrsval;
+        $exttool = $oldcrsexttool;
     } else {
-        $itemvalue = $newcrsval;
+        $exttool = $newcrsexttool;
     }
-    if ($itemvalue eq '') {
-        $status = $titles{'used'}.': '.
-                  '<span style="font-style:italic">'.$domdefdisplay.'</span>';
-    } elsif ($item eq 'crseditors') {
-        $status = $titles{'cour'}.': '.
-                  '<span style="font-style:italic">'.join(', ', map { $titles{$_}; } split(/,/,$itemvalue)).'</span>';
+    if ($exttool eq '') {
+        $status = $titles{'used'}.': <span style="font-style:italic">'.$domdefdisplay.'</span>';
     } else {
-        $status = $titles{'cour'}.': '.
-                  '<span style="font-style:italic">'.$optiontext->{$itemvalue}.'</span>';
+        $status = $titles{'cour'}.': <span style="font-style:italic">'.$titles{$exttool}.'</span>';
     }
     if ($error) {
         $r->print('<p class="LC_warning">'.&mt('An error occurred when saving your changes').'</p>');
     }
     $r->print('<p>');
     if ($nochange) {
-        $r->print($resulttext{'nochg'});
+        $r->print(&mt('External Tool permissions unchanged'));
     } elsif ($change) {
-        $r->print($resulttext{'chg'});
+        $r->print(&mt('External Tool permissions changed'));
     }
     $r->print('<br />'.$status.'</p>'.
               &hidden_form_elements().'</form>');
     my @actions =
-        ('<a href="javascript:changePage(document.process'.$item.','."'menu'".')">'.
+        ('<a href="javascript:changePage(document.processexttool,'."'menu'".')">'.
          $titles{'back'}.'</a>');
     $r->print('<br />'.&Apache::lonhtmlcommon::actionbox(\@actions));
     return;
@@ -2742,7 +2808,7 @@ ENDSCRIPT
         &js_escape(\$alert);
         my $regexp = ' /^\s*\d+\s*$/';
 
-        $js .= <<"ENDSCRIPT";
+        $js .= <<"ENDSCRIPT"; 
 
 function verify_postsubmit() {
     var optionsElement = document.setpostsubmit.postsubmit;
@@ -2774,7 +2840,7 @@ function verify_postsubmit() {
             }
         } else {
             if (currval == 0) {
-               verified = 'ok';
+               verified = 'ok'; 
             } else {
                alert('$invalid');
                return false;
@@ -2792,7 +2858,7 @@ function togglePostsubmit(caller) {
     var optionsElement = document.setpostsubmit.postsubmit;
     if (document.getElementById(caller)) {
         var divitem = document.getElementById(caller);
-        var optionsElement = document.setpostsubmit.postsubmit;
+        var optionsElement = document.setpostsubmit.postsubmit; 
         if (optionsElement.length) {
             var currval;
             for (var i=0; i<optionsElement.length; i++) {
@@ -2812,25 +2878,40 @@ function togglePostsubmit(caller) {
 
 ENDSCRIPT
 
-    } elsif (($phase eq 'setltiauth') || ($phase eq 'setexttool') ||
-             ($phase eq 'setcrseditors')) {
+    } elsif ($phase eq 'setltiauth') {
         $js .= <<"ENDJS";
-function toggleOptions(form,phase) {
-    var radioname;
-    var divid;
-    if (phase == 'setltiauth') {
-        radioname = 'ltiauthset';
-        divid = 'crsltiauth';
-    } else if (phase == 'setexttool') {
-        radioname = 'exttoolset';
-        divid = 'crsexttool';
-    } else if (phase == 'setcrsauthor') {
-        radioname = 'crsauthorset';
-        divid = 'crscrsauthor';
-    } else if (phase == 'setcrseditors') {
-        radioname = 'crseditorsset';
-        divid = 'crscrseditors';
+function toggleLTIOptions(form) {
+    var radioname = 'ltiauthset';
+    var divid = 'crsltiauth';
+    var num = form.elements[radioname].length;
+    if (num) {
+        var setvis = '';
+        for (var i=0; i<num; i++) {
+            if (form.elements[radioname][i].checked) {
+                if (form.elements[radioname][i].value == 'course') {
+                    if (document.getElementById(divid)) {
+                        document.getElementById(divid).style.display = 'inline-block';
+                    }
+                    setvis = 1;
+                }
+                break;
+            }
+        }
+        if (!setvis) {
+            if (document.getElementById(divid)) {
+                document.getElementById(divid).style.display = 'none';
+            }
+        }
     }
+    return;
+}
+
+ENDJS
+    } elsif ($phase eq 'setexttool') {
+        $js .= <<"ENDJS";
+function toggleExtToolOptions(form) {
+    var radioname = 'exttoolset';
+    var divid = 'crsexttool';
     var num = form.elements[radioname].length;
     if (num) {
         var setvis = '';
@@ -2865,9 +2946,13 @@ ENDJS
         $starthash = {
            add_entries => {'onload' => "hide_searching(); courseSet(document.filterpicker.official, 'load');"},
                      };
-    } elsif ($env{'form.phase'} =~ /^set(ltiauth|exttool|crseditors)$/) {
+    } elsif ($env{'form.phase'} eq 'setltiauth') {
+        $starthash = {
+           add_entries => {'onload' => "toggleLTIOptions(document.setltiauth);"},
+                     };
+    } elsif ($env{'form.phase'} eq 'setexttool') {
         $starthash = {
-           add_entries => {'onload' => "toggleOptions(document.$env{'form.phase'},'$env{'form.phase'}');"},
+           add_entries => {'onload' => "toggleExtToolOptions(document.setexttool);"},
                      };
     }
     $r->print(&Apache::loncommon::start_page('View/Modify Course/Community Settings',
@@ -2876,6 +2961,8 @@ ENDJS
     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';
     }
@@ -2968,15 +3055,14 @@ sub course_settings_descrip {
 }
 
 sub hidden_form_elements {
-    my $hidden_elements =
+    my $hidden_elements = 
       &Apache::lonhtmlcommon::echo_form_input(['gosearch','updater','coursecode',
           'prevphase','numlocalcc','courseowner','login','coursequota','intarg',
           'locarg','krbarg','krbver','counter','hidefromcat','usecategory',
           'threshold','postsubmit','postsubtimeout','defaultcredits','uploadquota',
           'selfenrollmgrdc','selfenrollmgrcc','action','state','currsec_st',
           'sections','newsec','mysqltables','nopasswdchg','ltiauth','ltiauthset',
-          'exttoolset','exttool','crseditorsset','crseditors'],
-          ['^selfenrollmgr_','^selfenroll_'])."\n".
+          'exttoolset','exttool'],['^selfenrollmgr_','^selfenroll_'])."\n".
           '<input type="hidden" name="prevphase" value="'.$env{'form.phase'}.'" />';
     return $hidden_elements;
 }
@@ -3013,8 +3099,6 @@ sub get_permission {
             processltiauth    => 'edit',
             setexttool        => 'edit',
             processexttool    => 'edit',
-            setcrseditors     => 'edit',
-            processcrseditors => 'edit',
         );
         if ($passwdconf{'crsownerchg'}) {
             $permission{passwdchg} = 'edit';
@@ -3032,7 +3116,6 @@ sub get_permission {
             adhocrole     => 'custom',
             setltiauth    => 'view',
             setexttool    => 'view',
-            setcrseditors => 'view',
         );
         if ($passwdconf{'crsownerchg'}) {
             $permission{passwdchg} = 'view';
@@ -3098,9 +3181,11 @@ 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";
-            }
+            } 
             &Apache::lonhtmlcommon::add_breadcrumb
             ({href=>"javascript:changePage(document.$firstform,'')",
               text=>"Course/Community search"},
@@ -3114,6 +3199,8 @@ sub handler {
                     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';
                     }
@@ -3136,11 +3223,11 @@ sub handler {
                         my ($cdom,$cnum) = split(/_/,$env{'form.pickedcourse'});
                         my ($readonly,$linktext);
                         if ($permission->{$phase} eq 'view') {
-                           $readonly = 1;
+                           $readonly = 1; 
                         }
                         if (($phase eq 'setquota') && ($permission->{'setquota'})) {
                             if ($permission->{'setquota'} eq 'view') {
-                                $linktext = 'Set quota';
+                                $linktext = 'Set quota'; 
                             } else {
                                 $linktext = 'Display quota';
                             }
@@ -3148,7 +3235,7 @@ sub handler {
                             ({href=>"javascript:changePage(document.$phase,'$phase')",
                               text=>$linktext});
                             &print_setquota($r,$cdom,$cnum,$cdesc,$type,$readonly);
-                        } elsif (($phase eq 'processquota') && ($permission->{'processquota'})) {
+                        } elsif (($phase eq 'processquota') && ($permission->{'processquota'})) { 
                             &Apache::lonhtmlcommon::add_breadcrumb
                             ({href=>"javascript:changePage(document.$phase,'setquota')",
                               text=>"Set quota"});
@@ -3227,7 +3314,7 @@ sub handler {
                             if (!exists($env{'form.state'})) {
                                 &print_selfenrollconfig($r,$type,$cdesc,$coursehash,$readonly);
                             } elsif ($env{'form.state'} eq 'done') {
-                                &Apache::lonhtmlcommon::add_breadcrumb
+                                &Apache::lonhtmlcommon::add_breadcrumb 
                                 ({href=>"javascript:changePage(document.$phase,'$phase')",
                                   text=>"Result"});
                                 &modify_selfenrollconfig($r,$type,$cdesc,$coursehash);
@@ -3235,45 +3322,35 @@ sub handler {
                         } elsif (($phase eq 'setltiauth') && ($permission->{'setltiauth'})) {
                             &Apache::lonhtmlcommon::add_breadcrumb
                             ({href=>"javascript:changePage(document.$phase,'$phase')",
-                              text=>"Authentication post-LTI launch"});
-                            &print_default_overrides($r,$cdom,$cnum,$cdesc,$type,$readonly,'ltiauth');
+                              text=>"Requirement for re-authentication for LTI launch of deep-linked item"});
+                            &print_set_ltiauth($r,$cdom,$cnum,$cdesc,$type,$readonly);
                         } elsif (($phase eq 'processltiauth') && ($permission->{'processltiauth'})) {
                             &Apache::lonhtmlcommon::add_breadcrumb
                             ({href=>"javascript:changePage(document.$phase,'setltiauth')",
-                              text=>"Authentication post-LTI launch"},
+                              text=>"Requirement for re-authentication for LTI launch of deep-linked item"},
                              {href=>"javascript:changePage(document.$phase,'$phase')",
                               text=>"Result"});
-                            &modify_default_overrides($r,$cdom,$cnum,$cdesc,$domdesc,$type,'ltiauth');
+                            &modify_ltiauth($r,$cdom,$cnum,$cdesc,$domdesc,$type);
                         } elsif (($phase eq 'setexttool') && ($permission->{'setexttool'})) {
                             &Apache::lonhtmlcommon::add_breadcrumb
                             ({href=>"javascript:changePage(document.$phase,'$phase')",
                               text=>"External Tool permission"});
-                            &print_default_overrides($r,$cdom,$cnum,$cdesc,$type,$readonly,'exttool');
+                            &print_set_exttool($r,$cdom,$cnum,$cdesc,$type,$readonly);
                         } elsif (($phase eq 'processexttool') && ($permission->{'processexttool'})) {
                             &Apache::lonhtmlcommon::add_breadcrumb
                             ({href=>"javascript:changePage(document.$phase,'setexttool')",
                               text=>"External Tool permission"},
                              {href=>"javascript:changePage(document.$phase,'$phase')",
                               text=>"Result"});
-                            &modify_default_overrides($r,$cdom,$cnum,$cdesc,$domdesc,$type,'exttool');
-                        } elsif (($phase eq 'setcrseditors') && ($permission->{'setcrseditors'})) {
-                            &Apache::lonhtmlcommon::add_breadcrumb
-                            ({href=>"javascript:changePage(document.$phase,'$phase')",
-                              text=>"Available course resource editors"});
-                            &print_default_overrides($r,$cdom,$cnum,$cdesc,$type,$readonly,'crseditors');
-                        } elsif (($phase eq 'processcrseditors') && ($permission->{'processcrseditors'})) {
-                            &Apache::lonhtmlcommon::add_breadcrumb
-                            ({href=>"javascript:changePage(document.$phase,'setcrseditors')",
-                              text=>"Available course resource editors"},
-                             {href=>"javascript:changePage(document.$phase,'$phase')",
-                              text=>"Result"});
-                            &modify_default_overrides($r,$cdom,$cnum,$cdesc,$domdesc,$type,'crseditors');
+                            &modify_exttool($r,$cdom,$cnum,$cdesc,$domdesc,$type);
                         }
                     }
                 } else {
                     $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'));
                     }