--- loncom/interface/lonmodifycourse.pm	2016/04/04 01:12:11	1.82
+++ loncom/interface/lonmodifycourse.pm	2016/07/24 14:34:59	1.84
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # handler for DC-only modifiable course settings
 #
-# $Id: lonmodifycourse.pm,v 1.82 2016/04/04 01:12:11 raeburn Exp $
+# $Id: lonmodifycourse.pm,v 1.84 2016/07/24 14:34:59 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -41,7 +41,7 @@ use LONCAPA qw(:DEFAULT :match);
 
 sub get_dc_settable {
     my ($type,$cdom) = @_;
-    if (($type eq 'Community') || ($type eq 'Placement')) {
+    if ($type eq 'Community') {
         return ('courseowner','selfenrollmgrdc','selfenrollmgrcc');
     } else {
         my @items = ('courseowner','coursecode','authtype','autharg','selfenrollmgrdc','selfenrollmgrcc');
@@ -55,7 +55,7 @@ sub get_dc_settable {
 sub autoenroll_keys {
     my $internals = ['coursecode','courseowner','authtype','autharg','defaultcredits',
                      'autoadds','autodrops','autostart','autoend','sectionnums',
-                     'crosslistings','co-owners'];
+                     'crosslistings','co-owners','autodropfailsafe'];
     my $accessdates = ['default_enrollment_start_date','default_enrollment_end_date'];
     return ($internals,$accessdates);
 }
@@ -124,14 +124,13 @@ sub get_enrollment_settings {
                 $enrollvar{$type} =~ s/,/, /g;
             } elsif ($type eq "authtype"
                      || $type eq "autharg"    || $type eq "coursecode"
-                     || $type eq "crosslistings") {
+                     || $type eq "crosslistings" || $type eq "selfenrollmgr"
+                     || $type eq "autodropfailsafe") {
                 $enrollvar{$type} = $settings{$item};
             } elsif ($type eq 'defaultcredits') {
                 if (&showcredits($cdom)) {
                     $enrollvar{$type} = $settings{$item};
                 }
-            } elsif ($type eq 'selfenrollmgr') {
-                $enrollvar{$type} = $settings{$item};    
             } elsif ($type eq 'courseowner') {
                 if ($settings{$item} =~ /^[^:]+:[^:]+$/) {
                     $enrollvar{$type} = $settings{$item};
@@ -268,9 +267,7 @@ sub print_modification_menu {
         $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 eq 'Placement') {
-            $setparams_text = 'View/Modify course owner and self-enrollment';
-        } elsif (&showcredits($dom)) {
+        if (($type ne 'Placement') && (&showcredits($dom))) {
             $setparams_text = 'View/Modify course owner, institutional code, default authentication, credits, and self-enrollment';
         } else {
             $setparams_text = 'View/Modify course owner, institutional code, default authentication, and self-enrollment';
@@ -375,10 +372,6 @@ sub print_modification_menu {
     if ($type eq 'Community') {
         $menu_html .= '<li>'.&mt('Community owner (permitted to assign Coordinator roles in the community).').'</li>'."\n".
                       '<li>'.&mt('Override defaults for who configures self-enrollment for this specific community').'</li>'."\n";
-    } elsif ($type eq 'Placement') {
-        $menu_html .= '<li>'.&mt('Course owner (permitted to assign Course Coordinator roles in the course).').'</li>'."\n".
-                      '<li>'.&mt('Override defaults for who configures self-enrollment for this specific course.').'</li>'."\n";
-
     } 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";
@@ -853,31 +846,27 @@ sub print_course_modification_page {
         $r->print(&Apache::lonhtmlcommon::row_title(
                   &Apache::loncommon::help_open_topic('Modify_Community_Owner').
                   '&nbsp;'.&mt('Community Owner'))."\n");
-    } elsif ($crstype eq 'Placement') {
-         $r->print(&Apache::lonhtmlcommon::row_title(
-                   &Apache::loncommon::help_open_topic('Modify_Course_Owner').
-                      '&nbsp;'.&mt('Course Owner'))."\n");
     } else {
         $r->print(&Apache::lonhtmlcommon::row_title(
                       &Apache::loncommon::help_open_topic('Modify_Course_Instcode').
                       '&nbsp;'.&mt('Course Code'))."\n".
                   '<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').
                       '&nbsp;'.&mt('Credits (students)'))."\n".
                       '<input type="text" size="3" name="defaultcredits" value="'.$enrollvar{'defaultcredits'}.'" />'.
                       &Apache::lonhtmlcommon::row_closure());
-         }
-         $r->print(&Apache::lonhtmlcommon::row_title(
-                       &Apache::loncommon::help_open_topic('Modify_Course_Defaultauth').
-                       '&nbsp;'.&mt('Default Authentication method'))."\n".
-                   $authenitems."\n".
-                   &Apache::lonhtmlcommon::row_closure().
-                   &Apache::lonhtmlcommon::row_title(
-                   &Apache::loncommon::help_open_topic('Modify_Course_Owner').
-                      '&nbsp;'.&mt('Course Owner'))."\n");
+        }
+        $r->print(&Apache::lonhtmlcommon::row_title(
+                      &Apache::loncommon::help_open_topic('Modify_Course_Defaultauth').
+                      '&nbsp;'.&mt('Default Authentication method'))."\n".
+                  $authenitems."\n".
+                  &Apache::lonhtmlcommon::row_closure().
+                  &Apache::lonhtmlcommon::row_title(
+                  &Apache::loncommon::help_open_topic('Modify_Course_Owner').
+                     '&nbsp;'.&mt('Course Owner'))."\n");
     }
     my ($cctitle,$rolename,$currmanages,$ccchecked,$dcchecked,$defaultchecked);
     my ($selfenrollrows,$selfenrolltitles) = &Apache::lonuserutils::get_selfenroll_titles();
@@ -931,7 +920,7 @@ sub print_course_modification_page {
               '<br />'.&Apache::lonhtmlcommon::row_closure(1).
               &Apache::lonhtmlcommon::end_pick_box().'</p><p>'.$hidden_elements.
               '<input type="button" onclick="javascript:changePage(this.form,'."'processparms'".');');
-    if (($crstype eq 'Community') || ($crstype eq 'Placement')) {
+    if ($crstype eq 'Community') {
         $r->print('this.form.submit();"');
     } else {
         $r->print('javascript:verify_message(this.form);"');
@@ -1070,9 +1059,6 @@ sub modify_course {
     if ($type eq 'Community') {
         %changed = ( owner  => 0 );
         $ccrole = 'co';
-    } elsif ($type eq 'Placement') {
-        %changed = ( owner  => 0 );
-        $ccrole = 'cc';
     } else {
         %changed = ( code  => 0,
                      owner => 0,
@@ -1125,7 +1111,7 @@ sub modify_course {
             }
         }
 
-        if (&showcredits($cdom) && exists($env{'form.defaultcredits'})) {
+        if (($type ne 'Placement') && (&showcredits($cdom) && exists($env{'form.defaultcredits'}))) {
             $newattr{'defaultcredits'} =~ s/[^\d\.]//g;
             $newattr{'defaultcredits'}=$env{'form.defaultcredits'};
         }
@@ -2144,6 +2130,7 @@ sub course_settings_descrip {
                       'sectionnums' => 'Course section number:LON-CAPA section',
                       'crosslistings' => 'Crosslisted class:LON-CAPA section',
                       'defaultcredits' => 'Credits',
+                      'autodropfailsafe' => "Failsafe section enrollment count",
                       'selfenrollmgrdc'  => "Course-specific self-enrollment configuration by Domain Coordinator",
                       'selfenrollmgrcc'  => "Course-specific self-enrollment configuration by Course personnel",