--- loncom/interface/lonconfigsettings.pm	2023/03/19 16:05:48	1.68
+++ loncom/interface/lonconfigsettings.pm	2024/07/14 16:05:27	1.73
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Handler to set domain-wide configuration settings
 #
-# $Id: lonconfigsettings.pm,v 1.68 2023/03/19 16:05:48 raeburn Exp $
+# $Id: lonconfigsettings.pm,v 1.73 2024/07/14 16:05:27 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -93,7 +93,7 @@ sub print_header {
                             var gotcode = 0;
                             for (var i=0; i<codes.length; i++) {
                                 if (codes[i].selectedIndex != 0) {
-                                     gotcode = 1; 
+                                     gotcode = 1;
                                      break;
                                 }
                             }
@@ -232,7 +232,7 @@ $jscript
                     $onload .= "toggleEmailOptions(document.display,'cancreate_emailoptions','$hascustom',".
                                                        "'cancreate_emaildomain','default');";
                 }
-            } 
+            }
             if (grep(/^contacts$/,@actions)) {
                 my $customclass = 'LC_helpdesk_override';
                 my $optionsprefix = 'LC_options_helpdesk_';
@@ -346,7 +346,8 @@ $jscript
                                     $onload .= "toggleChgSecret(document.display,'$num','secret','linkprot');";
                                 }
                             }
-                            $onload .= "toggleLinkProtExtra(document.display,'returnurl','divurlparam','1','inline-block','$num');";
+                            $onload .= "toggleLinkProtExtra(document.display,'returnurl','divurlparam','1','inline-block','$num');".
+                                       "toggleLinkProtExtra(document.display,'passback','passbackparam','1','inline-block','$num');";
                             if ($ltiauth) {
                                 $onload .= "toggleLinkProtExtra(document.display,'requser','optional','1','block','$num');".
                                            "toggleLinkProtExtra(document.display,'mapuser','userfield','other','inline-block','$num');";
@@ -389,6 +390,9 @@ $jscript
             if (grep(/^localization$/,@actions)) {
                 $onload .= "toggleTimeZone();";
             }
+            if (grep(/^grading$/,@actions)) {
+                $onload .= 'toggleGrading(document.display);toggleHiddenTotalsSec(document.display);';
+            }
             if ($onload) {
                 my %loaditems = (
                                   'onload' => $onload,
@@ -503,7 +507,7 @@ ENDJS
             &Apache::loncommon::passwd_validation_js($name.'secretinput',$dom,$name,'add');
         if ($numrules) {
             $js .= <<ENDJS
-${name}secretinput = document.display.elements['${name}_add_secret'].value;
+${name}secretinput = document.display.elements['${name}_secret_add'].value;
 if (document.display.elements['${name}_add'].checked) {
     $intargjs
 }
@@ -512,7 +516,7 @@ ENDJS
         }
         $js .= <<ENDJS;
 uncheckLinkProtMakeVis('$name','visible','add');
-document.display.elements['${name}_add_secret'].type = 'password';
+document.display.elements['${name}_secret_add'].type = 'password';
 
 ENDJS
     }
@@ -569,7 +573,7 @@ sub print_footer {
         if ($showbutton) {
             $r->print('<p><input type="button" name="store" value="'.
                       $button_text.'" onclick='.$onclick.' /></p>');
-        } 
+        }
     }
     if ($phase eq 'process') {
         $r->print('</form>');
@@ -652,8 +656,8 @@ sub make_changes {
     if ($context eq 'course') {
         $footer_text = 'Back to display/edit settings'; 
     }
-    &print_footer($r,$phase,'display',$footer_text,\@actions,$container,$parm_permission);
     $r->print('</p>');
+    &print_footer($r,$phase,'display',$footer_text,\@actions,$container,$parm_permission);
     return \%lastact;
 }
 
@@ -680,7 +684,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') || ($item eq 'ssl')) {
@@ -708,7 +712,7 @@ sub display_settings {
                                 my $inststatus = {
                                                    inststatustypes => $usertypes,
                                                    inststatusorder => $types,
-                                                   inststatusguest => [], 
+                                                   inststatusguest => [],
                                                  };
                                 if (ref($values->{defaults}) eq 'HASH') {
                                     $settings = {%{$inststatus},%{$values->{'defaults'}}};