erp5_deferred_style: send a notification if there has error when generate report

bt5/erp5_deferred_style/SkinTemplateItem/portal_skins/erp5_deferred_style_core/Base_renderSimpleView.py

@@ -14,30 +14,46 @@ if skin_name and skin_name != 'None': # make_query serializes None as 'None'
   portal.portal_skins.changeSkin(skin_name)
 
 with portal.Localizer.translationContext(localizer_language):
-  report_data = getattr(context, deferred_style_dialog_method)(**params)
+  try:
+    report_data = getattr(context, deferred_style_dialog_method)(**params)
 
-  attachment_name_list = [x[len(' filename='):] for x in (request.RESPONSE.getHeader(
-                        'content-disposition') or '').split(';')
-                            if x.startswith(' filename=')]
-  if attachment_name_list:
-    attachment_name, = attachment_name_list
-  else:
-    assert 'inline' in (request.RESPONSE.getHeader('content-disposition') or '')
-    attachment_name = 'index.html'
-  if attachment_name.startswith('"'):
-    attachment_name = attachment_name[1:]
-  if attachment_name.endswith('"'):
-    attachment_name = attachment_name[:-1]
-  attachment_list = (
-    {'mime_type': (request.RESPONSE.getHeader('content-type') or 'application/octet-stream;').split(';')[0],
-     'content': '%s' % report_data,
-     'name': attachment_name},)
+    attachment_name_list = [x[len(' filename='):] for x in (request.RESPONSE.getHeader(
+                          'content-disposition') or '').split(';')
+                              if x.startswith(' filename=')]
+    if attachment_name_list:
+      attachment_name, = attachment_name_list
+    else:
+      assert 'inline' in (request.RESPONSE.getHeader('content-disposition') or '')
+      attachment_name = 'index.html'
+    if attachment_name.startswith('"'):
+      attachment_name = attachment_name[1:]
+    if attachment_name.endswith('"'):
+      attachment_name = attachment_name[:-1]
+    attachment_list = (
+      {'mime_type': (request.RESPONSE.getHeader('content-type') or 'application/octet-stream;').split(';')[0],
+      'content': '%s' % report_data,
+      'name': attachment_name},)
 
-  getattr(portal, notify_report_complete_script_id)(
-    user_name=user_name,
-    subject=str(translateString(attachment_name.rsplit('.', 1)[0])),
-    message='',
-    attachment_list=attachment_list,
-    format=report_format,
-    **notify_report_complete_kwargs
-  )
+    getattr(portal, notify_report_complete_script_id)(
+      user_name=user_name,
+      subject=str(translateString(attachment_name.rsplit('.', 1)[0])),
+      message='',
+      attachment_list=attachment_list,
+      format=report_format,
+      **notify_report_complete_kwargs
+    )
+  except Exception, Error:
+    if type(report_data) is dict and report_data.pop('status', 0) == -2:
+      message = report_data.pop('message')
+      subject = report_data.pop('subject')
+    else:
+      message = 'Unknown Error, Please Retry'
+      subject = 'ERP5: Deferred Report'
+    getattr(portal, notify_report_complete_script_id)(
+      user_name=user_name,
+      subject=subject,
+      attachment_list = [],
+      message=message,
+      format=report_format
+    )
+    context.log(Error)