[Erp5-report] r28428 - in /erp5/trunk/products/ERP5/Tool: BuilderTool.py LogMixin.py

[nobody at svn.erp5.org]

Author: jerome
Date: Tue Aug 18 08:48:20 2009
New Revision: 28428

URL: http://svn.erp5.org?rev=28428&view=rev
Log:
fix bogus security declarations

Modified:
    erp5/trunk/products/ERP5/Tool/BuilderTool.py
    erp5/trunk/products/ERP5/Tool/LogMixin.py

Modified: erp5/trunk/products/ERP5/Tool/BuilderTool.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5/Tool/BuilderTool.py?rev=28428&r1=28427&r2=28428&view=diff
==============================================================================
--- erp5/trunk/products/ERP5/Tool/BuilderTool.py [utf8] (original)
+++ erp5/trunk/products/ERP5/Tool/BuilderTool.py [utf8] Tue Aug 18 08:48:20 2009
@@ -62,7 +62,8 @@
 
   security = ClassSecurityInfo()
 
-  security.declareProtected( Permissions.AccessContentsInformation, 'build')
+  security.declareProtected(Permissions.AccessContentsInformation,
+                            'getBuilderValueList')
   def getBuilderValueList(self, business_process_list=None,
         trade_phase_list=None):
     """Returns sorted builder list with proper condition"""
@@ -86,7 +87,7 @@
     # FIXME: what kind of sorting to use?
     return sorted(builder_value_list)
 
-  security.declareProtected( Permissions.View, 'build')
+  security.declareProtected(Permissions.AccessContentsInformation, 'build')
   def build(self, input_movement_list=None, existing_delivery_list=None,
       business_process_list=None, trade_phase_list=None):
     """Informs all builders to be build or invoke building

Modified: erp5/trunk/products/ERP5/Tool/LogMixin.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5/Tool/LogMixin.py?rev=28428&r1=28427&r2=28428&view=diff
==============================================================================
--- erp5/trunk/products/ERP5/Tool/LogMixin.py [utf8] (original)
+++ erp5/trunk/products/ERP5/Tool/LogMixin.py [utf8] Tue Aug 18 08:48:20 2009
@@ -50,7 +50,7 @@
   """
   security = ClassSecurityInfo()
 
-  security.declareProtected('getLogFile', Permissions.ManagePortal)
+  security.declareProtected(Permissions.ManagePortal, 'getLogFile')
   def getLogFile(self, file_name, account=None):
     """
       Returns the raw log file (as they are)
@@ -58,7 +58,8 @@
     """
     raise NotImplementedError
 
-  security.declareProtected('getLogFileNameList', Permissions.ManagePortal)
+  security.declareProtected(Permissions.ManagePortal,
+                            'getLogFileNameList')
   def getLogFileNameList(self, log_name, account=None,
                         domain=None, user_name=None, 
                         from_line=None, to_line=None,
@@ -71,14 +72,14 @@
     """
     raise NotImplementedError
 
-  security.declareProtected('getLogNameList', Permissions.ManagePortal)
+  security.declareProtected(Permissions.ManagePortal, 'getLogNameList')
   def getLogNameList(self, account=None):
     """
       Returns the list of log names.
     """
     raise NotImplementedError
 
-  security.declareProtected('parseLogLine', Permissions.ManagePortal)
+  security.declareProtected(Permissions.ManagePortal, 'parseLogLine')
   def parseLogLine(self, log_name, log_line ):
     """
       Parses the line and returns a dict