From b04e36c2738f33bf00a8968073b7395fc2cd04ff Mon Sep 17 00:00:00 2001
From: "bugzilla%arlen.demon.co.uk"
 <bugzilla%arlen.demon.co.uk@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Sun, 24 Oct 2004 17:41:57 +0000
Subject: [PATCH] Bug 260364 Fixup radiogroups to use new .value functionality
 of radio.xml - mail patch p=me r=mscott

git-svn-id: svn://10.0.0.236/trunk@164285 18797224-902f-48f8-a5cc-f745e15eee43
---
 .../prefwindow/content/pref-masterpass.xul     |  8 ++++----
 .../mail/extensions/smime/content/am-smime.js  | 18 +++---------------
 2 files changed, 7 insertions(+), 19 deletions(-)

diff --git a/mozilla/mail/components/prefwindow/content/pref-masterpass.xul b/mozilla/mail/components/prefwindow/content/pref-masterpass.xul
index 1543cd84316..971ca3e804e 100644
--- a/mozilla/mail/components/prefwindow/content/pref-masterpass.xul
+++ b/mozilla/mail/components/prefwindow/content/pref-masterpass.xul
@@ -30,7 +30,7 @@
       el.setAttribute("value", gPrefService.getIntPref(el.getAttribute("prefstring")) );
 
       el = document.getElementById('passwordAskTimes');
-      el.selectedItem = el.getElementsByAttribute('value', gPrefService.getIntPref(el.getAttribute("prefstring")))[0];
+      el.value = gPrefService.getIntPref(el.getAttribute("prefstring"));
 
       onMasterPasswordLoad();
     }
@@ -93,12 +93,12 @@
                   flex="1">
         <!-- note that these values are different than what NSS uses, which
              are (0, -1, 1) respectively -->
-        <radio value="0" label="&managepassword.askfirsttime;" id="askFirstTime"
+        <radio value="0" label="&managepassword.askfirsttime;"
                style="margin: 0px;" oncommand="changePasswordSettings(false);"/>
-        <radio value="1" label="&managepassword.askeverytime;" id="askEveryTime"
+        <radio value="1" label="&managepassword.askeverytime;"
                style="margin: 0px;" oncommand="changePasswordSettings(false);"/>
         <hbox align="center">
-          <radio value="2" label="&managepassword.asktimeout;" id="askTimeout"
+          <radio value="2" label="&managepassword.asktimeout;"
                  style="margin: 0px;" oncommand="changePasswordSettings(true);"/>
           <textbox id="passwordTimeout" size="4"
                    preftype="int"
diff --git a/mozilla/mail/extensions/smime/content/am-smime.js b/mozilla/mail/extensions/smime/content/am-smime.js
index e0417c6df27..8818c5a2118 100644
--- a/mozilla/mail/extensions/smime/content/am-smime.js
+++ b/mozilla/mail/extensions/smime/content/am-smime.js
@@ -62,19 +62,7 @@ function onInit()
 
   gEncryptionCertName.value = gIdentity.getUnicharAttribute("encryption_cert_name");
 
-  var selectedItemId = null;
-  var encryptionPolicy = gIdentity.getIntAttribute("encryptionpolicy");
-  switch (encryptionPolicy)
-  {
-    case 2:
-      selectedItemId = 'encrypt_mail_always';
-      break;
-    default:
-      selectedItemId = 'encrypt_mail_never';
-      break;
-  }
-
-  gEncryptionChoices.selectedItem = document.getElementById(selectedItemId);
+  gEncryptionChoices.value = gIdentity.getIntAttribute("encryptionpolicy");
     
   if (!gEncryptionCertName.value)
   {
@@ -112,7 +100,7 @@ function onPreInit(account, accountValues)
 function onSave()
 {
   // find out which radio for the encryption radio group is selected and set that on our hidden encryptionChoice pref....
-  var newValue = gEncryptionChoices.selectedItem.value;
+  var newValue = gEncryptionChoices.value;
   gHiddenEncryptionPolicy.setAttribute('value', newValue);
   gIdentity.setIntAttribute("encryptionpolicy", newValue);
   gIdentity.setUnicharAttribute("encryption_cert_name", gEncryptionCertName.value);
@@ -343,7 +331,7 @@ function enableEncryptionControls(do_enable)
   else {
     gEncryptAlways.setAttribute("disabled", "true");
     gNeverEncrypt.setAttribute("disabled", "true");
-    gEncryptionChoices.selectedItem = document.getElementById('encrypt_mail_never');
+    gEncryptionChoices.value = 0;
   }
 }