From 3c552ac62851a450e3801a32cca4ad1310570c3e Mon Sep 17 00:00:00 2001
From: Simon Kornblith <simon@simonster.com>
Date: Mon, 5 Sep 2011 06:29:35 +0000
Subject: [PATCH] Addresses #597, Don't iterate Array with for...in

---
 .../zotero/tools/testTranslators/translatorTester.js | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/chrome/content/zotero/tools/testTranslators/translatorTester.js b/chrome/content/zotero/tools/testTranslators/translatorTester.js
index daae11b92..8c5a9043c 100644
--- a/chrome/content/zotero/tools/testTranslators/translatorTester.js
+++ b/chrome/content/zotero/tools/testTranslators/translatorTester.js
@@ -23,7 +23,7 @@
     ***** END LICENSE BLOCK *****
 */
 
-const Zotero_TranslatorTester_IGNORE_FIELDS = ["complete", "accessDate", "checkFields"];
+var Zotero_TranslatorTester_IGNORE_FIELDS = ["complete", "accessDate", "checkFields"];
 
 /**
  * A tool to run unit tests for a given translator
@@ -66,7 +66,7 @@ Zotero_TranslatorTester = function(translator, type, debug) {
 			Zotero.logError(e);
 		}
 		
-		for(var i in testObject) {
+		for(var i=0, n=testObject.length; i<n; i++) {
 			if(testObject[i].type === type) {
 				this.tests.push(testObject[i]);
 				this.pending.push(testObject[i]);
@@ -93,7 +93,7 @@ Zotero_TranslatorTester._sanitizeItem = function(item, forSave) {
 	}
 	
 	// remove fields to be ignored
-	for(var j in Zotero_TranslatorTester_IGNORE_FIELDS) {
+	for(var j=0, n=Zotero_TranslatorTester_IGNORE_FIELDS.length; j<n; j++) {
 		if(forSave) {
 			delete item[Zotero_TranslatorTester_IGNORE_FIELDS[j]]
 		} else {
@@ -285,7 +285,7 @@ Zotero_TranslatorTester.prototype._checkResult = function(test, translate, retur
 			return;
 		}
 		
-		for(var i in test.items) {
+		for(var i=0, n=test.items.length; i<n; i++) {
 			var testItem = Zotero_TranslatorTester._sanitizeItem(test.items[i]);
 			var translatedItem = Zotero_TranslatorTester._sanitizeItem(translate.newItems[i]);
 			
@@ -348,7 +348,9 @@ Zotero_TranslatorTester.prototype._createTest = function(translate, multipleMode
 	if(multipleMode) {
 		var items = "multiple";
 	} else {
-		for(var i in translate.newItems) Zotero_TranslatorTester._sanitizeItem(translate.newItems[i], true);
+		for(var i=0, n=translate.newItems; i<n; i++) {
+			Zotero_TranslatorTester._sanitizeItem(translate.newItems[i], true);
+		}
 		var items = translate.newItems;
 	}