[open-ils-commits] r15520 - trunk/Open-ILS/web/js/dojo/fieldmapper (miker)

svn at svn.open-ils.org svn at svn.open-ils.org
Fri Feb 12 11:24:24 EST 2010


Author: miker
Date: 2010-02-12 11:24:21 -0500 (Fri, 12 Feb 2010)
New Revision: 15520

Modified:
   trunk/Open-ILS/web/js/dojo/fieldmapper/hash.js
Log:
Addressing https://bugs.launchpad.net/evergreen/+bug/520632 -- not strictly required here, but for(;;) is better form for array access

Modified: trunk/Open-ILS/web/js/dojo/fieldmapper/hash.js
===================================================================
--- trunk/Open-ILS/web/js/dojo/fieldmapper/hash.js	2010-02-12 16:23:07 UTC (rev 15519)
+++ trunk/Open-ILS/web/js/dojo/fieldmapper/hash.js	2010-02-12 16:24:21 UTC (rev 15520)
@@ -41,7 +41,7 @@
 		}
 
 		if (virtFields && virtFields.length > 0) {
-			for (var i in virtFields) {
+			for (var i = 0; i < virtFields.length; i++) {
 				if (!_hash[virtFields[i]])
 					_hash[virtFields[i]] = null;
 			}



More information about the open-ils-commits mailing list