--- loncom/interface/lonconfigsettings.pm	2016/08/04 17:34:35	1.21.4.8
+++ loncom/interface/lonconfigsettings.pm	2014/04/23 10:11:26	1.29
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Handler to set domain-wide configuration settings
 #
-# $Id: lonconfigsettings.pm,v 1.21.4.8 2016/08/04 17:34:35 raeburn Exp $
+# $Id: lonconfigsettings.pm,v 1.29 2014/04/23 10:11:26 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -35,20 +35,16 @@ use Apache::lonnet;
 use Apache::loncommon();
 use Apache::lonhtmlcommon();
 use Apache::lonlocal;
-use Apache::courseclassifier();
-use LONCAPA qw(:DEFAULT :match);
+use Apache::lonparmset();
 
 sub print_header {
     my ($r,$phase,$context,$jscript,$container) = @_;
-    my ($pagetitle,$brcrumtitle,$action,$call_category_check,$instcode_check,
-        $crstype,@actions,@code_order);
-    if ($phase eq 'display') {
-        @actions = &Apache::loncommon::get_env_multiple('form.actions');
-    }
+    my ($pagetitle,$brcrumtitle,$action,$call_category_check,$crstype);
     if ($context eq 'domain') {
         ($pagetitle, $brcrumtitle) = ('View/Modify Domain Settings','View/Modify Domain Settings');
         $action = '/adm/domainprefs';
         if ($phase eq 'display') {
+            my @actions = &Apache::loncommon::get_env_multiple('form.actions');
             if (grep(/^coursecategories$/,@actions)) {
                 $call_category_check = qq|
     if (formname == document.display) {
@@ -67,58 +63,8 @@ sub print_header {
             ($pagetitle,$brcrumtitle) = ('Course Configuration','Course Configuration');
         }
         $action = '/adm/courseprefs';
-        if ($phase eq 'display') {
-            if (grep(/^courseinfo$/,@actions)) {
-                my %codedefaults;
-                &Apache::lonnet::auto_instcode_defaults($env{'request.role.domain'},\%codedefaults,
-                                                        \@code_order);
-                if (@code_order) {
-                   my $noinstcodestr = &mt('You indicated cloning based on category, but did not select any categories.');
-                   &js_escape(\$noinstcodestr);
-                   $instcode_check = <<"ENDSCRIPT";
-    if (formname == document.display) {
-        if (formname.cloners_instcode.length) {
-            for (var j=0; j<formname.cloners_instcode.length; j++) {
-                if (formname.cloners_instcode[j].checked) {
-                    if (formname.cloners_instcode[j].value == 1) {
-                        var codes;
-                        if (document.getElementsByClassName) {
-                            codes = document.getElementsByClassName('LC_cloners_instcodes');
-                        } else {
-                            codes = getElementsByClassName(document.body,'LC_cloners_instcodes');
-                        }
-                        if (codes.length) {
-                            var gotcode = 0;
-                            for (var i=0; i<codes.length; i++) {
-                                if (codes[i].selectedIndex != 0) {
-                                     gotcode = 1;
-                                     break;
-                                }
-                            }
-                            if (!gotcode) {
-                                for (var k=0; k<formname.cloners_instcode.length; k++) {
-                                    if (formname.cloners_instcode[k].value == 0) {
-                                        formname.cloners_instcode[k].checked = true;
-                                    }
-                                }
-                                toggleCloners(document.display.cloners_instcode);
-                                alert('$noinstcodestr');
-                                return false;
-                            }
-                        }
-                    }
-                }
-            }
-        }
-    }
-
-ENDSCRIPT
-                }
-            }
-        }
     }
     my $alert = &mt('You must select at least one functionality type to display.');
-    &js_escape(\$alert);
     my $js = '
 <script type="text/javascript">
 // <![CDATA[
@@ -145,15 +91,15 @@ function changePage(formname,newphase) {
             return;
         }
     }
-    '.$instcode_check.$call_category_check.'
+    '.$call_category_check.'
     formname.submit();
 }'."\n";
     if ($phase eq 'pickactions') {
-        $js .= &Apache::lonhtmlcommon::color_picker();
+	$js .= &Apache::lonhtmlcommon::color_picker();
         $js .=
             &Apache::lonhtmlcommon::set_form_elements({actions => 'checkbox'})."\n";
     } elsif ($phase eq 'display') {
-        $js .= &Apache::lonhtmlcommon::color_picker();
+	$js .= &Apache::lonhtmlcommon::color_picker();
         $js .= &color_pick_js()."\n";
     }
     $js .= &Apache::loncommon::viewport_size_js().'
@@ -174,25 +120,6 @@ $jscript
                     'onload' => "setFormElements(document.pickactions);",
                         );
         $additem = {'add_entries' => \%loaditems,};
-    } elsif ($phase eq 'display') {
-        if ($context eq 'domain') {
-            if (grep(/^coursedefaults$/,@actions)) {
-                my %loaditems = (
-                    '             onload' => "toggleDisplay(document.display,'cloneinstcode');".
-                                             "toggleDisplay(document.display,'credits');".
-                                             "toggleDisplay(document.display,'studentsubmission');",
-                                );
-                $additem = {'add_entries' => \%loaditems,};
-            }
-        } elsif ($context eq 'course') {
-            if (grep(/^courseinfo$/,@actions)) {
-                if (@code_order) {
-                    $additem = {
-                                   add_entries => {'onload' => "courseSet('','load');toggleCloners(document.display.cloners_instcode);"},
-                               };
-                }
-            }
-        }
     }
     $r->print(&Apache::loncommon::start_page($pagetitle,$js,$additem));
     $r->print(&Apache::lonhtmlcommon::breadcrumbs($brcrumtitle));
@@ -203,6 +130,9 @@ $jscript
 <input type="hidden" name="pres_value" />
 </form>
 ');
+    if ($container) {
+       &Apache::lonparmset::startSettingsScreen($r,$container,$crstype);
+    }
     $r->print('<form method="post" name="'.$phase.'" action="'.$action.'"'.
               ' enctype="multipart/form-data">');
     return;
@@ -239,6 +169,9 @@ sub print_footer {
     }
     if ($phase eq 'process') {
         $r->print('</form>');
+        if ($container) {
+           &Apache::lonparmset::endSettingsScreen($r);
+        }
         $r->print(&Apache::loncommon::end_page());
     }
     return;
@@ -322,11 +255,7 @@ sub display_settings {
     &Apache::lonhtmlcommon::add_breadcrumb
         ({href=>"javascript:changePage(document.$phase,'display')",
           text=>"Display/Edit Settings"});
-    my $instcode;
-    if (ref($values) eq 'HASH') {
-        $instcode = $values->{'internal.coursecode'};
-    }
-    &print_header($r,$phase,$context,$jscript,$container,$instcode);
+    &print_header($r,$phase,$context,$jscript,$container);
     my $divwidth = 900;
     if ((ref($prefs_order) eq 'ARRAY') && (ref($prefs) eq 'HASH') && (ref($values) eq 'HASH')) { 
         if (@actions > 0) {
@@ -337,7 +266,7 @@ sub display_settings {
                     push(@items,$item);
                     if ($context eq 'domain') {
                         my $settings;
-                        if (ref($values) eq 'HASH') {
+                        if (ref($values) eq 'HASH') { 
                             $settings = $values->{$item};
                         }
                         if ($item eq 'usersessions') {
@@ -362,7 +291,7 @@ sub display_settings {
                                 my $inststatus = {
                                                    inststatustypes => $usertypes,
                                                    inststatusorder => $types,
-                                                   inststatusguest => [],
+                                                   inststatusguest => [], 
                                                  };
                                 if (ref($values->{defaults}) eq 'HASH') {
                                     $settings = {%{$inststatus},%{$values->{'defaults'}}};
@@ -395,6 +324,9 @@ sub display_settings {
         }
         $r->print('</form>');
     }
+    if ($container) {
+        &Apache::lonparmset::endSettingsScreen($r);
+    }
     $r->print(&Apache::loncommon::end_page());
     return;
 }
@@ -451,6 +383,9 @@ sub display_choices {
     }
     $r->print(&print_footer($r,$phase,'display','Display',undef,$container));
     $r->print('</form>');
+    if ($container) {
+        &Apache::lonparmset::endSettingsScreen($r);
+    }
     $r->print(&Apache::loncommon::end_page());
     return;
 }