[Erp5-report] r39521 arnaud.fontaine - in /erp5/trunk/products/ERP5Type: dynamic/ tests/

[nobody at svn.erp5.org]

Author: arnaud.fontaine
Date: Tue Oct 26 09:22:13 2010
New Revision: 39521

URL: http://svn.erp5.org?rev=39521&view=rev
Log:
Fix generation of accessor holders for Document ZODB Property Sheets
and typos in ZODB Property Sheets test


Modified:
    erp5/trunk/products/ERP5Type/dynamic/portal_type_class.py
    erp5/trunk/products/ERP5Type/tests/testDynamicClassGeneration.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=39521&r1=39520&r2=39521&view=diff
==============================================================================
--- erp5/trunk/products/ERP5Type/dynamic/portal_type_class.py [utf8] (original)
+++ erp5/trunk/products/ERP5Type/dynamic/portal_type_class.py [utf8] Tue Oct 26 09:22:13 2010
@@ -163,7 +163,8 @@ def generatePortalTypeClass(portal_type_
   # _fillAccessorHolderList(
   #   accessor_holder_list,
   #   site.portal_property_sheets.createFilesystemPropertySheetAccessorHolder,
-  #   [ klass.__name__ for klass in getClassPropertyList(type_class) ],
+  #   [ property_sheet.__name__ for property_sheet in \
+  #     getClassPropertyList(klass) ],
   #   erp5.filesystem_accessor_holder,
   #   FilesystemPropertySheet)
 

Modified: erp5/trunk/products/ERP5Type/tests/testDynamicClassGeneration.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5Type/tests/testDynamicClassGeneration.py?rev=39521&r1=39520&r2=39521&view=diff
==============================================================================
--- erp5/trunk/products/ERP5Type/tests/testDynamicClassGeneration.py [utf8] (original)
+++ erp5/trunk/products/ERP5Type/tests/testDynamicClassGeneration.py [utf8] Tue Oct 26 09:22:13 2010
@@ -274,16 +274,16 @@ class TestZodbPropertySheet(ERP5TypeTest
       self._newCategoryExistenceConstraint()
 
       # Create all the test Properties
-      for op in ('change', 'delete', 'assign'):
-        self._newStandardProperty(op)
-        self._newAcquiredProperty(op)
-        self._newCategoryProperty(op)
-        self._newDynamicCategoryProperty(op)
+      for operation_type in ('change', 'delete', 'assign'):
+        self._newStandardProperty(operation_type)
+        self._newAcquiredProperty(operation_type)
+        self._newCategoryProperty(operation_type)
+        self._newDynamicCategoryProperty(operation_type)
 
     # Bind all test properties to this instance, so they can be
     # accessed easily in further tests
-    for prop in self.test_property_sheet.contentValues():
-      setattr(self, prop.getReference(), prop)
+    for property in self.test_property_sheet.contentValues():
+      setattr(self, property.getReference(), property)
 
     # Create a Portal Type for the tests, this is necessary, otherwise
     # there will be no accessor holder generated
@@ -313,7 +313,7 @@ class TestZodbPropertySheet(ERP5TypeTest
     self.tic()
 
     # Ensure that erp5.acessor_holder is empty
-    from Products.ERP5Type.Dynamic.portaltypeclass import \
+    from Products.ERP5Type.dynamic.portal_type_class import \
         synchronizeDynamicModules
 
     synchronizeDynamicModules(portal, force=True)