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

[rhmessaging-commits at lists.jboss.org]

Author: justi9
Date: 2008-05-08 16:08:31 -0400 (Thu, 08 May 2008)
New Revision: 2014

Modified:
   mgmt/cumin/python/cumin/broker.py
   mgmt/cumin/python/wooly/__init__.py
Log:
Because the API could be confusing, get rid of the return value for
Page.pop_current_frame.



Modified: mgmt/cumin/python/cumin/broker.py
===================================================================
--- mgmt/cumin/python/cumin/broker.py	2008-05-08 20:04:06 UTC (rev 2013)
+++ mgmt/cumin/python/cumin/broker.py	2008-05-08 20:08:31 UTC (rev 2014)
@@ -573,8 +573,8 @@
 class BrokerSetAdd(BrokerSetForm):
     def process_cancel(self, session):
         branch = session.branch()
-        frame = self.page.pop_current_frame(branch)
-        frame.show_view(branch)
+        self.page.pop_current_frame(branch)
+        self.page.get_current_frame(branch).show_view(branch)
         self.page.set_redirect_url(session, branch.marshal())
 
     def process_submit(self, session):

Modified: mgmt/cumin/python/wooly/__init__.py
===================================================================
--- mgmt/cumin/python/wooly/__init__.py	2008-05-08 20:04:06 UTC (rev 2013)
+++ mgmt/cumin/python/wooly/__init__.py	2008-05-08 20:08:31 UTC (rev 2014)
@@ -328,7 +328,7 @@
     def pop_current_frame(self, session):
         frame = self.get_current_frame(session)
         #print "Popping current frame", frame
-        return self.set_current_frame(session, frame.frame)
+        self.set_current_frame(session, frame.frame)
 
     def set_default_frame(self, frame):
         self.current_frame.default = frame