[rhmessaging-commits] rhmessaging commits: r1549 - mgmt/cumin/python/cumin.

[rhmessaging-commits at lists.jboss.org]

Author: justi9
Date: 2008-01-11 10:14:21 -0500 (Fri, 11 Jan 2008)
New Revision: 1549

Modified:
   mgmt/cumin/python/cumin/broker.py
Log:
Fixes bulk broker unregister.



Modified: mgmt/cumin/python/cumin/broker.py
===================================================================
--- mgmt/cumin/python/cumin/broker.py	2008-01-11 11:41:52 UTC (rev 1548)
+++ mgmt/cumin/python/cumin/broker.py	2008-01-11 15:14:21 UTC (rev 1549)
@@ -724,13 +724,15 @@
     def __init__(self, app, name):
         super(BrokerSetRemove, self).__init__(app, name)
 
-        self.param = BrokerParameter(app, "param")
-        self.add_parameter(self.param)
-        self.add_form_parameter(self.param)
+        # Note that the params below are not "form params", so we
+        # don't use add_form_parameter to exclude them from the
+        # marshalled set
 
-        self.brokers = ListParameter(app, "id", self.param)
+        param = BrokerParameter(app, "param")
+        self.add_parameter(param)
+
+        self.brokers = ListParameter(app, "id", param)
         self.add_parameter(self.brokers)
-        self.add_form_parameter(self.brokers)
 
     def get_title(self, session, model):
         return "Unregister Brokers"
@@ -746,11 +748,8 @@
     def process_submit(self, session, model):
         brokers = self.brokers.get(session)
 
-        print "submit with brokers", brokers
-
         for broker in brokers:
             try:
-                print "destroying broker reg", broker
                 broker.destroySelf()
             except Exception, e:
                 # In an extra ui step, note any errors that arose