[LON-CAPA-cvs] cvs: loncom /interface lonuserutils.pm

[raeburn]

raeburn		Tue Aug  1 07:48:21 2023 EDT

  Modified files:              
    /loncom/interface	lonuserutils.pm 
  Log:
  - Submission of "Course settings modifiable only by Domain Coordinator"
    form should still work when there is only one authentication type which 
    may be set as the default, but it is not set.
  - Scope for @in_domain array.
  
  
Index: loncom/interface/lonuserutils.pm
diff -u loncom/interface/lonuserutils.pm:1.214 loncom/interface/lonuserutils.pm:1.215
--- loncom/interface/lonuserutils.pm:1.214	Tue Jun 20 14:03:52 2023
+++ loncom/interface/lonuserutils.pm	Tue Aug  1 07:48:21 2023
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Utility functions for managing LON-CAPA user accounts
 #
-# $Id: lonuserutils.pm,v 1.214 2023/06/20 14:03:52 raeburn Exp $
+# $Id: lonuserutils.pm,v 1.215 2023/08/01 07:48:21 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -1065,7 +1065,7 @@
 ";
     } elsif ($mode eq 'modifycourse') {
         $auth_checks .= "
-    if (vf.elements[current.argfield].value == null || vf.elements[current.argfield].value == '') {
+    if ((current.argfield !== null) && (current.argfield !== undefined) && (current.argfield !== '') && (vf.elements[current.argfield].value == null || vf.elements[current.argfield].value == '')) {
 ";
     }
     if ( ($mode eq 'createcourse') || ($mode eq 'modifycourse') ) {
@@ -7243,7 +7243,7 @@
         }
     }
     if ($settings{'internal.selfenrollmgrdc'} ne '') {
-        my @in_domain = split(/,/,$settings{'internal.selfenrollmgrdc'});
+        @in_domain = split(/,/,$settings{'internal.selfenrollmgrdc'});
         my @diffs = &Apache::loncommon::compare_arrays(\@in_domain,$possconfigs);
         unless (@diffs) {
             return (\@in_course,\@in_domain);