Commit 7373b4cf authored by Yoshinori Okuji's avatar Yoshinori Okuji

Mark bad names that must be fixed.

git-svn-id: https://svn.erp5.org/repos/public/erp5/trunk@39392 20353a03-c40f-0410-a6d1-a30d3c3de9de
parent 349fd4a5
...@@ -3,6 +3,7 @@ from ExtensionClass import Base as ExtensionBase ...@@ -3,6 +3,7 @@ from ExtensionClass import Base as ExtensionBase
from zLOG import LOG, ERROR, BLATHER from zLOG import LOG, ERROR, BLATHER
# FIXME: bad name
def lazyclass(name, portal_type_class_attr_getter): def lazyclass(name, portal_type_class_attr_getter):
def load(self, attr): def load(self, attr):
klass = None klass = None
......
...@@ -42,6 +42,7 @@ from Products.ERP5Type import PropertySheet as FilesystemPropertySheet ...@@ -42,6 +42,7 @@ from Products.ERP5Type import PropertySheet as FilesystemPropertySheet
from ExtensionClass import Base as ExtensionBase from ExtensionClass import Base as ExtensionBase
from zLOG import LOG, ERROR, INFO from zLOG import LOG, ERROR, INFO
# FIXME: bad name
def _import_class(classpath): def _import_class(classpath):
try: try:
module_path, class_name = classpath.rsplit('.', 1) module_path, class_name = classpath.rsplit('.', 1)
...@@ -56,6 +57,7 @@ def _import_class(classpath): ...@@ -56,6 +57,7 @@ def _import_class(classpath):
except StandardError: except StandardError:
raise ImportError('Could not import document class %s' % classpath) raise ImportError('Could not import document class %s' % classpath)
# FIXME: bad name
def _create_accessor_holder_class(property_sheet_tool, def _create_accessor_holder_class(property_sheet_tool,
property_sheet_module, property_sheet_module,
property_sheet_name): property_sheet_name):
...@@ -64,6 +66,7 @@ def _create_accessor_holder_class(property_sheet_tool, ...@@ -64,6 +66,7 @@ def _create_accessor_holder_class(property_sheet_tool,
generate its accessor holder generate its accessor holder
""" """
# FIXME: bad name
def _fill_accessor_holder_list(accessor_holder_list, def _fill_accessor_holder_list(accessor_holder_list,
create_accessor_holder_func, create_accessor_holder_func,
property_sheet_name_set, property_sheet_name_set,
...@@ -101,6 +104,7 @@ def _fill_accessor_holder_list(accessor_holder_list, ...@@ -101,6 +104,7 @@ def _fill_accessor_holder_list(accessor_holder_list,
"Created accessor holder for %s in %s" % (property_sheet_name, "Created accessor holder for %s in %s" % (property_sheet_name,
accessor_holder_module)) accessor_holder_module))
# FIXME: bad name
def portal_type_factory(portal_type_name): def portal_type_factory(portal_type_name):
""" """
Given a portal type, look up in Types Tool the corresponding Given a portal type, look up in Types Tool the corresponding
...@@ -224,6 +228,7 @@ def initializeDynamicModules(): ...@@ -224,6 +228,7 @@ def initializeDynamicModules():
XXX: there should be only one accessor_holder once the code is XXX: there should be only one accessor_holder once the code is
stable and all the Property Sheets have been migrated stable and all the Property Sheets have been migrated
""" """
# FIXME: bad name
def portal_type_loader(portal_type_name): def portal_type_loader(portal_type_name):
""" """
Returns a lazily-loaded "portal-type as a class" Returns a lazily-loaded "portal-type as a class"
...@@ -245,6 +250,7 @@ def initializeDynamicModules(): ...@@ -245,6 +250,7 @@ def initializeDynamicModules():
erp5.portal_type = portal_type_container erp5.portal_type = portal_type_container
# FIXME: bad name
def temp_portal_type_loader(portal_type_name): def temp_portal_type_loader(portal_type_name):
""" """
Returns a class suitable for a temporary portal type Returns a class suitable for a temporary portal type
...@@ -280,6 +286,7 @@ def initializeDynamicModules(): ...@@ -280,6 +286,7 @@ def initializeDynamicModules():
erp5.temp_portal_type = dynamicmodule.dynamicmodule('erp5.temp_portal_type', erp5.temp_portal_type = dynamicmodule.dynamicmodule('erp5.temp_portal_type',
temp_portal_type_loader) temp_portal_type_loader)
# FIXME: bad name
def _clear_accessor_holder_module(module): def _clear_accessor_holder_module(module):
""" """
Clear the given accessor holder module (either for filesystem or Clear the given accessor holder module (either for filesystem or
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment