--- loncom/interface/lonpopulate.pm	2004/08/23 16:53:06	1.19
+++ loncom/interface/lonpopulate.pm	2004/09/13 16:21:30	1.20
@@ -1,5 +1,5 @@
 # automated enrollment configuration handler
-# $Id: lonpopulate.pm,v 1.19 2004/08/23 16:53:06 raeburn Exp $
+# $Id: lonpopulate.pm,v 1.20 2004/09/13 16:21:30 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -118,39 +118,28 @@ function process(calling,numauto,nummanu
      }
  }
  if (calling == "viewclass") {
-     document.forms.studentform.state.value = "process";
      var totcheck = 0
+     var numchk = 0
      if (numauto > 0) {
-        for (var i=0; i<document.studentform.chgauto.length; i++) {
-            if (document.studentform.chgauto[i].checked == true) {
-                totcheck ++
-            }
-        }
+         numchk = countChecked(document.studentform.chgauto);
+         totcheck = totcheck + numchk
      }
      if (nummanual > 0) {
-        for (var i=0; i<document.studentform.chgmanual.length; i++) {
-            if (document.studentform.chgmanual[i].checked == true) {
-                totcheck ++
-            }
-        }
+         numchk = countChecked(document.studentform.chgmanual);
+         totcheck = totcheck + numchk
      }
      if (numlock > 0) {
-        for (var i=0; i<document.studentform.lockchg.length; i++) {
-            if (document.studentform.lockchg[i].checked == true) {
-                totcheck ++
-            }
-        }
+         numchk = countChecked(document.studentform.lockchg);
+         totcheck = totcheck + numchk
      }
      if (numunlock > 0) {
-        for (var i=0; i<document.studentform.unlockchg.length; i++) {
-            if (document.studentform.unlockchg[i].checked == true) {
-                totcheck ++
-            }
-        }
+         numchk = countChecked(document.studentform.unlockchg);
+         totcheck = totcheck + numchk
      }
      if (totcheck > 0) {
-        checker = 1
-     } else {
+        document.forms.studentform.state.value = "process";
+     }
+     if (totcheck == 0) {
         alert("You must check at least one checkbox, before proceeding to the next page")
         checker = 0
      }
@@ -163,14 +152,41 @@ function process(calling,numauto,nummanu
     if ($action eq 'viewclass') {
         $scripttag .= qq|
 function checkAll(field) {
-    for (i = 0; i < field.length; i++)
-        field[i].checked = true ;
+    if (field.length > 0) {
+        for (i = 0; i < field.length; i++) {
+            field[i].checked = true ;
+        }
+    } else {
+        field.checked = true
+    }
 }
 
 function uncheckAll(field) {
-    for (i = 0; i < field.length; i++)
-        field[i].checked = false ;
+    if (field.length > 0) {
+        for (i = 0; i < field.length; i++) {
+            field[i].checked = false ;
+        }
+    } else {
+        field.checked = false ;
+    }
+}
+
+function countChecked(field) {
+    var count = 0;
+    if (field.length > 0) {
+        for (var i=0; i<field.length; i++) {
+            if (field[i].checked == true) {
+                count ++
+            }
+        }
+    } else {
+        if (field.checked == true) {
+            count ++
+        }
+    }
+    return count
 }
+
 |;
     }
     $scripttag .= qq|