InitialiseClass implemented on prop_holder for security....

InitialiseClass implemented on prop_holder for security. Base.aq_related_generated = 1 moves in safe place


git-svn-id: https://svn.erp5.org/repos/public/erp5/trunk@3315 20353a03-c40f-0410-a6d1-a30d3c3de9de

product/ERP5Type/Base.py

@@ -177,10 +177,7 @@ def initializePortalTypeDynamicProperties(self, klass, ptype):
     prop_holder._properties = prop_list
     prop_holder._categories = cat_list
     prop_holder._constraints = constraint_list
-    if hasattr(klass, 'security'):
-      prop_holder.security = klass.security # Is this OK for security XXX ?
-    else:
-      prop_holder.security = ClassSecurityInfo() # Is this OK for security XXX ?
+    prop_holder.security = ClassSecurityInfo() # We create a new security info object
     from Utils import initializeDefaultProperties
     initializeDefaultProperties([prop_holder], object=self)
     #LOG('initializeDefaultProperties: %s' % ptype, 0, str(prop_holder.__dict__))
@@ -256,7 +253,8 @@ def initializePortalTypeDynamicProperties(self, klass, ptype):
                       method = WorkflowMethod(method, method_id)
                       setattr(prop_holder, method_id, method)      
                       
-    # We can now associate it
+    # We can now associate it after initialising security
+    InitializeClass(prop_holder)
     Base.aq_portal_type[ptype] = prop_holder
 
 
@@ -363,7 +361,6 @@ class Base( CopyContainer, PortalContent, ActiveObject, ERP5PropertyManager ):
     # Generate Related Accessors
     if not Base.aq_related_generated:
       from Utils import createRelatedValueAccessors
-      Base.aq_related_generated = 1
       generated = 1
       portal_categories = getToolByName(self, 'portal_categories', None)
       generated_bid = {}
@@ -374,6 +371,7 @@ class Base( CopyContainer, PortalContent, ActiveObject, ERP5PropertyManager ):
               #LOG( "Create createRelatedValueAccessors %s" % bid,0,'')
               createRelatedValueAccessors(Base, bid)
               generated_bid[bid] = 1
+      Base.aq_related_generated = 1
 
     # Always try to return something after generation
     if generated: