[rhmessaging-commits] rhmessaging commits: r2345 - in mgmt/trunk/cumin/python: wooly and 1 other directory.

[rhmessaging-commits at lists.jboss.org]

Author: justi9
Date: 2008-08-21 17:42:18 -0400 (Thu, 21 Aug 2008)
New Revision: 2345

Modified:
   mgmt/trunk/cumin/python/cumin/brokergroup.py
   mgmt/trunk/cumin/python/wooly/forms.py
Log:
Fix some accidental literal False's in the xml output

Modified: mgmt/trunk/cumin/python/cumin/brokergroup.py
===================================================================
--- mgmt/trunk/cumin/python/cumin/brokergroup.py	2008-08-21 20:33:43 UTC (rev 2344)
+++ mgmt/trunk/cumin/python/cumin/brokergroup.py	2008-08-21 21:42:18 UTC (rev 2345)
@@ -58,7 +58,8 @@
         return group.name
 
     def render_item_checked_attr(self, session, group):
-        return group in self.param.get(session) and "checked=\"checked\""
+        if group in self.param.get(session):
+            return "checked=\"checked\""
 
 class BrokerGroupCheckboxField(FormField):
     def __init__(self, app, name, form):

Modified: mgmt/trunk/cumin/python/wooly/forms.py
===================================================================
--- mgmt/trunk/cumin/python/wooly/forms.py	2008-08-21 20:33:43 UTC (rev 2344)
+++ mgmt/trunk/cumin/python/wooly/forms.py	2008-08-21 21:42:18 UTC (rev 2345)
@@ -174,7 +174,8 @@
         self.add_parameter(self.get_parameter())
 
     def render_checked_attr(self, session, *args):
-        return self.get(session) and "checked=\"checked\""
+        if self.get(session):
+            return "checked=\"checked\""
 
 class RadioInput(FormInput):
     def __init__(self, app, name, form):