Merge pull request #229 from Tecnativa/9.0-account_financial_report_qweb-fix_importerror

[FIX][account_financial_report_qweb] Fix ImportError.

account_financial_report_qweb/README.rst

@@ -50,6 +50,7 @@ Contributors
 * Francesco Apruzzese <opencode@e-ware.org>
 * Lorenzo Battistini <lorenzo.battistini@agilebg.com>
 * Julien Coux <julien.coux@camptocamp.com>
+* Jairo Llopis <jairo.llopis@tecnativa.com>
 
 Much of the work in this module was done at a sprint in Sorrento, Italy in
 April 2016.

account_financial_report_qweb/__openerp__.py

@@ -5,7 +5,7 @@
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
 {
     'name': 'QWeb Financial Reports',
-    'version': '9.0.1.0.0',
+    'version': '9.0.1.0.1',
     'category': 'Reporting',
     'summary': 'OCA Financial Reports',
     'author': 'Camptocamp SA,'
@@ -35,8 +35,11 @@
         'report/templates/trial_balance.xml',
         'view/account_view.xml'
     ],
-    'test': [
+    'external_dependencies': {
+        "python": [
+            "xlsxwriter",
         ],
+    },
     'installable': True,
     'application': True,
     'auto_install': False,

account_financial_report_qweb/report/abstract_report_xlsx.py

@@ -2,9 +2,16 @@
 # Author: Julien Coux
 # Copyright 2016 Camptocamp SA
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
+try:
+    import xlsxwriter
+except ImportError:  # pragma: no cover
+    import logging
+    _logger = logging.getLogger(__name__)
+    _logger.info("Missing dependency: xlsxwriter.")
+    _logger.debug("ImportError details:", exc_info=True)
 
 from cStringIO import StringIO
-import xlsxwriter
+
 from openerp.addons.report_xlsx.report.report_xlsx import ReportXlsx