From 7373b4cf3afd120c3ad08e85e48c8e6a43006b56 Mon Sep 17 00:00:00 2001
From: Yoshinori Okuji <yo@nexedi.com>
Date: Wed, 20 Oct 2010 11:40:15 +0000
Subject: [PATCH] Mark bad names that must be fixed.

git-svn-id: https://svn.erp5.org/repos/public/erp5/trunk@39392 20353a03-c40f-0410-a6d1-a30d3c3de9de
---
 product/ERP5Type/Dynamic/lazyclass.py       | 1 +
 product/ERP5Type/Dynamic/portaltypeclass.py | 7 +++++++
 2 files changed, 8 insertions(+)

diff --git a/product/ERP5Type/Dynamic/lazyclass.py b/product/ERP5Type/Dynamic/lazyclass.py
index 52be5601af..1e658196e5 100644
--- a/product/ERP5Type/Dynamic/lazyclass.py
+++ b/product/ERP5Type/Dynamic/lazyclass.py
@@ -3,6 +3,7 @@ from ExtensionClass import Base as ExtensionBase
 
 from zLOG import LOG, ERROR, BLATHER
 
+# FIXME: bad name
 def lazyclass(name, portal_type_class_attr_getter):
     def load(self, attr):
         klass = None
diff --git a/product/ERP5Type/Dynamic/portaltypeclass.py b/product/ERP5Type/Dynamic/portaltypeclass.py
index 5098a16c69..cbd1e8294a 100644
--- a/product/ERP5Type/Dynamic/portaltypeclass.py
+++ b/product/ERP5Type/Dynamic/portaltypeclass.py
@@ -42,6 +42,7 @@ from Products.ERP5Type import PropertySheet as FilesystemPropertySheet
 from ExtensionClass import Base as ExtensionBase
 from zLOG import LOG, ERROR, INFO
 
+# FIXME: bad name
 def _import_class(classpath):
   try:
     module_path, class_name = classpath.rsplit('.', 1)
@@ -56,6 +57,7 @@ def _import_class(classpath):
   except StandardError:
     raise ImportError('Could not import document class %s' % classpath)
 
+# FIXME: bad name
 def _create_accessor_holder_class(property_sheet_tool,
                                   property_sheet_module,
                                   property_sheet_name):
@@ -64,6 +66,7 @@ def _create_accessor_holder_class(property_sheet_tool,
   generate its accessor holder
   """
 
+# FIXME: bad name
 def _fill_accessor_holder_list(accessor_holder_list,
                                create_accessor_holder_func,
                                property_sheet_name_set,
@@ -101,6 +104,7 @@ def _fill_accessor_holder_list(accessor_holder_list,
             "Created accessor holder for %s in %s" % (property_sheet_name,
                                                       accessor_holder_module))
 
+# FIXME: bad name
 def portal_type_factory(portal_type_name):
   """
   Given a portal type, look up in Types Tool the corresponding
@@ -224,6 +228,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):
     """
     Returns a lazily-loaded "portal-type as a class"
@@ -245,6 +250,7 @@ def initializeDynamicModules():
 
   erp5.portal_type = portal_type_container
 
+  # FIXME: bad name
   def temp_portal_type_loader(portal_type_name):
     """
     Returns a class suitable for a temporary portal type
@@ -280,6 +286,7 @@ def initializeDynamicModules():
   erp5.temp_portal_type = dynamicmodule.dynamicmodule('erp5.temp_portal_type',
                                                    temp_portal_type_loader)
 
+# FIXME: bad name
 def _clear_accessor_holder_module(module):
   """
   Clear the given accessor holder module (either for filesystem or
-- 
2.30.9