refine other rackspace checkboxes JavaScript props
authorDenis Ovsienko <denis@ovsienko.info>
Fri, 2 Nov 2018 19:06:14 +0000 (19:06 +0000)
committerDenis Ovsienko <denis@ovsienko.info>
Fri, 2 Nov 2018 20:29:22 +0000 (20:29 +0000)
wwwroot/js/racktables.js

index 3b318fd..facff16 100644 (file)
@@ -2,39 +2,31 @@
 
 // Used for (un)checking an entire row of rackspace atoms
 function toggleRowOfAtoms (rackId, rowId) {
-       var checkboxId;
        var toSet;
+       var element;
        toSet = null;
-       for (var i=0; i<=2; i++) {
-               checkboxId = "atom_" + rackId + "_" + rowId + "_" + i;
-
-               // Abort if the box is disabled
-               if (document.getElementById(checkboxId).disabled == true) continue;
-
-               // Box isn't disabled, toggle it
-               if (toSet == null) {
-                       toSet = !document.getElementById(checkboxId).checked;
+       for (var columnId=0; columnId<=2; columnId++) {
+               element = document.getElementById("atom_" + rackId + "_" + rowId + "_" + columnId);
+               if (element != null && ! element.disabled) {
+                       if (toSet == null)
+                               toSet = !element.checked;
+                       element.checked = toSet;
                }
-               document.getElementById(checkboxId).checked = toSet;
        }
 }
 
 // Used for (un)checking an entire column of rackspace atoms
 function toggleColumnOfAtoms (rackId, columnId, numRows) {
-       var checkboxId;
        var toSet;
+       var element;
        toSet = null;
-       for (var i=1; i<numRows+1; i++) {
-               checkboxId = "atom_" + rackId + "_" + i + "_" + columnId;
-
-               // Abort if the box is disabled
-               if (document.getElementById(checkboxId).disabled == true) continue;
-
-               // Box isn't disabled, toggle it
-               if (toSet == null) {
-                       toSet = !document.getElementById(checkboxId).checked;
+       for (var rowId=1; rowId<numRows+1; rowId++) {
+               element = document.getElementById("atom_" + rackId + "_" + rowId + "_" + columnId);
+               if (element != null && ! element.disabled) {
+                       if (toSet == null)
+                               toSet = !element.checked;
+                       element.checked = toSet;
                }
-               document.getElementById(checkboxId).checked = toSet;
        }
 }