← Back to team overview

openerp-community-reviewer team mailing list archive

[Merge] lp:~agilebg/account-financial-report/fix_account_move_line_report_xls into lp:account-financial-report

 

Alex Comba - Agile BG has proposed merging lp:~agilebg/account-financial-report/fix_account_move_line_report_xls into lp:account-financial-report.

Requested reviews:
  Account Report Core Editors (account-report-core-editor)
Related bugs:
  Bug #1275730 in Account - Financial Report: "ImportError: No module named report_xls"
  https://bugs.launchpad.net/account-financial-report/+bug/1275730

For more details, see:
https://code.launchpad.net/~agilebg/account-financial-report/fix_account_move_line_report_xls/+merge/204500

Fixing for https://bugs.launchpad.net/account-financial-report/+bug/1275730
-- 
https://code.launchpad.net/~agilebg/account-financial-report/fix_account_move_line_report_xls/+merge/204500
Your team Account Report Core Editors is requested to review the proposed merge of lp:~agilebg/account-financial-report/fix_account_move_line_report_xls 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 14:15:59 +0000
@@ -19,6 +19,11 @@
 #    along with this program. If not, see <http://www.gnu.org/licenses/>.
 #
 ##############################################################################
-
-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')


Follow ups