From 6f39031ed6eb8767215211836613ce4e4ca968e6 Mon Sep 17 00:00:00 2001
From: Dan Stillman <dstillman@zotero.org>
Date: Wed, 28 Mar 2007 18:10:00 +0000
Subject: [PATCH] Addresses #597, Don't iterate Array with for...in

---
 chrome/content/zotero/itemPane.js | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/chrome/content/zotero/itemPane.js b/chrome/content/zotero/itemPane.js
index 7acfbd09a..d205d5ea5 100644
--- a/chrome/content/zotero/itemPane.js
+++ b/chrome/content/zotero/itemPane.js
@@ -252,8 +252,8 @@ var ZoteroItemPane = new function()
 			}
 			
 			var creatorTypes = Zotero.CreatorTypes.getTypesForItemType(_itemBeingEdited.getType());
-			var localized = [];
-			for (var i in creatorTypes)
+			var localized = {};
+			for (var i=0; i<creatorTypes.length; i++)
 			{
 				localized[creatorTypes[i]['name']]
 					= Zotero.getString('creatorTypes.' + creatorTypes[i]['name']);
@@ -509,9 +509,9 @@ var ZoteroItemPane = new function()
 		// Generate list of localized field names for display in pop-up
 		if (fieldsToDelete) {
 			var fieldNames = "";
-			for each(var fieldID in fieldsToDelete) {
+			for (var i=0; i<fieldsToDelete.length; i++) {
 				fieldNames += "\n - " +
-					Zotero.ItemFields.getLocalizedString(_itemBeingEdited.getType(), fieldID);
+					Zotero.ItemFields.getLocalizedString(_itemBeingEdited.getType(), fieldsToDelete[i]);
 			}
 			
 			var promptService = Components.classes["@mozilla.org/embedcomp/prompt-service;1"]