guard dynamically generated tester and preference methods.

product/ERP5Form/PreferenceTool.py

@@ -88,6 +88,20 @@ class PreferenceMethod(Method):
             cache_factory='erp5_ui_long')
     return _getPreference(default, *args, **kw)
 
+  class __roles__:
+    @staticmethod
+    def rolesForPermissionOn(ob):
+      self = ob.__self__
+      name = '%s__roles__' % ob.__name__
+      # we explictly call _aq_dynamic to prevent acquiering the attribute
+      # from container
+      roles = getattr(self.__class__, name, self)
+      if roles is self:
+        roles = self._aq_dynamic(name)
+        if roles is None:
+          return rolesForPermissionOn(None, self, ('Manager',),
+                                      '_Access_contents_information_Permission')
+      return getattr(roles, '__of__', lambda aq_parent: roles)(self)
 
 class PreferenceTool(BaseTool):
   """

product/ERP5Type/Accessor/Accessor.py

@@ -66,3 +66,18 @@ class Accessor(Method):
       # Returns a reindexing alias
       from .Alias import ReindexAlias
       return ReindexAlias(id, self.__name__)
+
+    class __roles__:
+      @staticmethod
+      def rolesForPermissionOn(ob):
+        self = ob.__self__
+        name = '%s__roles__' % ob.__name__
+        # we explictly call _aq_dynamic to prevent acquiering the attribute
+        # from container
+        roles = getattr(self.__class__, name, self)
+        if roles is self:
+          roles = self._aq_dynamic(name)
+          if roles is None:
+            return rolesForPermissionOn(None, self, ('Manager',),
+                                        '_Access_contents_information_Permission')
+        return getattr(roles, '__of__', lambda aq_parent: roles)(self)

product/ERP5Type/Accessor/Base.py

@@ -187,22 +187,6 @@ class Getter(Method):
 
     psyco.bind(__call__)
 
-    class __roles__:
-      @staticmethod
-      def rolesForPermissionOn(ob):
-        self = ob.__self__
-        name = '%s__roles__' % ob.__name__
-        # we explictly call _aq_dynamic to prevent acquiering the attribute
-        # from container
-        roles = getattr(self.__class__, name, self)
-        if roles is self:
-          roles = self._aq_dynamic(name)
-          if roles is None:
-            return rolesForPermissionOn(None, self, ('Manager',),
-                                        '_Access_contents_information_Permission')
-        return getattr(roles, '__of__', lambda aq_parent: roles)(self)
-
-
 class Tester(Method):
     """
       Tests if an attribute value exists