From 904fe05142175382d0c702990aa1f06fa6282ffb Mon Sep 17 00:00:00 2001
From: Arnaud Fontaine <arnaud.fontaine@nexedi.com>
Date: Thu, 4 Nov 2010 06:23:43 +0000
Subject: [PATCH] Re-raise the exception on setDefaultProperties failure and
 improve logging of the error

git-svn-id: https://svn.erp5.org/repos/public/erp5/trunk@39876 20353a03-c40f-0410-a6d1-a30d3c3de9de
---
 product/ERP5Type/Tool/PropertySheetTool.py | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/product/ERP5Type/Tool/PropertySheetTool.py b/product/ERP5Type/Tool/PropertySheetTool.py
index 609ce663ef..bc8b4f932a 100644
--- a/product/ERP5Type/Tool/PropertySheetTool.py
+++ b/product/ERP5Type/Tool/PropertySheetTool.py
@@ -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(
-- 
2.30.9