openerp-community-reviewer team mailing list archive

[Romain Deheele - Camptocamp <romain.deheele@xxxxxxxxxxxxxx>]

Romain Deheele - Camptocamp has proposed merging lp:~camptocamp/account-financial-report/7.0-fix-crash-without-report_xls-rde into lp:account-financial-report.

Requested reviews:
  Account Report Core Editors (account-report-core-editor)

For more details, see:
https://code.launchpad.net/~camptocamp/account-financial-report/7.0-fix-crash-without-report_xls-rde/+merge/204432

Hello,

It's a readjustment. 
Rev 67 has removed check commited in rev65 to avoid a crash if this branch is in addons path but report_xls is not available.

Regards,

Romain
-- 
https://code.launchpad.net/~camptocamp/account-financial-report/7.0-fix-crash-without-report_xls-rde/+merge/204432
Your team Account Report Core Editors is requested to review the proposed merge of lp:~camptocamp/account-financial-report/7.0-fix-crash-without-report_xls-rde into lp:account-financial-report.

=== modified file 'account_move_line_report_xls/__init__.py'
--- account_move_line_report_xls/__init__.py	2014-01-09 11:07:08 +0000
+++ account_move_line_report_xls/__init__.py	2014-02-03 09:21:20 +0000
@@ -20,5 +20,10 @@
 #
 ##############################################################################
 
-from . import account_move_line
-from . import report
+try:
+    from . import account_move_line
+    from . import report
+except ImportError:
+    import logging
+    logging.getLogger('openerp.module').warning('report_xls not available in addons path. account_financial_report_webkit_xls will not be usable')
+