From aa18c2596d00966935616c5f7dffe3fe8bb5b0d2 Mon Sep 17 00:00:00 2001
From: "smeredith%netscape.com" <smeredith%netscape.com>
Date: Tue, 14 May 2002 16:51:12 +0000
Subject: [PATCH] Fixed bug getting the pretty value from bool prefs with
 choices.

---
 cck/driver/dlgeditprefstr.cpp | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/cck/driver/dlgeditprefstr.cpp b/cck/driver/dlgeditprefstr.cpp
index 67c70a5fb72..a59e3711f43 100644
--- a/cck/driver/dlgeditprefstr.cpp
+++ b/cck/driver/dlgeditprefstr.cpp
@@ -134,22 +134,22 @@ BOOL CDlgEditPrefStr::OnInitDialog()
 // Caller only has to get value from m_strValue for all types.
 void CDlgEditPrefStr::OnOK() 
 {
-  // For bool types, convert back to the data string.
-  if (m_strType.CompareNoCase("bool") == 0)
-  {
-    if (m_checkValue.GetCheck())
-      m_editValue.SetWindowText("true");
-    else
-      m_editValue.SetWindowText("false");
-  }	  
   // For choose, return the selected list string.
-  else if (m_bChoose)
+  if (m_bChoose)
   {
     CString strSelectString;
     m_listValue.GetLBText(m_listValue.GetCurSel(), strSelectString);
     m_editValue.SetWindowText(strSelectString);
   }
 	
+  // For bool types, convert back to the data string.
+  else if (m_strType.CompareNoCase("bool") == 0)
+  {
+    if (m_checkValue.GetCheck())
+      m_editValue.SetWindowText("true");
+    else
+      m_editValue.SetWindowText("false");
+  }	  
   else // string, int type
   {