From: miker <miker@dcc99617-32d9-48b4-a31d-7c20da2025e4>
Date: Fri, 12 Feb 2010 15:49:32 +0000 (+0000)
Subject: Addressing https://bugs.launchpad.net/evergreen/+bug/520632 -- not strictly required... 
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=f7daa86aa8cbcf32ce63747ba195b60b96cb16b2;p=Evergreen.git

Addressing https://bugs.launchpad.net/evergreen/+bug/520632 -- not strictly required here, but for(;;) is better form for array access

git-svn-id: svn://svn.open-ils.org/ILS/trunk@15512 dcc99617-32d9-48b4-a31d-7c20da2025e4
---

diff --git a/Open-ILS/web/js/dojo/MARC/Field.js b/Open-ILS/web/js/dojo/MARC/Field.js
index b9b9e592c4..7242a8945b 100644
--- a/Open-ILS/web/js/dojo/MARC/Field.js
+++ b/Open-ILS/web/js/dojo/MARC/Field.js
@@ -68,16 +68,16 @@ if(!dojo._hasResource["MARC.Field"]) {
                 args.pos = [ args.pos ];
             }
 
-            for (var i in args.code) {
+            for (var i = 0; i < args.code.length; i++) {
                 var sub_pos = {};
-                for (var j in me.subfields) {
+                for (var j = 0; j < me.subfields; j++) {
                     if (me.subfields[j][0] == args.code[i]) {
 
                         if (!sub_pos[args.code[i]]) sub_pos[args.code[j]] = 0;
                         else sub_pos[args.code[i]]++;
 
                         if (args.pos) {
-                            for (var k in args.pos) {
+                            for (var k = 0; k < args.pos.length; k++) {
                                 if (sub_pos[args.code[i]] == args.pos[k]) me.subfields.splice(j,1);
                             }
                         } else if (args.match && me.subfields[j][1].match( args.match )) {