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

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