Take into accounts most remarks of @lasley

Remove <data> in views
Protect import of py3o libs
Remove dep on base module
Other small changes

report_py3o/NEWS

@@ -1,4 +0,0 @@
-report_py3o 1.3
-
-Production release
-

report_py3o/__openerp__.py

@@ -2,7 +2,7 @@
 # Copyright 2013 XCG Consulting (http://odoo.consulting)
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 {
-    'name': 'LibreOffice Report Engine',
+    'name': 'Py3o Report Engine',
     'summary': 'Reporting engine based on Libreoffice (ODT -> ODT, '
                'ODT -> PDF, ODT -> DOC, ODT -> DOCX, ODS -> ODS, etc.)',
     'version': '9.0.1.0.0',
@@ -10,10 +10,7 @@
     'license': 'AGPL-3',
     'author': 'XCG Consulting,Odoo Community Association (OCA)',
     'website': 'http://odoo.consulting/',
-    'depends': [
-        'base',
-        'report',
-    ],
+    'depends': ['report'],
     'external_dependencies': {
         'python': ['py3o.template',
                    'py3o.formats']

report_py3o/models/ir_report.py

@@ -2,15 +2,22 @@
 # Copyright 2013 XCG Consulting (http://odoo.consulting)
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 import os
-from py3o.formats import Formats
 from openerp import api, fields, models, _
 from openerp.report.interface import report_int
 from openerp.exceptions import ValidationError
 from openerp import addons
 from ..py3o_parser import Py3oParser
+import logging
+
+logger = logging.getLogger(__name__)
+
+try:
+    from py3o.formats import Formats
+except ImportError:
+    logger.debug('Cannot import py3o.formats')
 
 
-class ReportXml(models.Model):
+class IrActionsReportXml(models.Model):
     """ Inherit from ir.actions.report.xml to allow customizing the template
     file. The user cam chose a template from a list.
     The list is configurable in the configuration tab, see py3o_template.py
@@ -22,8 +29,8 @@ class ReportXml(models.Model):
     @api.constrains("py3o_filetype", "report_type")
     def _check_py3o_filetype(self):
         if self.report_type == "py3o" and not self.py3o_filetype:
-            raise ValidationError(
-                "Field 'Output Format' is required for Py3O report")
+            raise ValidationError(_(
+                "Field 'Output Format' is required for Py3O report"))
 
     @api.one
     @api.constrains("py3o_is_local_fusion", "py3o_server_id",
@@ -32,9 +39,9 @@ class ReportXml(models.Model):
         is_native = Formats().get_format(self.py3o_filetype)
         if ((not is_native or not self.py3o_is_local_fusion) and
                 not self.py3o_server_id):
-            raise ValidationError(
+            raise ValidationError(_(
                 "Can not use not native format in local fusion. "
-                "Please specify a Fusion Server")
+                "Please specify a Fusion Server"))
 
     @api.model
     def _get_py3o_filetypes(self):
@@ -116,4 +123,4 @@ class ReportXml(models.Model):
         if new_report:
             return new_report
         else:
-            return super(ReportXml, self)._lookup_report(cr, name)
+            return super(IrActionsReportXml, self)._lookup_report(cr, name)

report_py3o/py3o_parser.py

@@ -9,14 +9,23 @@ import sys
 from base64 import b64decode
 import requests
 from tempfile import NamedTemporaryFile
-from py3o.template.helpers import Py3oConvertor
-from py3o.template import Template
-from py3o.formats import Formats
-
 from openerp import _
 from openerp import exceptions
 from openerp.report.report_sxw import report_sxw
 from openerp import registry
+import logging
+
+logger = logging.getLogger(__name__)
+
+try:
+    from py3o.template.helpers import Py3oConvertor
+    from py3o.template import Template
+except ImportError:
+    logger.debug('Cannot import py3o.template')
+try:
+    from py3o.formats import Formats
+except ImportError:
+    logger.debug('Cannot import py3o.formats')
 
 
 _extender_functions = {}

report_py3o/views/ir_report.xml

@@ -1,6 +1,5 @@
 <?xml version="1.0" encoding="utf-8"?>
 <odoo>
-    <data>
 
     <!-- Inherit from base.act_report_xml_view to add py3o-related settings. -->
 
@@ -11,10 +10,10 @@
         <field name="arch" type="xml">
 
             <xpath expr="//page[@name='security']" position="before">
-                    <page string="LibreOffice Template"
+                <page string="LibreOffice Template" name="py3o_tab"
                     attrs="{'invisible': [('report_type', '!=', 'py3o')]}">
 
-                        <group>
+                    <group name="py3o_params">
                         <field name="py3o_filetype" />
                         <field name="py3o_is_local_fusion"/>
                         <field name="py3o_server_id" />
@@ -28,5 +27,5 @@
 
         </field>
     </record>
-    </data>
+
 </odoo>

report_py3o/views/menu.xml

@@ -1,8 +1,8 @@
 <?xml version="1.0" encoding="utf-8"?>
 <odoo>
-    <data>
+
     <menuitem id="py3o_config_menu"
         name="Py3o"
         parent="report.reporting_menuitem" />
-    </data>
+
 </odoo>

report_py3o/views/py3o_server.xml

@@ -1,12 +1,12 @@
 <?xml version="1.0" encoding="utf-8"?>
 <odoo>
-    <data>
+
     <record id="py3o_server_configuration_form_view" model="ir.ui.view">
         <field name="name">py3o.server.configuration.form.view</field>
         <field name="model">py3o.server</field>
         <field name="arch" type="xml">
             <form string="Py3o Server Configuration">
-                    <group>
+                <group name="main">
                     <field name="url" widget="url"/>
                     <field name="is_active" />
                 </group>
@@ -34,5 +34,5 @@
     <menuitem id="py3o_server_configuration_menu"
         parent="py3o_config_menu"
         action="py3o_server_configuration_action" />
-    </data>
+
 </odoo>

report_py3o/views/py3o_template.xml

@@ -1,11 +1,11 @@
 <?xml version="1.0" encoding="utf-8"?>
 <odoo>
-    <data>
+
     <record id="py3o_template_configuration_search_view" model="ir.ui.view">
         <field name="name">py3o.template.configuration.search.view</field>
         <field name="model">py3o.template</field>
         <field name="arch" type="xml">
-                <search string="Py3o Templates Configuration">
+            <search string="Py3o Templates">
                 <field name="name" />
                 <field name="filetype" />
                 <group string="Group By" name="groupby">
@@ -20,8 +20,8 @@
         <field name="name">py3o.template.configuration.form.view</field>
         <field name="model">py3o.template</field>
         <field name="arch" type="xml">
-                <form string="Py3o Templates Configuration">
-                    <group>
+            <form string="Py3o Templates">
+                <group name="main">
                     <field name="name" />
                     <field name="filetype" />
                     <field name="py3o_template_data" />
@@ -34,7 +34,7 @@
         <field name="name">py3o.template.configuration.tree.view</field>
         <field name="model">py3o.template</field>
         <field name="arch" type="xml">
-                <tree string="Py3o Templates Configuration">
+            <tree string="Py3o Templates">
                 <field name="name" />
                 <field name="filetype" />
             </tree>
@@ -50,5 +50,5 @@
     <menuitem id="py3o_template_configuration_menu"
         parent="py3o_config_menu"
         action="py3o_template_configuration_action" />
-    </data>
+
 </odoo>