[Erp5-report] r39413 nicolas.dumazet - /erp5/trunk/products/ERP5Type/Dynamic/

nobody at svn.erp5.org nobody at svn.erp5.org
Thu Oct 21 03:49:17 CEST 2010


Author: nicolas.dumazet
Date: Thu Oct 21 03:49:16 2010
New Revision: 39413

URL: http://svn.erp5.org?rev=39413&view=rev
Log:
naming fixes

Modified:
    erp5/trunk/products/ERP5Type/Dynamic/portaltypeclass.py

Modified: erp5/trunk/products/ERP5Type/Dynamic/portaltypeclass.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5Type/Dynamic/portaltypeclass.py?rev=39413&r1=39412&r2=39413&view=diff
==============================================================================
--- erp5/trunk/products/ERP5Type/Dynamic/portaltypeclass.py [utf8] (original)
+++ erp5/trunk/products/ERP5Type/Dynamic/portaltypeclass.py [utf8] Thu Oct 21 03:49:16 2010
@@ -232,8 +232,7 @@ def initializeDynamicModules():
   XXX: there should be only one accessor_holder once the code is
        stable and all the Property Sheets have been migrated
   """
-  # FIXME: bad name
-  def portal_type_loader(portal_type_name):
+  def portalTypeLoader(portal_type_name):
     """
     Returns a lazily-loaded "portal-type as a class"
     """
@@ -250,12 +249,11 @@ def initializeDynamicModules():
   sys.modules["erp5.filesystem_accessor_holder"] = erp5.filesystem_accessor_holder
 
   portal_type_container = newDynamicModule('erp5.portal_type',
-                                           portal_type_loader)
+                                           portalTypeLoader)
 
   erp5.portal_type = portal_type_container
 
-  # FIXME: bad name
-  def temp_portal_type_loader(portal_type_name):
+  def tempPortalTypeLoader(portal_type_name):
     """
     Returns a class suitable for a temporary portal type
 
@@ -288,7 +286,7 @@ def initializeDynamicModules():
     return TempDocument
 
   erp5.temp_portal_type = newDynamicModule('erp5.temp_portal_type',
-                                           temp_portal_type_loader)
+                                           tempPortalTypeLoader)
 
 def _clearAccessorHolderModule(module):
   """




More information about the Erp5-report mailing list