Bug 530142 - Account Settings copies "Reply-To Address" between accounts when selected. r=IanN,r=Standard8
--- a/mailnews/base/prefs/content/AccountManager.js
+++ b/mailnews/base/prefs/content/AccountManager.js
@@ -932,22 +932,17 @@ function getFormElementValue(formElement
if (formElement.value) {
var localfile = Components.classes["@mozilla.org/file/local;1"].createInstance(Components.interfaces.nsILocalFile);
localfile.initWithPath(formElement.value);
return localfile;
}
return null;
}
- if (type == "text") {
- var val = formElement.getAttribute("value");
- if (val) return val;
- return null;
- }
- if ("value" in formElement) {
+ if ((type == "textbox") || ("value" in formElement)) {
return formElement.value;
}
return null;
}
catch (ex) {
dump("getFormElementValue failed, ex="+ex+"\n");
}
return null;
@@ -1000,22 +995,22 @@ function setFormElementValue(formElement
} else {
if ("defaultValue" in formElement)
formElement.value = formElement.defaultValue;
else
formElement.value = "";
}
}
-
- else if (type == "text") {
- if (value == null || value == undefined)
- formElement.removeAttribute("value");
- else
- formElement.setAttribute("value",value);
+ else if (type == "textbox") {
+ if (value == null || value == undefined) {
+ formElement.value = null;
+ } else {
+ formElement.value = value;
+ }
}
// let the form figure out what to do with it
else {
if (value == undefined) {
if ("defaultValue" in formElement && formElement.defaultValue)
formElement.value = formElement.defaultValue;
}