[CMF-checkins] CVS: CMF/CMFCore - ActionProviderBase.py:1.19

[Chris McDonough]

Update of /cvs-repository/CMF/CMFCore
In directory cvs.zope.org:/tmp/cvs-serv10905/CMFCore

Modified Files:
	ActionProviderBase.py 
Log Message:
Add bug note.


=== CMF/CMFCore/ActionProviderBase.py 1.18 => 1.19 ===
--- CMF/CMFCore/ActionProviderBase.py:1.18	Tue Sep  2 13:36:46 2003
+++ CMF/CMFCore/ActionProviderBase.py	Sun Oct 12 20:04:30 2003
@@ -60,6 +60,19 @@
     def manage_editActionsForm( self, REQUEST, manage_tabs_message=None ):
 
         """ Show the 'Actions' management tab.
+
+        Bug note: Note that this method, when called by Management.py's
+        manage_workspace (when it is the first action of an object),
+        will be called with REQUEST equal to 'self' and
+        manage_tabs_message equal to the REQUEST.  This causes the
+        request to be printed by manage_tabs, because it considers
+        'manage_tabs_message' to be true.  The fix would be to remove
+        this method and put it all into DTML (probably easiest) or to
+        make manage_workspace sniff at the method it obtains to make sure
+        it's a DTMLFile before passing along self and REQUEST. I don't
+        care too much either way, and I'm too lazy to fix it at the moment.
+        - chrism
+        
         """
         actions = []