--- loncom/interface/lonconfigsettings.pm	2023/03/19 16:05:48	1.68
+++ loncom/interface/lonconfigsettings.pm	2025/03/15 01:03:33	1.74
@@ -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.74 2025/03/15 01:03:33 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_';
@@ -269,6 +269,7 @@ $jscript
             if (grep(/^ltitools$/,@actions)) {
                 $onload .= "toggleLTITools(document.display,'passback','add');".
                            "toggleLTITools(document.display,'roster','add');".
+                           "toggleLTITools(document.display,'returnurl','add');".
                            "toggleLTITools(document.display,'user','add');";
                 if (ref($values) eq 'HASH') {
                     if (ref($values->{'ltitools'}) eq 'HASH') {
@@ -276,6 +277,7 @@ $jscript
                         for (my $i=0; $i<$numltitools; $i++) {
                             $onload .= "toggleLTITools(document.display,'passback','$i');".
                                        "toggleLTITools(document.display,'roster','$i');".
+                                       "toggleLTITools(document.display,'returnurl','$i');".
                                        "toggleLTITools(document.display,'user','$i');";
                         }
                     }
@@ -346,7 +348,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 +392,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 +509,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 +518,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 +575,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 +658,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 +686,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 +714,7 @@ sub display_settings {
                                 my $inststatus = {
                                                    inststatustypes => $usertypes,
                                                    inststatusorder => $types,
-                                                   inststatusguest => [], 
+                                                   inststatusguest => [],
                                                  };
                                 if (ref($values->{defaults}) eq 'HASH') {
                                     $settings = {%{$inststatus},%{$values->{'defaults'}}};
@@ -1114,9 +1120,13 @@ sub ltitools_toggle_js {
 function toggleLTITools(form,setting,item) {
     var radioname = '';
     var divid = '';
-    if ((setting == 'passback') || (setting == 'roster')) {
+    if ((setting == 'passback') || (setting == 'roster') || (setting == 'returnurl')) {
         radioname = 'ltitools_'+setting+'_'+item;
-        divid = 'ltitools_'+setting+'time_'+item;
+        if (setting == 'returnurl') {
+            divid = 'ltitools_course'+setting+'_'+item;
+        } else {
+            divid = 'ltitools_'+setting+'time_'+item;
+        }
         var num = form.elements[radioname].length;
         if (num) {
             var setvis = '';