[FIX] regression on shortcut button on partner form view after PR #41

add many2many_tags_email.js file for mail_usability that replaces addons-mail-fix.diff patch

account_analytic_analysis_usability/__init__.py

@@ -0,0 +1,3 @@
+# -*- coding: utf-8 -*-
+
+from . import account_analytic_analysis

account_analytic_analysis_usability/__openerp__.py

@@ -0,0 +1,29 @@
+# -*- coding: utf-8 -*-
+# © 2017 Akretion (Alexis de Lattre <alexis.delattre@akretion.com>)
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+
+{
+    'name': 'Account Analytic Analysis Usability',
+    'version': '8.0.1.0.0',
+    'category': 'Accounting & Finance',
+    'license': 'AGPL-3',
+    'summary': 'Usability improvements on Account Analytic Analysis',
+    'description': """
+Account Analytic Analysis Usability
+===================================
+
+Usability improvements include:
+
+* add next invoice date in tree view of contrats (and add a group by)
+
+This module has been written by Alexis de Lattre from Akretion
+<alexis.delattre@akretion.com>.
+    """,
+    'author': 'Akretion',
+    'website': 'http://www.akretion.com',
+    'depends': ['account_analytic_analysis'],
+    'data': [
+        'analytic_view.xml',
+    ],
+    'installable': True,
+}

account_analytic_analysis_usability/account_analytic_analysis.py

@@ -0,0 +1,14 @@
+# -*- coding: utf-8 -*-
+# © 2017 Akretion (Alexis de Lattre <alexis.delattre@akretion.com>)
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+
+from openerp import models, fields
+
+
+class AccountAnalyticAccount(models.Model):
+    _inherit = "account.analytic.account"
+
+    recurring_next_date = fields.Date(track_visibility='onchange')
+    recurring_rule_type = fields.Selection(track_visibility='onchange')
+    recurring_interval = fields.Integer(track_visibility='onchange')
+    recurring_invoices = fields.Boolean(track_visibility='onchange')

account_analytic_analysis_usability/analytic_view.xml

@@ -0,0 +1,40 @@
+<?xml version="1.0" encoding="utf-8"?>
+<!--
+  © 2017 Akretion (Alexis de Lattre <alexis.delattre@akretion.com>)
+  License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+-->
+
+<openerp>
+<data>
+
+<record id="view_account_analytic_account_tree_c2c_3" model="ir.ui.view">
+    <field name="name">usability.analytic_analysis.account.analytic.account.tree.contract</field>
+    <field name="model">account.analytic.account</field>
+    <field name="inherit_id" ref="account_analytic_analysis.view_account_analytic_account_tree_c2c_3"/>
+    <field name="groups_id" eval="[(4, ref('account.group_account_invoice'))]"/>
+    <field name="arch" type="xml">
+        <field name="date" position="after">
+            <field name="recurring_rule_type"/>
+            <field name="recurring_next_date"/>
+        </field>
+    </field>
+</record>
+
+<record id="view_account_analytic_account_overdue_search" model="ir.ui.view">
+    <field name="name">usability.analytic_analysis.account.analytic.account.search</field>
+    <field name="model">account.analytic.account</field>
+    <field name="inherit_id" ref="account_analytic_analysis.view_account_analytic_account_overdue_search"/>
+    <field name="arch" type="xml">
+        <filter context="{'group_by' : 'pricelist_id'}" position="after">
+            <filter name="recurring_rule_type_groupby" string="Invoicing Frequency" context="{'group_by': 'recurring_rule_type'}"/>
+            <filter name="recurring_next_date_groupby" string="Next Invoice Date" context="{'group_by': 'recurring_next_date'}"/>
+        </filter>
+        <filter name="cancelled" position="after">
+            <separator/>
+            <filter name="recurring_invoices" string="Recurring Invoicing" domain="[('recurring_invoices', '=', True)]"/>
+        </filter>
+    </field>
+</record>
+
+</data>
+</openerp>

account_cutoff_accrual_picking_ods/__init__.py

@@ -0,0 +1 @@
+# -*- coding: utf-8 -*-

account_cutoff_accrual_picking_ods/__openerp__.py

@@ -0,0 +1,24 @@
+# -*- coding: utf-8 -*-
+# © 2016 Akretion (Alexis de Lattre <alexis.delattre@akretion.com>)
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+
+{
+    'name': 'Account Cutoff Accrual Picking ODS',
+    'version': '8.0.0.1.0',
+    'category': 'Tools',
+    'license': 'AGPL-3',
+    'summary': 'Adds an Aeroo ODS report on cutoff accrual',
+    'description': """
+Account Cutoff Accrual Picking ODS
+==================================
+
+This module will add an Aeroo ODS report on Accrued Revenue and Accrued Expense.
+
+This module has been written by Alexis de Lattre from Akretion <alexis.delattre@akretion.com>.
+    """,
+    'author': "Akretion",
+    'website': 'http://www.akretion.com',
+    'depends': ['account_cutoff_accrual_picking', 'report_aeroo'],
+    'data': ['report.xml'],
+    'installable': True,
+}

account_cutoff_accrual_picking_ods/report.xml

@@ -0,0 +1,27 @@
+<?xml version="1.0" encoding="utf-8"?>
+<openerp>
+<data>
+
+
+<record id="account_cutoff_accrual_picking_ods" model="ir.actions.report.xml">
+    <field name="name">Cutoff Accrual ODS</field>
+    <field name="model">account.cutoff</field>
+    <field name="report_name">account.cutoff.accrual.picking.ods</field>
+    <field name="report_type">aeroo</field>
+    <field name="in_format">oo-ods</field>
+    <field name="report_rml">account_cutoff_accrual_picking_ods/cutoff_accrual.ods</field>
+    <field name="parser_state">default</field>
+    <field name="tml_source">file</field>
+    <field name="out_format" ref="report_aeroo.report_mimetypes_ods_ods"/>
+</record>
+
+<record id="account_cutoff_accrual_picking_ods_button" model="ir.values">
+    <field name="name">Cutoff Accrual ODS</field>
+    <field name="model">account.cutoff</field>
+    <field name="key2">client_print_multi</field>
+    <field name="value" eval="'ir.actions.report.xml,%d'%account_cutoff_accrual_picking_ods"/>
+</record>
+
+
+</data>
+</openerp>

account_fiscal_position_payable_receivable/__init__.py

@@ -0,0 +1,3 @@
+# -*- coding: utf-8 -*-
+
+from . import account_fiscal_position

account_fiscal_position_payable_receivable/__openerp__.py

@@ -0,0 +1,26 @@
+# -*- coding: utf-8 -*-
+# © 2016 Akretion (Alexis de Lattre <alexis.delattre@akretion.com>)
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+
+{
+    'name': 'Account Fiscal Position Payable Receivable',
+    'version': '8.0.1.0.0',
+    'category': 'Accounting & Finance',
+    'license': 'AGPL-3',
+    'summary': 'Configure payable/receivable accounts on fiscal positions',
+    'description': """
+Account Fiscal Position Payable Receivable
+==========================================
+
+This module allows to configure a special *Partner Receivable Account* and a special *Partner Payable Account* on fiscal positions. This is used in the onchange of the fiscal position of partners.
+
+This module has been written by Alexis de Lattre from Akretion <alexis.delattre@akretion.com>.
+    """,
+    'author': "Akretion",
+    'website': 'http://www.akretion.com',
+    'depends': ['account'],
+    'data': [
+        'account_fiscal_position_view.xml',
+    ],
+    'installable': True,
+}

account_fiscal_position_payable_receivable/account_fiscal_position.py

@@ -0,0 +1,35 @@
+# -*- coding: utf-8 -*-
+# © 2016 Akretion (Alexis de Lattre <alexis.delattre@akretion.com>)
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+
+from openerp import models, fields, api
+
+
+class AccountFiscalPosition(models.Model):
+    _inherit = 'account.fiscal.position'
+
+    receivable_account_id = fields.Many2one(
+        'account.account', string='Partner Receivable Account',
+        company_dependent=True, domain=[('type', '=', 'receivable')])
+    payable_account_id = fields.Many2one(
+        'account.account', string='Partner Payable Account',
+        company_dependent=True, domain=[('type', '=', 'payable')])
+
+
+class ResPartner(models.Model):
+    _inherit = 'res.partner'
+
+    @api.onchange('property_account_position')
+    def fiscal_position_receivable_payable_change(self):
+        fp = self.property_account_position
+        ipo = self.env['ir.property']
+        if fp.receivable_account_id:
+            self.property_account_receivable = fp.receivable_account_id
+        else:
+            self.property_account_receivable = ipo.get(
+                'property_account_receivable', 'res.partner')
+        if fp.payable_account_id:
+            self.property_account_payable = fp.payable_account_id
+        else:
+            self.property_account_payable = ipo.get(
+                'property_account_payable', 'res.partner')

account_fiscal_position_payable_receivable/account_fiscal_position_view.xml

@@ -0,0 +1,28 @@
+<?xml version="1.0" encoding="utf-8"?>
+<!--
+  © 2016 Akretion (Alexis de Lattre <alexis.delattre@akretion.com>)
+  License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+-->
+
+<openerp>
+<data>
+
+
+<record id="view_account_position_form" model="ir.ui.view">
+    <field name="name">receivable_payable.fiscal_position_form</field>
+    <field name="model">account.fiscal.position</field>
+    <field name="inherit_id" ref="account.view_account_position_form" />
+    <field name="arch" type="xml">
+        <xpath expr="//field[@name='country_group_id']/.." position="after">
+            <group string="Payable and Receivable Accounts"
+                    name="payable_receivable_account" col="4">
+                <field name="receivable_account_id"/>
+                <field name="payable_account_id"/>
+            </group>
+        </xpath>
+    </field>
+</record>
+
+
+</data>
+</openerp>

account_invoice_update_wizard/__init__.py

@@ -0,0 +1,3 @@
+# -*- coding: utf-8 -*-
+
+from . import wizard

account_invoice_update_wizard/__openerp__.py

@@ -0,0 +1,34 @@
+# -*- coding: utf-8 -*-
+# © 2017 Akretion (Alexis de Lattre <alexis.delattre@akretion.com>)
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+
+{
+    'name': 'Account Invoice Update Wizard',
+    'version': '8.0.1.0.0',
+    'category': 'Accounting & Finance',
+    'license': 'AGPL-3',
+    'summary': 'Wizard to update non-legal fields of an open/paid invoice',
+    'description': """
+Account Invoice Update Wizard
+=============================
+
+This module adds a button *Update Invoice* on Customer and Supplier invoices in Open or Paid state. This button starts a wizard which allows the user to update non-legal fields of the invoice:
+
+* Source Document
+* Reference/Description
+* Payment terms (update allowed only to a payment term with same number of terms of the same amount and on invoices without any payment)
+* Bank Account
+* Salesman
+* Notes
+* Description of invoice lines
+
+    """,
+    'author': 'Akretion',
+    'website': 'http://www.akretion.com',
+    'depends': ['account'],
+    'data': [
+        'wizard/account_invoice_update_view.xml',
+        'views/account_invoice.xml',
+        ],
+    'installable': True,
+}

account_invoice_update_wizard/views/account_invoice.xml

@@ -0,0 +1,31 @@
+<?xml version="1.0" encoding="utf-8"?>
+<!--
+  © 2017 Akretion (Alexis de Lattre <alexis.delattre@akretion.com>)
+  License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+-->
+
+<openerp>
+<data>
+
+<record id="invoice_supplier_form" model="ir.ui.view">
+    <field name="model">account.invoice</field>
+    <field name="inherit_id" ref="account.invoice_supplier_form"/>
+    <field name="arch" type="xml">
+        <button name="invoice_cancel" position="before">
+            <button name="%(account_invoice_update_action)d" type="action" string="Update Invoice" states="open,paid" groups="account.group_account_invoice"/>
+        </button>
+    </field>
+</record>
+
+<record id="invoice_form" model="ir.ui.view">
+    <field name="model">account.invoice</field>
+    <field name="inherit_id" ref="account.invoice_form"/>
+    <field name="arch" type="xml">
+        <button name="invoice_cancel" position="before">
+            <button name="%(account_invoice_update_action)d" type="action" string="Update Invoice" states="open,paid" groups="account.group_account_invoice"/>
+        </button>
+    </field>
+</record>
+
+</data>
+</openerp>

account_invoice_update_wizard/wizard/__init__.py

@@ -0,0 +1,3 @@
+# -*- coding: utf-8 -*-
+
+from . import account_invoice_update

account_invoice_update_wizard/wizard/account_invoice_update.py

@@ -0,0 +1,206 @@
+# -*- coding: utf-8 -*-
+# © 2017 Akretion (Alexis de Lattre <alexis.delattre@akretion.com>)
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+
+from openerp import models, fields, api, _
+from openerp.exceptions import Warning as UserError
+import openerp.addons.decimal_precision as dp
+
+
+class AccountInvoiceUpdate(models.TransientModel):
+    _name = 'account.invoice.update'
+    _description = 'Wizard to update non-legal fields of invoice'
+
+    invoice_id = fields.Many2one(
+        'account.invoice', string='Invoice', required=True,
+        readonly=True)
+    type = fields.Selection(related='invoice_id.type', readonly=True)
+    company_id = fields.Many2one(
+        related='invoice_id.company_id', readonly=True)
+    commercial_partner_id = fields.Many2one(
+        related='invoice_id.commercial_partner_id', readonly=True)
+    user_id = fields.Many2one('res.users', string='Salesperson')
+    # I use the same field name as the original invoice field name
+    # even if it the name is "bad"
+    # Updating payment_term will not work if you use
+    # the OCA module account_constraints (you will just get an error)
+    payment_term = fields.Many2one(
+        'account.payment.term', string='Payment Terms')
+    reference = fields.Char(string='Invoice Reference')
+    name = fields.Char(string='Reference/Description')
+    origin = fields.Char(string='Source Document')
+    comment = fields.Text('Additional Information')
+    partner_bank_id = fields.Many2one(
+        'res.partner.bank', string='Bank Account')
+    line_ids = fields.One2many(
+        'account.invoice.line.update', 'parent_id', string='Invoice Lines')
+
+    @api.model
+    def _simple_fields2update(self):
+        '''List boolean, date, datetime, char, text fields'''
+        return ['reference', 'name', 'origin', 'comment']
+
+    @api.model
+    def _m2o_fields2update(self):
+        return ['payment_term', 'user_id', 'partner_bank_id']
+
+    @api.model
+    def _prepare_default_get(self, invoice):
+        res = {'invoice_id': invoice.id, 'line_ids': []}
+        for sfield in self._simple_fields2update():
+            res[sfield] = invoice[sfield]
+        for m2ofield in self._m2o_fields2update():
+            res[m2ofield] = invoice[m2ofield].id or False
+        for line in invoice.invoice_line:
+            res['line_ids'].append({
+                'invoice_line_id': line.id,
+                'name': line.name,
+                'quantity': line.quantity,
+                'price_subtotal': line.price_subtotal,
+                })
+        return res
+
+    @api.model
+    def default_get(self, fields_list):
+        res = super(AccountInvoiceUpdate, self).default_get(fields_list)
+        assert self._context.get('active_model') == 'account.invoice',\
+            'active_model should be account.invoice'
+        inv = self.env['account.invoice'].browse(self._context['active_id'])
+        res = self._prepare_default_get(inv)
+        return res
+
+    @api.onchange('type')
+    def type_on_change(self):
+        res = {'domain': {}}
+        if self.type in ('out_invoice', 'out_refund'):
+            res['domain']['partner_bank_id'] =\
+                "[('partner_id.ref_companies', 'in', [company_id])]"
+        else:
+            res['domain']['partner_bank_id'] =\
+                "[('partner_id', '=', commercial_partner_id)]"
+        return res
+
+    @api.multi
+    def _prepare_invoice(self):
+        vals = {}
+        inv = self.invoice_id
+        for sfield in self._simple_fields2update():
+            if self[sfield] != inv[sfield]:
+                vals[sfield] = self[sfield]
+        for m2ofield in self._m2o_fields2update():
+            if self[m2ofield] != inv[m2ofield]:
+                vals[m2ofield] = self[m2ofield].id or False
+        if 'payment_term' in vals:
+            pterm_list = self.payment_term.compute(
+                value=1, date_ref=inv.date_invoice)[0]
+            if pterm_list:
+                vals['date_due'] = max(line[0] for line in pterm_list)
+        return vals
+
+    @api.model
+    def _prepare_invoice_line(self, line):
+        vals = {}
+        if line.name != line.invoice_line_id.name:
+            vals['name'] = line.name
+        return vals
+
+    @api.multi
+    def _prepare_move(self):
+        mvals = {}
+        inv = self.invoice_id
+        ini_ref = inv.move_id.ref
+        ref = inv.reference or inv.name
+        if ini_ref != ref:
+            mvals['ref'] = ref
+        return mvals
+
+    @api.multi
+    def _update_payment_term_move(self):
+        self.ensure_one()
+        inv = self.invoice_id
+        if (
+                self.payment_term and
+                self.payment_term != inv.payment_term and
+                inv.move_id and
+                inv.move_id.period_id.state == 'draft'):
+            # I don't update pay term when the invoice is partially (or fully)
+            # paid because if you have a payment term with several lines
+            # of the same amount, you would also have to take into account
+            # the reconcile marks to put the new maturity date on the right
+            # lines
+            if inv.payment_ids:
+                raise UserError(_(
+                    "This wizard doesn't support the update of payment "
+                    "terms on an invoice which is partially or fully "
+                    "paid."))
+            prec = self.env['decimal.precision'].precision_get('Account')
+            term_res = self.payment_term.compute(
+                inv.amount_total, inv.date_invoice)[0]
+            new_pterm = {}  # key = int(amount * 100), value = [date1, date2]
+            for entry in term_res:
+                amount = int(entry[1] * 10 * prec)
+                if amount in new_pterm:
+                    new_pterm[amount].append(entry[0])
+                else:
+                    new_pterm[amount] = [entry[0]]
+            mlines = {}  # key = int(amount * 100), value : [line1, line2]
+            for line in inv.move_id.line_id:
+                if line.account_id == inv.account_id:
+                    amount = int(abs(line.credit - line.debit) * 10 * prec)
+                    if amount in mlines:
+                        mlines[amount].append(line)
+                    else:
+                        mlines[amount] = [line]
+            for iamount, lines in mlines.iteritems():
+                if len(lines) != len(new_pterm.get(iamount, [])):
+                    raise UserError(_(
+                        "The original payment term '%s' doesn't have the "
+                        "same terms (number of terms and/or amount) as the "
+                        "new payment term '%s'. You can only switch to a "
+                        "payment term that has the same number of terms "
+                        "with the same amount.") % (
+                        inv.payment_term.name, self.payment_term.name))
+                for line in lines:
+                    line.date_maturity = new_pterm[iamount].pop()
+
+    @api.multi
+    def run(self):
+        self.ensure_one()
+        inv = self.invoice_id
+        updated = False
+        # re-write date_maturity on move line
+        self._update_payment_term_move()
+        ivals = self._prepare_invoice()
+        if ivals:
+            updated = True
+            inv.write(ivals)
+        for line in self.line_ids:
+            ilvals = self._prepare_invoice_line(line)
+            if ilvals:
+                updated = True
+                line.invoice_line_id.write(ilvals)
+        if inv.move_id and inv.move_id.period_id.state == 'draft':
+            mvals = self._prepare_move()
+            if mvals:
+                inv.move_id.write(mvals)
+        if updated:
+            inv.message_post(_(
+                'Non-legal fields of invoice updated via the Invoice Update '
+                'wizard.'))
+        return True
+
+
+class AccountInvoiceLineUpdate(models.TransientModel):
+    _name = 'account.invoice.line.update'
+    _description = 'Update non-legal fields of invoice lines'
+
+    parent_id = fields.Many2one(
+        'account.invoice.update', string='Wizard', ondelete='cascade')
+    invoice_line_id = fields.Many2one(
+        'account.invoice.line', string='Invoice Line', readonly=True)
+    name = fields.Text(string='Description', required=True)
+    quantity = fields.Float(
+        string='Quantity', digits=dp.get_precision('Product Unit of Measure'),
+        readonly=True)
+    price_subtotal = fields.Float(
+        string='Amount', readonly=True, digits=dp.get_precision('Account'))

account_invoice_update_wizard/wizard/account_invoice_update_view.xml

@@ -0,0 +1,54 @@
+<?xml version="1.0" encoding="utf-8"?>
+<!--
+  © 2017 Akretion (Alexis de Lattre <alexis.delattre@akretion.com>)
+  License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+-->
+
+<openerp>
+<data>
+
+<record id="account_invoice_update_form" model="ir.ui.view">
+    <field name="model">account.invoice.update</field>
+    <field name="arch" type="xml">
+        <form string="Update Invoice Wizard">
+            <group name="main">
+                <field name="invoice_id" invisible="1"/>
+                <field name="type" invisible="1"/>
+                <field name="company_id" invisible="1"/>
+                <field name="commercial_partner_id" invisible="1"/>
+                <field name="reference" attrs="{'invisible': [('type', 'not in', ('in_invoice', 'in_refund'))]}"/>
+                <field name="origin"/>
+                <field name="name"/>
+                <field name="payment_term" widget="selection"/>
+                <field name="partner_bank_id"/>
+                <field name="user_id"/>
+                <field name="comment"/>
+            </group>
+            <group name="lines">
+                <field name="line_ids" nolabel="1">
+                    <tree editable="bottom">
+                        <field name="invoice_line_id" invisible="1"/>
+                        <field name="name"/>
+                        <field name="quantity"/>
+                        <field name="price_subtotal"/>
+                    </tree>
+                </field>
+            </group>
+            <footer>
+                <button name="run" type="object" class="oe_highlight" string="Update"/>
+                or 
+                <button special="cancel" string="Cancel" class="oe_link"/>
+            </footer>
+        </form>
+    </field>
+</record>
+
+<record id="account_invoice_update_action" model="ir.actions.act_window">
+    <field name="name">Invoice Update Wizard</field>
+    <field name="res_model">account.invoice.update</field>
+    <field name="view_mode">form</field>
+    <field name="target">new</field>
+</record>
+
+</data>
+</openerp>

account_move_line_filter_wizard/wizard/account_move_line_filter.py

@@ -53,7 +53,10 @@ class AccountMoveLineFilterWizard(models.TransientModel):
         self.ensure_one()
         action = self.env['ir.actions.act_window'].for_xml_id(
             'account', 'action_account_moves_all_a')
-        action['context'] = {'search_default_account_id': [self.account_id.id]}
+        action['context'] = {
+            'search_default_account_id': [self.account_id.id],
+            'journal_show_code_only': True,
+            }
         if self.partner_id:
             action['context']['search_default_partner_id'] =\
                 [self.partner_id.id]

account_move_line_start_end_dates_xls/account_move_line.py

@@ -21,9 +21,16 @@
 ##############################################################################
 
 import xlwt
+import logging
 from openerp import models, api
-from openerp.addons.report_xls.utils import _render
-from openerp.addons.report_xls.report_xls import report_xls
+
+_logger = logging.getLogger(__name__)
+
+try:
+    from openerp.addons.report_xls.utils import _render
+    from openerp.addons.report_xls.report_xls import report_xls
+except (ImportError, IOError) as err:
+    _logger.debug(err)
 
 
 class AccountMoveLine(models.Model):

account_usability/__init__.py

@@ -1,3 +1,4 @@
 # -*- encoding: utf-8 -*-
 
 from . import account
+from . import wizard

account_usability/__openerp__.py

@@ -23,7 +23,7 @@
 
 {
     'name': 'Account Usability',
-    'version': '0.1',
+    'version': '0.3',
     'category': 'Accounting & Finance',
     'license': 'AGPL-3',
     'summary': 'Small usability enhancements in account module',
@@ -49,6 +49,9 @@ This module has been written by Alexis de Lattre from Akretion <alexis.delattre@
     'website': 'http://www.akretion.com',
     'depends': ['account'],
     'conflicts': ['account_invoice_overdue_filter'],
-    'data': ['account_view.xml'],
+    'data': [
+        'account_view.xml',
+        'wizard/account_invoice_mark_sent_view.xml',
+        ],
     'installable': True,
 }

account_usability/account.py

@@ -23,10 +23,13 @@
 from openerp import models, fields, api, _
 from openerp.tools import float_compare
 from openerp.exceptions import Warning as UserError
+import logging
+logger = logging.getLogger(__name__)
 
 
 class AccountInvoice(models.Model):
     _inherit = 'account.invoice'
+    _order = "date_invoice desc, number desc, id desc"
 
     origin = fields.Char(track_visibility='onchange')
     supplier_invoice_number = fields.Char(track_visibility='onchange')
@@ -41,18 +44,86 @@ class AccountInvoice(models.Model):
     journal_id = fields.Many2one(track_visibility='onchange')
     partner_bank_id = fields.Many2one(track_visibility='onchange')
     fiscal_position = fields.Many2one(track_visibility='onchange')
+    # has_attachment is useful for those who use attachment to archive
+    # supplier invoices. It allows them to find supplier invoices
+    # that don't have any attachment
+    has_attachment = fields.Boolean(
+        compute='_compute_has_attachment',
+        search='_search_has_attachment', readonly=True)
 
     @api.multi
-    def action_move_create(self):
-        today = fields.Date.context_today(self)
-        for invoice in self:
-            if invoice.date_invoice and invoice.date_invoice > today:
-                raise UserError(_(
-                    "You cannot validate the invoice of '%s' "
-                    " with an invoice date (%s) in the future !") % (
-                        invoice.partner_id.name_get()[0][1],
-                        invoice.date_invoice))
-        return super(AccountInvoice, self).action_move_create()
+    def _compute_has_attachment(self):
+        iao = self.env['ir.attachment']
+        for inv in self:
+            if iao.search([
+                    ('res_model', '=', 'account.invoice'),
+                    ('res_id', '=', inv.id),
+                    ('type', '=', 'binary'),
+                    ('company_id', '=', inv.company_id.id)], limit=1):
+                inv.has_attachment = True
+            else:
+                inv.has_attachment = False
+
+    def _search_has_attachment(self, operator, value):
+        att_inv_ids = {}
+        if operator == '=':
+            search_res = self.env['ir.attachment'].search_read([
+                ('res_model', '=', 'account.invoice'),
+                ('type', '=', 'binary'),
+                ('res_id', '!=', False)], ['res_id'])
+            for att in search_res:
+                att_inv_ids[att['res_id']] = True
+        res = [('id', value and 'in' or 'not in', att_inv_ids.keys())]
+        return res
+
+    @api.multi
+    def onchange_payment_term_date_invoice(
+            self, payment_term_id, date_invoice):
+        res = super(AccountInvoice, self).onchange_payment_term_date_invoice(
+            payment_term_id, date_invoice)
+        if res and isinstance(res, dict) and 'value' in res:
+            res['value']['period_id'] = False
+        return res
+
+    # I really hate to see a "/" in the 'name' field of the account.move.line
+    # generated from customer invoices linked to the partners' account because:
+    # 1) the label of an account move line is an important field, we can't
+    #    write a rubbish '/' in it !
+    # 2) the 'name' field of the account.move.line is used in the overdue
+    #    letter and '/' is not meaningful for our customer !
+    @api.multi
+    def action_number(self):
+        res = super(AccountInvoice, self).action_number()
+        for inv in self:
+            self._cr.execute(
+                "UPDATE account_move_line SET name= "
+                "CASE WHEN name='/' THEN %s "
+                "ELSE %s||' - '||name END "
+                "WHERE move_id=%s", (inv.number, inv.number, inv.move_id.id))
+            self.invalidate_cache()
+        return res
+
+
+class AccountInvoiceLine(models.Model):
+    _inherit = 'account.invoice.line'
+
+    # In the 'account' module, we have related stored field for:
+    # company_id, partner_id
+    currency_id = fields.Many2one(
+        related='invoice_id.currency_id', readonly=True, store=True)
+    invoice_type = fields.Selection(
+        related='invoice_id.type', store=True, readonly=True)
+    date_invoice = fields.Date(
+        related='invoice_id.date_invoice', store=True, readonly=True)
+    commercial_partner_id = fields.Many2one(
+        related='invoice_id.partner_id.commercial_partner_id',
+        store=True, readonly=True)
+    state = fields.Selection(
+        related='invoice_id.state', store=True, readonly=True,
+        string='Invoice State')
+    invoice_number = fields.Char(
+        related='invoice_id.move_id.name', store=True, readonly=True,
+        string='Invoice Number')
 
 
 class AccountFiscalYear(models.Model):
@@ -91,10 +162,51 @@ class AccountAccount(models.Model):
         else:
             return super(AccountAccount, self).name_get()
 
+    def _check_account_type(self, cr, uid, ids, context=None):
+        '''Disable this native python constraint, because we want to be able
+        to configure payable/receivable accounts with an account type
+        with close_method == 'balance' in order to have opening entries with
+        fewer lines. It is not a problem because we always use
+        account_financial_report_webkit which doesn't take the detailed
+        opening entries into account.'''
+        return True
+
+    _constraints = [
+        # The method name must be exactly the same as the native
+        # method, in order to override it
+        (_check_account_type, 'No error message', ['user_type', 'type']),
+        ]
+
+    @api.model
+    def check_account_hierarchy(self):
+        '''designed to be called by a script'''
+        accounts = self.env['account.account'].search([])
+        parent_accounts = self
+        for account in accounts:
+            if account.parent_id and account.parent_id not in parent_accounts:
+                parent_accounts += account.parent_id
+        err_msg = []
+        view_user_type = self.env.ref('account.data_account_type_view')
+        for pacc in parent_accounts:
+            if pacc.type != 'view':
+                err_msg.append(_(
+                    'Parent account %s should have type=view '
+                    '(current type=%s)') % (pacc.code, pacc.type))
+            if pacc.user_type != view_user_type:
+                err_msg.append(_(
+                    'Parent account %s should have user_type=view (current '
+                    '(user_type=%s)') % (pacc.code, pacc.user_type.name))
+        if err_msg:
+            raise UserError('\n'.join(err_msg))
+
 
 class AccountAnalyticAccount(models.Model):
     _inherit = 'account.analytic.account'
 
+    invoice_line_ids = fields.One2many(
+        'account.invoice.line', 'account_analytic_id', 'Invoice Lines',
+        readonly=True)
+
     @api.multi
     def name_get(self):
         if self._context.get('analytic_account_show_code_only'):
@@ -111,11 +223,55 @@ class AccountAnalyticAccount(models.Model):
 class AccountMove(models.Model):
     _inherit = 'account.move'
 
+    # When ref is too long, the PDF general ledger report becomes
+    # unreadable
+    ref = fields.Char(size=32)
+    # If you want to migrate existing data :
+    # update account_move set ref=left(ref,32) where ref is not null;
+    # update account_move_line set ref=left(ref,32) where ref is not null;
+
+    # Allow to duplicate an account.move that belongs to a closed period
+    period_id = fields.Many2one(copy=False)
+    date = fields.Date(copy=False)
+
     @api.onchange('date')
     def date_onchange(self):
         if self.date:
             self.period_id = self.env['account.period'].find(self.date)
 
+    @api.model
+    def delete_move_no_lines(self):
+        '''designed to be called by a script'''
+        moves_no_lines = self.search([('line_id', '=', False)])
+        inv_moves_sr = self.env['account.invoice'].search_read(
+            [('move_id', '!=', False)], ['move_id'])
+        move2inv = {}  # key=move_id, value=invoice_id
+        invoice_move_no_line = {}  # key=ID, value=number
+        deleted_move_ids = []
+        for inv_move_sr in inv_moves_sr:
+            move2inv[inv_move_sr['move_id'][0]] = inv_move_sr['id']
+        for move in moves_no_lines:
+            for l in move.line_id:
+                raise UserError(_('Move %d has a line !') % move.id)
+            if move.id not in move2inv:
+                if move.state == 'posted':
+                    move.state = 'draft'
+                deleted_move_ids.append(move.id)
+                move.unlink()
+            else:
+                invoice_move_no_line[move2inv[move.id]] = move.name
+        if deleted_move_ids:
+            logger.info(
+                'Account move IDs %s have been deleted because they '
+                'had 0 lines', deleted_move_ids)
+        else:
+            logger.info('0 moves without lines found')
+        if invoice_move_no_line:
+            for inv_id, inv_number in invoice_move_no_line.iteritems():
+                logger.info(
+                    'Invoice ID %d number %s has a move with 0 lines',
+                    inv_id, inv_number)
+
 
 class AccountMoveLine(models.Model):
     _inherit = 'account.move.line'
@@ -149,6 +305,28 @@ class AccountMoveLine(models.Model):
             else:
                 self.credit = amount_company_currency
 
+    analytic_account_id = fields.Many2one(
+        domain=[('type', 'not in', ('view', 'template'))])
+
+
+class AccountBankStatement(models.Model):
+    _inherit = 'account.bank.statement'
+
+    start_date = fields.Date(
+        compute='_compute_dates', string='Start Date', readonly=True,
+        store=True)
+    end_date = fields.Date(
+        compute='_compute_dates', string='End Date', readonly=True,
+        store=True)
+
+    @api.multi
+    @api.depends('line_ids.date')
+    def _compute_dates(self):
+        for st in self:
+            dates = [line.date for line in st.line_ids]
+            st.start_date = dates and min(dates) or False
+            st.end_date = dates and max(dates) or False
+
 
 class AccountBankStatementLine(models.Model):
     _inherit = 'account.bank.statement.line'
@@ -196,6 +374,7 @@ class AccountBankStatementLine(models.Model):
             raise UserError(_(
                 'No journal entry linked to this bank statement line.'))
 
+
 class ResPartner(models.Model):
     _inherit = 'res.partner'
 

account_usability/account_view.xml

@@ -38,6 +38,11 @@
         <field name="period_id" position="attributes">
             <attribute name="groups"></attribute>
         </field>
+        <!-- move sent field and make it visible -->
+        <field name="sent" position="replace"/>
+        <field name="move_id" position="after">
+            <field name="sent"/>
+        </field>
     </field>
 </record>
 
@@ -61,6 +66,123 @@
         <filter name="unpaid" position="after">
             <filter name="overdue" string="Overdue"
                 domain="[('state', '=', 'open'), ('date_due', '&lt;', current_date)]"/>
+            <separator/>
+            <filter name="to_send" string="To Send" domain="[('sent', '=', False), ('state', 'in', ('open', 'paid'))]"/>
+            <filter name="sent" string="Sent" domain="[('sent', '=', True)]"/>
+            <separator/>
+            <filter name="no_attachment" string="Missing Attachment" domain="[('has_attachment', '=', False)]"/>
+        </filter>
+    </field>
+</record>
+
+<!-- Having a menu entry on invoice lines is often very usefull for odoo user:
+they can search in their lines, etc...
+So I enhance the generic views and add actions, but I don't add menu entries here ;
+the creation of the corresponding menu entry should be done in the customer-specific
+module -->
+<record id="view_invoice_line_tree" model="ir.ui.view">
+    <field name="name">account_usability.invoice_line_tree</field>
+    <field name="model">account.invoice.line</field>
+    <field name="inherit_id" ref="account.view_invoice_line_tree"/>
+    <field name="arch" type="xml">
+        <field name="name" position="before">
+            <field name="partner_id" invisible="not context.get('show_invoice_fields')"/>
+            <field name="date_invoice" invisible="not context.get('show_invoice_fields')"/>
+            <field name="invoice_number" invisible="not context.get('show_invoice_fields')"/>
+        </field>
+        <field name="price_subtotal" position="after">
+            <field name="currency_id"/>
+            <field name="state" invisible="not context.get('show_invoice_fields')"/>
+            <field name="invoice_type" invisible="1"/>
+        </field>
+        <xpath expr="/tree" position="attributes">
+            <attribute name="edit">0</attribute>
+            <attribute name="create">0</attribute>
+        </xpath>
+    </field>
+</record>
+
+<record id="account_invoice_line_search" model="ir.ui.view">
+    <field name="name">account_usability.invoice_line_search</field>
+    <field name="model">account.invoice.line</field>
+    <field name="arch" type="xml">
+        <search string="Search Invoice Lines">
+            <field name="partner_id"/>
+            <field name="product_id"/>
+            <field name="account_id"/>
+            <field name="invoice_number"/>
+            <field name="name"/>
+            <filter name="out_invoice" string="Customer Invoices"
+                domain="[('invoice_type', '=', 'out_invoice')]"/>
+            <filter name="out_refund" string="Customer Refunds"
+                domain="[('invoice_type', '=', 'out_refund')]"/>
+            <filter name="in_invoice" string="Supplier Invoices"
+                domain="[('invoice_type', '=', 'in_invoice')]"/>
+            <filter name="in_refund" string="Supplier Refunds"
+                domain="[('invoice_type', '=', 'in_refund')]"/>
+            <separator/>
+            <filter name="draft" string="Draft" domain="[('state', '=', 'draft')]"/>
+            <filter name="unpaid" string="Not Paid" domain="[('state', '=', 'open')]"/>
+            <filter name="paid" string="Paid" domain="[('state', '=', 'paid')]"/>
+
+            <group string="Group By" name="groupby">
+                <filter name="partner_groupby" string="Partner"
+                    context="{'group_by': 'partner_id'}"/>
+                <filter name="commercial_partner_group_by" string="Commercial Partner"
+                    context="{'group_by': 'commercial_partner_id'}"/>
+                <filter name="date_groupby" string="Invoice Date"
+                    context="{'group_by': 'date_invoice'}"/>
+                <filter name="product_groupby" string="Product"
+                    context="{'group_by': 'product_id'}"/>
+                <filter name="account_groupby" string="Account"
+                    context="{'group_by': 'account_id'}"/>
+            </group>
+        </search>
+    </field>
+</record>
+
+<record id="out_invoice_line_action" model="ir.actions.act_window">
+    <field name="name">Customer Invoice Lines</field>
+    <field name="res_model">account.invoice.line</field>
+    <field name="view_mode">tree,form</field>
+    <field name="domain">[('invoice_type', '=', 'out_invoice')]</field>
+    <field name="context">{'show_invoice_fields': True}</field>
+</record>
+
+<record id="out_refund_line_action" model="ir.actions.act_window">
+    <field name="name">Customer Refund Lines</field>
+    <field name="res_model">account.invoice.line</field>
+    <field name="view_mode">tree,form</field>
+    <field name="domain">[('invoice_type', '=', 'out_refund')]</field>
+    <field name="context">{'show_invoice_fields': True}</field>
+</record>
+
+<record id="in_invoice_line_action" model="ir.actions.act_window">
+    <field name="name">Supplier Invoice Lines</field>
+    <field name="res_model">account.invoice.line</field>
+    <field name="view_mode">tree,form</field>
+    <field name="domain">[('invoice_type', '=', 'in_invoice')]</field>
+    <field name="context">{'show_invoice_fields': True}</field>
+</record>
+
+<record id="in_refund_line_action" model="ir.actions.act_window">
+    <field name="name">Supplier Refund Lines</field>
+    <field name="res_model">account.invoice.line</field>
+    <field name="view_mode">tree,form</field>
+    <field name="domain">[('invoice_type', '=', 'in_refund')]</field>
+    <field name="context">{'show_invoice_fields': True}</field>
+</record>
+
+<record id="view_account_invoice_report_search" model="ir.ui.view">
+    <field name="name">usability.account.invoice.report.search</field>
+    <field name="model">account.invoice.report</field>
+    <field name="inherit_id" ref="account.view_account_invoice_report_search"/>
+    <field name="arch" type="xml">
+        <field name="categ_id" position="after">
+            <field name="product_id"/>
+        </field>
+        <filter name="category_product" position="after">
+            <filter string="Product" name="product_group_by" context="{'group_by': 'product_id'}"/>
         </filter>
     </field>
 </record>
@@ -68,10 +190,24 @@
 <!-- model account.move.line / Journal Items -->
 <record id="account.action_account_moves_all_a" model="ir.actions.act_window">
     <field name="limit">200</field>
+    <!-- add graph -->
+    <field name="view_mode">tree_account_move_line_quickadd,form,graph</field>
     <!-- Win space, because there are already many columns -->
     <field name="context">{'journal_show_code_only': True}</field>
 </record>
 
+<record id="account_move_line_graph" model="ir.ui.view">
+    <field name="name">usability.account.move.line.graph</field>
+    <field name="model">account.move.line</field>
+    <field name="inherit_id" ref="account.account_move_line_graph"/>
+    <field name="arch" type="xml">
+        <graph position="attributes">
+            <!-- pivot by default instead of bar -->
+            <attribute name="type">pivot</attribute>
+        </graph>
+    </field>
+</record>
+
 <!-- model account.move / Journal Entries -->
 <record id="account.action_move_journal_line" model="ir.actions.act_window">
     <field name="limit">200</field>
@@ -101,6 +237,7 @@
     <field name="arch" type="xml">
         <field name="partner_id" position="after">
             <field name="reconcile_ref" />
+            <field name="debit" filter_domain="['|', ('debit', '=', self), ('credit', '=', self)]" string="Debit or Credit"/>
         </field>
         <filter name="unreconciled" position="before">
             <filter name="reconciled" string="Fully Reconciled" domain="[('reconcile_id', '!=', False)]"/>
@@ -128,10 +265,18 @@
     <field name="name">account_usability.account_move_line_tree</field>
     <field name="model">account.move.line</field>
     <field name="inherit_id" ref="account.view_move_line_tree"/>
+    <field name="priority">1</field> <!-- to be compatible with account_analytic_plans which removes the field analytic_account_id and has a priority of 2 -->
     <field name="arch" type="xml">
         <field name="analytic_account_id" position="attributes">
             <attribute name="invisible"></attribute>
         </field>
+        <field name="reconcile_ref" position="attributes">
+            <attribute name="invisible">1</attribute>
+        </field>
+        <field name="reconcile_ref" position="after">
+            <field name="reconcile_id"/>
+            <field name="reconcile_partial_id"/>
+        </field>
     </field>
 </record>
 
@@ -164,6 +309,12 @@
     <field name="inherit_id" ref="account.partner_view_button_journal_item_count"/>
     <field name="arch" type="xml">
         <button name="%(account.action_account_moves_all_tree)d" position="after">
+            <button name="show_receivable_account" type="object"
+                attrs="{'invisible': [('customer', '=', False)]}"
+                icon="fa-list" class="oe_stat_button">
+                <field string="Journal Items" name="journal_item_count"
+                    widget="statinfo"/>
+            </button>            
             <button name="show_payable_account" type="object"
                 attrs="{'invisible': [('supplier', '=', False)]}"
                 icon="fa-list" class="oe_stat_button">
@@ -172,9 +323,7 @@
             </button>
         </button>
         <button name="%(account.action_account_moves_all_tree)d" position="attributes">
-            <attribute name="type">object</attribute>
-            <attribute name="name">show_receivable_account</attribute>
-            <attribute name="attrs">{'invisible': [('customer', '=', False)]}</attribute>
+            <attribute name="invisible">True</attribute>
         </button>
         <field name="journal_item_count" position="attributes">
             <attribute name="string">Receivable Account</attribute>
@@ -182,6 +331,12 @@
     </field>
 </record>
 
+<!-- When you click on the "Contract" button of partner form view,
+the newly created analytic accounts should be of type contract ! -->
+<record id="account.action_open_partner_analytic_accounts" model="ir.actions.act_window">
+    <field name="context">{'search_default_partner_id': [active_id], 'default_partner_id': active_id, 'default_type': 'contract'}</field>
+</record>
+
 <record id="view_bank_statement_form" model="ir.ui.view">
     <field name="name">usability.account.bank.statement.form</field>
     <field name="model">account.bank.statement</field>
@@ -193,6 +348,65 @@
                 string="View Account Move" icon="gtk-redo"
                 attrs="{'invisible': [('journal_entry_id', '=', False)]}"/>
         </xpath>
+        <field name="company_id" position="before">
+            <field name="start_date"/>
+            <field name="end_date"/>
+        </field>
+    </field>
+</record>
+
+<record id="view_bank_statement_tree" model="ir.ui.view">
+    <field name="name">usability.account.bank.statement.tree</field>
+    <field name="model">account.bank.statement</field>
+    <field name="inherit_id" ref="account.view_bank_statement_tree"/>
+    <field name="arch" type="xml">
+        <field name="period_id" position="attributes">
+            <attribute name="invisible">1</attribute>
+        </field>
+        <field name="date" position="attributes">
+            <attribute name="invisible">1</attribute>
+        </field>
+        <field name="journal_id" position="after">
+            <field name="start_date"/>
+            <field name="end_date"/>
+        </field>
+    </field>
+</record>
+
+<record id="view_bank_statement_search" model="ir.ui.view">
+    <field name="name">usability.account.bank.statement.search</field>
+    <field name="model">account.bank.statement</field>
+    <field name="inherit_id" ref="account.view_bank_statement_search"/>
+    <field name="arch" type="xml">
+        <field name="date" position="attributes">
+            <attribute name="invisible">1</attribute>
+        </field>
+        <filter context="{'group_by': 'period_id'}" position="attributes">
+            <attribute name="invisible">1</attribute>
+        </filter>
+        <filter context="{'group_by': 'period_id'}" position="after">
+            <filter name="start_date_groupby" string="Start Date"
+                context="{'group_by': 'start_date'}"/>
+            <filter name="end_date_groupby" string="End Date"
+                context="{'group_by': 'end_date'}"/>
+        </filter>
+        <field name="date" position="after">
+            <field name="start_date"/>
+            <field name="end_date"/>
+        </field>
+    </field>
+</record>
+
+<record id="view_account_analytic_account_form" model="ir.ui.view">
+    <field name="name">analytic.analytic.account.form</field>
+    <field name="model">account.analytic.account</field>
+    <field name="inherit_id" ref="analytic.view_account_analytic_account_form"/>
+    <field name="arch" type="xml">
+        <notebook position="inside">
+            <page name="invoice_lines" string="Related Invoice Lines" attrs="{'invisible': [('type', 'not in', ('contract', 'normal'))]}">
+                <field name="invoice_line_ids" nolabel="1" context="{'show_invoice_fields': 1}"/>
+            </page>
+        </notebook>
     </field>
 </record>
 

account_usability/migrations/8.0.0.2/pre-migration.py

@@ -0,0 +1,17 @@
+# -*- coding: utf-8 -*-
+# © 2016 Akretion (Alexis de Lattre <alexis.delattre@akretion.com>)
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+
+def migrate(cr, version):
+    if not version:
+        return
+
+    # cf the inherit of action_number() of account.invoice
+    # in account_usability/account.py
+    cr.execute(
+        "UPDATE account_move_line SET name= "
+        "CASE WHEN account_move_line.name='/' THEN account_move.name "
+        "ELSE account_move.name||' - '||account_move_line.name END "
+        "FROM account_move WHERE account_move_line.move_id = account_move.id "
+        "AND account_move_line.journal_id in "
+        "(SELECT id FROM account_journal WHERE type in ('sale', 'sale_refund'))")

account_usability/migrations/8.0.0.3/pre-migration.py

@@ -0,0 +1,16 @@
+# -*- coding: utf-8 -*-
+# © 2016 Akretion (Alexis de Lattre <alexis.delattre@akretion.com>)
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+
+def migrate(cr, version):
+    if not version:
+        return
+
+    # cf the inherit of action_number() of account.invoice
+    # in account_usability/account.py
+    cr.execute(
+        "UPDATE account_move_line SET name=account_move.name "
+        "FROM account_move WHERE account_move_line.move_id = account_move.id "
+        "AND account_move_line.name='/' "
+        "AND account_move_line.journal_id in "
+        "(SELECT id FROM account_journal WHERE type in ('purchase', 'purchase_refund'))")

account_usability/wizard/__init__.py

@@ -0,0 +1,3 @@
+# -*- encoding: utf-8 -*-
+
+from . import account_invoice_mark_sent

account_usability/wizard/account_invoice_mark_sent.py

@@ -0,0 +1,23 @@
+# -*- coding: utf-8 -*-
+# © 2017 Akretion (Alexis de Lattre <alexis.delattre@akretion.com>)
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+
+from openerp import models, api
+import logging
+logger = logging.getLogger(__name__)
+
+
+class AccountInvoiceMarkSent(models.TransientModel):
+    _name = 'account.invoice.mark.sent'
+    _description = 'Mark invoices as sent'
+
+    @api.multi
+    def run(self):
+        assert self.env.context.get('active_model') == 'account.invoice',\
+            'Source model must be invoices'
+        assert self.env.context.get('active_ids'), 'No invoices selected'
+        invoices = self.env['account.invoice'].browse(
+            self.env.context.get('active_ids'))
+        invoices.write({'sent': True})
+        logger.info('Marking invoices with ID %s as sent', invoices.ids)
+        return

account_usability/wizard/account_invoice_mark_sent_view.xml

@@ -0,0 +1,36 @@
+<?xml version="1.0" encoding="utf-8"?>
+<!--
+    © 2017 Akretion (Alexis de Lattre <alexis.delattre@akretion.com>)
+    The licence is in the file __openerp__.py
+-->
+
+<openerp>
+<data>
+
+<record id="account_invoice_mark_sent_form" model="ir.ui.view">
+    <field name="name">account.invoice.mark.sent.form</field>
+    <field name="model">account.invoice.mark.sent</field>
+    <field name="arch"  type="xml">
+        <form string="Mark invoices as sent">
+            <p>
+                This wizard will mark as sent all the selected invoices.
+            </p>
+            <footer>
+                <button type="object" name="run" string="Mark as Sent" class="oe_highlight"/>
+                <button special="cancel" string="Cancel" class="oe_link"/>
+            </footer>
+        </form>
+    </field>
+</record>
+
+<act_window id="account_invoice_mark_sent_action"
+            multi="True"
+            key2="client_action_multi"
+            name="Mark as Sent"
+            res_model="account.invoice.mark.sent"
+            src_model="account.invoice"
+            view_mode="form"
+            target="new" />
+
+</data>
+</openerp>

base_usability/module_view.xml

@@ -12,6 +12,9 @@
         <field name="model">ir.module.module</field>
         <field name="inherit_id" ref="base.view_module_filter"/>
         <field name="arch" type="xml">
+            <xpath expr="//filter[@name='extra']" position="after">
+                <filter name="installable" string="Installable" domain="[('state', '!=', 'uninstallable')]"/>
+            </xpath>
             <xpath expr="//filter[@string='Category']" position="after">
                 <filter string="State" domain="[]" context="{'group_by':'state'}"/>
             </xpath>
@@ -20,7 +23,7 @@
 
     <record id="base.open_module_tree" model="ir.actions.act_window">
         <field name="res_model">ir.module.module</field>
-        <field name="context">{}</field>
+        <field name="context">{'search_default_installable': 1}</field>
     </record>
 
 </data>

base_usability/partner.py

@@ -60,3 +60,21 @@ class Partner(models.Model):
             else:
                 name_title = ' '.join([title, name_title])
         self.name_title = name_title
+
+    @api.model
+    def name_search(self, name, args=None, operator='ilike', limit=100):
+        if args is None:
+            args = []
+        if name:
+            # only filter on name and ref not in email
+            args += [
+                '|', ('display_name', 'ilike', name), ('ref', 'ilike', name)]
+        res = super(Partner, self).name_search(
+            name, args=args, operator=operator, limit=limit)
+        return [(pid, val.replace('\n', ' ')) for pid, val in res]
+
+
+class ResPartnerCategory(models.Model):
+    _inherit = 'res.partner.category'
+
+    name = fields.Char(translate=False)

crm_usability/__init__.py

@@ -0,0 +1,4 @@
+# -*- coding: utf-8 -*-
+
+from . import wizard
+from . import crm

crm_usability/__openerp__.py

@@ -0,0 +1,35 @@
+# -*- coding: utf-8 -*-
+# © 2016 Akretion (http://www.akretion.com)
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+# @author Alexis de Lattre <alexis.delattre@akretion.com>
+
+{
+    'name': 'CRM Usability',
+    'version': '8.0.1.0.1',
+    'category': 'Customer Relationship Management',
+    'license': 'AGPL-3',
+    'summary': 'CRM usability enhancements',
+    'description': """
+CRM Usability
+=============
+
+The usability improvements include :
+
+* Adds multi-company record rules on crm.lead.
+* Some enhancements in the *Merge Partners* wizard:
+  * take into account the unaccent option of the server config file
+  * add optional group by on 'customer' and 'supplier' (active by default)
+
+This module has been written by Alexis de Lattre from Akretion
+<alexis.delattre@akretion.com>.
+    """,
+    'author': 'Akretion',
+    'website': 'http://www.akretion.com',
+    'depends': ['crm'],
+    'data': [
+        'crm_view.xml',
+        'wizard/base_partner_merge_view.xml',
+        'security/rule.xml',
+        ],
+    'installable': True,
+}

crm_usability/crm.py

@@ -0,0 +1,12 @@
+# -*- coding: utf-8 -*-
+# © 2017 Akretion (http://www.akretion.com)
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+# @author Alexis de Lattre <alexis.delattre@akretion.com>
+
+from openerp import models, fields
+
+
+class CrmCaseCateg(models.Model):
+    _inherit = 'crm.case.categ'
+
+    name = fields.Char(translate=False)

crm_usability/crm_view.xml

@@ -0,0 +1,37 @@
+<?xml version="1.0" encoding="utf-8"?>
+<!--
+   © 2017 Akretion (http://www.akretion.com/)
+   @author: Alexis de Lattre <alexis.delattre@akretion.com>
+   License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+-->
+
+<openerp>
+<data>
+
+
+<record id="crm_case_form_view_oppor" model="ir.ui.view">
+    <field name="name">usability.crm.opportunity.form</field>
+    <field name="model">crm.lead</field>
+    <field name="inherit_id" ref="crm.crm_case_form_view_oppor"/>
+    <field name="arch" type="xml">
+        <field name="active" position="before">
+            <field name="company_id" groups="base.group_multi_company"
+                widget="selection"/>
+        </field>
+    </field>
+</record>
+
+<record id="crm_case_inbound_phone_tree_view" model="ir.ui.view">
+    <field name="name">crm_usability.crm.phonecall.tree.inbound</field>
+    <field name="model">crm.phonecall</field>
+    <field name="inherit_id" ref="crm.crm_case_inbound_phone_tree_view"/>
+    <field name="arch" type="xml">
+        <field name="state" position="attributes">
+            <attribute name="invisible">0</attribute>
+        </field>
+    </field>
+</record>
+
+
+</data>
+</openerp>

crm_usability/security/rule.xml

@@ -0,0 +1,14 @@
+<?xml version="1.0" encoding="utf-8"?>
+<openerp>
+<data noupdate="1">
+
+
+<record id="crm_lead_company_rule" model="ir.rule">
+    <field name="name">Lead/opportunity multi-company rule</field>
+    <field name="model_id" ref="crm.model_crm_lead"/>
+    <field name="domain_force">['|', ('company_id', '=', False), ('company_id', 'child_of', [user.company_id.id])]</field>
+</record>
+
+
+</data>
+</openerp>

crm_usability/wizard/__init__.py

@@ -0,0 +1,3 @@
+# -*- coding: utf-8 -*-
+
+from . import base_partner_merge

crm_usability/wizard/base_partner_merge.py

@@ -0,0 +1,29 @@
+# -*- coding: utf-8 -*-
+# © 2016 Akretion (http://www.akretion.com)
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+# @author Alexis de Lattre <alexis.delattre@akretion.com>
+
+from openerp import models, fields, tools
+
+
+class MergePartnerAutomatic(models.TransientModel):
+    _inherit = 'base.partner.merge.automatic.wizard'
+
+    group_by_customer = fields.Boolean('Customer', default=True)
+    group_by_supplier = fields.Boolean('Supplier', default=True)
+
+    def _generate_query(self, fields, maximum_group=100):
+        sql = super(MergePartnerAutomatic, self)._generate_query(
+            fields, maximum_group=maximum_group)
+        name_sql_original = 'lower(name)'
+        if name_sql_original in sql:
+            if tools.config.get('unaccent', False):
+                sql = sql.replace(
+                    name_sql_original,
+                    "unaccent(lower(replace(name, ' ', '')))")
+            else:
+                sql = sql.replace(
+                    name_sql_original,
+                    "lower(replace(name, ' ', ''))")
+
+        return sql

crm_usability/wizard/base_partner_merge_view.xml

@@ -0,0 +1,23 @@
+<?xml version="1.0" encoding="UTF-8"?>
+
+<openerp>
+<data>
+
+<record id="base_partner_merge_automatic_wizard_form" model="ir.ui.view">
+    <field name="name">rvip.base.partner.merge.automatic.wizard.form</field>
+    <field name="model">base.partner.merge.automatic.wizard</field>
+    <field name="inherit_id" ref="crm.base_partner_merge_automatic_wizard_form"/>
+    <field name="arch" type="xml">
+        <field name="group_by_is_company" position="after">
+            <field name="group_by_customer"/>
+            <field name="group_by_supplier"/>
+        </field>
+        <xpath expr="//field[@name='partner_ids']/tree/field[@name='is_company']" position="after">
+            <field name="customer"/>
+            <field name="supplier"/>
+        </xpath>
+    </field>
+</record>
+
+</data>
+</openerp>

delivery_usability/__init__.py

@@ -0,0 +1 @@
+# -*- coding: utf-8 -*-

delivery_usability/__openerp__.py

@@ -0,0 +1,30 @@
+# -*- coding: utf-8 -*-
+# Copyright 2017 Akretion (http://www.akretion.com).
+# @author Sébastien BEAU <sebastien.beau@akretion.com>
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+
+{
+    "name": "Delivery usability",
+    "summary": "Delivery Usability",
+    "version": "8.0.1.0.0",
+    "category": "Warehouse",
+    "website": "www.akretion.com",
+    "author": " Akretion",
+    "license": "AGPL-3",
+    "application": False,
+    "installable": True,
+    "external_dependencies": {
+        "python": [],
+        "bin": [],
+    },
+    "depends": [
+        "delivery",
+    ],
+    "data": [
+        "views/stock_picking_view.xml",
+    ],
+    "demo": [
+    ],
+    "qweb": [
+    ]
+}

delivery_usability/views/stock_picking_view.xml

@@ -0,0 +1,20 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<openerp>
+    <data>
+
+<record id="view_picking_withcarrier_out_form" model="ir.ui.view">
+    <field name="model">stock.picking</field>
+    <field name="inherit_id" ref="delivery.view_picking_withcarrier_out_form"/>
+    <field name="priority" eval="100"/>
+    <field name="arch" type="xml">
+        <xpath expr="//field[@name='carrier_id']/.." position="replace">
+        </xpath>
+        <field name="backorder_id" position="after">
+            <field name="carrier_id"/>
+            <field name="carrier_tracking_ref"/>
+        </field>
+    </field>
+</record>
+
+    </data>
+</openerp>

hr_holidays_lunch_voucher/__init__.py

@@ -0,0 +1,7 @@
+# -*- coding: utf-8 -*-
+
+from . import company
+from . import hr_holidays
+from . import hr_employee
+from . import lunch_voucher_attribution
+from . import wizard

hr_holidays_lunch_voucher/__openerp__.py

@@ -0,0 +1,28 @@
+# -*- coding: utf-8 -*-
+# © 2017 Akretion (http://www.akretion.com)
+# @author Alexis de Lattre <alexis.delattre@akretion.com>
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+
+{
+    'name': 'HR Holidays Lunch Voucher',
+    'version': '10.0.1.0.0',
+    'category': 'Human Resources',
+    'license': 'AGPL-3',
+    'summary': 'Manage lunch vouchers in holidays requests',
+    'description': '',
+    'author': 'Akretion',
+    'website': 'http://www.akretion.com',
+    'depends': ['hr_holidays_usability', 'purchase'],
+    'data': [
+        'security/ir.model.access.csv',
+        'security/lunch_voucher_security.xml',
+        'product_data.xml',
+        'company_view.xml',
+        'wizard/lunch_voucher_purchase_view.xml',
+        'lunch_voucher_attribution_view.xml',
+        'hr_employee_view.xml',
+        'hr_holidays_view.xml',
+        'wizard/hr_holidays_post_view.xml',
+        ],
+    'installable': True,
+}

hr_holidays_lunch_voucher/company.py

@@ -0,0 +1,20 @@
+# -*- coding: utf-8 -*-
+# © 2017 Akretion (http://www.akretion.com)
+# @author Alexis de Lattre <alexis.delattre@akretion.com>
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+
+from openerp import models, fields
+import openerp.addons.decimal_precision as dp
+
+
+class ResCompany(models.Model):
+    _inherit = 'res.company'
+
+    lunch_voucher_product_id = fields.Many2one(
+        'product.product', string='Lunch Voucher Product',
+        ondelete='restrict')
+    lunch_voucher_employer_price = fields.Float(
+        'Lunch Voucher Employer Price', digits=dp.get_precision('Account'))
+
+    # Add constrain to check that lunch_voucher_employer_price is between
+    # 50% and 60% of the price of lunch_voucher_product_id for France

hr_holidays_lunch_voucher/company_view.xml

@@ -0,0 +1,29 @@
+<?xml version="1.0" encoding="utf-8"?>
+<!--
+  © 2017 Akretion France (www.akretion.com)
+  @author: Alexis de Lattre <alexis.delattre@akretion.com>
+  License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+-->
+
+<openerp>
+<data>
+
+
+<record id="view_company_form" model="ir.ui.view">
+    <field name="name">lunch_voucher.company.form</field>
+    <field name="model">res.company</field>
+    <field name="inherit_id" ref="base.view_company_form"/>
+    <field name="arch"  type="xml">
+        <group name="account_grp" position="after">
+            <group name="lunch_voucher" string="Lunch Vouchers">
+                <field name="lunch_voucher_product_id"/>
+                <!-- <field name="lunch_voucher_po_type"/> -->
+                <field name="lunch_voucher_employer_price"/>
+            </group>
+        </group>
+    </field>
+</record>
+
+
+</data>
+</openerp>

hr_holidays_lunch_voucher/hr_employee.py

@@ -0,0 +1,12 @@
+# -*- coding: utf-8 -*-
+# © 2017 Akretion (http://www.akretion.com)
+# @author Alexis de Lattre <alexis.delattre@akretion.com>
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+
+from openerp import models, fields
+
+
+class HrEmployee(models.Model):
+    _inherit = 'hr.employee'
+
+    lunch_voucher = fields.Boolean(string="Has Lunch Vouchers", default=True)

hr_holidays_lunch_voucher/hr_employee_view.xml

@@ -0,0 +1,25 @@
+<?xml version="1.0" encoding="utf-8"?>
+<!--
+  © 2017 Akretion France (www.akretion.com)
+  @author: Alexis de Lattre <alexis.delattre@akretion.com>
+  License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+-->
+
+<openerp>
+<data>
+
+
+<record id="view_employee_form_leave_inherit" model="ir.ui.view">
+    <field name="name">lunch_voucher.hr.employee.form</field>
+    <field name="model">hr.employee</field>
+    <field name="inherit_id" ref="hr_holidays_usability.view_employee_form_leave_inherit"/>
+    <field name="arch"  type="xml">
+        <group name="holidays" position="inside">
+            <field name="lunch_voucher"/>
+        </group>
+    </field>
+</record>
+
+
+</data>
+</openerp>

hr_holidays_lunch_voucher/hr_holidays.py

@@ -0,0 +1,45 @@
+# -*- coding: utf-8 -*-
+# © 2017 Akretion (http://www.akretion.com)
+# @author Alexis de Lattre <alexis.delattre@akretion.com>
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+
+from openerp import models, fields, api
+from dateutil.relativedelta import relativedelta
+
+
+class HrHolidays(models.Model):
+    _inherit = 'hr.holidays'
+
+    lunch_voucher_id = fields.Many2one(
+        'lunch.voucher.attribution',
+        string="Related Lunch Voucher Attribution")
+    lunch_voucher_remove_qty = fields.Integer(
+        compute='_compute_lunch_voucher_remove_qty', readonly=True,
+        store=True, string='Lunch Vouchers to Deduct')
+
+    @api.depends('employee_id', 'vacation_date_from', 'vacation_date_to')
+    @api.multi
+    def _compute_lunch_voucher_remove_qty(self):
+        hhpo = self.env['hr.holidays.public']
+        for hol in self:
+            qty = 0
+            if (
+                    hol.type == 'remove' and
+                    hol.vacation_date_from and
+                    hol.vacation_date_to):
+                start_date_dt = fields.Date.from_string(hol.vacation_date_from)
+                end_date_str = hol.vacation_date_to
+                date_dt = start_date_dt
+                # Remove 1 full LV when vacation_time_from == noon
+                # and also when vacation_time_to == noon
+                while True:
+                    if (
+                            date_dt.weekday() not in (5, 6) and
+                            not hhpo.is_public_holiday(
+                                date_dt, hol.employee_id.id)):
+                        qty += 1
+                    date_dt += relativedelta(days=1)
+                    date_str = fields.Date.to_string(date_dt)
+                    if date_str > end_date_str:
+                        break
+            hol.lunch_voucher_remove_qty = qty

hr_holidays_lunch_voucher/hr_holidays_view.xml

@@ -0,0 +1,39 @@
+<?xml version="1.0" encoding="utf-8"?>
+<!--
+  © 2017 Akretion (http://www.akretion.com/)
+  @author: Alexis de Lattre <alexis.delattre@akretion.com>
+  License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+-->
+
+<openerp>
+<data>
+
+
+<record id="edit_holiday_new" model="ir.ui.view">
+    <field name="name">hr_holidays_lunch_voucher.leave_request_form</field>
+    <field name="model">hr.holidays</field>
+    <field name="inherit_id" ref="hr_holidays_usability.edit_holiday_new"/>
+    <field name="arch" type="xml">
+        <group name="counters" position="after">
+            <group name="lunch_vouchers" string="Lunch Vouchers" attrs="{'invisible': [('type', '!=', 'remove')]}">
+                <field name="lunch_voucher_remove_qty"/>
+                <field name="lunch_voucher_id"/>
+            </group>
+        </group>
+    </field>
+</record>
+
+<record id="view_holiday" model="ir.ui.view">
+    <field name="name">hr_holidays_lunch_voucher.leave_request_tree</field>
+    <field name="model">hr.holidays</field>
+    <field name="inherit_id" ref="hr_holidays_usability.view_holiday"/>
+    <field name="arch" type="xml">
+        <field name="number_of_days" position="after">
+             <field name="lunch_voucher_remove_qty" string="Lunch Vouchers -=" sum="Total"/>
+        </field>
+    </field>
+</record>
+
+
+</data>
+</openerp>

hr_holidays_lunch_voucher/lunch_voucher_attribution.py

@@ -0,0 +1,45 @@
+# -*- coding: utf-8 -*-
+# © 2017 Akretion (http://www.akretion.com)
+# @author Alexis de Lattre <alexis.delattre@akretion.com>
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+
+from openerp import models, fields, api
+
+
+class LunchVoucherAttribution(models.Model):
+    _name = 'lunch.voucher.attribution'
+    _description = 'Lunch Voucher Attribution'
+    _order = 'date desc'
+
+    employee_id = fields.Many2one(
+        'hr.employee', string='Employee', ondelete='restrict',
+        required=True, readonly=True)
+    company_id = fields.Many2one(
+        related='employee_id.resource_id.company_id', readonly=True,
+        store=True)
+    date = fields.Date('Attribution Date', readonly=True)
+    purchase_id = fields.Many2one(
+        'purchase.order', 'Purchase Order',
+        readonly=True)
+    month_work_days = fields.Integer(
+        'Month Work Days',
+        help="Number of work days of the month (without taking into "
+        "account the holidays)")
+    no_lunch_days = fields.Integer(
+        compute='_compute_qty', string='No Lunch Days',
+        readonly=True, store=True)
+    qty = fields.Integer(
+        compute='_compute_qty', readonly=True, store=True,
+        string='Lunch Voucher Quantity')
+    holiday_ids = fields.One2many(
+        'hr.holidays', 'lunch_voucher_id', readonly=True)
+
+    @api.depends('month_work_days', 'holiday_ids.lunch_voucher_remove_qty')
+    @api.multi
+    def _compute_qty(self):
+        for rec in self:
+            no_lunch_days = 0
+            for hol in rec.holiday_ids:
+                no_lunch_days += hol.lunch_voucher_remove_qty
+            rec.no_lunch_days = no_lunch_days
+            rec.qty = rec.month_work_days - no_lunch_days

hr_holidays_lunch_voucher/lunch_voucher_attribution_view.xml

@@ -0,0 +1,97 @@
+<?xml version="1.0" encoding="utf-8"?>
+<!--
+  © 2017 Akretion France (www.akretion.com)
+  @author: Alexis de Lattre <alexis.delattre@akretion.com>
+  License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+-->
+
+<openerp>
+<data>
+
+
+<record id="lunch_voucher_attribution_form" model="ir.ui.view">
+    <field name="name">lunch.voucher.attribution.form</field>
+    <field name="model">lunch.voucher.attribution</field>
+    <field name="arch"  type="xml">
+        <form string="Lunch Vouchers Attribution">
+            <group name="main">
+                <field name="employee_id"/>
+                <field name="company_id" groups="base.group_multi_company"/>
+                <field name="date"/>
+                <field name="month_work_days"/>
+                <field name="no_lunch_days"/>
+                <field name="qty"/>
+                <field name="purchase_id"/>
+            </group>
+            <group name="holidays" string="Related Holidays">
+                <field name="holiday_ids" nolabel="1" context="{'tree_view_ref': 'hr_holidays_lunch_voucher.view_holiday', 'form_view_ref': 'hr_holidays_lunch_voucher.edit_holiday_new'}"/>
+            </group>
+        </form>
+    </field>
+</record>
+
+<record id="lunch_voucher_attribution_tree" model="ir.ui.view">
+    <field name="name">lunch.voucher.attribution.tree</field>
+    <field name="model">lunch.voucher.attribution</field>
+    <field name="arch"  type="xml">
+        <tree string="Lunch Vouchers Attribution">
+            <field name="date"/>
+            <field name="employee_id"/>
+            <field name="month_work_days"/>
+            <field name="no_lunch_days" sum="Total"/>
+            <field name="qty" sum="Total"/>
+            <field name="purchase_id"/>
+            <field name="company_id" groups="base.group_multi_company"/>
+        </tree>
+    </field>
+</record>
+
+<record id="lunch_voucher_attribution_search" model="ir.ui.view">
+    <field name="name">lunch.voucher.attribution.search</field>
+    <field name="model">lunch.voucher.attribution</field>
+    <field name="arch"  type="xml">
+        <search string="Search Lunch Vouchers Attribution">
+            <field name="date"/>
+            <field name="employee_id"/>
+            <filter name="purchased" string="Purchased" domain="[('purchase_id', '!=', False)]"/>
+            <filter name="to_purchase" string="To Purchase" domain="[('purchase_id', '=', False)]"/>
+            <group string="Group By" name="groupby">
+                <filter name="employee_groupby" string="Employee" context="{'group_by': 'employee_id'}"/>
+                <filter name="purchase_order_groupby" string="Purchase Order" context="{'group_by': 'purchase_id'}"/>
+            </group>
+        </search>
+    </field>
+</record>
+
+<record id="lunch_voucher_attribution_graph" model="ir.ui.view">
+    <field name="name">lunch.voucher.attribution.graph</field>
+    <field name="model">lunch.voucher.attribution</field>
+    <field name="arch"  type="xml">
+        <graph string="Lunch Vouchers Attribution" type="pivot">
+            <field name="date" type="row" interval="month"/>
+            <field name="qty" type="measure"/>
+        </graph>
+    </field>
+</record>
+
+<record id="lunch_voucher_attribution_action" model="ir.actions.act_window">
+    <field name="name">Lunch Voucher Attribution</field>
+    <field name="res_model">lunch.voucher.attribution</field>
+    <field name="view_mode">tree,form,graph</field>
+</record>
+
+<menuitem id="lunch_voucher_attribution_menu"
+    action="lunch_voucher_attribution_action"
+    parent="hr_holidays.menu_open_ask_holidays" sequence="200"/>
+
+<act_window id="lunch_voucher_attribution_action_create_po"
+            multi="True"
+            key2="client_action_multi"
+            name="Create Purchase Order"
+            res_model="lunch.voucher.purchase"
+            src_model="lunch.voucher.attribution"
+            view_mode="form"
+            target="new" />
+
+</data>
+</openerp>

hr_holidays_lunch_voucher/product_data.xml

@@ -0,0 +1,27 @@
+<?xml version="1.0" encoding="utf-8"?>
+
+<openerp>
+<data noupdate="1">
+
+
+<record id="lunch_voucher_product" model="product.product">
+    <field name="name">Lunch Vouchers</field>
+    <field name="categ_id" ref="product.product_category_all"/>
+    <field name="sale_ok" eval="False"/>
+    <field name="purchase_ok" eval="True"/>
+    <field name="list_price">0</field>
+    <field name="standard_price">8</field>
+    <field name="type">service</field> <!-- For those who want to manage stock of lunch vouchers, they can switch type to product manually -->
+    <field name="uom_id" ref="product.product_uom_unit"/>
+    <field name="uom_po_id" ref="product.product_uom_unit"/>
+    <field name="supplier_taxes_id" eval="False"/>
+    <field name="taxes_id" eval="False"/>
+</record>
+
+<record id="base.main_company" model="res.company">
+    <field name="lunch_voucher_product_id" ref="lunch_voucher_product"/>
+</record>
+
+
+</data>
+</openerp>

hr_holidays_lunch_voucher/security/ir.model.access.csv

@@ -0,0 +1,3 @@
+id,name,model_id:id,group_id:id,perm_read,perm_write,perm_create,perm_unlink
+access_lunch_voucher_attribution_full,Full access on lunch_voucher_attribution,model_lunch_voucher_attribution,base.group_hr_manager,1,1,1,1
+access_lunch_voucher_attribution_read,Read access on lunch_voucher_attribution,model_lunch_voucher_attribution,base.group_user,1,0,0,0

hr_holidays_lunch_voucher/security/lunch_voucher_security.xml

@@ -0,0 +1,28 @@
+<?xml version="1.0" encoding="utf-8"?>
+<openerp>
+<data noupdate="1">
+
+
+<record id="lunch_voucher_attribution_employee" model="ir.rule">
+    <field name="name">Personal Lunch Voucher Attributions</field>
+    <field name="model_id" ref="hr_holidays_lunch_voucher.model_lunch_voucher_attribution"/>
+    <field name="domain_force">[('employee_id.user_id', '=', user.id)]</field>
+    <field name="groups" eval="[(4, ref('base.group_user'))]"/>
+</record>
+
+<record id="lunch_voucher_attribution_officer" model="ir.rule">
+    <field name="name">HR Officer sees all Lunch Voucher Attributions</field>
+    <field name="model_id" ref="hr_holidays_lunch_voucher.model_lunch_voucher_attribution"/>
+    <field name="domain_force">[(1, '=', 1)]</field>
+    <field name="groups" eval="[(4, ref('base.group_hr_manager'))]"/>
+</record>
+
+<record id="lunch_voucher_attribution_multicompany_rule" model="ir.rule">
+    <field name="name">Lunch Voucher Attribution multi-company</field>
+    <field name="model_id" ref="hr_holidays_lunch_voucher.model_lunch_voucher_attribution"/>
+    <field name="domain_force">['|', ('company_id', '=', False), ('company_id', 'child_of', [user.company_id.id])]</field>
+</record>
+
+
+</data>
+</openerp>

hr_holidays_lunch_voucher/wizard/__init__.py

@@ -0,0 +1,4 @@
+# -*- encoding: utf-8 -*-
+
+from . import hr_holidays_post
+from . import lunch_voucher_purchase

hr_holidays_lunch_voucher/wizard/hr_holidays_post.py

@@ -0,0 +1,74 @@
+# -*- coding: utf-8 -*-
+# © 2017 Akretion (http://www.akretion.com)
+# @author Alexis de Lattre <alexis.delattre@akretion.com>
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+
+from openerp import models, fields, api
+from dateutil.relativedelta import relativedelta
+import logging
+
+logger = logging.getLogger(__name__)
+
+
+class HrHolidaysPost(models.TransientModel):
+    _inherit = 'hr.holidays.post'
+
+    lunch_voucher_po = fields.Boolean(
+        string='Generate Lunch Voucher Purchase Order')
+    work_days = fields.Integer(string='Work Days')
+
+    @api.model
+    def default_get(self, fields_list):
+        res = super(HrHolidaysPost, self).default_get(fields_list)
+        hhpo = self.env['hr.holidays.public']
+        company = self.env.user.company_id
+        if company.lunch_voucher_product_id:
+            res['lunch_voucher_po'] = True
+        today = fields.Date.context_today(self)
+        today_dt = fields.Date.from_string(today)
+        cur_month = today_dt.month
+        # last day of month
+        date_dt = today_dt + relativedelta(day=31)
+        work_days = date_dt.day
+        logger.info('Number of days in the month: %d', work_days)
+        # from last day of month to the first
+        while date_dt.month == cur_month:
+            if hhpo.is_public_holiday(date_dt):
+                work_days -= 1
+                logger.info(
+                    "%s is a bank holiday, don't count", date_dt)
+            # if it's a saturday/sunday
+            elif date_dt.weekday() in (5, 6):
+                work_days -= 1
+                logger.info(
+                    "%s is a saturday/sunday, don't count", date_dt)
+            date_dt += relativedelta(days=-1)
+        logger.info('Number of work days in the month: %d', work_days)
+        res['work_days'] = work_days
+        return res
+
+    @api.multi
+    def run(self):
+        self.ensure_one()
+        lvao = self.env['lunch.voucher.attribution']
+        today = fields.Date.context_today(self)
+        employees = self.env['hr.employee'].search([
+            ('lunch_voucher', '=', True),
+            ('company_id', '=', self.env.user.company_id.id),
+            ])
+        lv_dict = {}
+        for employee in employees:
+            lv_dict[employee.id] = []
+        for hol in self.holidays_to_post_ids:
+            if not hol.lunch_voucher_id and hol.employee_id.id in lv_dict:
+                lv_dict[hol.employee_id.id].append(hol.id)
+        for employee_id, hol_ids in lv_dict.iteritems():
+            vals = {
+                'employee_id': employee_id,
+                'date': today,
+                'month_work_days': self.work_days,
+                }
+            if hol_ids:
+                vals['holiday_ids'] = [(6, 0, hol_ids)]
+            lvao.create(vals)
+        return super(HrHolidaysPost, self).run()

hr_holidays_lunch_voucher/wizard/hr_holidays_post_view.xml

@@ -0,0 +1,27 @@
+<?xml version="1.0" encoding="utf-8"?>
+<!--
+  © 2017 Akretion (http://www.akretion.com/)
+  @author: Alexis de Lattre <alexis.delattre@akretion.com>
+  License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+-->
+
+<openerp>
+<data>
+
+
+<record id="hr_holidays_post_form" model="ir.ui.view">
+    <field name="name">hr_holidays_post_form</field>
+    <field name="model">hr.holidays.post</field>
+    <field name="inherit_id" ref="hr_holidays_usability.hr_holidays_post_form"/>
+    <field name="arch" type="xml">
+        <field name="holidays_to_post_ids" position="before">
+            <field name="lunch_voucher_po" states="done"/>
+            <field name="work_days"
+                attrs="{'invisible': ['|', ('state', '!=', 'done'), ('lunch_voucher_po', '=', False)]}"/>
+        </field>
+    </field>
+</record>
+
+
+</data>
+</openerp>

hr_holidays_lunch_voucher/wizard/lunch_voucher_purchase.py

@@ -0,0 +1,77 @@
+# -*- coding: utf-8 -*-
+# © 2017 Akretion (http://www.akretion.com)
+# @author Alexis de Lattre <alexis.delattre@akretion.com>
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+
+from openerp import models, api, _
+from openerp.exceptions import Warning as UserError
+
+
+class LunchVoucherPurchase(models.TransientModel):
+    _name = 'lunch.voucher.purchase'
+    _description = 'Purchase Lunch Vouchers Wizard'
+
+    @api.multi
+    def run(self):
+        self.ensure_one()
+        company = self.env.user.company_id
+        if not company.lunch_voucher_product_id:
+            raise UserError(_(
+                "Lunch Voucher Product not configured on company %s")
+                % company.name)
+        if not company.lunch_voucher_product_id.seller_id:
+            raise UserError(_(
+                "Missing supplier on Product '%s'.")
+                % company.lunch_voucher_product_id.name)
+
+        poo = self.env['purchase.order']
+        polo = self.env['purchase.order.line']
+        lvao = self.env['lunch.voucher.attribution']
+        assert self._context.get('active_model') ==\
+            'lunch.voucher.attribution', 'wrong source model'
+        assert self._context.get('active_ids'), 'missing active_ids in ctx'
+        lvouchers = lvao.browse(self._context['active_ids'])
+        total_qty = 0
+        for lvoucher in lvouchers:
+            if lvoucher.purchase_id:
+                raise UserError(_(
+                    "One of the Lunch Voucher Attributions you selected "
+                    "related to employee '%s' is already linked to a "
+                    "purchase order.") % lvoucher.employee_id.name)
+            if lvoucher.qty < 0:
+                raise UserError(_(
+                    "One of the Lunch Voucher Attributions you selected "
+                    "related to employee '%s' has a negative quantity.")
+                    % lvoucher.employee_id.name)
+            total_qty += lvoucher.qty
+
+        supplier = company.lunch_voucher_product_id.seller_id
+        pick_type_id = poo.default_get(['picking_type_id'])['picking_type_id']
+        onchange_ptype_vals = poo.browse(False).onchange_picking_type_id(
+            pick_type_id)
+        vals = onchange_ptype_vals['value']
+        onchange_vals = poo.browse(False).onchange_partner_id(supplier.id)
+        vals.update(onchange_vals['value'])
+        vals['partner_id'] = supplier.id
+
+        product = company.lunch_voucher_product_id
+        onchange_product_vals = polo.browse(False).onchange_product_id(
+            vals.get('pricelist_id'), product.id, total_qty, False,
+            supplier.id, fiscal_position_id=vals.get('fiscal_position_id'))
+        lvals = onchange_product_vals['value']
+        lvals['product_id'] = product.id
+        lvals['product_qty'] = total_qty
+        if lvals['taxes_id']:
+            lvals['taxes_id'] = [(6, 0, lvals['taxes_id'])]
+        vals['order_line'] = [(0, 0, lvals)]
+        po = poo.create(vals)
+        lvouchers.write({'purchase_id': po.id})
+
+        action = self.env['ir.actions.act_window'].for_xml_id(
+            'purchase', 'purchase_rfq')
+        action.update({
+            'res_id': po.id,
+            'view_mode': 'form,tree',
+            'views': False,
+            })
+        return action

hr_holidays_lunch_voucher/wizard/lunch_voucher_purchase_view.xml

@@ -0,0 +1,36 @@
+<?xml version="1.0" encoding="utf-8"?>
+<!--
+  © 2017 Akretion (http://www.akretion.com/)
+  @author Alexis de Lattre <alexis.delattre@akretion.com>
+  License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+-->
+
+<openerp>
+<data>
+
+<record id="lunch_voucher_purchase_form" model="ir.ui.view">
+    <field name="name">lunch_voucher_purchase_form</field>
+    <field name="model">lunch.voucher.purchase</field>
+    <field name="arch" type="xml">
+        <form string="Create Purchase Order of Lunch Vouchers">
+            <group name="main">
+                <label string="Click on the button below to create the purchase order for the selected lunch vouchers attributions." colspan="2"/>
+            </group>
+            <footer>
+                <button name="run" type="object" string="Create"
+                    class="oe_highlight"/>
+                <button special="cancel" string="Cancel" class="oe_link"/>
+            </footer>
+        </form>
+    </field>
+</record>
+
+<record id="lunch_voucher_purchase_action" model="ir.actions.act_window">
+    <field name="name">Create PO for Lunch Vouchers</field>
+    <field name="res_model">lunch.voucher.purchase</field>
+    <field name="view_mode">form</field>
+    <field name="target">new</field>
+</record>
+
+</data>
+</openerp>

hr_holidays_lunch_voucher_natixis/__init__.py

@@ -0,0 +1,4 @@
+# -*- coding: utf-8 -*-
+
+from . import company
+from . import wizard

hr_holidays_lunch_voucher_natixis/__openerp__.py

@@ -0,0 +1,20 @@
+# -*- coding: utf-8 -*-
+# © 2017 Akretion (http://www.akretion.com)
+# @author Alexis de Lattre <alexis.delattre@akretion.com>
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+
+{
+    'name': 'HR Holidays Lunch Voucher Natixis',
+    'version': '10.0.1.0.0',
+    'category': 'Human Resources',
+    'license': 'AGPL-3',
+    'summary': 'Generate order file for Natixis lunch vouchers',
+    'description': '',
+    'author': 'Akretion',
+    'website': 'http://www.akretion.com',
+    'depends': ['hr_holidays_lunch_voucher'],
+    'data': [
+        'company_view.xml',
+        ],
+    'installable': True,
+}

hr_holidays_lunch_voucher_natixis/company.py

@@ -0,0 +1,15 @@
+# -*- coding: utf-8 -*-
+# © 2017 Akretion (http://www.akretion.com)
+# @author Alexis de Lattre <alexis.delattre@akretion.com>
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+
+from openerp import models, fields
+
+
+class ResCompany(models.Model):
+    _inherit = 'res.company'
+
+    lunch_voucher_natixis_customer_code = fields.Char(
+        string='Natixis Customer Ref', size=7)
+    lunch_voucher_natixis_delivery_code = fields.Char(
+        string='Natixis Delivery Code', size=7)

hr_holidays_lunch_voucher_natixis/company_view.xml

@@ -0,0 +1,26 @@
+<?xml version="1.0" encoding="utf-8"?>
+<!--
+  © 2017 Akretion France (www.akretion.com)
+  @author: Alexis de Lattre <alexis.delattre@akretion.com>
+  License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+-->
+
+<openerp>
+<data>
+
+
+<record id="view_company_form" model="ir.ui.view">
+    <field name="name">natixis.lunch_voucher.company.form</field>
+    <field name="model">res.company</field>
+    <field name="inherit_id" ref="hr_holidays_lunch_voucher.view_company_form"/>
+    <field name="arch"  type="xml">
+        <group name="lunch_voucher" position="inside">
+            <field name="lunch_voucher_natixis_customer_code"/>
+            <field name="lunch_voucher_natixis_delivery_code"/>
+        </group>
+    </field>
+</record>
+
+
+</data>
+</openerp>

hr_holidays_lunch_voucher_natixis/wizard/__init__.py

@@ -0,0 +1,3 @@
+# -*- encoding: utf-8 -*-
+
+from . import lunch_voucher_purchase

hr_holidays_lunch_voucher_natixis/wizard/lunch_voucher_purchase.py

@@ -0,0 +1,92 @@
+# -*- coding: utf-8 -*-
+# © 2017 Akretion (http://www.akretion.com)
+# @author Alexis de Lattre <alexis.delattre@akretion.com>
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+
+from openerp import models, fields, api, _
+from openerp.exceptions import Warning as UserError
+from openerp.tools import float_is_zero
+
+
+class LunchVoucherPurchase(models.TransientModel):
+    _inherit = 'lunch.voucher.purchase'
+
+    @api.multi
+    def run(self):
+        self.ensure_one()
+        action = super(LunchVoucherPurchase, self).run()
+        company = self.env.user.company_id
+        # force_company to read standard_price properly
+        company = company.with_context(force_company=company.id)
+        lvao = self.env['lunch.voucher.attribution']
+        assert self._context.get('active_model')\
+            == 'lunch.voucher.attribution', 'wrong source model'
+        assert self._context.get('active_ids'), 'missing active_ids in ctx'
+        if not company.lunch_voucher_natixis_customer_code:
+            raise UserError(_(
+                "Missing Natixis Customer Ref on company '%s'.")
+                % company.name)
+        if not company.lunch_voucher_natixis_delivery_code:
+            raise UserError(_(
+                "Missing Natixis Delivery Code on company '%s'.")
+                % company.name)
+        if len(company.lunch_voucher_natixis_customer_code) != 7:
+            raise UserError(_(
+                "Natixis Customer Ref '%s' on company '%s' should "
+                "have 7 characters/digits.")
+                % (company.lunch_voucher_natixis_customer_code, company.name))
+        if len(company.lunch_voucher_natixis_delivery_code) != 7:
+            raise UserError(_(
+                "Natixis Delivery Code on company '%s' should "
+                "have 7 characters/digits.")
+                % (company.lunch_voucher_natixis_delivery_code, company.name))
+        if float_is_zero(
+                company.lunch_voucher_employer_price,
+                precision_digits=2):
+            raise UserError(_(
+                "Lunch Voucher Employer Price not set on company '%s'.")
+                % company.name)
+        if float_is_zero(
+                company.lunch_voucher_product_id.standard_price,
+                precision_digits=2):
+            raise UserError(_(
+                "Lunch Voucher Standard Price is not set on product '%s'.")
+                % company.lunch_voucher_product_id.display_name)
+        lvouchers = lvao.browse(self._context['active_ids'])
+        of = u''
+        tmp = {}
+        for lvoucher in lvouchers:
+            if lvoucher.qty > 0:
+                if lvoucher.qty not in tmp:
+                    tmp[lvoucher.qty] = 1
+                else:
+                    tmp[lvoucher.qty] += 1
+        for vouchers_per_pack, pack_qty in tmp.iteritems():
+            if vouchers_per_pack > 99:
+                raise UserError(_(
+                    "Cannot order more than 99 vouchers per pack"))
+            line = u'%s%s%s%s%s%s%s%s\n' % (
+                company.lunch_voucher_natixis_delivery_code,
+                company.lunch_voucher_natixis_customer_code,
+                unicode(pack_qty).zfill(3),
+                unicode(vouchers_per_pack).zfill(2),
+                unicode(pack_qty * vouchers_per_pack).zfill(5),
+                '{:05.2f}'.format(
+                    company.lunch_voucher_product_id.standard_price),
+                '{:05.2f}'.format(company.lunch_voucher_employer_price),
+                ' ' * 64)
+            of += line
+        today_dt = fields.Date.from_string(
+            fields.Date.context_today(self))
+        filename = 'E%s_%s.txt' % (
+            company.lunch_voucher_natixis_customer_code,
+            today_dt.strftime('%d%m%Y'))
+        self.env['ir.attachment'].create({
+            'name': filename,
+            'res_id': action['res_id'],
+            'res_model': 'purchase.order',
+            'datas': of.encode('base64'),
+            'datas_fname': filename,
+            'type': 'binary',
+            })
+        return action

hr_holidays_usability/hr_holidays.py

@@ -184,6 +184,9 @@ class HrHolidays(models.Model):
 
     vacation_date_from = fields.Date(
         string='First Day of Vacation', track_visibility='onchange',
+        readonly=True, states={
+            'draft': [('readonly', False)],
+            'confirm': [('readonly', False)]},
         help="Enter the first day of vacation. For example, if "
         "you leave one full calendar week, the first day of vacation "
         "is Monday morning (and not Friday of the week before)")
@@ -191,11 +194,16 @@ class HrHolidays(models.Model):
         ('morning', 'Morning'),
         ('noon', 'Noon'),
         ], string="Start of Vacation", track_visibility='onchange',
-        default='morning',
+        default='morning', readonly=True, states={
+            'draft': [('readonly', False)],
+            'confirm': [('readonly', False)]},
         help="For example, if you leave one full calendar week, "
         "the first day of vacation is Monday Morning")
     vacation_date_to = fields.Date(
         string='Last Day of Vacation', track_visibility='onchange',
+        readonly=True, states={
+            'draft': [('readonly', False)],
+            'confirm': [('readonly', False)]},
         help="Enter the last day of vacation. For example, if you "
         "leave one full calendar week, the last day of vacation is "
         "Friday evening (and not Monday of the week after)")
@@ -203,7 +211,9 @@ class HrHolidays(models.Model):
         ('noon', 'Noon'),
         ('evening', 'Evening'),
         ], string="End of Vacation", track_visibility='onchange',
-        default='evening',
+        default='evening', readonly=True, states={
+            'draft': [('readonly', False)],
+            'confirm': [('readonly', False)]},
         help="For example, if you leave one full calendar week, "
         "the end of vacation is Friday Evening")
     current_leaves_taken = fields.Float(
@@ -228,8 +238,13 @@ class HrHolidays(models.Model):
     # The 'name' field is displayed publicly in the calendar
     # So the label should not be 'Description' but 'Public Title'
     name = fields.Char(
-        string='Public Title', help="Warning: this title is shown publicly in the "
+        string='Public Title',
+        help="Warning: this title is shown publicly in the "
         "calendar. Don't write private/personnal information in this field.")
+    # by default, there is no company_id field on hr.holidays !
+    company_id = fields.Many2one(
+        related='employee_id.resource_id.company_id', store=True,
+        readonly=True)
 
     @api.one
     @api.constrains(
@@ -381,6 +396,19 @@ class HrHolidays(models.Model):
         res = super(HrHolidays, self).holidays_validate()
         return res
 
+    @api.multi
+    def holidays_refuse(self):
+        for holi in self:
+            if (
+                    holi.user_id == self.env.user and
+                    not self.pool['res.users'].has_group(
+                        self._cr, self._uid, 'base.group_hr_manager')):
+                raise UserError(_(
+                    "You cannot refuse your own Leave or Allocation "
+                    "holiday request '%s'.")
+                    % holi.name)
+        return super(HrHolidays, self).holidays_refuse()
+
 
 class ResCompany(models.Model):
     _inherit = 'res.company'

hr_holidays_usability/hr_holidays_mail.xml

@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="utf-8"?>
 <openerp>
-<data>
+<data noupdate="0">
 
 <record id="ir_actions_server_hr_holidays_confirm_mail" model="ir.actions.server">
     <field name="name">Auto-email confirmed leave</field>
@@ -59,6 +59,9 @@ if not object.no_email_notification:
     <field name="action_id" ref="ir_actions_server_hr_holidays_refuse_mail"/>
 </record>
 
+</data>
+<data noupdate="1">
+
 <record id="email_template_hr_holidays" model="email.template">
     <field name="name">Holidays email template</field>
     <field name="model_id" ref="hr_holidays.model_hr_holidays"/>

hr_holidays_usability/hr_holidays_view.xml

@@ -61,6 +61,7 @@ hr_holidays.edit_holiday_new is used for both leaves and allocation -->
             </group>
         </field>
         <field name="department_id" position="after">
+            <field name="company_id" groups="base.group_multi_company"/>
             <field name="posted_date" groups="base.group_hr_manager"/>
         </field>
     </field>
@@ -90,6 +91,7 @@ hr_holidays.edit_holiday_new is used for both leaves and allocation -->
         </field>
         <field name="holiday_status_id" position="after">
             <field name="posted_date" groups="base.group_hr_manager"/>
+            <field name="company_id" groups="base.group_multi_company"/>
         </field>
     </field>
 </record>

hr_holidays_usability/security/holiday_security.xml

@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="utf-8"?>
 <openerp>
-<data noupdate="0">
+<data noupdate="1">
 
 <!--
 Employee : only see his holidays
@@ -41,5 +41,12 @@ Manager = person that administrates the holidays process : can see everything, d
     <field name="groups" eval="[(4, ref('base.group_hr_manager'))]"/>
 </record>
 
+<record id="hr_holidays_multicompany_rule" model="ir.rule">
+    <field name="name">Holidays multi-company</field>
+    <field name="model_id" ref="hr_holidays.model_hr_holidays"/>
+    <field name="domain_force">['|', ('company_id', '=', False), ('company_id', 'child_of', [user.company_id.id])]</field>
+</record>
+
+
 </data>
 </openerp>

hr_holidays_usability/wizard/hr_holidays_mass_allocation.py

@@ -31,7 +31,9 @@ class HrHolidaysMassAllocation(models.TransientModel):
     @api.model
     def _default_employees(self):
         return self.env['hr.employee'].search([
-            ('holiday_exclude_mass_allocation', '=', False)])
+            ('holiday_exclude_mass_allocation', '=', False),
+            ('company_id', '=', self.env.user.company_id.id),
+            ])
 
     @api.model
     def _get_default_holiday_status(self):

hr_holidays_usability/wizard/hr_holidays_post.py

@@ -56,6 +56,7 @@ class HrHolidaysPost(models.TransientModel):
             ('state', '=', 'validate'),
             ('posted_date', '=', False),
             ('vacation_date_to', '<=', self.before_date),
+            ('company_id', '=', self.env.user.company_id.id),
             ])
         self.write({
             'state': 'done',
@@ -73,9 +74,11 @@ class HrHolidaysPost(models.TransientModel):
         # because, after the write, it doesn't have a value any more !!!
         holidays_to_post = self.holidays_to_post_ids
         today = fields.Date.context_today(self)
-        if not self.holidays_to_post_ids:
-            raise Warning(
-                _('No leave request to post.'))
+        # Disable the raise below to make the module "hr_holidays_lunch_voucher"
+        # work even when nobody took holidays on the current month
+        # if not self.holidays_to_post_ids:
+        #    raise Warning(
+        #        _('No leave request to post.'))
         self.holidays_to_post_ids.write({'posted_date': today})
         view_id = self.env.ref('hr_holidays_usability.hr_holiday_graph').id
         action = {

hr_holidays_usability/wizard/hr_holidays_post_view.xml

@@ -12,13 +12,13 @@
     <field name="name">hr_holidays_post_form</field>
     <field name="model">hr.holidays.post</field>
     <field name="arch" type="xml">
-        <form string="Post Leaves" version="7.0">
+        <form string="Post Leaves">
             <group name="main" string="Leave Requests to Post">
                 <field name="state" invisible="1"/>
                 <field name="before_date" states="draft"/>
                 <field name="holidays_to_post_ids" nolabel="1"
                     context="{'tree_view_ref': 'hr_holidays.view_holiday'}"
-                    states="done"/>
+                    states="done" colspan="2"/>
             </group>
             <footer>
                 <button name="select_date" type="object" string="Get Holiday Requests"

hr_usability/__init__.py

@@ -1,23 +1,3 @@
-# -*- encoding: utf-8 -*-
-##############################################################################
-#
-#    HR Usability module for Odoo
-#    Copyright (C) 2015 Akretion (http://www.akretion.com)
-#    @author Alexis de Lattre <alexis.delattre@akretion.com>
-#
-#    This program is free software: you can redistribute it and/or modify
-#    it under the terms of the GNU Affero General Public License as
-#    published by the Free Software Foundation, either version 3 of the
-#    License, or (at your option) any later version.
-#
-#    This program is distributed in the hope that it will be useful,
-#    but WITHOUT ANY WARRANTY; without even the implied warranty of
-#    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-#    GNU Affero General Public License for more details.
-#
-#    You should have received a copy of the GNU Affero General Public License
-#    along with this program.  If not, see <http://www.gnu.org/licenses/>.
-#
-##############################################################################
-
+# -*- coding: utf-8 -*-
 
+from . import hr

hr_usability/hr.py

@@ -0,0 +1,46 @@
+# -*- coding: utf-8 -*-
+##############################################################################
+#
+#    HR Usability module for Odoo
+#    Copyright (C) 2016 Akretion (http://www.akretion.com)
+#    @author Alexis de Lattre <alexis.delattre@akretion.com>
+#
+#    This program is free software: you can redistribute it and/or modify
+#    it under the terms of the GNU Affero General Public License as
+#    published by the Free Software Foundation, either version 3 of the
+#    License, or (at your option) any later version.
+#
+#    This program is distributed in the hope that it will be useful,
+#    but WITHOUT ANY WARRANTY; without even the implied warranty of
+#    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+#    GNU Affero General Public License for more details.
+#
+#    You should have received a copy of the GNU Affero General Public License
+#    along with this program.  If not, see <http://www.gnu.org/licenses/>.
+#
+##############################################################################
+
+from openerp import models, fields, api
+
+
+class HrEmployee(models.Model):
+    _inherit = 'hr.employee'
+
+    country_id = fields.Many2one(track_visibility='onchange')
+    birthday = fields.Date(track_visibility='onchange')
+    ssnid = fields.Char(track_visibility='onchange')
+    sinid = fields.Char(track_visibility='onchange')
+    identification_id = fields.Char(track_visibility='onchange')
+    otherid = fields.Char(track_visibility='onchange')
+    gender = fields.Selection(track_visibility='onchange')
+    marital = fields.Selection(track_visibility='onchange')
+    department_id = fields.Many2one(track_visibility='onchange')
+    address_id = fields.Many2one(track_visibility='onchange')
+    address_home_id = fields.Many2one(track_visibility='onchange')
+    work_phone = fields.Char(track_visibility='onchange')
+    mobile_phone = fields.Char(track_visibility='onchange')
+    work_email = fields.Char(track_visibility='onchange')
+    parent_id = fields.Many2one(track_visibility='onchange')
+    coach_id = fields.Many2one(track_visibility='onchange')
+    job_id = fields.Many2one(track_visibility='onchange')
+    passport_id = fields.Char(track_visibility='onchange')

l10n_fr_fix_thousands_sep/res_lang.py

@@ -28,10 +28,10 @@ class ResLang(models.Model):
 
     @api.v7
     def init(self, cr):
-        narrow_no_break_space = u'\u202f'
+        no_break_space = u'\u00A0'
         cr.execute(
             "UPDATE res_lang SET thousands_sep=%s, grouping='[3,0]' "
-            "WHERE code='fr_FR'", (narrow_no_break_space, ))
+            "WHERE code='fr_FR'", (no_break_space, ))
         cr.execute(
             "UPDATE res_lang SET grouping='[3,0]' WHERE code='en_US'")
         return True

mail_usability/__openerp__.py

@@ -23,7 +23,7 @@
 
 {
     'name': 'Mail Usability',
-    'version': '0.1',
+    'version': '8.0.1.0.0',
     'category': 'Base',
     'license': 'AGPL-3',
     'summary': 'Usability improvements on mails',
@@ -37,10 +37,11 @@ Small usability improvements on mails:
 
 * remove 'sent by' in notification footer
 
+* add a new entry *All Messages Except Notifications* to the field *Receive Inbox Notifications by Email* of partners (becomes the default value)
     """,
     'author': 'Akretion',
     'website': 'http://www.akretion.com',
     'depends': ['mail'],
-    'data': [],
+    'data': ['views/mail.xml'],
     'installable': True,
 }

mail_usability/mail.py

@@ -20,7 +20,16 @@
 #
 ##############################################################################
 
-from openerp import models, api
+from openerp import models, fields, api
+
+
+class ResPartner(models.Model):
+    _inherit = 'res.partner'
+
+    notify_email = fields.Selection(
+        selection_add=[
+            ('all_except_notification', 'All Messages Except Notifications')],
+        default='all_except_notification')
 
 
 class MailMail(models.Model):
@@ -47,6 +56,19 @@ class MailNotification(models.Model):
         footer = super(MailNotification, self).get_signature_footer(
             cr, uid, user_id, res_model=res_model, res_id=res_id,
             context=context, user_signature=user_signature)
-        footer = footer[:footer.find('\n\n<br /><small>Sent by ')]
-        footer = footer[:footer.find(u'\n\n<br /><small>Envoyé par ')]
+        footer = footer[:footer.find('\n<br /><small>Sent by ')]
+        footer = footer[:footer.find(u'\n<br /><small>Envoyé par ')]
         return footer
+
+    @api.multi
+    def get_partners_to_email(self, message):
+        notify_pids = super(MailNotification, self).get_partners_to_email(
+            message)
+        for notif in self:
+            if (
+                    notif.partner_id.id in notify_pids and
+                    message.type == 'notification' and
+                    notif.partner_id.notify_email ==
+                    'all_except_notification'):
+                notify_pids.remove(notif.partner_id.id)
+        return notify_pids

mail_usability/static/src/js/many2many_tags_email.js

@@ -0,0 +1,92 @@
+openerp_FieldMany2ManyTagsEmail = function(instance) {
+var _t = instance.web._t;
+
+/**
+ * Extend of FieldMany2ManyTags widget method.
+ * When the user add a partner and the partner don't have an email, open a popup to purpose to add an email.
+ * The user can choose to add an email or cancel and close the popup.
+ */
+instance.web.form.FieldMany2ManyTagsEmail = instance.web.form.FieldMany2ManyTags.extend({
+
+    start: function() {
+        this.mutex = new openerp.Mutex();
+
+        // This widget will indirectly trigger a change:value to it's parent widget
+        // when setting the value of valid partners. For this reason we have to keep an
+        // internal state of the last value in order to compute the effective value changes.
+        this.last_processed_value = [];
+
+        this.on("change:value", this, this.on_change_value_check);
+        // Restore pop-up when trying to send to a partner without email
+        this.on_change_value_check();
+        this._super.apply(this, arguments);
+    },
+
+    on_change_value_check : function () {
+        var self = this;
+        var values = this.get('value').slice(0);  // Clone the array
+
+        // We only validate partners emails in case the value is not empty
+        // and is different from the last processed value
+        var effective_change = _.difference(values, self.last_processed_value).length;
+        if (values.length && effective_change) {
+            this.mutex.exec(function() {
+                return self._check_email_popup(values);
+            });
+        }
+    },
+
+    _check_email_popup: function (ids) {
+        var self = this;
+        var valid_partners;
+
+        new instance.web.Model('res.partner').call("search", [[
+                ["id", "in", ids],
+                ["email", "=", false],
+                ["notify_email", "in", ['always', 'all_except_notification']]]],
+                {context: this.build_context()})
+            .then(function (record_ids) {
+                var popups_deferreds = [];
+                self.valid_partners = _.difference(ids, record_ids);
+
+                // Propose the user to correct invalid partners
+                _.each(record_ids, function (id) {
+                    var popup_def = $.Deferred();
+                    popups_deferreds.push(popup_def);
+
+                    var pop = new instance.web.form.FormOpenPopup(self);
+                    pop.show_element(
+                        'res.partner',
+                        id,
+                        self.build_context(),
+                        {
+                            title: _t("Please complete partner's informations and Email"),
+                        }
+                    );
+                    pop.on('write_completed', self, function () {
+                        self.valid_partners.push(id);
+                    });
+                    pop.on('closed', self, function () {
+                        popup_def.resolve();
+                    });
+                });
+                return $.when.apply($, popups_deferreds).then(function() {
+                    // All popups have been processed for the given ids
+                    // It is now time to set the final value with valid partners ids.
+                    var filtered_value = _.uniq(self.valid_partners);
+                    self.last_processed_value = filtered_value;
+                    self.set({'value': filtered_value});
+                });
+            });
+    },
+});
+
+
+/**
+ * Registry of form fields
+ */
+instance.web.form.widgets = instance.web.form.widgets.extend({
+    'many2many_tags_email' : 'instance.web.form.FieldMany2ManyTagsEmail',
+});
+
+};

mail_usability/views/mail.xml

@@ -0,0 +1,11 @@
+<?xml version="1.0" encoding="utf-8"?>
+
+<openerp>
+    <data>
+        <template id="assets_backend" name="mail usability assets" inherit_id="web.assets_backend">
+            <xpath expr="." position="inside">
+                <script type="text/javascript" src="/mail_usability/static/src/js/many2many_tags_email.js"></script>
+            </xpath>
+        </template>
+    </data>
+</openerp>

partner_aged_open_invoices/partner.py

@@ -40,31 +40,27 @@ class ResPartner(models.Model):
     def open_aged_open_invoices_report(self):
         aoiwo = self.env['aged.open.invoices.webkit']
         afo = self.env['account.fiscalyear']
-        # Force 'date_from' to the first day of the company's life
-        # in order to get the full picture of the partner's account
-        # (we could also do that via a filter by period, but, in this case,
-        # account_financial_report_webkit doesn't accept "until_date = today" ;
-        # until_date has to be the end date of the last period
+        # Filter by period (and not by date) to get
+        # the report à nouveau
         fy_years = afo.search([], order='date_start')
         date_from = fy_years[0].date_start
         fy_id = aoiwo._get_fiscalyear()
         filter_change = aoiwo.onchange_filter(
-            filter='filter_date', fiscalyear_id=fy_id)
+            filter='filter_period', fiscalyear_id=fy_id)
         vals = {
             'fiscalyear_id': fy_id,
-            'filter': 'filter_date',
+            'filter': 'filter_period',
             'partner_ids': [(6, 0, [self.commercial_partner_id.id])],
             'target_move': 'all',
             }
         vals.update(filter_change['value'])
-        vals['date_from'] = date_from
         wizard = aoiwo.create(vals)
         data = {'form': {
             'chart_account_id': wizard.chart_account_id.id,
             'filter': vals['filter'],
-            'date_from': vals['date_from'],
-            'date_to': vals['date_to'],
-            'period_to': False,
+            'period_from': vals['period_from'],
+            'period_to': vals['period_to'],
+            'date_to': False,
             'fiscalyear_id': vals['fiscalyear_id'],
             'partner_ids': vals['partner_ids'][0][2],
             'target_move': vals['target_move'],

partner_search/partner_view.xml

@@ -14,10 +14,20 @@
                 <field name="phone" string="Phones"
                        filter_domain="['|', ('phone', 'like', self), ('mobile', 'like', self)]"/>
             </field>
-            <!-- Display the real field names of the search -->
+
+            <!-- We split the search of name, ref / email in two
+                 as searching in email can return a lot of unwanted result
+                 for exemple when you sell on marketplace and search for a customer
+                 amazon you will have all customer that have bought throught the marketplace -->
+
             <field name="name" position="attributes">
-                <attribute name="string">Name, Ref, Email</attribute>
+                <attribute name="string">Name, Ref</attribute>
+                <attribute name="filter_domain">['|', ('display_name','ilike',self), ('ref','=',self)]</attribute>
             </field>
+            <field name="name" position="after">
+                <field name="email"/>
+            </field>
+
         </field>
     </record>
 

partner_split_menu/__init__.py

@@ -0,0 +1 @@
+# -*- coding: utf-8 -*-

partner_split_menu/__openerp__.py

@@ -0,0 +1,32 @@
+# -*- coding: utf-8 -*-
+# Copyright 2017 Akretion (http://www.akretion.com).
+# @author Sébastien BEAU <sebastien.beau@akretion.com>
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+
+
+{
+    "name": "Partner Menu Split",
+    "summary": "Split the customer menu in two",
+    "version": "8.0.0.0",
+    "category": "Uncategorized",
+    "website": "www.akretion.com",
+    "author": " Akretion",
+    "license": "AGPL-3",
+    "application": False,
+    "installable": True,
+    "external_dependencies": {
+        "python": [],
+        "bin": [],
+    },
+    "depends": [
+        "base",
+        "purchase",
+    ],
+    "data": [
+        "views/res_partner_view.xml",
+    ],
+    "demo": [
+    ],
+    "qweb": [
+    ]
+}

partner_split_menu/views/res_partner_view.xml

@@ -0,0 +1,81 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<openerp>
+    <data>
+
+<record id="base.action_partner_customer_form" model="ir.actions.act_window">
+    <field name="domain">[('parent_id', '=', False)]</field>
+    <field name="view_mode">tree,form,kanban</field>
+</record>
+
+<record id="base.action_partner_form" model="ir.actions.act_window">
+    <field name="domain">[('parent_id', '=', False)]</field>
+    <field name="view_mode">tree,form,kanban</field>
+</record>
+
+<record id="base.action_partner_form_view1" model="ir.actions.act_window.view">
+    <field eval="3" name="sequence"/>
+</record>
+
+<record id="action_partner_customer_contact_form" model="ir.actions.act_window">
+    <field name="name">Contact</field>
+    <field name="type">ir.actions.act_window</field>
+    <field name="res_model">res.partner</field>
+    <field name="view_type">form</field>
+    <field name="view_mode">tree,form,kanban</field>
+    <field name="domain">[('parent_id', '!=', False)]</field>
+    <field name="context">{'default_customer':1, 'search_default_customer':1}</field>
+    <field name="filter" eval="True"/>
+    <field name="help" type="html">
+      <p class="oe_view_nocontent_create">
+        Click to add a contact in your address book.
+      </p><p>
+        Odoo helps you easily track all activities related to
+        a customer: discussions, history of business opportunities,
+        documents, etc.
+      </p>
+    </field>
+</record>
+
+<menuitem
+    id="menu_partner_form"
+    parent="base.menu_sales"
+    action="action_partner_customer_contact_form"
+    sequence="2"/>
+
+
+<record id="base.action_partner_supplier_form" model="ir.actions.act_window">
+    <field name="domain">[('parent_id', '=', False)]</field>
+    <field name="view_mode">tree,form,kanban</field>
+</record>
+
+
+<record id="action_partner_supplier_contact_form" model="ir.actions.act_window">
+    <field name="name">Contact</field>
+    <field name="type">ir.actions.act_window</field>
+    <field name="res_model">res.partner</field>
+    <field name="view_type">form</field>
+    <field name="view_mode">tree,form,kanban</field>
+    <field name="domain">[('parent_id', '!=', False)]</field>
+    <field name="context">{'default_supplier':1, 'search_default_supplier':1}</field>
+    <field name="filter" eval="True"/>
+    <field name="help" type="html">
+      <p class="oe_view_nocontent_create">
+        Click to add a contact in your address book.
+      </p><p>
+        Odoo helps you easily track all activities related to
+        a supplier: discussions, history of business opportunities,
+        documents, etc.
+      </p>
+    </field>
+</record>
+
+<menuitem
+    id="menu_partner_supplier_form"
+    parent="purchase.menu_procurement_management"
+    action="action_partner_supplier_contact_form"
+    sequence="20"/>
+
+
+
+    </data>
+</openerp>

procurement_suggest/wizard/procurement_suggest.py

@@ -41,6 +41,7 @@ class ProcurementSuggestionGenerate(models.TransientModel):
     location_id = fields.Many2one(
         'stock.location', string='Stock Location', required=True,
         default=lambda self: self.env.ref('stock.stock_location_stock'))
+        # TODO This code must be fixed in case of multicompany project
 
     @api.model
     def _compute_procurement_qty(self, orderpoint):

product_expiry_simple/__init__.py

@@ -0,0 +1,3 @@
+# -*- encoding: utf-8 -*-
+
+from . import stock

product_expiry_simple/__openerp__.py

@@ -0,0 +1,34 @@
+# -*- coding: utf-8 -*-
+# © 2017 Akretion (Alexis de Lattre <alexis.delattre@akretion.com>)
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+
+{
+    'name': 'Product Expiry Simple',
+    'version': '8.0.1.0.0',
+    'category': 'Product',
+    'license': 'AGPL-3',
+    'summary': 'Simpler and better alternative to the official product_expiry module',
+    'description': """
+Product Expiry Simple
+=====================
+
+This module is similar to the official *product_expiry* modules, but much simpler and much better:
+
+* Only one expiry date field instead of 4 !
+* date field instead of datetime
+* No automatic computing of expiry date based on a delay configured on product (not needed in most companies)
+* colored production lot and stock quant tree views depending on expiry dates
+* ability to show stats about expiry dates on quants pivot table (thanks to related stored field on stock.quant)
+
+This module has been written by Alexis de Lattre from Akretion <alexis.delattre@akretion.com>.
+    """,
+    'author': 'Akretion',
+    'website': 'http://www.akretion.com',
+    'depends': ['stock'],
+    'conflicts': ['product_expiry'],
+    'data': [
+        'stock_view.xml',
+        'product_removal_data.xml',
+        ],
+    'installable': True,
+}

product_expiry_simple/i18n/fr.po

@@ -0,0 +1,53 @@
+# Translation of Odoo Server.
+# This file contains the translation of the following modules:
+#	* product_expiry_simple
+#
+msgid ""
+msgstr ""
+"Project-Id-Version: Odoo Server 8.0\n"
+"Report-Msgid-Bugs-To: \n"
+"POT-Creation-Date: 2017-02-14 13:19+0000\n"
+"PO-Revision-Date: 2017-02-14 13:19+0000\n"
+"Last-Translator: <>\n"
+"Language-Team: \n"
+"MIME-Version: 1.0\n"
+"Content-Type: text/plain; charset=UTF-8\n"
+"Content-Transfer-Encoding: \n"
+"Plural-Forms: \n"
+
+#. module: product_expiry_simple
+#: view:stock.production.lot:product_expiry_simple.search_product_lot_filter
+#: view:stock.quant:product_expiry_simple.quant_search_view
+msgid "Expired"
+msgstr "Périmé"
+
+#. module: product_expiry_simple
+#: view:stock.production.lot:product_expiry_simple.search_product_lot_filter
+#: field:stock.production.lot,expiry_date:0
+#: view:stock.quant:product_expiry_simple.quant_search_view
+#: field:stock.quant,expiry_date:0
+msgid "Expiry Date"
+msgstr "Date de péremption"
+
+#. module: product_expiry_simple
+#: model:ir.model,name:product_expiry_simple.model_stock_production_lot
+msgid "Lot/Serial"
+msgstr "Lot/N° série"
+
+#. module: product_expiry_simple
+#: view:stock.production.lot:product_expiry_simple.search_product_lot_filter
+#: view:stock.quant:product_expiry_simple.quant_search_view
+msgid "Not Expired"
+msgstr "Non périmé"
+
+#. module: product_expiry_simple
+#: model:ir.model,name:product_expiry_simple.model_stock_quant
+msgid "Quants"
+msgstr "Quants"
+
+#. module: product_expiry_simple
+#: view:stock.production.lot:product_expiry_simple.view_production_lot_tree
+#: view:stock.quant:product_expiry_simple.view_stock_quant_tree
+msgid "red:expiry_date and expiry_date < current_date;green:expiry_date >= current_date"
+msgstr "red:expiry_date and expiry_date < current_date;green:expiry_date >= current_date"
+

product_expiry_simple/product_removal_data.xml

@@ -0,0 +1,11 @@
+<?xml version="1.0" ?>
+<openerp>
+<data>
+
+<record id="removal_fefo" model="product.removal">
+    <field name="name">First Expiry First Out (FEFO)</field>
+    <field name="method">fefo</field>
+</record>
+
+</data>
+</openerp>

product_expiry_simple/stock.py

@@ -0,0 +1,43 @@
+# -*- coding: utf-8 -*-
+# © 2017 Akretion (Alexis de Lattre <alexis.delattre@akretion.com>)
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+
+from openerp import models, fields, api
+
+
+class StockProductionLot(models.Model):
+    _inherit = 'stock.production.lot'
+    _rec_name = 'display_name'
+
+    expiry_date = fields.Date(string='Expiry Date')
+    display_name = fields.Char(
+        compute='compute_display_name_field',
+        string='Lot/Serial Number Display', store=True, readonly=True)
+
+    @api.multi
+    @api.depends('name', 'expiry_date')
+    def compute_display_name_field(self):
+        for lot in self:
+            dname = lot.name
+            if lot.expiry_date:
+                dname = '[%s] %s' % (lot.expiry_date, dname)
+            lot.display_name = dname
+
+
+class StockQuant(models.Model):
+    _inherit = 'stock.quant'
+
+    expiry_date = fields.Date(
+        related='lot_id.expiry_date', store=True, readonly=True)
+
+    # method copy/pasted from the official product_expiry module
+    # © Odoo SA
+    @api.model
+    def apply_removal_strategy(
+            self, location, product, qty, domain, removal_strategy):
+        if removal_strategy == 'fefo':
+            order = 'expiry_date, location_id, package_id, lot_id, in_date, id'
+            return self._quants_get_order(
+                location, product, qty, domain, order)
+        return super(StockQuant, self).apply_removal_strategy(
+            location, product, qty, domain, removal_strategy)