From bb660c17f763b69d9341ffb462f1fb93ac2f90cf Mon Sep 17 00:00:00 2001
From: "alecf%netscape.com"
 <alecf%netscape.com@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Wed, 10 May 2000 22:30:54 +0000
Subject: [PATCH] fix excess undefineds coming from the .defaultValue attribute

git-svn-id: svn://10.0.0.236/trunk@69076 18797224-902f-48f8-a5cc-f745e15eee43
---
 .../prefs/resources/content/AccountManager.js    | 16 +++++++++++-----
 1 file changed, 11 insertions(+), 5 deletions(-)

diff --git a/mozilla/mailnews/base/prefs/resources/content/AccountManager.js b/mozilla/mailnews/base/prefs/resources/content/AccountManager.js
index 00a3e462164..3fd275757fb 100644
--- a/mozilla/mailnews/base/prefs/resources/content/AccountManager.js
+++ b/mozilla/mailnews/base/prefs/resources/content/AccountManager.js
@@ -572,7 +572,10 @@ function setFormElementValue(formElement, value) {
   var type = formElement.tagName;
   if (type == "checkbox") {
     if (value == undefined) {
-      formElement.checked = formElement.defaultChecked;
+      if (formElement.defaultChecked)
+        formElement.checked = formElement.defaultChecked;
+      else
+        formElement.checked = false;
     } else {
       if (formElement.getAttribute("reversed"))
         formElement.checked = !value;
@@ -605,15 +608,18 @@ function setFormElementValue(formElement, value) {
       } catch (ex) {
         dump("Still need to fix uninitialized filespec problem!\n");
       }
-    } else
-      formElement.value = formElement.defaultValue;
+    } else {
+      if (formElement.defaultValue)
+        formElement.value = formElement.defaultValue;
 
   }
 
   // let the form figure out what to do with it
   else {
-    if (value == undefined)
-      formElement.value = formElement.defaultValue;
+    if (value == undefined) {
+      if (formElement.defaultValue)
+        formElement.value = formElement.defaultValue;
+    }
     else
       formElement.value = value;
   }