From 392adc2f706ed5175d026a3b8b18c61d736ad592 Mon Sep 17 00:00:00 2001
From: "sspitzer%netscape.com" <sspitzer%netscape.com>
Date: Wed, 31 Jan 2001 03:00:56 +0000
Subject: [PATCH] fix for #58567.  fix js warnings.  r=jag,sr=sspitzer thanks
 to mao (Mark Olson) for the fix.

---
 mailnews/base/resources/content/subscribe.js | 43 ++++++++------------
 1 file changed, 17 insertions(+), 26 deletions(-)

diff --git a/mailnews/base/resources/content/subscribe.js b/mailnews/base/resources/content/subscribe.js
index dfeb4b92dc60..58baa8dc14ab 100644
--- a/mailnews/base/resources/content/subscribe.js
+++ b/mailnews/base/resources/content/subscribe.js
@@ -251,38 +251,29 @@ function changeTableRecord(server, name, state) {
 
 function StateChanged(name,state)
 {
-	//dump("StateChanged(" + name + "," + state + ")\n");
-	if (gChangeTable[gServerURI] == undefined) {
-        gChangeTable[gServerURI] = {};
-		gChangeTable[gServerURI][name] = state;
-	}
-	else {
-        if (gChangeTable[gServerURI][name] == undefined) {
-            gChangeTable[gServerURI][name] = state;
-        }
-        else {
-		    var oldValue = gChangeTable[gServerURI][name];
-		    if (oldValue != state) {
-			    gChangeTable[gServerURI][name] = undefined;
-		    }
-        }
-	}
+  if (gServerURI in gChangeTable) {
+    if (name in gChangeTable[gServerURI]) {
+      var oldValue = gChangeTable[gServerURI][name];
+      if (oldValue != state)
+        delete gChangeTable[gServerURI][name];
+    }
+    else {
+      gChangeTable[gServerURI][name] = state;
+    }
+  }
+  else {
+    gChangeTable[gServerURI] = {};
+    gChangeTable[gServerURI][name] = state;
+  }
 }
 
 function SetSubscribeState(state)
 {
-  //dump("SetSubscribedState()\n");
- 
   try {
-	//dump("subscribe button clicked\n");
-	
 	var groupList = gSubscribeTree.selectedItems;
-	for (i=0;i<groupList.length;i++) {
-		group = groupList[i];
-		uri = group.getAttribute('id');
-		//dump(uri + "\n");
-		name = group.getAttribute('name');
-		//dump(name + "\n");
+	for (var i=0;i<groupList.length;i++) {
+		var group = groupList[i];
+		var name = group.getAttribute('name');
 		SetState(name,state);
 	}
   }