[Erp5-report] r29981 - /erp5/trunk/products/ERP5Type/Utils.py

nobody at svn.erp5.org nobody at svn.erp5.org
Fri Oct 23 18:06:58 CEST 2009


Author: leonardo
Date: Fri Oct 23 18:06:56 2009
New Revision: 29981

URL: http://svn.erp5.org?rev=29981&view=rev
Log:
Small cleanup (approved by jm)

Modified:
    erp5/trunk/products/ERP5Type/Utils.py

Modified: erp5/trunk/products/ERP5Type/Utils.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5Type/Utils.py?rev=29981&r1=29980&r2=29981&view=diff
==============================================================================
--- erp5/trunk/products/ERP5Type/Utils.py [utf8] (original)
+++ erp5/trunk/products/ERP5Type/Utils.py [utf8] Fri Oct 23 18:06:56 2009
@@ -974,20 +974,15 @@
                        this_module,
                        global_hook,
                        document_module=None,
-                       document_classes=None, # XXX - Never used - must be likely removed
-                       object_classes=None,
-                       portal_tools=None,
-                       content_constructors=None,
-                       content_classes=None):
+                       document_classes=(), # XXX - Never used - must be likely removed
+                       object_classes=(),
+                       portal_tools=(),
+                       content_constructors=(),
+                       content_classes=()):
   """
     This function does all the initialization steps required
     for a Zope / CMF Product
   """
-  if document_classes is None: document_classes = []
-  if object_classes is None: object_classes = []
-  if portal_tools is None: portal_tools = []
-  if content_constructors is None: content_constructors = []
-  if content_classes is None: content_classes = []
   product_name = this_module.__name__.split('.')[-1]
 
   # Define content constructors for Document content classes (RAD)
@@ -1069,19 +1064,12 @@
 
   # Register Objets
   for c in object_classes:
-    if hasattr(c, 'icon'):
-      icon = getattr(c, 'icon')
-    else:
-      icon = None
-    if hasattr(c, 'permission_type'):
-      context.registerClass( c,
-                           constructors = c.constructors,
-                           permission = c.permission_type,
-                           icon = icon)
-    else:
-      context.registerClass( c,
-                           constructors = c.constructors,
-                           icon = icon)
+    icon = getattr(c, 'icon', None)
+    permission = getattr(c, 'permission_type', None)
+    context.registerClass(c,
+                          constructors = c.constructors,
+                          permission = permission,
+                          icon = icon)
 
 class ConstraintNotFound(Exception):
   pass




More information about the Erp5-report mailing list