[Erp5-report] r39876 arnaud.fontaine - /erp5/trunk/products/ERP5Type/Tool/PropertySheetTool.py

[nobody at svn.erp5.org]

Author: arnaud.fontaine
Date: Thu Nov  4 07:23:43 2010
New Revision: 39876

URL: http://svn.erp5.org?rev=39876&view=rev
Log:
Re-raise the exception on setDefaultProperties failure and improve
logging of the error


Modified:
    erp5/trunk/products/ERP5Type/Tool/PropertySheetTool.py

Modified: erp5/trunk/products/ERP5Type/Tool/PropertySheetTool.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5Type/Tool/PropertySheetTool.py?rev=39876&r1=39875&r2=39876&view=diff
==============================================================================
--- erp5/trunk/products/ERP5Type/Tool/PropertySheetTool.py [utf8] (original)
+++ erp5/trunk/products/ERP5Type/Tool/PropertySheetTool.py [utf8] Thu Nov  4 07:23:43 2010
@@ -27,6 +27,7 @@
 #
 ##############################################################################
 
+import sys
 import transaction
 
 from AccessControl import ClassSecurityInfo
@@ -185,14 +186,12 @@ class PropertySheetTool(BaseTool):
     try:
       setDefaultProperties(property_holder, portal=self.getPortalObject())
     except:
-      import traceback
       LOG("Tool.PropertySheetTool", ERROR,
-          "Could not generate accessor holder class for %s (module=%s): %s" %\
-          (property_sheet_id,
-           accessor_holder_module_name,
-           traceback.format_exc()))
+          "Could not generate accessor holder class for %s (module=%s)" % \
+          (property_sheet_id, accessor_holder_module_name),
+          error=sys.exc_info())
 
-      return None
+      raise
 
     # Create the new accessor holder class and set its module properly
     accessor_holder_class = type(property_sheet_id, (object,), dict(