[Erp5-report] r42910 nicolas.dumazet - /erp5/trunk/products/ERP5Type/dynamic/

[nobody at svn.erp5.org]

Author: nicolas.dumazet
Date: Wed Feb  2 01:27:53 2011
New Revision: 42910

URL: http://svn.erp5.org?rev=42910&view=rev
Log:
if we're creating accessors for Base Category, the accessor holder was already
created. Move the test up, before accessor creation.

The accessor holder name is also BaseCategory instead of "Base Category".

After this, we should get back all related category accessors.

Modified:
    erp5/trunk/products/ERP5Type/dynamic/portal_type_class.py

Modified: erp5/trunk/products/ERP5Type/dynamic/portal_type_class.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5Type/dynamic/portal_type_class.py?rev=42910&r1=42909&r2=42910&view=diff
==============================================================================
--- erp5/trunk/products/ERP5Type/dynamic/portal_type_class.py [utf8] (original)
+++ erp5/trunk/products/ERP5Type/dynamic/portal_type_class.py [utf8] Wed Feb  2 01:27:53 2011
@@ -284,24 +284,25 @@ def generatePortalTypeClass(site, portal
     import erp5
 
     if property_sheet_set:
-      # Initialize ZODB Property Sheets accessor holders
-      _fillAccessorHolderList(
-        accessor_holder_list,
-        property_sheet_tool.createZodbPropertySheetAccessorHolder,
-        property_sheet_set,
-        erp5.accessor_holder,
-        property_sheet_tool)
-
       if "Base" in property_sheet_set:
         accessor_holder = None
         # useless if Base Category is not yet here
-        if hasattr(erp5.accessor_holder, "Base Category"):
+        # or if we're currently generating accessors for Base Categories
+        if hasattr(erp5.accessor_holder, "BaseCategory"):
           accessor_holder = _generateBaseAccessorHolder(
                               site,
                               erp5.accessor_holder)
         if accessor_holder is not None:
           accessor_holder_list.append(accessor_holder)
 
+      # Initialize ZODB Property Sheets accessor holders
+      _fillAccessorHolderList(
+        accessor_holder_list,
+        property_sheet_tool.createZodbPropertySheetAccessorHolder,
+        property_sheet_set,
+        erp5.accessor_holder,
+        property_sheet_tool)
+
       # XXX a hook to add per-portal type accessor holders maybe?
       if portal_type_name == "Preference Tool":
         accessor_holder = _generatePreferenceToolAccessorHolder(