[Erp5-report] r7467 - /erp5/trunk/products/ERP5Form/PreferenceTool.py

nobody at svn.erp5.org nobody at svn.erp5.org
Tue May 23 22:27:08 CEST 2006


Author: jerome
Date: Tue May 23 22:26:58 2006
New Revision: 7467

URL: http://svn.erp5.org?rev=7467&view=rev
Log:
Search catalog using portal_type argument.
use getProperty instead of getPreferenceState, because it's safer (if
this method fail, we cannot use view interface)


Modified:
    erp5/trunk/products/ERP5Form/PreferenceTool.py

Modified: erp5/trunk/products/ERP5Form/PreferenceTool.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5Form/PreferenceTool.py?rev=7467&r1=7466&r2=7467&view=diff
==============================================================================
--- erp5/trunk/products/ERP5Form/PreferenceTool.py (original)
+++ erp5/trunk/products/ERP5Form/PreferenceTool.py Tue May 23 22:26:58 2006
@@ -155,9 +155,10 @@
     #                  or better solution
     user = getToolByName(self, 'portal_membership').getAuthenticatedMember()
     user_is_manager = 'Manager' in user.getRolesInContext(self)
-    for pref in self.searchFolder(spec=('ERP5 Preference', )) :
+    for pref in self.searchFolder(portal_type='Preference') :
       pref = pref.getObject()
-      if pref is not None and pref.getPreferenceState() == 'enabled' :
+      if pref is not None and pref.getProperty('preference_state',
+                                               'broken') == 'enabled' :
         # XXX quick workaround so that manager only see user preference
         # they actually own.
         if user_is_manager and pref.getPriority() == Priority.USER :




More information about the Erp5-report mailing list