Bug 450747: Minor JS cleanup for the "edit multiple bugs" page

Patch by Max Kanat-Alexander <mkanat@bugzilla.org> r=mkanat, a=mkanat

Bug 450747: Minor JS cleanup for the "edit multiple bugs" page
877c8ef6 · mkanat%bugzilla.org · fc0f80b2 · 877c8ef6
Commit 877c8ef6 authored Nov 09, 2009 by mkanat%bugzilla.org
Hide whitespace changes
Inline Side-by-side

Showing with 8 additions and 5 deletions

edit-multiple.html.tmpl template/en/default/list/edit-multiple.html.tmpl +8 -5

No files found.
--- a/template/en/default/list/edit-multiple.html.tmpl
+++ b/template/en/default/list/edit-multiple.html.tmpl
@@ -28,12 +28,15 @@
 <input type="hidden" name="token" value="[% token FILTER html %]">
 <script type="text/javascript">
-  var numelements = document.forms.changeform.elements.length;
  function SetCheckboxes(value) {
-      var item;
+      var elements = document.forms.changeform.getElementsByTagName('input'),
-      for (var i=0 ; i<numelements ; i++) {
+          numelements = elements.length,
-          item = document.forms.changeform.elements[i];
+          item, i;
-          item.checked = value;
+      for (i = 0; i < numelements; i++) {
+          item = elements[i];
+          if (item.type === 'checkbox' && item.name.match(/^id_/)) {
+            item.checked = value;
+          }
      }
  }
  document.write(' <input type="button" name="uncheck_all" value="Uncheck All" onclick="SetCheckboxes(false);">');