Merge pull request #9 from akretion/py3o-usability

Small usability improvements
8 years ago · c41dd469b9
13 changed files with 240 additions and 159 deletions
--- a/report_py3o/NEWS
+++ b/report_py3o/NEWS
@ -1,4 +0,0 @@
-report_py3o 1.3
-
-Production release
-
--- a/report_py3o/README.md
+++ b/report_py3o/README.md
@ -1,18 +0,0 @@
-### README ###
-
-
-### LibreOffice Report Engine for Odoo/OpenERP ###
-
-
-This module allows you to convert any type of report into LibreOffice/OpenOffice.
-The benefits is that you have control over your documents and you can 
-modify them as needed.
-
-
-Requirements
-============
-
-The py3o.template package is required; install it with:
-    easy_install -UZ py3o.template
-
-
--- a/report_py3o/README.rst
+++ b/report_py3o/README.rst
@ -0,0 +1,94 @@
+.. image:: https://img.shields.io/badge/licence-AGPL--3-blue.svg
+   :target: http://www.gnu.org/licenses/agpl-3.0-standalone.html
+   :alt: License: AGPL-3
+
+===========
+Report Py3o
+===========
+
+The py3o reporting engine is a reporting engine for Odoo based on `Libreoffice <http://www.libreoffice.org/>`_:
+
+* the report is created with Libreoffice (ODT or ODS),
+* the report is stored on the server in OpenDocument format (.odt or .ods file)
+* the report is sent to the user in OpenDocument format or in any output format supported by Libreoffice (PDF, HTML, DOC, DOCX, Docbook, XLS, etc.)
+
+The key advantages of a Libreoffice-based reporting engine are:
+
+* no need to be a developper to create or modify a report: the report is created and modified with Libreoffice. So this reporting engine has a fully WYSIWYG report developpment tool!
+
+* For a PDF report in A4/Letter format, it's easier to develop it with a tool such as Libreoffice that is designed to create A4/Letter documents than to develop it in HTML/CSS.
+
+* If you want your users to be able to modify the document after its generation by Odoo, just configure the document with ODT output (or DOC or DOCX) and the user will be able to modify the document with Libreoffice (or Word) after its generation by Odoo.
+
+* Easy development of spreadsheet reports in ODS format (XLS output possible).
+
+This reporting engine is an alternative to `Aeroo <https://github.com/aeroo/aeroo_reports>`_: these 2 reporting engines have similar features but their codes are completely different.
+
+Installation
+============
+
+You must install 2 additionnal python libs:
+
+.. code::
+
+  pip install py3o.template
+  pip install py3o.formats
+
+If you want to convert the ODT or ODS report in another format, you need several additionnal components and Python libs:
+
+* `Py3o Fusion server <https://bitbucket.org/faide/py3o.fusion>`_
+* `Py3o render server <https://bitbucket.org/faide/py3o.renderserver>`_
+* Libreoffice started in the background in headless mode.
+
+TODO : continue
+
+Configuration
+=============
+
+If you want to convert the report in another format, go to the menu *Configuration > Technical > Reports > Py3o > Py3o Servers* and create a new Py3o server with its URL (for example:  http://localhost:8765/form).
+
+TODO: continue
+
+Usage
+=====
+
+.. image:: https://odoo-community.org/website/image/ir.attachment/5784_f2813bd/datas
+   :alt: Try me on Runbot
+   :target: https://runbot.odoo-community.org/runbot/143/9.0
+
+Known issues / Roadmap
+======================
+
+* generate barcode ?
+
+Bug Tracker
+===========
+
+Bugs are tracked on `GitHub Issues
+<https://github.com/OCA/reporting-engine/issues>`_. In case of trouble, please
+check there if your issue has already been reported. If you spotted it first,
+help us smashing it by providing a detailed and welcomed feedback.
+
+Credits
+=======
+
+Contributors
+------------
+
+* Florent Aide (`XCG Consulting <http://odoo.consulting/>`_)
+* Laurent Mignon (Acsone)
+
+Maintainer
+----------
+
+.. image:: https://odoo-community.org/logo.png
+   :alt: Odoo Community Association
+   :target: https://odoo-community.org
+
+This module is maintained by the OCA.
+
+OCA, or the Odoo Community Association, is a nonprofit organization whose
+mission is to support the collaborative development of Odoo features and
+promote its widespread use.
+
+To contribute to this module, please visit https://odoo-community.org.
--- a/report_py3o/openerp.py
+++ b/report_py3o/openerp.py
@ -2,22 +2,15 @@
 # Copyright 2013 XCG Consulting (http://odoo.consulting)
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 {
-    'name': 'LibreOffice Report Engine',
-    'description': '''
-Generation of LibreOffice/OpenOffice reports using LibreOffice/OpenOffice
-templates.
-
-The py3o.template package is required; install it with:
-    pip install py3o.template
-''',
+    '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',
    'category': 'Reporting',
-    'author': 'XCG Consulting',
+    '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']
--- a/report_py3o/models/ir_report.py
+++ b/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__)

-class ReportXml(models.Model):
+try:
+    from py3o.formats import Formats
+except ImportError:
+    logger.debug('Cannot import py3o.formats')
+
+
+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):
@ -55,14 +62,14 @@ class ReportXml(models.Model):
        'py3o.template',
        "Template")
    py3o_is_local_fusion = fields.Boolean(
-        "Local fusion",
+        "Local Fusion",
        help="Native formats will be processed without a server. "
             "You must use this mode if you call methods on your model into "
             "the template.",
        default=True)
    py3o_server_id = fields.Many2one(
        "py3o.server",
-        "Fusion server")
+        "Fusion Server")
    module = fields.Char(
        "Module",
        help="The implementer module that provides this report")
@ -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)
--- a/report_py3o/models/py3o_server.py
+++ b/report_py3o/models/py3o_server.py
@ -6,6 +6,10 @@ from openerp import fields, models

 class Py3oServer(models.Model):
    _name = 'py3o.server'
+    _rec_name = 'url'

-    url = fields.Char("URL", required=True)
+    url = fields.Char(
+        "Py3o Fusion Server URL", required=True,
+        help="If your Py3o Fusion server is on the same machine and runs "
+        "on the default port, the URL is http://localhost:8765/form")
    is_active = fields.Boolean("Active", default=True)
--- a/report_py3o/models/py3o_template.py
+++ b/report_py3o/models/py3o_template.py
@ -8,7 +8,7 @@ class Py3oTemplate(models.Model):
    _name = 'py3o.template'

    name = fields.Char(required=True)
-    py3o_template_data = fields.Binary("LibreOffice template")
+    py3o_template_data = fields.Binary("LibreOffice Template")
    filetype = fields.Selection(
        selection=[
            ('odt', "ODF Text Document"),
--- a/report_py3o/py3o_parser.py
+++ b/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 = {}
--- a/report_py3o/views/ir_report.xml
+++ b/report_py3o/views/ir_report.xml
@ -1,31 +1,31 @@
+<?xml version="1.0" encoding="utf-8"?>
 <odoo>
-    <data>

-        <!-- Inherit from base.act_report_xml_view to add py3o-related settings. -->
+    <!-- Inherit from base.act_report_xml_view to add py3o-related settings. -->

-        <record id="py3o_report_view" model="ir.ui.view">
-            <field name="name">py3o_report_view</field>
-            <field name="model">ir.actions.report.xml</field>
-            <field name="inherit_id" ref="base.act_report_xml_view" />
-            <field name="arch" type="xml">
+    <record id="py3o_report_view" model="ir.ui.view">
+        <field name="name">py3o_report_view</field>
+        <field name="model">ir.actions.report.xml</field>
+        <field name="inherit_id" ref="base.act_report_xml_view" />
+        <field name="arch" type="xml">

-                <xpath expr="//page[@name='security']" position="before">
-                    <page string="LibreOffice template"
-                        attrs="{'invisible': [('report_type', '!=', 'py3o')]}">
+            <xpath expr="//page[@name='security']" position="before">
+                <page string="LibreOffice Template" name="py3o_tab"
+                    attrs="{'invisible': [('report_type', '!=', 'py3o')]}">

-                        <group>
-                            <field name="py3o_filetype" />
-                            <field name="py3o_is_local_fusion"/>
-                            <field name="py3o_server_id" />
-                            <field name="py3o_template_id" />
-                            <field name="module" />
-                            <field name="py3o_template_fallback" />
-                        </group>
+                    <group name="py3o_params">
+                        <field name="py3o_filetype" />
+                        <field name="py3o_is_local_fusion"/>
+                        <field name="py3o_server_id" />
+                        <field name="py3o_template_id" />
+                        <field name="module" />
+                        <field name="py3o_template_fallback" />
+                    </group>

-                    </page>
-                </xpath>
+                </page>
+            </xpath>
+
+        </field>
+    </record>

-            </field>
-        </record>
-    </data>
 </odoo>
--- a/report_py3o/views/menu.xml
+++ b/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>
+
+    <menuitem id="py3o_config_menu"
+        name="Py3o"
+        parent="report.reporting_menuitem" />
+
 </odoo>
--- a/report_py3o/views/py3o_server.xml
+++ b/report_py3o/views/py3o_server.xml
@ -1,42 +1,38 @@
 <?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" version="7.0">
-                    <sheet>
-                        <group>
-                            <field name="url" />
-                            <field name="is_active" />
-                        </group>
-                    </sheet>
-                </form>
-            </field>
-        </record>

-        <record id="py3o_server_configuration_tree_view" model="ir.ui.view">
-            <field name="name">py3o.server.configuration.tree.view</field>
-            <field name="model">py3o.server</field>
-            <field name="arch" type="xml">
-                <tree string="Py3o Server Configuration" version="7.0">
-                    <field name="url" />
+    <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 name="main">
+                    <field name="url" widget="url"/>
                    <field name="is_active" />
-                </tree>
-            </field>
-        </record>
+                </group>
+            </form>
+        </field>
+    </record>

-        <record id="py3o_server_configuration_action" model="ir.actions.act_window">
-            <field name="name">py3o.server.configuration.action</field>
-            <field name="res_model">py3o.server</field>
-            <field name="view_type">form</field>
-            <field name="view_mode">tree,form</field>
-        </record>
+    <record id="py3o_server_configuration_tree_view" model="ir.ui.view">
+        <field name="name">py3o.server.configuration.tree.view</field>
+        <field name="model">py3o.server</field>
+        <field name="arch" type="xml">
+            <tree string="Py3o Servers Configuration">
+                <field name="url" />
+                <field name="is_active" />
+            </tree>
+        </field>
+    </record>
+
+    <record id="py3o_server_configuration_action" model="ir.actions.act_window">
+        <field name="name">Py3o Servers</field>
+        <field name="res_model">py3o.server</field>
+        <field name="view_mode">tree,form</field>
+    </record>
+
+    <menuitem id="py3o_server_configuration_menu"
+        parent="py3o_config_menu"
+        action="py3o_server_configuration_action" />

-        <menuitem id="py3o_server_configuration_menu"
-            name="Py3o Server Configuration"
-            parent="py3o_config_menu"
-            action="py3o_server_configuration_action" />
-    </data>
 </odoo>
--- a/report_py3o/views/py3o_template.xml
+++ b/report_py3o/views/py3o_template.xml
@ -1,54 +1,54 @@
 <?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" version="7.0">
-                    <field name="name" />
-                    <field name="filetype" />
-                </search>
-            </field>
-        </record>

-        <record id="py3o_template_configuration_form_view" model="ir.ui.view">
-            <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" version="7.0">
-                    <sheet>
-                        <group>
-                            <field name="name" />
-                            <field name="filetype" />
-                            <field name="py3o_template_data" />
-                        </group>
-                    </sheet>
-                </form>
-            </field>
-        </record>
+    <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">
+                <field name="name" />
+                <field name="filetype" />
+                <group string="Group By" name="groupby">
+                    <filter name="filetype_groupby" string="File Type"
+                        context="{'group_by': 'filetype'}"/>
+                </group>
+            </search>
+        </field>
+    </record>

-        <record id="py3o_template_configuration_tree_view" model="ir.ui.view">
-            <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" version="7.0">
+    <record id="py3o_template_configuration_form_view" model="ir.ui.view">
+        <field name="name">py3o.template.configuration.form.view</field>
+        <field name="model">py3o.template</field>
+        <field name="arch" type="xml">
+            <form string="Py3o Templates">
+                <group name="main">
                    <field name="name" />
                    <field name="filetype" />
-                </tree>
-            </field>
-        </record>
+                    <field name="py3o_template_data" />
+                </group>
+            </form>
+        </field>
+    </record>
+
+    <record id="py3o_template_configuration_tree_view" model="ir.ui.view">
+        <field name="name">py3o.template.configuration.tree.view</field>
+        <field name="model">py3o.template</field>
+        <field name="arch" type="xml">
+            <tree string="Py3o Templates">
+                <field name="name" />
+                <field name="filetype" />
+            </tree>
+        </field>
+    </record>
+
+    <record id="py3o_template_configuration_action" model="ir.actions.act_window">
+        <field name="name">Py3o Templates</field>
+        <field name="res_model">py3o.template</field>
+        <field name="view_mode">tree,form</field>
+    </record>

-        <record id="py3o_template_configuration_action" model="ir.actions.act_window">
-            <field name="name">Py3o Templates Configuration</field>
-            <field name="res_model">py3o.template</field>
-            <field name="view_type">form</field>
-            <field name="view_mode">tree,form</field>
-        </record>
+    <menuitem id="py3o_template_configuration_menu"
+        parent="py3o_config_menu"
+        action="py3o_template_configuration_action" />

-        <menuitem id="py3o_template_configuration_menu"
-            name="Py3o Templates"
-            parent="py3o_config_menu"
-            action="py3o_template_configuration_action" />
-    </data>
 </odoo>