[Erp5-report] r43206 arnaud.fontaine - /erp5/trunk/products/ERP5Type/Tool/PropertySheetTool.py

[nobody at svn.erp5.org]

Author: arnaud.fontaine
Date: Wed Feb  9 07:22:55 2011
New Revision: 43206

URL: http://svn.erp5.org?rev=43206&view=rev
Log:
When adding a new category in a ZODB Property Sheet, the reference has
not been filled yet, thus re-generated accessors could have failed


Modified:
    erp5/trunk/products/ERP5Type/Tool/PropertySheetTool.py

Modified: erp5/trunk/products/ERP5Type/Tool/PropertySheetTool.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5Type/Tool/PropertySheetTool.py?rev=43206&r1=43205&r2=43206&view=diff
==============================================================================
--- erp5/trunk/products/ERP5Type/Tool/PropertySheetTool.py [utf8] (original)
+++ erp5/trunk/products/ERP5Type/Tool/PropertySheetTool.py [utf8] Wed Feb  9 07:22:55 2011
@@ -221,9 +221,19 @@ class PropertySheetTool(BaseTool):
     categories = []
 
     for property in property_sheet.contentValues():
-      portal_type = property.getPortalType()
       property_definition = property.exportToFilesystemDefinition()
 
+      # If a category doesn't have a name yet or the constraint class
+      # returned is None, then just skip it
+      if property_definition is None:
+        LOG("Tool.PropertySheetTool", INFO,
+            "Skipping property with ID '%s' in Property Sheet '%s'" % \
+            (property.getId(), property_sheet.getId()))
+
+        continue
+
+      portal_type = property.getPortalType()
+
       if portal_type == "Category Property" or \
          portal_type == "Dynamic Category Property":
         categories.append(property_definition)