--- loncom/interface/lonparmset.pm	2022/05/27 18:17:11	1.522.2.28.4.5
+++ loncom/interface/lonparmset.pm	2022/05/31 14:31:45	1.522.2.28.4.6
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Handler to set parameters for assessments
 #
-# $Id: lonparmset.pm,v 1.522.2.28.4.5 2022/05/27 18:17:11 raeburn Exp $
+# $Id: lonparmset.pm,v 1.522.2.28.4.6 2022/05/31 14:31:45 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -971,6 +971,7 @@ function validateParms() {
                             if (document.parmform.elements['set_'+identifier].value) {
                                 possdeny = ','+possdeny;
                             }
+                            document.parmform.elements['set_'+identifier].value += possdeny;
                         }
                     }
                 } else if (deeplinkRegExp.test(name)) {
@@ -1076,8 +1077,8 @@ function validateParms() {
                             }
                             document.parmform.elements['set_'+identifier].value += posslinkmenu;
                         }
-                    } else if (dlTargetRegExp.text(name)) {
-                        var identifier =  name.replace(dlTargetExp,'');
+                    } else if (dlTargetRegExp.test(name)) {
+                        var identifier =  name.replace(dlTargetRegExp,'');
                         var idx = document.parmform.elements[i].selectedIndex;
                         if (idx > 0) {
                             var possdeeplink = document.parmform.elements[i].options[idx].value
@@ -4467,6 +4468,8 @@ sub newoverview {
 '.
             &Apache::lonhtmlcommon::resize_scrollbox_js('params')."\n".
             &showhide_js()."\n".
+            &validateparms_js()."\n".
+            &ipacc_boxes_js()."\n".
             &done_proctor_js()."\n".
             &deeplink_js()."\n".
 '// ]]>
@@ -4477,7 +4480,7 @@ sub newoverview {
     my $breadcrumbs = &Apache::lonhtmlcommon::breadcrumbs('Overview');
     $r->print($start_page.$breadcrumbs);
     $r->print(<<ENDOVER);
-<form method="post" action="/adm/parmset?action=newoverview" name="parmform">
+<form method="post" action="/adm/parmset?action=newoverview" name="parmform" onsubmit="return validateParms();">
 ENDOVER
     my @ids=();
     my %typep=();
@@ -4662,6 +4665,8 @@ sub overview {
     my $js = '<script type="text/javascript">'."\n".
              '// <![CDATA['."\n".
              &done_proctor_js()."\n".
+             &validateparms_js()."\n".
+             &ipacc_boxes_js()."\n".
              &deeplink_js()."\n".
              '// ]]>'."\n".
              '</script>'."\n";
@@ -4674,7 +4679,7 @@ sub overview {
     my $start_page=&Apache::loncommon::start_page('Modify Parameters',$js);
     my $breadcrumbs = &Apache::lonhtmlcommon::breadcrumbs('Overview');
     $r->print($start_page.$breadcrumbs);
-    $r->print('<form method="post" action="/adm/parmset?action=setoverview" name="parmform">');
+    $r->print('<form method="post" action="/adm/parmset?action=setoverview" name="parmform" onsubmit="return validateParms();">');
 
 # Store modified
     unless ($readonly) {