From da9a6acf535795eddcf797424b67476caff3f4bb Mon Sep 17 00:00:00 2001
From: Yusei Tahara <yusei@nexedi.com>
Date: Mon, 26 May 2008 17:37:41 +0000
Subject: [PATCH] log portal_transforms failure.

git-svn-id: https://svn.erp5.org/repos/public/erp5/trunk@21145 20353a03-c40f-0410-a6d1-a30d3c3de9de
---
 product/ERP5/Document/TextDocument.py | 15 +++++++++++----
 1 file changed, 11 insertions(+), 4 deletions(-)

diff --git a/product/ERP5/Document/TextDocument.py b/product/ERP5/Document/TextDocument.py
index 252d8ebda9..098739e099 100644
--- a/product/ERP5/Document/TextDocument.py
+++ b/product/ERP5/Document/TextDocument.py
@@ -27,6 +27,7 @@
 ##############################################################################
 
 from AccessControl import ClassSecurityInfo
+from zLOG import LOG, WARNING
 from Products.ERP5Type.Base import WorkflowMethod
 from Products.CMFCore.utils import getToolByName
 from Products.CMFCore.utils import _setCacheHeaders, _ViewEmulator
@@ -162,10 +163,16 @@ class TextDocument(Document, TextContent):
       text_content = self.getTextContent()
       if text_content is not None:
         portal_transforms = getToolByName(self, 'portal_transforms')
-        return mime_type, portal_transforms.convertTo(mime_type,
-                                                      text_content,
-                                                      object = self,
-                                                      mimetype = src_mimetype)
+        result = portal_transforms.convertTo(mime_type,
+                                             text_content,
+                                             object=self,
+                                             mimetype=src_mimetype)
+        if result is None:
+            # portal_transforms fails to convert.
+            LOG('TextDocument.convert', WARNING,
+                'portal_transforms failed to convert to text: %r' % self)
+            result = ''
+        return mime_type, result
       else:
         # text_content is not set, return empty string instead of None
         return mime_type, ''
-- 
2.30.9