[ADD] separator on filter view

I don't use stock_packaging_usability_pp from OCA/stock-logistics-tracking because the feature is native since v11 (I didn't figure it out when I ported stock_packaging_usability_ul from v10 to v14 under the new name stock_packaging_usability_ul
I also improved the code to clean it up and use the native field for measured weight

account_bank_reconciliation_summary_xlsx/README.rst

@@ -0,0 +1,27 @@
+===============================
+Bank Reconciliation Report XLSX
+===============================
+
+In Odoo v13+, a bank reconciliation report is not really needed because all the payments executed that are not debited/credited on the bank account are in separate waiting accounts. But accountants want a bank reconciliation report, so this module adds one, even if it is quite different from a classic bank reconciliation report.
+
+Configuration
+=============
+
+This module doesn't require any configuration.
+
+Usage
+=====
+
+You can launch the Bank Reconciliation Report wizard from:
+
+* the menu *Accounting > Reports > Bank > Bank Reconciliation*,
+* the invoicing dashboard: on a bank journal, click on the options, then select *Bank Reconciliation*.
+* the form view of a bank statement: click on the button *Bank Reconciliation Report*.
+
+Credits
+=======
+
+Contributors
+------------
+
+* Alexis de Lattre <alexis.delattre@akretion.com>

account_bank_reconciliation_summary_xlsx/__init__.py

@@ -1 +1,2 @@
+from . import report
 from . import wizard

account_bank_reconciliation_summary_xlsx/__manifest__.py

@@ -0,0 +1,21 @@
+# Copyright 2017-2023 Akretion France (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": "Bank Reconciliation Report",
+    "version": "14.0.1.0.0",
+    "license": "AGPL-3",
+    "author": "Akretion",
+    "website": "https://github.com/akretion/odoo-usability",
+    "summary": "Bank reconciliation XLSX report",
+    "depends": ["account", "report_xlsx"],
+    "data": [
+        "report/report.xml",
+        "wizard/bank_reconciliation_report_wizard_view.xml",
+        "views/account_bank_statement.xml",
+        "views/account_journal.xml",
+        "security/ir.model.access.csv",
+    ],
+    "installable": True,
+}

account_bank_reconciliation_summary_xlsx/i18n/fr.po

@@ -0,0 +1,213 @@
+# Translation of Odoo Server.
+# This file contains the translation of the following modules:
+# 	* account_bank_reconciliation_summary_xlsx
+#
+msgid ""
+msgstr ""
+"Project-Id-Version: Odoo Server 14.0\n"
+"Report-Msgid-Bugs-To: \n"
+"POT-Creation-Date: 2023-01-13 10:31+0000\n"
+"PO-Revision-Date: 2023-01-13 10:31+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: account_bank_reconciliation_summary_xlsx
+#: code:addons/account_bank_reconciliation_summary_xlsx/report/bank_reconciliation_xlsx.py:0
+#, python-format
+msgid "Amount"
+msgstr "Montant"
+
+#. module: account_bank_reconciliation_summary_xlsx
+#: code:addons/account_bank_reconciliation_summary_xlsx/report/bank_reconciliation_xlsx.py:0
+#, python-format
+msgid "Balance %s:"
+msgstr "Solde %s :"
+
+#. module: account_bank_reconciliation_summary_xlsx
+#: model:ir.model.fields,field_description:account_bank_reconciliation_summary_xlsx.field_bank_reconciliation_report_wizard__journal_ids
+msgid "Bank Journals"
+msgstr "Journaux de banque"
+
+#. module: account_bank_reconciliation_summary_xlsx
+#: model:ir.actions.act_window,name:account_bank_reconciliation_summary_xlsx.bank_reconciliation_report_wizard_action
+#: model:ir.ui.menu,name:account_bank_reconciliation_summary_xlsx.bank_reconciliation_report_wizard_menu
+#: model_terms:ir.ui.view,arch_db:account_bank_reconciliation_summary_xlsx.account_journal_dashboard_kanban_view
+msgid "Bank Reconciliation"
+msgstr "Rapprochement bancaire"
+
+#. module: account_bank_reconciliation_summary_xlsx
+#: code:addons/account_bank_reconciliation_summary_xlsx/report/bank_reconciliation_xlsx.py:0
+#: model_terms:ir.ui.view,arch_db:account_bank_reconciliation_summary_xlsx.view_bank_statement_form
+#, python-format
+msgid "Bank Reconciliation Report"
+msgstr "Rapport de rapprochement bancaire"
+
+#. module: account_bank_reconciliation_summary_xlsx
+#: model:ir.model,name:account_bank_reconciliation_summary_xlsx.model_bank_reconciliation_report_wizard
+msgid "Bank Reconciliation Report Wizard"
+msgstr "Assistant rapport de rapprochement bancaire"
+
+#. module: account_bank_reconciliation_summary_xlsx
+#: model:ir.actions.report,name:account_bank_reconciliation_summary_xlsx.bank_reconciliation_xlsx
+msgid "Bank Reconciliation XLSX"
+msgstr "Rapprochement bancaire XLSX"
+
+#. module: account_bank_reconciliation_summary_xlsx
+#: model:ir.model,name:account_bank_reconciliation_summary_xlsx.model_report_bank_reconciliation_xlsx
+msgid "Bank Reconciliation XLSX Report"
+msgstr "Rapport de rapprochement bancaire XLSX"
+
+#. module: account_bank_reconciliation_summary_xlsx
+#: model:ir.ui.menu,name:account_bank_reconciliation_summary_xlsx.menu_report_bank_root
+msgid "Bank Reports"
+msgstr "Rapports bancaires"
+
+#. module: account_bank_reconciliation_summary_xlsx
+#: model_terms:ir.ui.view,arch_db:account_bank_reconciliation_summary_xlsx.bank_reconciliation_report_wizard_form
+msgid "Cancel"
+msgstr "Annuler"
+
+#. module: account_bank_reconciliation_summary_xlsx
+#: code:addons/account_bank_reconciliation_summary_xlsx/report/bank_reconciliation_xlsx.py:0
+#: model:ir.model.fields,field_description:account_bank_reconciliation_summary_xlsx.field_bank_reconciliation_report_wizard__company_id
+#, python-format
+msgid "Company"
+msgstr "Société"
+
+#. module: account_bank_reconciliation_summary_xlsx
+#: code:addons/account_bank_reconciliation_summary_xlsx/report/bank_reconciliation_xlsx.py:0
+#, python-format
+msgid "Counter-part"
+msgstr "Contre partie"
+
+#. module: account_bank_reconciliation_summary_xlsx
+#: model:ir.model.fields,field_description:account_bank_reconciliation_summary_xlsx.field_bank_reconciliation_report_wizard__create_uid
+msgid "Created by"
+msgstr "Créé par"
+
+#. module: account_bank_reconciliation_summary_xlsx
+#: model:ir.model.fields,field_description:account_bank_reconciliation_summary_xlsx.field_bank_reconciliation_report_wizard__create_date
+msgid "Created on"
+msgstr "Créé le"
+
+#. module: account_bank_reconciliation_summary_xlsx
+#: code:addons/account_bank_reconciliation_summary_xlsx/report/bank_reconciliation_xlsx.py:0
+#: code:addons/account_bank_reconciliation_summary_xlsx/report/bank_reconciliation_xlsx.py:0
+#: model:ir.model.fields,field_description:account_bank_reconciliation_summary_xlsx.field_bank_reconciliation_report_wizard__date
+#, python-format
+msgid "Date"
+msgstr ""
+
+#. module: account_bank_reconciliation_summary_xlsx
+#: model:ir.model.fields,field_description:account_bank_reconciliation_summary_xlsx.field_bank_reconciliation_report_wizard__display_name
+#: model:ir.model.fields,field_description:account_bank_reconciliation_summary_xlsx.field_report_bank_reconciliation_xlsx__display_name
+msgid "Display Name"
+msgstr ""
+
+#. module: account_bank_reconciliation_summary_xlsx
+#: model:ir.model.fields.selection,name:account_bank_reconciliation_summary_xlsx.selection__bank_reconciliation_report_wizard__move_state__draft_posted
+msgid "Draft and Posted Entries"
+msgstr "Écritures brouillon et comptabilisées"
+
+#. module: account_bank_reconciliation_summary_xlsx
+#: code:addons/account_bank_reconciliation_summary_xlsx/report/bank_reconciliation_xlsx.py:0
+#: model:ir.model.fields,field_description:account_bank_reconciliation_summary_xlsx.field_bank_reconciliation_report_wizard__move_state
+#, python-format
+msgid "Entries"
+msgstr "Écritures"
+
+#. module: account_bank_reconciliation_summary_xlsx
+#: model_terms:ir.ui.view,arch_db:account_bank_reconciliation_summary_xlsx.bank_reconciliation_report_wizard_form
+msgid "Export XLSX"
+msgstr ""
+
+#. module: account_bank_reconciliation_summary_xlsx
+#: code:addons/account_bank_reconciliation_summary_xlsx/report/bank_reconciliation_xlsx.py:0
+#, python-format
+msgid "Generated on %s"
+msgstr "Généré le %s"
+
+#. module: account_bank_reconciliation_summary_xlsx
+#: model:ir.model.fields,field_description:account_bank_reconciliation_summary_xlsx.field_bank_reconciliation_report_wizard__id
+#: model:ir.model.fields,field_description:account_bank_reconciliation_summary_xlsx.field_report_bank_reconciliation_xlsx__id
+msgid "ID"
+msgstr ""
+
+#. module: account_bank_reconciliation_summary_xlsx
+#: code:addons/account_bank_reconciliation_summary_xlsx/report/bank_reconciliation_xlsx.py:0
+#, python-format
+msgid "Journal"
+msgstr ""
+
+#. module: account_bank_reconciliation_summary_xlsx
+#: code:addons/account_bank_reconciliation_summary_xlsx/report/bank_reconciliation_xlsx.py:0
+#, python-format
+msgid "Label"
+msgstr "Libellé"
+
+#. module: account_bank_reconciliation_summary_xlsx
+#: model:ir.model.fields,field_description:account_bank_reconciliation_summary_xlsx.field_bank_reconciliation_report_wizard____last_update
+#: model:ir.model.fields,field_description:account_bank_reconciliation_summary_xlsx.field_report_bank_reconciliation_xlsx____last_update
+msgid "Last Modified on"
+msgstr ""
+
+#. module: account_bank_reconciliation_summary_xlsx
+#: model:ir.model.fields,field_description:account_bank_reconciliation_summary_xlsx.field_bank_reconciliation_report_wizard__write_uid
+msgid "Last Updated by"
+msgstr ""
+
+#. module: account_bank_reconciliation_summary_xlsx
+#: model:ir.model.fields,field_description:account_bank_reconciliation_summary_xlsx.field_bank_reconciliation_report_wizard__write_date
+msgid "Last Updated on"
+msgstr ""
+
+#. module: account_bank_reconciliation_summary_xlsx
+#: code:addons/account_bank_reconciliation_summary_xlsx/report/bank_reconciliation_xlsx.py:0
+#, python-format
+msgid "Move Number"
+msgstr "Numéro de pièce"
+
+#. module: account_bank_reconciliation_summary_xlsx
+#: code:addons/account_bank_reconciliation_summary_xlsx/report/bank_reconciliation_xlsx.py:0
+#, python-format
+msgid "No bank journal selected."
+msgstr "Aucun journal de banque sélectionné."
+
+#. module: account_bank_reconciliation_summary_xlsx
+#: code:addons/account_bank_reconciliation_summary_xlsx/report/bank_reconciliation_xlsx.py:0
+#, python-format
+msgid "None"
+msgstr "Aucun"
+
+#. module: account_bank_reconciliation_summary_xlsx
+#: code:addons/account_bank_reconciliation_summary_xlsx/report/bank_reconciliation_xlsx.py:0
+#, python-format
+msgid "Partner"
+msgstr "Partenaire"
+
+#. module: account_bank_reconciliation_summary_xlsx
+#: model:ir.model.fields.selection,name:account_bank_reconciliation_summary_xlsx.selection__bank_reconciliation_report_wizard__move_state__posted
+msgid "Posted Entries"
+msgstr "Écritures comptabilisées"
+
+#. module: account_bank_reconciliation_summary_xlsx
+#: code:addons/account_bank_reconciliation_summary_xlsx/report/bank_reconciliation_xlsx.py:0
+#, python-format
+msgid "Ref."
+msgstr "Réf."
+
+#. module: account_bank_reconciliation_summary_xlsx
+#: code:addons/account_bank_reconciliation_summary_xlsx/report/bank_reconciliation_xlsx.py:0
+#, python-format
+msgid "Sub-total:"
+msgstr "Sous-total :"
+
+#. module: account_bank_reconciliation_summary_xlsx
+#: code:addons/account_bank_reconciliation_summary_xlsx/report/bank_reconciliation_xlsx.py:0
+#, python-format
+msgid "TOTAL:"
+msgstr "TOTAL :"

account_bank_reconciliation_summary_xlsx/report/__init__.py

@@ -0,0 +1 @@
+from . import bank_reconciliation_xlsx

account_bank_reconciliation_summary_xlsx/report/bank_reconciliation_xlsx.py

@@ -0,0 +1,296 @@
+# Copyright 2017-2023 Akretion France (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 odoo import _, models
+from odoo.exceptions import UserError
+from datetime import datetime
+from odoo.tools.misc import format_datetime
+import pytz
+
+
+class BankReconciliationXlsx(models.AbstractModel):
+    _name = "report.bank.reconciliation.xlsx"
+    _description = "Bank Reconciliation XLSX Report"
+    _inherit = "report.report_xlsx.abstract"
+
+    def _domain_add_move_state(self, wizard, domain):
+        if wizard.move_state == 'posted':
+            domain.append(('parent_state', '=', 'posted'))
+        elif wizard.move_state == 'draft_posted':
+            domain.append(('parent_state', 'in', ('draft', 'posted')))
+
+    def _get_account_balance(self, account, wizard):
+        domain = [
+            ('account_id', '=', account.id),
+            ('date', '<=', wizard.date),
+            ('company_id', '=', wizard.company_id.id),
+            ]
+        self._domain_add_move_state(wizard, domain)
+        res_rg = self.env['account.move.line'].read_group(domain, ['balance:sum'], [])
+        account_bal = res_rg and res_rg[0].get('balance', 0.0) or 0.0
+        return account_bal
+
+    def _prepare_payment_move_lines(self, journal, account, wizard, unreconciled_only=True):
+        domain = [
+            ("company_id", "=", wizard.company_id.id),
+            ("account_id", "=", account.id),
+            ("journal_id", "=", journal.id),
+            ("date", "<=", wizard.date),
+            ]
+        if unreconciled_only:
+            limit_datetime_naive = datetime.combine(wizard.date, datetime.max.time())
+            tz = pytz.timezone(self.env.user.tz)
+            limit_datetime_aware = tz.localize(limit_datetime_naive)
+            limit_datetime_utc = limit_datetime_aware.astimezone(pytz.utc)
+            limit_datetime = limit_datetime_utc.replace(tzinfo=None)
+            domain += [
+                '|', ('full_reconcile_id', '=', False),
+                ('full_reconcile_id.create_date', '>', limit_datetime)]
+        self._domain_add_move_state(wizard, domain)
+        mlines = self.env["account.move.line"].search(domain)
+        res = []
+        for mline in mlines:
+            move = mline.move_id
+            cpart = []
+            for line in move.line_ids:
+                if (
+                    line.account_id != account
+                    and line.account_id.code not in cpart
+                ):
+                    cpart.append(line.account_id.code)
+            counterpart = " ,".join(cpart)
+            res.append(
+                {
+                    "date": mline.date,
+                    "ref": move.ref or "",
+                    "label": mline.name,
+                    "partner": mline.partner_id.display_name or "",
+                    "amount": mline.balance,
+                    "move_name": move.name,
+                    "counterpart": counterpart,
+                }
+            )
+        return res
+
+    def _write_move_lines_block(self, jdi, row, account, add2total=True):
+        sheet = jdi['sheet']
+        style = jdi['style']
+        style_suffix = not add2total and '_warn' or ''
+        subtotal = 0.0
+        mlines = self._prepare_payment_move_lines(jdi['journal'], account, jdi['wizard'])
+        if mlines or add2total:
+            sheet.write(row, 0, '%s  %s' % (account.name, account.code), style['title' + style_suffix])
+            sheet.write(row, 1, "", style['title' + style_suffix])
+
+        if not mlines:
+            if add2total:
+                sheet.write(row, 2, _("None"), style['none'])
+            else:
+                return
+        else:
+            row += 1
+            col_labels = [
+                _("Date"),
+                _("Partner"),
+                _("Amount"),
+                _("Move Number"),
+                _("Counter-part"),
+                _("Ref."),
+                _("Label"),
+            ]
+            col = 0
+            for col_label in col_labels:
+                sheet.write(row, col, col_label, style['col_header'])
+                col += 1
+            row += 1
+            start_line = row + 1
+            for mline in mlines:
+                sheet.write(row, 0, mline["date"], style['regular_date'])
+                sheet.write(row, 1, mline["partner"], style['regular'])
+                sheet.write(row, 2, mline["amount"], style['currency'])
+                sheet.write(row, 3, mline["move_name"], style['regular'])
+                sheet.write(row, 4, mline["counterpart"], style['regular'])
+                sheet.write(row, 5, mline["ref"], style['regular'])
+                sheet.write(row, 6, mline["label"], style['regular'])
+                subtotal += mline["amount"]
+                row += 1
+            end_line = row
+
+            for col in range(1):
+                sheet.write(row, col, "", style['title' + style_suffix])
+            sheet.write(row, 1, _("Sub-total:") + ' ', style['title_right' + style_suffix])
+
+            formula = '=SUM(%s%d:%s%d)' % (
+                jdi['total_col'], start_line, jdi['total_col'], end_line)
+            sheet.write_formula(row, 2, formula, style['currency_bg' + style_suffix], subtotal)
+            if add2total:
+                jdi['total'] += subtotal
+                jdi['total_formula'] += '+%s%d' % (jdi['total_col'], row + 1)
+        return row
+
+    def generate_xlsx_report(self, workbook, data, wizard):
+        if not wizard.journal_ids:
+            raise UserError(_("No bank journal selected."))
+        date_dt = wizard.date
+        company = wizard.company_id
+        style = self._get_style(workbook, company)
+        move_state_label = dict(
+            wizard.fields_get('move_state', 'selection')['move_state']['selection'])
+        generated_on_label = _('Generated on %s') % format_datetime(
+            self.env, datetime.utcnow())
+        for journal in wizard.journal_ids:
+            row = 0
+            sheet = workbook.add_worksheet(journal.code or journal.name)
+            jdi = {
+                'wizard': wizard,
+                'journal': journal,
+                'style': style,
+                'sheet': sheet,
+                'total': 0.0,
+                'total_formula': '=',
+                'total_col': 'C',
+                }
+            sheet.write(
+                row,
+                0,
+                _("Bank Reconciliation Report"),
+                style['doc_title'],
+            )
+            row += 1
+            sheet.write(row, 0, generated_on_label, style['small'])
+            sheet.set_row(0, 26)
+            sheet.set_column(0, 0, 10)
+            sheet.set_column(1, 1, 35)
+            sheet.set_column(2, 2, 15)
+            sheet.set_column(3, 3, 15)
+            sheet.set_column(4, 4, 25)
+            sheet.set_column(5, 5, 30)
+            sheet.set_column(6, 6, 60)
+            row += 3
+            sheet.write(row, 0, _("Company"), style['wizard_field'])
+            sheet.write(row, 1, wizard.company_id.display_name, style['wizard_value'])
+            row += 1
+            sheet.write(row, 0, _("Date"), style['wizard_field'])
+            sheet.write(row, 1, date_dt, style['wizard_value_date'])
+            row += 1
+            sheet.write(row, 0, _("Journal"), style['wizard_field'])
+            sheet.write(row, 1, journal.display_name, style['wizard_value'])
+            row += 1
+            sheet.write(row, 0, _("Entries"), style['wizard_field'])
+            sheet.write(row, 1, move_state_label[wizard.move_state], style['wizard_value'])
+
+            # 1) Show balance of bank account
+            row += 3
+            bank_account = journal.default_account_id
+            for col in range(1):
+                sheet.write(row, col, "", style['title'])
+            sheet.write(row, 1, _("Balance %s:") % bank_account.code + ' ', style['title_right'])
+            account_bal = self._get_account_balance(bank_account, wizard)
+
+            sheet.write(row, 2, account_bal, style['currency_bg'])
+            jdi['total'] += account_bal
+            jdi['total_formula'] += '%s%d' % (jdi['total_col'], row + 1)
+
+            row += 2
+            # 2) Show payment lines IN (debit)
+            debit_account = journal.payment_debit_account_id
+            row = self._write_move_lines_block(jdi, row, debit_account)
+            row += 2
+            # 3) Show payment lines OUT (credit)
+            credit_account = journal.payment_credit_account_id
+            row = self._write_move_lines_block(jdi, row, credit_account)
+            row += 2
+
+            for col in range(1):
+                sheet.write(row, col, "", style['title'])
+            sheet.write(row, 1, _("TOTAL:") + ' ', style['title_right'])
+            sheet.write_formula(
+                row, 2, jdi['total_formula'], style['currency_bg'], jdi['total'])
+            row += 3
+
+            # 4) Show suspense account lines
+            row = self._write_move_lines_block(
+                jdi, row, journal.suspense_account_id, add2total=False)
+
+    def _get_style(self, workbook, company):
+        style = {}
+        font_size = 10
+        light_grey = "#eeeeee"
+        title_blue = "#e6e6fa"
+        subtotal_orange = "#ffcc00"
+        title_warn = "#ff9999"
+        subtotal_warn = "#ffff99"
+        light_purple = "#ffdeff"
+        lang_code = self.env.user.lang
+        lang = False
+        if lang_code:
+            lang = self.env["res.lang"].search([("code", "=", lang_code)])
+        if not lang:
+            lang = self.env["res.lang"].search([], limit=1)
+        xls_date_format = (
+            lang.date_format.replace("%Y", "yyyy")
+            .replace("%m", "mm")
+            .replace("%d", "dd")
+            .replace("%y", "yy")
+        )
+
+        style['doc_title'] = workbook.add_format(
+            {"bold": True, "font_size": font_size + 4})
+        style['small'] = workbook.add_format({"font_size": font_size - 3})
+        style['col_header'] = workbook.add_format(
+            {
+                "bold": True,
+                "bg_color": light_grey,
+                "text_wrap": True,
+                "font_size": font_size,
+                "align": "center",
+            }
+        )
+        title_style = {
+            "bold": True,
+            "bg_color": title_blue,
+            "font_size": font_size,
+            "align": "left",
+            }
+        style['title_right'] = workbook.add_format(dict(title_style, align="right"))
+        style['title'] = workbook.add_format(dict(title_style))
+        style['wizard_field'] = workbook.add_format(dict(title_style, bg_color=light_grey))
+        wizard_value_style = {
+                "bg_color": light_purple,
+                "bold": True,
+                "font_size": font_size,
+                "align": "left",
+            }
+
+        style['wizard_value'] = workbook.add_format(wizard_value_style)
+        style['wizard_value_date'] = workbook.add_format(
+            dict(wizard_value_style, num_format=xls_date_format))
+        style['none'] = workbook.add_format(
+            {"bold": True, "font_size": font_size, "align": "right", "bg_color": subtotal_orange}
+        )
+        # WARN for suspense account
+        style['title_warn'] = workbook.add_format(
+            dict(title_style, align="left", bg_color=title_warn))
+        style['title_right_warn'] = workbook.add_format(
+            dict(title_style, align="right", bg_color=title_warn))
+        style['regular'] = workbook.add_format({"font_size": font_size})
+        if "%" in xls_date_format:
+            # fallback
+            xls_date_format = "yyyy-mm-dd"
+        style['regular_date'] = workbook.add_format(
+            {"num_format": xls_date_format, "font_size": font_size, "align": "left"}
+        )
+        cur_format = "#,##0.00 %s" % (
+            company.currency_id.symbol or company.currency_id.name
+        )
+        # It seems that Excel replaces automatically the decimal
+        # and thousand separator by those of the language under which
+        # Excel runs
+        currency_style = {"num_format": cur_format, "font_size": font_size}
+        style['currency'] = workbook.add_format(currency_style)
+        style['currency_bg'] = workbook.add_format(
+            dict(currency_style, bg_color=subtotal_orange))
+        style['currency_bg_warn'] = workbook.add_format(
+            dict(currency_style, bg_color=subtotal_warn))
+        return style

account_bank_reconciliation_summary_xlsx/report/report.xml

@@ -0,0 +1,18 @@
+<?xml version="1.0" encoding="utf-8" ?>
+<!--
+  Copyright 2017-2023 Akretion France (http://www.akretion.com/)
+  @author: Alexis de Lattre <alexis.delattre@akretion.com>
+  License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+-->
+<odoo>
+
+<record id="bank_reconciliation_xlsx" model="ir.actions.report">
+    <field name="name">Bank Reconciliation XLSX</field>
+    <field name="model">bank.reconciliation.report.wizard</field>
+    <field name="report_type">xlsx</field>
+    <field name="report_name">bank.reconciliation.xlsx</field>
+    <field name="report_file">bank.reconciliation.xlsx</field>
+    <!-- print_report_name doesn't work here... -->
+    <field name="print_report_name">'bank_reconciliation-%s' % (object.date)</field>
+</record>
+</odoo>

account_bank_reconciliation_summary_xlsx/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_bank_reconciliation_report_wizard_user,Full access on bank.reconciliation.report.wizard,model_bank_reconciliation_report_wizard,account.group_account_user,1,1,1,1
+access_bank_reconciliation_report_wizard_readonly,Full access on bank.reconciliation.report.wizard,model_bank_reconciliation_report_wizard,account.group_account_readonly,1,1,1,1

account_bank_reconciliation_summary_xlsx/views/account_bank_statement.xml

@@ -0,0 +1,23 @@
+<?xml version="1.0" encoding="utf-8" ?>
+<!--
+  Copyright 2017-2020 Akretion France (http://www.akretion.com/)
+  @author: Alexis de Lattre <alexis.delattre@akretion.com>
+  License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+-->
+<odoo>
+    <record id="view_bank_statement_form" model="ir.ui.view">
+        <field name="name">bank_rec_summary.account.bank.statement.form</field>
+        <field name="model">account.bank.statement</field>
+        <field name="inherit_id" ref="account.view_bank_statement_form" />
+        <field name="arch" type="xml">
+            <button name="button_reprocess" position="after">
+                <button
+                    name="%(bank_reconciliation_report_wizard_action)d"
+                    type="action"
+                    string="Bank Reconciliation Report"
+                    context="{'default_journal_ids': [journal_id]}"
+                />
+            </button>
+        </field>
+    </record>
+</odoo>

account_bank_reconciliation_summary_xlsx/views/account_journal.xml

@@ -0,0 +1,28 @@
+<?xml version="1.0" encoding="utf-8" ?>
+<!--
+  Copyright 2018-2023 Akretion France (http://www.akretion.com/)
+  @author: Alexis de Lattre <alexis.delattre@akretion.com>
+  License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+-->
+<odoo>
+    <!-- Accounting Dashboard -->
+    <record id="account_journal_dashboard_kanban_view" model="ir.ui.view">
+        <field
+            name="name"
+        >bank_reconciliation_summarry.account_journal_dashboard</field>
+        <field name="model">account.journal</field>
+        <field name="inherit_id" ref="account.account_journal_dashboard_kanban_view" />
+        <field name="arch" type="xml">
+            <xpath expr="//a[@name='open_collect_money']/.." position="before">
+                <div name="bank_reconciliation_report">
+                    <a
+                        role="menuitem"
+                        type="action"
+                        name="%(bank_reconciliation_report_wizard_action)d"
+                        context="{'default_journal_ids': [active_id]}"
+                        >Bank Reconciliation</a>
+                </div>
+            </xpath>
+        </field>
+    </record>
+</odoo>

account_bank_reconciliation_summary_xlsx/wizard/__init__.py

@@ -0,0 +1 @@
+from . import bank_reconciliation_report_wizard

account_bank_reconciliation_summary_xlsx/wizard/bank_reconciliation_report_wizard.py

@@ -0,0 +1,42 @@
+# Copyright 2017-2023 Akretion France (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 odoo import api, fields, models
+
+
+class BankReconciliationReportWizard(models.TransientModel):
+    _name = "bank.reconciliation.report.wizard"
+    _description = "Bank Reconciliation Report Wizard"
+    _check_company_auto = True
+
+    company_id = fields.Many2one(
+        'res.company', string='Company',
+        ondelete='cascade', required=True,
+        default=lambda self: self.env.company)
+    date = fields.Date(required=True, default=fields.Date.context_today)
+    move_state = fields.Selection(
+        [("posted", "Posted Entries"), ("draft_posted", "Draft and Posted Entries")],
+        string="Entries",
+        required=True,
+        default="posted",
+    )
+    journal_ids = fields.Many2many(
+        "account.journal",
+        string="Bank Journals",
+        domain="[('type', '=', 'bank'), ('company_id', '=', company_id)]",
+        required=True,
+        check_company=True,
+        default=lambda self: self._default_journal_ids(),
+    )
+
+    @api.model
+    def _default_journal_ids(self):
+        journals = self.env["account.journal"].search(
+            [
+                ("type", "=", "bank"),
+                ("bank_account_id", "!=", False),
+                ("company_id", "=", self.env.company.id),
+            ]
+        )
+        return journals

account_bank_reconciliation_summary_xlsx/wizard/bank_reconciliation_report_wizard_view.xml

@@ -0,0 +1,49 @@
+<?xml version="1.0" encoding="utf-8" ?>
+<!--
+  Copyright 2017-2023 Akretion France (http://www.akretion.com/)
+  @author: Alexis de Lattre <alexis.delattre@akretion.com>
+  License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+-->
+<odoo>
+    <record id="bank_reconciliation_report_wizard_form" model="ir.ui.view">
+        <field name="name">bank.reconciliation.report.wizard.form</field>
+        <field name="model">bank.reconciliation.report.wizard</field>
+        <field name="arch" type="xml">
+            <form>
+                <group name="main">
+                    <field name="company_id" invisible="1" />
+                    <field name="date" />
+                    <field name="journal_ids" widget="many2many_tags" options="{'no_open': True, 'no_create': True}"/>
+                    <field name="move_state" widget="radio"/>
+                </group>
+                <footer>
+                    <button
+                        name="%(account_bank_reconciliation_summary_xlsx.bank_reconciliation_xlsx)d"
+                        string="Export XLSX"
+                        type="action"
+                        class="btn-primary"
+                    />
+                    <button special="cancel" string="Cancel" />
+                </footer>
+            </form>
+        </field>
+    </record>
+    <record id="bank_reconciliation_report_wizard_action" model="ir.actions.act_window">
+        <field name="name">Bank Reconciliation</field>
+        <field name="res_model">bank.reconciliation.report.wizard</field>
+        <field name="view_mode">form</field>
+        <field name="target">new</field>
+    </record>
+    <menuitem
+        id="menu_report_bank_root"
+        parent="account.menu_finance_reports"
+        name="Bank Reports"
+        sequence="12"
+    />
+    <menuitem
+        id="bank_reconciliation_report_wizard_menu"
+        action="bank_reconciliation_report_wizard_action"
+        parent="menu_report_bank_root"
+        sequence="10"
+    />
+</odoo>

account_invoice_margin/__init__.py

@@ -1,2 +1 @@
-from . import account_invoice
-from . import account_invoice_report
+from . import models

account_invoice_margin/__manifest__.py

@@ -4,7 +4,7 @@
 
 {
     'name': 'Account Invoice Margin',
-    'version': '12.0.1.0.0',
+    'version': '14.0.1.0.0',
     'category': 'Invoicing Management',
     'license': 'AGPL-3',
     'summary': 'Copy standard price on invoice line and compute margins',
@@ -15,10 +15,10 @@ This module has been written by Alexis de Lattre from Akretion
 <alexis.delattre@akretion.com>.
     """,
     'author': 'Akretion',
-    'website': 'http://www.akretion.com',
+    'website': 'https://github.com/akretion/odoo-usability',
     'depends': ['account'],
     'data': [
-        'account_invoice_view.xml',
+        'views/account_move.xml',
     ],
-    'installable': False,
+    'installable': True,
 }

account_invoice_margin/account_invoice.py

@@ -1,152 +0,0 @@
-# Copyright 2015-2019 Akretion France (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 odoo import api, fields, models
-import odoo.addons.decimal_precision as dp
-
-
-class AccountInvoiceLine(models.Model):
-    _inherit = 'account.invoice.line'
-
-    standard_price_company_currency = fields.Float(
-        string='Cost Price in Company Currency', readonly=True,
-        digits=dp.get_precision('Product Price'),
-        help="Cost price in company currency in the unit of measure "
-        "of the invoice line (which may be different from the unit "
-        "of measure of the product).")
-    standard_price_invoice_currency = fields.Float(
-        string='Cost Price in Invoice Currency', readonly=True,
-        compute='_compute_margin', store=True,
-        digits=dp.get_precision('Product Price'),
-        help="Cost price in invoice currency in the unit of measure "
-        "of the invoice line")
-    margin_invoice_currency = fields.Monetary(
-        string='Margin in Invoice Currency', readonly=True, store=True,
-        compute='_compute_margin', currency_field='currency_id')
-    margin_company_currency = fields.Monetary(
-        string='Margin in Company Currency', readonly=True, store=True,
-        compute='_compute_margin', currency_field='company_currency_id')
-    margin_rate = fields.Float(
-        string="Margin Rate", readonly=True, store=True,
-        compute='_compute_margin',
-        digits=(16, 2), help="Margin rate in percentage of the sale price")
-
-    @api.depends(
-        'standard_price_company_currency', 'invoice_id.currency_id',
-        'invoice_id.type', 'invoice_id.company_id',
-        'invoice_id.date_invoice', 'quantity', 'price_subtotal')
-    def _compute_margin(self):
-        for il in self:
-            standard_price_inv_cur = 0.0
-            margin_inv_cur = 0.0
-            margin_comp_cur = 0.0
-            margin_rate = 0.0
-            inv = il.invoice_id
-            if inv and inv.type in ('out_invoice', 'out_refund'):
-                # it works in _get_current_rate
-                # even if we set date = False in context
-                # standard_price_inv_cur is in the UoM of the invoice line
-                date = inv._get_currency_rate_date() or\
-                    fields.Date.context_today(self)
-                company = inv.company_id
-                company_currency = company.currency_id
-                standard_price_inv_cur =\
-                    company_currency._convert(
-                        il.standard_price_company_currency,
-                        inv.currency_id, company, date)
-                margin_inv_cur =\
-                    il.price_subtotal - il.quantity * standard_price_inv_cur
-                margin_comp_cur = inv.currency_id._convert(
-                    margin_inv_cur, company_currency, company, date)
-                if il.price_subtotal:
-                    margin_rate = 100 * margin_inv_cur / il.price_subtotal
-                # for a refund, margin should be negative
-                # but margin rate should stay positive
-                if inv.type == 'out_refund':
-                    margin_inv_cur *= -1
-                    margin_comp_cur *= -1
-            il.standard_price_invoice_currency = standard_price_inv_cur
-            il.margin_invoice_currency = margin_inv_cur
-            il.margin_company_currency = margin_comp_cur
-            il.margin_rate = margin_rate
-
-    # We want to copy standard_price on invoice line for customer
-    # invoice/refunds. We can't do that via on_change of product_id,
-    # because it is not always played when invoice is created from code
-    # => we inherit write/create
-    # We write standard_price_company_currency even on supplier invoice/refunds
-    # because we don't have access to the 'type' of the invoice
-    @api.model
-    def create(self, vals):
-        if vals.get('product_id'):
-            pp = self.env['product.product'].browse(vals['product_id'])
-            std_price = pp.standard_price
-            inv_uom_id = vals.get('uom_id')
-            if inv_uom_id and inv_uom_id != pp.uom_id.id:
-                inv_uom = self.env['uom.uom'].browse(inv_uom_id)
-                std_price = pp.uom_id._compute_price(
-                    std_price, inv_uom)
-            vals['standard_price_company_currency'] = std_price
-        return super(AccountInvoiceLine, self).create(vals)
-
-    def write(self, vals):
-        if not vals:
-            vals = {}
-        if 'product_id' in vals or 'uom_id' in vals:
-            for il in self:
-                if 'product_id' in vals:
-                    if vals.get('product_id'):
-                        pp = self.env['product.product'].browse(
-                            vals['product_id'])
-                    else:
-                        pp = False
-                else:
-                    pp = il.product_id or False
-                # uom_id is NOT a required field
-                if 'uom_id' in vals:
-                    if vals.get('uom_id'):
-                        inv_uom = self.env['uom.uom'].browse(
-                            vals['uom_id'])
-                    else:
-                        inv_uom = False
-                else:
-                    inv_uom = il.uom_id or False
-                std_price = 0.0
-                if pp:
-                    std_price = pp.standard_price
-                    if inv_uom and inv_uom != pp.uom_id:
-                        std_price = pp.uom_id._compute_price(
-                            std_price, inv_uom)
-                il.write({'standard_price_company_currency': std_price})
-        return super(AccountInvoiceLine, self).write(vals)
-
-
-class AccountInvoice(models.Model):
-    _inherit = 'account.invoice'
-
-    margin_invoice_currency = fields.Monetary(
-        string='Margin in Invoice Currency',
-        compute='_compute_margin', store=True, readonly=True,
-        currency_field='currency_id')
-    margin_company_currency = fields.Monetary(
-        string='Margin in Company Currency',
-        compute='_compute_margin', store=True, readonly=True,
-        currency_field='company_currency_id')
-
-    @api.depends(
-        'type',
-        'invoice_line_ids.margin_invoice_currency',
-        'invoice_line_ids.margin_company_currency')
-    def _compute_margin(self):
-        res = self.env['account.invoice.line'].read_group(
-            [('invoice_id', 'in', self.ids)],
-            ['invoice_id', 'margin_invoice_currency',
-             'margin_company_currency'],
-            ['invoice_id'])
-        for re in res:
-            if re['invoice_id']:
-                inv = self.browse(re['invoice_id'][0])
-                if inv.type in ('out_invoice', 'out_refund'):
-                    inv.margin_invoice_currency = re['margin_invoice_currency']
-                    inv.margin_company_currency = re['margin_company_currency']

account_invoice_margin/account_invoice_report.py

@@ -1,60 +0,0 @@
-# Copyright 2018-2019 Akretion France (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 odoo import api, fields, models
-
-
-class AccountInvoiceReport(models.Model):
-    _inherit = 'account.invoice.report'
-
-    margin = fields.Float(string='Margin', readonly=True)
-    # why digits=0 ??? Why is it like that in the native "account" module
-    user_currency_margin = fields.Float(
-        string="Margin", compute='_compute_user_currency_margin', digits=0)
-
-    _depends = {
-        'account.invoice': [
-            'account_id', 'amount_total_company_signed',
-            'commercial_partner_id', 'company_id',
-            'currency_id', 'date_due', 'date_invoice', 'fiscal_position_id',
-            'journal_id', 'number', 'partner_bank_id', 'partner_id',
-            'payment_term_id', 'residual', 'state', 'type', 'user_id',
-        ],
-        'account.invoice.line': [
-            'account_id', 'invoice_id', 'price_subtotal', 'product_id',
-            'quantity', 'uom_id', 'account_analytic_id',
-            'margin_company_currency',
-        ],
-        'product.product': ['product_tmpl_id'],
-        'product.template': ['categ_id'],
-        'uom.uom': ['category_id', 'factor', 'name', 'uom_type'],
-        'res.currency.rate': ['currency_id', 'name'],
-        'res.partner': ['country_id'],
-    }
-
-    @api.depends('currency_id', 'date', 'margin')
-    def _compute_user_currency_margin(self):
-        user_currency = self.env.user.company_id.currency_id
-        currency_rate = self.env['res.currency.rate'].search([
-            ('rate', '=', 1),
-            '|',
-            ('company_id', '=', self.env.user.company_id.id),
-            ('company_id', '=', False)], limit=1)
-        base_currency = currency_rate.currency_id
-        for record in self:
-            date = record.date or fields.Date.today()
-            company = record.company_id
-            record.user_currency_margin = base_currency._convert(
-                record.margin, user_currency, company, date)
-
-    # TODO check for refunds
-    def _sub_select(self):
-        select_str = super(AccountInvoiceReport, self)._sub_select()
-        select_str += ", SUM(ail.margin_company_currency) AS margin"
-        return select_str
-
-    def _select(self):
-        select_str = super(AccountInvoiceReport, self)._select()
-        select_str += ", sub.margin AS margin"
-        return select_str

account_invoice_margin/account_invoice_view.xml

@@ -1,51 +0,0 @@
-<?xml version="1.0" encoding="utf-8"?>
-<!--
-  © 2015-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).
--->
-
-<odoo>
-
-<record id="view_invoice_line_form" model="ir.ui.view">
-    <field name="name">margin.account.invoice.line.form</field>
-    <field name="model">account.invoice.line</field>
-    <field name="inherit_id" ref="account.view_invoice_line_form"/>
-    <field name="arch" type="xml">
-        <xpath expr="//field[@name='analytic_tag_ids']/.." position="inside">
-            <field name="standard_price_company_currency"
-                string="Cost Price in Comp. Cur."
-                groups="base.group_no_one"/>
-            <field name="standard_price_invoice_currency"
-                string="Cost Price in Inv. Cur."
-                groups="base.group_no_one"/>
-            <field name="margin_invoice_currency"
-                string="Margin in Inv. Cur."
-                groups="base.group_no_one"/>
-            <field name="margin_company_currency"
-                string="Margin in Comp. Cur."
-                groups="base.group_no_one"/>
-            <label for="margin_rate" groups="base.group_no_one"/>
-            <div name="margin_rate" groups="base.group_no_one">
-                <field name="margin_rate" class="oe_inline"/> %
-            </div>
-        </xpath>
-    </field>
-</record>
-
-<record id="invoice_form" model="ir.ui.view">
-    <field name="name">margin.account.invoice.form</field>
-    <field name="model">account.invoice</field>
-    <field name="inherit_id" ref="account.invoice_form"/>
-    <field name="arch" type="xml">
-        <field name="move_id" position="after">
-            <field name="margin_invoice_currency"
-                string="Margin in Inv. Cur." groups="base.group_no_one"/>
-            <field name="margin_company_currency"
-                string="Margin in Comp. Cur." groups="base.group_no_one"/>
-        </field>
-    </field>
-</record>
-
-
-</odoo>

account_invoice_margin/models/__init__.py

@@ -0,0 +1,2 @@
+from . import account_move
+from . import account_invoice_report

account_invoice_margin/models/account_invoice_report.py

@@ -0,0 +1,36 @@
+# Copyright 2018-2019 Akretion France (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 odoo import api, fields, models
+
+
+class AccountInvoiceReport(models.Model):
+    _inherit = 'account.invoice.report'
+
+    margin = fields.Float(string='Margin', readonly=True)
+
+    # added margin_company_currency on account.move.line
+    _depends = {
+        'account.move': [
+            'name', 'state', 'move_type', 'partner_id', 'invoice_user_id', 'fiscal_position_id',
+            'invoice_date', 'invoice_date_due', 'invoice_payment_term_id', 'partner_bank_id',
+        ],
+        'account.move.line': [
+            'quantity', 'price_subtotal', 'amount_residual', 'balance', 'amount_currency',
+            'move_id', 'product_id', 'product_uom_id', 'account_id', 'analytic_account_id',
+            'journal_id', 'company_id', 'currency_id', 'partner_id',
+            'margin_company_currency',
+        ],
+        'product.product': ['product_tmpl_id'],
+        'product.template': ['categ_id'],
+        'uom.uom': ['category_id', 'factor', 'name', 'uom_type'],
+        'res.currency.rate': ['currency_id', 'name'],
+        'res.partner': ['country_id'],
+    }
+
+    @api.model
+    def _select(self):
+        select_str = super()._select()
+        select_str += ", line.margin_company_currency * currency_table.rate AS margin"
+        return select_str

account_invoice_margin/models/account_move.py

@@ -0,0 +1,155 @@
+# Copyright 2015-2021 Akretion France (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 odoo import api, fields, models
+
+
+class AccountMoveLine(models.Model):
+    _inherit = 'account.move.line'
+
+    standard_price_company_currency = fields.Float(
+        string='Unit Cost Price in Company Currency', readonly=True,
+        digits='Product Price',
+        help="Unit Cost price in company currency in the unit of measure "
+        "of the invoice line (which may be different from the unit "
+        "of measure of the product).")
+    standard_price_invoice_currency = fields.Float(
+        string='Unit Cost Price in Invoice Currency',
+        compute='_compute_margin', store=True, digits='Product Price',
+        help="Unit Cost price in invoice currency in the unit of measure "
+        "of the invoice line.")
+    margin_invoice_currency = fields.Monetary(
+        string='Margin in Invoice Currency', store=True,
+        compute='_compute_margin', currency_field='currency_id')
+    margin_company_currency = fields.Monetary(
+        string='Margin in Company Currency', store=True,
+        compute='_compute_margin', currency_field='company_currency_id')
+    margin_rate = fields.Float(
+        string="Margin Rate", readonly=True, store=True,
+        compute='_compute_margin',
+        digits=(16, 2), help="Margin rate in percentage of the sale price")
+
+    @api.depends(
+        'standard_price_company_currency', 'move_id.currency_id',
+        'move_id.move_type', 'move_id.company_id',
+        'move_id.invoice_date', 'quantity', 'price_subtotal')
+    def _compute_margin(self):
+        for ml in self:
+            standard_price_inv_cur = 0.0
+            margin_inv_cur = 0.0
+            margin_comp_cur = 0.0
+            margin_rate = 0.0
+            move = ml.move_id
+            if move.move_type and move.move_type in ('out_invoice', 'out_refund'):
+                # it works in _get_current_rate
+                # even if we set date = False in context
+                # standard_price_inv_cur is in the UoM of the invoice line
+                date = move.date or fields.Date.context_today(self)
+                company = move.company_id
+                company_currency = company.currency_id
+                standard_price_inv_cur =\
+                    company_currency._convert(
+                        ml.standard_price_company_currency,
+                        ml.currency_id, company, date)
+                margin_inv_cur =\
+                    ml.price_subtotal - ml.quantity * standard_price_inv_cur
+                margin_comp_cur = move.currency_id._convert(
+                    margin_inv_cur, company_currency, company, date)
+                if ml.price_subtotal:
+                    margin_rate = 100 * margin_inv_cur / ml.price_subtotal
+                # for a refund, margin should be negative
+                # but margin rate should stay positive
+                if move.move_type == 'out_refund':
+                    margin_inv_cur *= -1
+                    margin_comp_cur *= -1
+            ml.standard_price_invoice_currency = standard_price_inv_cur
+            ml.margin_invoice_currency = margin_inv_cur
+            ml.margin_company_currency = margin_comp_cur
+            ml.margin_rate = margin_rate
+
+    # We want to copy standard_price on invoice line for customer
+    # invoice/refunds. We can't do that via on_change of product_id,
+    # because it is not always played when invoice is created from code
+    # => we inherit write/create
+    # We write standard_price_company_currency even on supplier invoice/refunds
+    # because we don't have access to the 'type' of the invoice
+    @api.model_create_multi
+    def create(self, vals_list):
+        for vals in vals_list:
+            if vals.get('product_id') and not vals.get('display_type'):
+                pp = self.env['product.product'].browse(vals['product_id'])
+                std_price = pp.standard_price
+                inv_uom_id = vals.get('product_uom_id')
+                if inv_uom_id and inv_uom_id != pp.uom_id.id:
+                    inv_uom = self.env['uom.uom'].browse(inv_uom_id)
+                    std_price = pp.uom_id._compute_price(
+                        std_price, inv_uom)
+                vals['standard_price_company_currency'] = std_price
+        return super().create(vals_list)
+
+    def write(self, vals):
+        if not vals:
+            vals = {}
+        if 'product_id' in vals or 'product_uom_id' in vals:
+            for il in self:
+                if 'product_id' in vals:
+                    if vals.get('product_id'):
+                        pp = self.env['product.product'].browse(
+                            vals['product_id'])
+                    else:
+                        pp = False
+                else:
+                    pp = il.product_id or False
+                # uom_id is NOT a required field
+                if 'product_uom_id' in vals:
+                    if vals.get('product_uom_id'):
+                        inv_uom = self.env['uom.uom'].browse(
+                            vals['product_uom_id'])
+                    else:
+                        inv_uom = False
+                else:
+                    inv_uom = il.uom_id or False
+                std_price = 0.0
+                if pp:
+                    std_price = pp.standard_price
+                    if inv_uom and inv_uom != pp.uom_id:
+                        std_price = pp.uom_id._compute_price(
+                            std_price, inv_uom)
+                il.write({'standard_price_company_currency': std_price})
+        return super().write(vals)
+
+
+class AccountMove(models.Model):
+    _inherit = 'account.move'
+
+    margin_invoice_currency = fields.Monetary(
+        string='Margin in Invoice Currency',
+        compute='_compute_margin', store=True,
+        currency_field='currency_id')
+    margin_company_currency = fields.Monetary(
+        string='Margin in Company Currency',
+        compute='_compute_margin', store=True,
+        currency_field='company_currency_id')
+
+    @api.depends(
+        'move_type',
+        'invoice_line_ids.margin_invoice_currency',
+        'invoice_line_ids.margin_company_currency')
+    def _compute_margin(self):
+        rg_res = self.env['account.move.line'].read_group(
+            [
+                ('move_id', 'in', self.ids),
+                ('display_type', '=', False),
+                ('exclude_from_invoice_tab', '=', False),
+                ('move_id.move_type', 'in', ('out_invoice', 'out_refund')),
+            ],
+            ['move_id', 'margin_invoice_currency:sum', 'margin_company_currency:sum'],
+            ['move_id'])
+        mapped_data = dict([(x['move_id'][0], {
+            'margin_invoice_currency': x['margin_invoice_currency'],
+            'margin_company_currency': x['margin_company_currency'],
+            }) for x in rg_res])
+        for move in self:
+            move.margin_invoice_currency = mapped_data.get(move.id, {}).get('margin_invoice_currency')
+            move.margin_company_currency = mapped_data.get(move.id, {}).get('margin_company_currency')

account_invoice_margin/views/account_move.xml

@@ -0,0 +1,55 @@
+<?xml version="1.0" encoding="utf-8"?>
+<!--
+  Copyright 2015-2024 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).
+-->
+
+<odoo>
+
+<record id="view_move_form" model="ir.ui.view">
+    <field name="name">margin.account.move.form</field>
+    <field name="model">account.move</field>
+    <field name="inherit_id" ref="account.view_move_form"/>
+    <field name="arch" type="xml">
+        <group name="sale_info_group" position="inside">
+            <field name="margin_invoice_currency"
+                groups="base.group_no_one"
+                attrs="{'invisible': [('move_type', 'not in', ('out_invoice', 'out_refund'))]}"/>
+            <field name="margin_company_currency"
+                groups="base.group_no_one"
+                attrs="{'invisible': [('move_type', 'not in', ('out_invoice', 'out_refund'))]}"/>
+        </group>
+        <xpath expr="//field[@name='invoice_line_ids']/tree/field[@name='price_total']" position="after">
+            <field name="standard_price_invoice_currency" optional="hide" attrs="{'column_invisible': [('parent.move_type', 'not in', ('out_invoice', 'out_refund'))]}"/>
+            <field name="margin_invoice_currency" optional="hide" attrs="{'column_invisible': [('parent.move_type', 'not in', ('out_invoice', 'out_refund'))]}"/>
+            <field name="margin_rate" optional="hide" string="Margin Rate (%)" attrs="{'column_invisible': [('parent.move_type', 'not in', ('out_invoice', 'out_refund'))]}"/>
+        </xpath>
+        <xpath expr="//field[@name='invoice_line_ids']/form//field[@name='price_total']/.." position="inside">
+            <field name="standard_price_company_currency"
+                groups="base.group_no_one" attrs="{'invisible': [('parent.move_type', 'not in', ('out_invoice', 'out_refund'))]}"/>
+            <field name="standard_price_invoice_currency"
+                groups="base.group_no_one" attrs="{'invisible': [('parent.move_type', 'not in', ('out_invoice', 'out_refund'))]}"/>
+            <field name="margin_invoice_currency"
+                groups="base.group_no_one" attrs="{'invisible': [('parent.move_type', 'not in', ('out_invoice', 'out_refund'))]}"/>
+            <field name="margin_company_currency"
+                groups="base.group_no_one" attrs="{'invisible': [('parent.move_type', 'not in', ('out_invoice', 'out_refund'))]}"/>
+            <label for="margin_rate" groups="base.group_no_one" attrs="{'invisible': [('parent.move_type', 'not in', ('out_invoice', 'out_refund'))]}"/>
+            <div name="margin_rate" groups="base.group_no_one" attrs="{'invisible': [('parent.move_type', 'not in', ('out_invoice', 'out_refund'))]}">
+                <field name="margin_rate" class="oe_inline"/> %
+            </div>
+        </xpath>
+    </field>
+</record>
+
+<record id="view_invoice_tree" model="ir.ui.view">
+    <field name="model">account.move</field>
+    <field name="inherit_id" ref="account.view_invoice_tree"/>
+    <field name="arch" type="xml">
+        <field name="amount_residual_signed" position="after">
+            <field name="margin_company_currency" optional="hide" sum="1" invisible="context.get('default_move_type') not in ('out_invoice', 'out_refund')" string="Margin"/>
+        </field>
+    </field>
+</record>
+
+</odoo>

account_invoice_update_wizard/i18n/account_invoice_update_wizard.pot

@@ -0,0 +1,243 @@
+# Translation of Odoo Server.
+# This file contains the translation of the following modules:
+# 	* account_invoice_update_wizard
+#
+msgid ""
+msgstr ""
+"Project-Id-Version: Odoo Server 14.0\n"
+"Report-Msgid-Bugs-To: \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: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_line_update__price_subtotal
+msgid "Amount"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_line_update__analytic_account_id
+msgid "Analytic Account"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_line_update__analytic_tag_ids
+msgid "Analytic Tags"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_update__partner_bank_id
+msgid "Bank Account"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model_terms:ir.ui.view,arch_db:account_invoice_update_wizard.account_invoice_update_form
+msgid "Bill Reference"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model_terms:ir.ui.view,arch_db:account_invoice_update_wizard.account_invoice_update_form
+msgid "Cancel"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_update__company_id
+msgid "Company"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_line_update__create_uid
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_update__create_uid
+msgid "Created by"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_line_update__create_date
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_update__create_date
+msgid "Created on"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_line_update__currency_id
+msgid "Currency"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model_terms:ir.ui.view,arch_db:account_invoice_update_wizard.account_invoice_update_form
+msgid "Customer Reference"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_line_update__name
+msgid "Description"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move__display_name
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_line_update__display_name
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_update__display_name
+msgid "Display Name"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_line_update__display_type
+msgid "Display Type"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move__id
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_line_update__id
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_update__id
+msgid "ID"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_update__invoice_id
+msgid "Invoice"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_line_update__invoice_line_id
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_update__line_ids
+msgid "Invoice Lines"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model:ir.actions.act_window,name:account_invoice_update_wizard.account_invoice_update_action
+msgid "Invoice Update Wizard"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model:ir.model,name:account_invoice_update_wizard.model_account_move
+msgid "Journal Entry"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move____last_update
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_line_update____last_update
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_update____last_update
+msgid "Last Modified on"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_line_update__write_uid
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_update__write_uid
+msgid "Last Updated by"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_line_update__write_date
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_update__write_date
+msgid "Last Updated on"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: code:addons/account_invoice_update_wizard/wizard/account_move_update.py:0
+#, python-format
+msgid "Non-legal fields of invoice updated via the Invoice Update wizard."
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields.selection,name:account_invoice_update_wizard.selection__account_move_line_update__display_type__line_note
+msgid "Note"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_update__partner_id
+msgid "Partner"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_update__invoice_payment_term_id
+msgid "Payment Term"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_line_update__quantity
+msgid "Quantity"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_update__ref
+msgid "Reference"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_update__user_id
+msgid "Salesperson"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields.selection,name:account_invoice_update_wizard.selection__account_move_line_update__display_type__line_section
+msgid "Section"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_line_update__sequence
+msgid "Sequence"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_update__invoice_origin
+msgid "Source Document"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,help:account_invoice_update_wizard.field_account_move_line_update__display_type
+msgid "Technical field for UX purpose."
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: code:addons/account_invoice_update_wizard/wizard/account_move_update.py:0
+#, python-format
+msgid ""
+"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."
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: code:addons/account_invoice_update_wizard/wizard/account_move_update.py:0
+#, python-format
+msgid ""
+"This wizard doesn't support the update of payment terms on an invoice which "
+"is partially or fully paid."
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_update__move_type
+msgid "Type"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model_terms:ir.ui.view,arch_db:account_invoice_update_wizard.account_invoice_update_form
+msgid "Update"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model_terms:ir.ui.view,arch_db:account_invoice_update_wizard.view_move_form_inherit
+msgid "Update Invoice"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model_terms:ir.ui.view,arch_db:account_invoice_update_wizard.account_invoice_update_form
+msgid "Update Invoice Wizard"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model:ir.model,name:account_invoice_update_wizard.model_account_move_line_update
+msgid "Update non-legal fields of invoice lines"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_line_update__parent_id
+msgid "Wizard"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model:ir.model,name:account_invoice_update_wizard.model_account_move_update
+msgid "Wizard to update non-legal fields of invoice"
+msgstr ""

account_invoice_update_wizard/i18n/fr.po

@@ -0,0 +1,250 @@
+# Translation of Odoo Server.
+# This file contains the translation of the following modules:
+# 	* account_invoice_update_wizard
+#
+msgid ""
+msgstr ""
+"Project-Id-Version: Odoo Server 14.0\n"
+"Report-Msgid-Bugs-To: \n"
+"Last-Translator: \n"
+"Language-Team: \n"
+"Language: \n"
+"MIME-Version: 1.0\n"
+"Content-Type: text/plain; charset=UTF-8\n"
+"Content-Transfer-Encoding: \n"
+"Plural-Forms: \n"
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_line_update__price_subtotal
+msgid "Amount"
+msgstr "Montant"
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_line_update__analytic_account_id
+msgid "Analytic Account"
+msgstr "Compte Analytique"
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_line_update__analytic_tag_ids
+msgid "Analytic Tags"
+msgstr "Tag Analytique"
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_update__partner_bank_id
+msgid "Bank Account"
+msgstr "Compte Bancaire"
+
+#. module: account_invoice_update_wizard
+#: model_terms:ir.ui.view,arch_db:account_invoice_update_wizard.account_invoice_update_form
+#, fuzzy
+msgid "Bill Reference"
+msgstr "Reference Client"
+
+#. module: account_invoice_update_wizard
+#: model_terms:ir.ui.view,arch_db:account_invoice_update_wizard.account_invoice_update_form
+msgid "Cancel"
+msgstr "Annuler"
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_update__company_id
+msgid "Company"
+msgstr "Société"
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_line_update__create_uid
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_update__create_uid
+msgid "Created by"
+msgstr "Créé par"
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_line_update__create_date
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_update__create_date
+msgid "Created on"
+msgstr "Créé le"
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_line_update__currency_id
+msgid "Currency"
+msgstr "Devise"
+
+#. module: account_invoice_update_wizard
+#: model_terms:ir.ui.view,arch_db:account_invoice_update_wizard.account_invoice_update_form
+#, fuzzy
+msgid "Customer Reference"
+msgstr "Reference Client"
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_line_update__name
+msgid "Description"
+msgstr "Description"
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move__display_name
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_line_update__display_name
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_update__display_name
+msgid "Display Name"
+msgstr "Nom"
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_line_update__display_type
+msgid "Display Type"
+msgstr "Type Affichage"
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move__id
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_line_update__id
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_update__id
+msgid "ID"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_update__invoice_id
+msgid "Invoice"
+msgstr "Facture"
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_line_update__invoice_line_id
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_update__line_ids
+msgid "Invoice Lines"
+msgstr "Ligne de factures"
+
+#. module: account_invoice_update_wizard
+#: model:ir.actions.act_window,name:account_invoice_update_wizard.account_invoice_update_action
+msgid "Invoice Update Wizard"
+msgstr "Assistance de mise à jour de la facture"
+
+#. module: account_invoice_update_wizard
+#: model:ir.model,name:account_invoice_update_wizard.model_account_move
+msgid "Journal Entry"
+msgstr "Entrée comptable"
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move____last_update
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_line_update____last_update
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_update____last_update
+msgid "Last Modified on"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_line_update__write_uid
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_update__write_uid
+msgid "Last Updated by"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_line_update__write_date
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_update__write_date
+msgid "Last Updated on"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: code:addons/account_invoice_update_wizard/wizard/account_move_update.py:0
+#, python-format
+msgid "Non-legal fields of invoice updated via the Invoice Update wizard."
+msgstr "Champs non légaux mis à jour via l'assistant"
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields.selection,name:account_invoice_update_wizard.selection__account_move_line_update__display_type__line_note
+msgid "Note"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_update__partner_id
+msgid "Partner"
+msgstr "Client"
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_update__invoice_payment_term_id
+msgid "Payment Term"
+msgstr "Condition de paiement"
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_line_update__quantity
+msgid "Quantity"
+msgstr "Quantité"
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_update__ref
+#, fuzzy
+msgid "Reference"
+msgstr "Reference Client"
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_update__user_id
+msgid "Salesperson"
+msgstr "Vendeur"
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields.selection,name:account_invoice_update_wizard.selection__account_move_line_update__display_type__line_section
+msgid "Section"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_line_update__sequence
+msgid "Sequence"
+msgstr "Sequence"
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_update__invoice_origin
+msgid "Source Document"
+msgstr "Origine du document"
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,help:account_invoice_update_wizard.field_account_move_line_update__display_type
+msgid "Technical field for UX purpose."
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: code:addons/account_invoice_update_wizard/wizard/account_move_update.py:0
+#, python-format
+msgid ""
+"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."
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: code:addons/account_invoice_update_wizard/wizard/account_move_update.py:0
+#, python-format
+msgid ""
+"This wizard doesn't support the update of payment terms on an invoice which "
+"is partially or fully paid."
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_update__move_type
+msgid "Type"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model_terms:ir.ui.view,arch_db:account_invoice_update_wizard.account_invoice_update_form
+msgid "Update"
+msgstr "Mettre à jour"
+
+#. module: account_invoice_update_wizard
+#: model_terms:ir.ui.view,arch_db:account_invoice_update_wizard.view_move_form_inherit
+msgid "Update Invoice"
+msgstr "Mettre à jour"
+
+#. module: account_invoice_update_wizard
+#: model_terms:ir.ui.view,arch_db:account_invoice_update_wizard.account_invoice_update_form
+msgid "Update Invoice Wizard"
+msgstr "Assistant de mise à jour"
+
+#. module: account_invoice_update_wizard
+#: model:ir.model,name:account_invoice_update_wizard.model_account_move_line_update
+msgid "Update non-legal fields of invoice lines"
+msgstr "Mettre à jour les champs non légaux des lignes de facture"
+
+#. module: account_invoice_update_wizard
+#: model:ir.model.fields,field_description:account_invoice_update_wizard.field_account_move_line_update__parent_id
+msgid "Wizard"
+msgstr ""
+
+#. module: account_invoice_update_wizard
+#: model:ir.model,name:account_invoice_update_wizard.model_account_move_update
+msgid "Wizard to update non-legal fields of invoice"
+msgstr "Assistant pour mettre à jours les champs non légaux"
+
+#~ msgid "Account"
+#~ msgstr "Compte"

account_invoice_update_wizard/views/account_move.xml

@@ -11,7 +11,8 @@
     <field name="inherit_id" ref="account.view_move_form"/>
     <field name="arch" type="xml">
         <button name="button_draft" position="before">
-            <button name="prepare_update_wizard" type="object" string="Update Invoice" states="posted" groups="account.group_account_invoice"/>
+            <button name="prepare_update_wizard" type="object" string="Update Invoice" groups="account.group_account_invoice" attrs="{'invisible': ['|', ('state', '!=', 'posted'), ('move_type', '=', 'entry')]}"/>
+            <button name="prepare_update_wizard" type="object" string="Update Entry" groups="account.group_account_invoice" attrs="{'invisible': ['|', ('state', '!=', 'posted'), ('move_type', '!=', 'entry')]}"/>
         </button>
     </field>
 </record>

account_invoice_update_wizard/wizard/account_move_update.py

@@ -21,6 +21,7 @@ class AccountMoveUpdate(models.TransientModel):
     invoice_payment_term_id = fields.Many2one(
         'account.payment.term', string='Payment Term')
     ref = fields.Char(string='Reference')  # field label is customized in the view
+    invoice_date = fields.Date()
     invoice_origin = fields.Char(string='Source Document')
     partner_bank_id = fields.Many2one(
         'res.partner.bank', string='Bank Account')
@@ -30,7 +31,7 @@ class AccountMoveUpdate(models.TransientModel):
     @api.model
     def _simple_fields2update(self):
         '''List boolean, date, datetime, char, text fields'''
-        return ['ref', 'invoice_origin']
+        return ['ref', 'invoice_origin', 'invoice_date']
 
     @api.model
     def _m2o_fields2update(self):
@@ -48,6 +49,7 @@ class AccountMoveUpdate(models.TransientModel):
             aa_tags = [(6, 0, aa_tags.ids)] if aa_tags else False
             res['line_ids'].append([0, 0, {
                 'invoice_line_id': line.id,
+                'sequence': line.sequence,
                 'name': line.name,
                 'quantity': line.quantity,
                 'price_subtotal': line.price_subtotal,
@@ -231,7 +233,9 @@ class AccountMoveUpdate(models.TransientModel):
 class AccountMoveLineUpdate(models.TransientModel):
     _name = 'account.move.line.update'
     _description = 'Update non-legal fields of invoice lines'
+    _order = "sequence, name"
 
+    sequence = fields.Integer()
     parent_id = fields.Many2one(
         'account.move.update', string='Wizard', ondelete='cascade')
     invoice_line_id = fields.Many2one(

account_invoice_update_wizard/wizard/account_move_update_view.xml

@@ -15,22 +15,25 @@
                     <field name="move_type" invisible="1"/>
                     <field name="company_id" invisible="1"/>
                     <field name="partner_id" invisible="1"/>
-                    <field string="Bill Reference" attrs="{'invisible': [('move_type', 'not in', ('in_invoice', 'in_refund'))]}" name="ref"/>
+                    <field string="Bill Date" attrs="{'invisible': [('move_type', 'not in', ('in_invoice', 'in_refund'))]}" name="invoice_date"/>
+                    <field string="Supplier Bill Reference" attrs="{'invisible': [('move_type', 'not in', ('in_invoice', 'in_refund'))]}" name="ref"/>
                     <field string="Customer Reference" attrs="{'invisible': [('move_type', 'not in', ('out_invoice', 'out_refund'))]}" name="ref"/>
-                    <field name="invoice_origin"/>
-                    <field name="invoice_payment_term_id" widget="selection"/>
-                    <field name="partner_bank_id"/>
-                    <field name="user_id" options="{'no_open': True, 'no_create': True, 'no_create_edit': True}"/>
+                    <field string="Ref" attrs="{'invisible': [('move_type', '!=', 'entry')]}" name="ref"/>
+                    <field name="invoice_origin" attrs="{'invisible': [('move_type', '=', 'entry')]}"/>
+                    <!-- update of payment term is broken -->
+                    <!-- <field name="invoice_payment_term_id" widget="selection"/>-->
+                    <field name="partner_bank_id" attrs="{'invisible': [('move_type', '=', 'entry')]}"/>
+                    <field name="user_id" options="{'no_open': True, 'no_create': True, 'no_create_edit': True}" attrs="{'invisible': [('move_type', '=', 'entry')]}"/>
                 </group>
                 <group name="lines">
-                    <field name="line_ids" nolabel="1">
+                    <field name="line_ids" nolabel="1" widget="section_and_note_one2many">
                         <tree editable="bottom" create="false" delete="false" edit="true">
                             <field name="invoice_line_id" invisible="1"/>
                             <field name="display_type" invisible="1"/>
                             <field name="currency_id" invisible="1"/>
                             <field name="name"/>
-                            <field name="quantity" attrs="{'invisible': [('display_type', '!=', False)]}"/>
-                            <field name="price_subtotal" attrs="{'invisible': [('display_type', '!=', False)]}"/>
+                            <field name="quantity" attrs="{'invisible': [('display_type', '!=', False)], 'column_invisible': [('parent.move_type', '=', 'entry')]}"/>
+                            <field name="price_subtotal" attrs="{'invisible': [('display_type', '!=', False)], 'column_invisible': [('parent.move_type', '=', 'entry')]}"/>
                             <field name="analytic_account_id" attrs="{'invisible': [('display_type', '!=', False)]}" groups="analytic.group_analytic_accounting"/>
                             <field name="analytic_tag_ids" attrs="{'invisible': [('display_type', '!=', False)]}" groups="analytic.group_analytic_tags" widget="many2many_tags"/>
                     </tree>

account_invoice_update_wizard_payment_mode/wizard/account_move_update_view.xml

@@ -12,7 +12,7 @@
     <field name="model">account.move.update</field>
     <field name="inherit_id" ref="account_invoice_update_wizard.account_invoice_update_form"/>
     <field name="arch" type="xml">
-        <field name="invoice_payment_term_id" position="after">
+        <field name="partner_bank_id" position="before">
             <field name="payment_mode_filter_type_domain" invisible="1"/>
             <field name="partner_bank_filter_type_domain" invisible="1"/>
             <field name="bank_account_required" invisible="1"/>

account_usability/__init__.py

@@ -1,2 +1,3 @@
 from . import models
 from . import wizard
+from .hooks import post_init_hook

account_usability/__manifest__.py

@@ -4,7 +4,7 @@
 
 {
     'name': 'Account Usability',
-    'version': '14.0.1.0.0',
+    'version': '14.0.1.1.0',
     'category': 'Accounting & Finance',
     'license': 'AGPL-3',
     'summary': 'Small usability enhancements in account module',
@@ -29,8 +29,8 @@
         'views/account_tax.xml',
         'views/product.xml',
         'views/res_config_settings.xml',
-        'views/res_partner.xml',
         'views/res_company.xml',
+        'views/res_partner.xml',
         'views/account_report.xml',
         'views/account_reconcile_model.xml',
         'wizard/account_invoice_mark_sent_view.xml',
@@ -41,4 +41,5 @@
         ],
     'qweb': ['static/src/xml/account_payment.xml'],
     'installable': True,
+    "post_init_hook": "post_init_hook",
 }

account_usability/hooks.py

@@ -0,0 +1,9 @@
+# Copyright 2022 Akretion (https://www.akretion.com).
+# @author Sébastien BEAU <sebastien.beau@akretion.com>
+# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
+
+from odoo import SUPERUSER_ID, api
+
+def post_init_hook(cr, registry):
+    env = api.Environment(cr, SUPERUSER_ID, {})
+    env["account.move.line"].update_matching_number()

account_usability/i18n/account_usability.pot

@@ -524,11 +524,6 @@ msgstr ""
 msgid "Recipient Bank"
 msgstr ""
 
-#. module: account_usability
-#: model:ir.model.fields,field_description:account_usability.field_account_move_line__reconcile_string
-msgid "Reconcile"
-msgstr ""
-
 #. module: account_usability
 #: model:ir.model.fields,field_description:account_usability.field_account_bank_statement_line__ref
 #: model:ir.model.fields,field_description:account_usability.field_account_move__ref

account_usability/migrations/14.0.1.1.0/post-migrate.py

@@ -0,0 +1,9 @@
+# Copyright 2020 ACSONE SA/NV
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+
+from odoo import SUPERUSER_ID, api
+
+
+def migrate(cr, version):
+    env = api.Environment(cr, SUPERUSER_ID, {})
+    env["account.move.line"].update_matching_number()

account_usability/models/account_bank_statement.py

@@ -43,13 +43,6 @@ class AccountBankStatement(models.Model):
             res.append((statement.id, name))
         return res
 
-    def button_reopen(self):
-        self = self.with_context(skip_undo_reconciliation=True)
-        return super().button_reopen()
-
-    def button_undo_reconciliation(self):
-        self.line_ids.button_undo_reconciliation()
-
 
 class AccountBankStatementLine(models.Model):
     _inherit = 'account.bank.statement.line'
@@ -96,9 +89,3 @@ class AccountBankStatementLine(models.Model):
             'res_id': self.move_id.id,
             })
         return action
-
-    def button_undo_reconciliation(self):
-        if self._context.get("skip_undo_reconciliation"):
-            return
-        else:
-            return super().button_undo_reconciliation()

account_usability/models/account_move.py

@@ -2,12 +2,18 @@
 # @author Alexis de Lattre <alexis.delattre@akretion.com>
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 
+from datetime import timedelta
+import logging
+
 from odoo import api, fields, models, _
+from odoo.exceptions import UserError, ValidationError
+from odoo.osv import expression
 from odoo.tools import float_is_zero
 from odoo.tools.misc import format_date
-from odoo.osv import expression
-from datetime import timedelta
-from odoo.exceptions import UserError
+from odoo.tools.safe_eval import safe_eval, time
+from collections import defaultdict
+
+_logger = logging.getLogger(__name__)
 
 
 class AccountMove(models.Model):
@@ -36,6 +42,34 @@ class AccountMove(models.Model):
         compute="_compute_sales_dates", readonly=True,
         help="This information appear on invoice qweb report "
              "(you may use it for your own report)")
+    # There is a native "blocked" field (bool) on account.move.line
+    # We want to have that field on invoices to improve usability
+    # while keeping compatibility with the standard Odoo datamodel
+    blocked = fields.Boolean(
+        compute="_compute_blocked",
+        inverse="_inverse_blocked",
+        store=True,
+        string="Dispute",
+        tracking=True,
+    )
+
+    @api.depends("line_ids", "line_ids.blocked")
+    def _compute_blocked(self):
+        for move in self:
+            move.blocked = any(
+                [
+                    l.blocked
+                    for l in move.line_ids
+                    if l.account_id.internal_type in ("payable", "receivable")
+                ]
+            )
+
+    def _inverse_blocked(self):
+        for move in self:
+            for line in move.line_ids.filtered(
+                lambda l: l.account_id.internal_type in ("payable", "receivable")
+            ):
+                line.blocked = move.blocked
 
     def _compute_has_discount(self):
         prec = self.env['decimal.precision'].precision_get('Discount')
@@ -108,6 +142,16 @@ class AccountMove(models.Model):
 #            self.invalidate_cache()
 #        return res
 
+    def _reverse_moves(self, default_values_list=None, cancel=False):
+        reverse_moves = super()._reverse_moves(
+            default_values_list=default_values_list, cancel=cancel)
+        # In the simple scenario 1 invoice -> 1 refund, we add a message in the chatter
+        # of the invoice and in the chatter of the refund
+        if len(self) == 1 and len(reverse_moves) == 1:
+            self.message_post(body=_("A reverse journal entry <a href=# data-oe-model=account.move data-oe-id=%d>%s</a> has been generated.") % (reverse_moves.id, reverse_moves.display_name))
+            reverse_moves.message_post(body=_("This journal entry has been generated as the reverse of <a href=# data-oe-model=account.move data-oe-id=%d>%s</a>.") % (self.id, self.display_name))
+        return reverse_moves
+
     def delete_lines_qty_zero(self):
         lines = self.env['account.move.line'].search([
             ('display_type', '=', False),
@@ -181,13 +225,11 @@ class AccountMove(models.Model):
                 move.suitable_journal_ids = self.env['account.journal'].search(domain)
 
     def button_draft(self):
+        # Get report name before reset to draft because name can be different.
+        report_names = self._get_invoice_attachment_name()
         super().button_draft()
         # Delete attached pdf invoice
-        try:
-            report_invoice = self.env['ir.actions.report']._get_report_from_name('account.report_invoice')
-        except IndexError:
-            report_invoice = False
-        if report_invoice and report_invoice.attachment:
+        if report_names:
             for move in self.filtered(lambda x: x.move_type in ('out_invoice', 'out_refund')):
                 # The pb is that the filename is dynamic and related to move.state
                 # in v12, the feature was native and they used that kind of code:
@@ -197,7 +239,7 @@ class AccountMove(models.Model):
                 # But do_in_draft() doesn't exists in v14
                 # If you know how we could do that, please update the code below
                 attachment = self.env['ir.attachment'].search([
-                    ('name', '=', self._get_invoice_attachment_name()),
+                    ('name', 'in', report_names[move.id]),
                     ('res_id', '=', move.id),
                     ('res_model', '=', self._name),
                     ('type', '=', 'binary'),
@@ -206,8 +248,22 @@ class AccountMove(models.Model):
                     attachment.unlink()
 
     def _get_invoice_attachment_name(self):
-        self.ensure_one()
-        return '%s.pdf' % (self.name and self.name.replace('/', '_') or 'INV')
+        report_names = defaultdict(list)
+        try:
+            report_invoice = self.env['ir.actions.report']._get_report_from_name('account.report_invoice')
+        except IndexError:
+            report_invoice = False
+        if report_invoice and report_invoice.attachment:
+            for move in self.filtered(lambda x: x.move_type in ('out_invoice', 'out_refund')):
+                report_names[move.id].append(safe_eval(report_invoice.print_report_name, {'object': self, 'time': time}))
+        try:
+            report_invoice = self.env['ir.actions.report']._get_report_from_name('account.report_invoice_with_payments')
+        except IndexError:
+            report_invoice = False
+        if report_invoice and report_invoice.attachment:
+            for move in self.filtered(lambda x: x.move_type in ('out_invoice', 'out_refund')):
+                report_names[move.id].append(safe_eval(report_invoice.print_report_name, {'object': self, 'time': time}))
+        return report_names
 
     def _get_accounting_date(self, invoice_date, has_tax):
         # On vendor bills/refunds, we want date = invoice_date unless
@@ -221,6 +277,40 @@ class AccountMove(models.Model):
             date = invoice_date
         return date
 
+    # I don't use account_invoice_supplier_ref_unique because it adds
+    # a field supplier_invoice_number on account.move instead of using the native field
+    # cf https://github.com/OCA/account-invoicing/issues/1484
+    # So I take inspiration from the code of account_invoice_supplier_ref_unique
+    # but I use the native "ref" field
+    @api.constrains("ref", "partner_id")
+    def _check_in_invoice_ref_unique_insensitive(self):
+        for move in self:
+            if move.ref and move.is_purchase_document(
+                include_receipts=True
+            ):
+                in_invoice_same_ref = self.search(
+                    [
+                        ("commercial_partner_id", "=", move.commercial_partner_id.id),
+                        ("move_type", "in", ("in_invoice", "in_refund")),
+                        ("company_id", "=", move.company_id.id),
+                        ("ref", "=ilike", move.ref),
+                        ("id", "!=", move.id),
+                    ],
+                    limit=1,
+                )
+                if in_invoice_same_ref:
+                    raise ValidationError(
+                        _(
+                            "An invoice already exists in Odoo with the same "
+                            "bill reference '%s' for the same supplier '%s': %s."
+                        )
+                        % (
+                            in_invoice_same_ref.ref,
+                            in_invoice_same_ref.partner_id.display_name,
+                            in_invoice_same_ref.display_name,
+                        )
+                    )
+
 
 class AccountMoveLine(models.Model):
     _inherit = 'account.move.line'
@@ -238,8 +328,6 @@ class AccountMoveLine(models.Model):
     full_reconcile_id = fields.Many2one(string='Full Reconcile')
     matched_debit_ids = fields.One2many(string='Partial Reconcile Debit')
     matched_credit_ids = fields.One2many(string='Partial Reconcile Credit')
-    reconcile_string = fields.Char(
-        compute='_compute_reconcile_string', string='Reconcile', store=True)
     # for optional display in tree view
     product_barcode = fields.Char(related='product_id.barcode', string="Product Barcode")
 
@@ -255,17 +343,21 @@ class AccountMoveLine(models.Model):
         })
         return action
 
-    @api.depends(
-            'full_reconcile_id', 'matched_debit_ids', 'matched_credit_ids')
-    def _compute_reconcile_string(self):
-        for line in self:
-            rec_str = False
-            if line.full_reconcile_id:
-                rec_str = line.full_reconcile_id.name
-            else:
-                rec_str = ', '.join([
-                    'a%d' % pr.id for pr in line.matched_debit_ids + line.matched_credit_ids])
-            line.reconcile_string = rec_str
+    def update_matching_number(self):
+        records = self.search([("matching_number", "=", "P")])
+        _logger.info(f"Update partial reconcile number for {len(records)} lines")
+        records._compute_matching_number()
+
+    def _compute_matching_number(self):
+        # TODO maybe it will be better to have the same maching_number for
+        # all partial so it will be easier to group by
+        super()._compute_matching_number()
+        for record in self:
+            if record.matching_number == "P":
+                record.matching_number = ", ".join([
+                    "a%d" % pr.id
+                    for pr in record.matched_debit_ids + record.matched_credit_ids
+                ])
 
     def _get_computed_name(self):
         # This is useful when you want to have the product code in a dedicated

account_usability/views/account_bank_statement.xml

@@ -13,13 +13,8 @@
     <field name="model">account.bank.statement</field>
     <field name="inherit_id" ref="account.view_bank_statement_form"/>
     <field name="arch" type="xml">
-        <button name="button_reopen" position="after">
-            <button
-                name="button_undo_reconciliation"
-                type="object"
-                confirm="Are you sure to unreconcile all the entries of the bank statement?"
-                states="open"
-                string="Unreconcile All"/>
+        <button name="button_reopen" position="attributes">
+            <attribute name="confirm">Are you sure ? Don't do 'Reset to New' if you just want to modify the bank journal entry of an existing statement line.</attribute>
         </button>
         <xpath expr="//field[@name='line_ids']/tree/button[@name='button_undo_reconciliation']" position="after">
             <field name="move_id" invisible="1"/>

account_usability/views/account_move.xml

@@ -42,12 +42,21 @@
             <attribute name="optional">show</attribute>
         </xpath>
         <xpath expr="//field[@name='line_ids']/tree/field[@name='tax_tag_ids']" position="after">
-            <field name="matching_number" optional="hide"/>
-            <field name="reconcile_string" optional="show"/>
+            <field name="matching_number" optional="show"/>
         </xpath>
         <xpath expr="//field[@name='invoice_line_ids']/tree/field[@name='product_id']" position="after">
             <field name="product_barcode" optional="hide"/>
         </xpath>
+        <field name="auto_post" position="before">
+            <field name="blocked"/>
+        </field>
+        <div role="alert" position="after">
+            <div id="warn_blocked" groups="account.group_account_invoice,account.group_account_readonly"
+                    class="alert alert-warning" role="alert" style="margin-bottom:0px;"
+                    attrs="{'invisible': ['|', ('move_type', 'not in', ('in_invoice', 'in_refund', 'out_invoice', 'out_refund')), ('blocked', '=', False)]}">
+                    This <field name="move_type"/> is marked as <b>disputed</b>.
+            </div>
+        </div>
     </field>
 </record>
 
@@ -72,6 +81,8 @@
              <filter name="sent" string="Sent" domain="[('is_move_sent', '=', True), ('move_type', 'in', ('out_invoice', 'out_refund'))]"/>
             <separator/>
             <filter name="no_attachment" string="Missing Attachment" domain="[('has_attachment', '=', False)]"/>
+            <separator/>
+            <filter name="dispute" string="Dispute" domain="[('blocked', '=', True)]"/>
         </filter>
         <filter name="salesperson" position="before">
             <filter name="commercial_partner_groupby" string="Commercial Partner" context="{'group_by': 'commercial_partner_id'}"/>
@@ -82,6 +93,23 @@
     </field>
 </record>
 
+<record id="view_move_line_form" model="ir.ui.view">
+    <field name="model">account.move.line</field>
+    <field name="inherit_id" ref="account.view_move_line_form"/>
+    <field name="arch" type="xml">
+        <!-- The field 'blocked' is alone in it's block
+        We don't want to display an empty block, so we put the attrs on the group
+        The drawback of this is that, if someone added a field in that group,
+        he won't see the field when internal_type is not payable/receivable -->
+        <xpath expr="//field[@name='blocked']/.." position="attributes">
+            <attribute name="attrs">{'invisible': [('account_internal_type', 'not in', ('payable', 'receivable'))]}</attribute>
+        </xpath>
+        <field name="company_id" position="after">
+            <field name="account_internal_type" invisible="1"/>
+        </field>
+    </field>
+</record>
+
 <record id="view_move_line_tree" model="ir.ui.view">
     <field name="model">account.move.line</field>
     <field name="inherit_id" ref="account.view_move_line_tree"/>
@@ -106,7 +134,7 @@
             <separator/>
         </filter>
         <field name="partner_id" position="after">
-            <field name="reconcile_string" />
+            <field name="matching_number" />
             <field name="debit" filter_domain="['|', ('debit', '=', self), ('credit', '=', self)]" string="Debit or Credit"/>
         </field>
         <filter name="unreconciled" position="before">
@@ -118,12 +146,27 @@
         <field name="name" position="attributes">
             <attribute name="string">Label, Reference, Account or Partner</attribute>
         </field>
+        <field name="name" position="before">
+            <field name="move_id" position="move"/>
+        </field>
         <field name="partner_id" position="attributes">
             <attribute name="domain">['|', ('parent_id', '=', False), ('is_company', '=', True)]</attribute>
         </field>
     </field>
 </record>
 
+<record id="view_move_line_pivot" model="ir.ui.view">
+    <field name="model">account.move.line</field>
+    <field name="inherit_id" ref="account.view_move_line_pivot"/>
+    <field name="arch" type="xml">
+        <!-- By default, date is split by month... but if you've been using Odoo for several years,
+            the pivot table becomes very big by default: so we split by year -->
+        <field name="date" position="attributes">
+            <attribute name="interval">year</attribute>
+        </field>
+    </field>
+</record>
+
 <record id="account.action_move_journal_line" model="ir.actions.act_window">
     <field name="context">{'default_move_type': 'entry', 'view_no_maturity': True}</field>
     <!-- Remove 'search_default_misc_filter': 1 -->

account_usability/views/res_partner.xml

@@ -1,23 +1,21 @@
 <?xml version="1.0" encoding="utf-8"?>
 <!--
-  Copyright 2017-2020 Akretion (http://www.akretion.com/)
+  Copyright 2024 Akretion France (http://www.akretion.com/)
   @author: Alexis de Lattre <alexis.delattre@akretion.com>
   License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 -->
 
 <odoo>
 
-
 <record id="view_partner_property_form" model="ir.ui.view">
-    <field name="name">account_usability.res.partner.form</field>
     <field name="model">res.partner</field>
     <field name="inherit_id" ref="account.view_partner_property_form"/>
     <field name="arch" type="xml">
-        <xpath expr="//field[@name='bank_ids']/tree/field[@name='acc_number']" position="after">
-            <field name="acc_type"/>
+        <xpath expr="//field[@name='bank_ids']/tree/field[@name='acc_holder_name']" position="attributes">
+            <attribute name="invisible">0</attribute>
+            <attribute name="optional">hide</attribute>
         </xpath>
     </field>
 </record>
 
-
 </odoo>

account_usability/wizard/account_move_reversal.py

@@ -12,13 +12,14 @@ class AccountMoveReversal(models.TransientModel):
 
     # Set default reversal date to original move + 1 day
     # and raise error if original move has already been reversed
+    # WARNING: this wizard is also used to generate refunds
     @api.model
     def default_get(self, fields_list):
         res = super().default_get(fields_list)
         assert self._context.get('active_model') == 'account.move'
         amo = self.env['account.move']
         moves = amo.browse(self._context['active_ids'])
-        if len(moves) == 1:
+        if len(moves) == 1 and moves.move_type not in ('out_invoice', 'in_invoice'):
             res['date'] = moves.date + relativedelta(days=1)
         reversed_move = amo.search([('reversed_entry_id', 'in', moves.ids)], limit=1)
         if reversed_move:

base_partner_one2many_phone/partner_phone.py

@@ -15,6 +15,7 @@ class ResPartnerPhone(models.Model):
     _name = 'res.partner.phone'
     _order = 'partner_id, type'
     _phone_name_sequence = 8
+    _phone_name_fields = ["phone"]
     _inherit = ['phone.validation.mixin']
     _description = 'Multiple emails and phones for partners'
 
@@ -73,7 +74,7 @@ class ResPartnerPhone(models.Model):
                 if self._context.get('callerid'):
                     name = pphone.partner_id.display_name
                 else:
-                    name = u'%s (%s)' % (pphone.phone, pphone.partner_id.name)
+                    name = '%s (%s)' % (pphone.phone, pphone.partner_id.name)
             else:
                 name = pphone.phone
             res.append((pphone.id, name))
@@ -104,6 +105,7 @@ class ResPartnerPhone(models.Model):
 
 class ResPartner(models.Model):
     _inherit = 'res.partner'
+    _phone_name_fields = []
 
     # in v10, we are supposed to have in DB E.164 format
     # with the current implementation, we have:
@@ -147,6 +149,8 @@ class ResPartner(models.Model):
         if vals.get(partner_field):
             vals['phone_ids'].append(
                 (0, 0, {'type': type, partner_phone_field: vals[partner_field]}))
+        if partner_field in vals:
+            vals.pop(partner_field)
 
     @api.model
     def create(self, vals):
@@ -155,7 +159,6 @@ class ResPartner(models.Model):
             self._update_create_vals(vals, '1_email_primary', 'email', 'email')
             self._update_create_vals(vals, '3_phone_primary', 'phone', 'phone')
             self._update_create_vals(vals, '5_mobile_primary', 'mobile', 'phone')
-            # self._update_create_vals(vals, '7_fax_primary', 'fax', 'phone')
         return super().create(vals)
 
     def _update_write_vals(
@@ -178,16 +181,16 @@ class ResPartner(models.Model):
             else:
                 if pphone:
                     vals['phone_ids'].append((2, pphone.id))
+            vals.pop(partner_field)
 
     def write(self, vals):
         if 'phone_ids' not in vals:
             for rec in self:
-                vals['phone_ids'] = []
-                rec._update_write_vals(vals, '1_email_primary', 'email', 'email')
-                rec._update_write_vals(vals, '3_phone_primary', 'phone', 'phone')
-                rec._update_write_vals(vals, '5_mobile_primary', 'mobile', 'phone')
-                rec._update_write_vals(vals, '7_fax_primary', 'fax', 'phone')
-                super(ResPartner, rec).write(vals)
+                cvals = dict(vals, phone_ids=[])
+                rec._update_write_vals(cvals, '1_email_primary', 'email', 'email')
+                rec._update_write_vals(cvals, '3_phone_primary', 'phone', 'phone')
+                rec._update_write_vals(cvals, '5_mobile_primary', 'mobile', 'phone')
+                super(ResPartner, rec).write(cvals)
             return True
         else:
             return super().write(vals)

base_partner_one2many_phone/partner_phone_view.xml

@@ -161,6 +161,12 @@
         <field name="name" position="attributes">
             <attribute name="filter_domain">['|', '|', ('display_name', 'ilike', self), ('ref', '=ilike', self + '%'), ('phone_ids.email', 'ilike', self)]</attribute>
         </field>
+        <field name="email" position="attributes">
+            <attribute name="filter_domain">[('phone_ids.email', 'ilike', self)]</attribute>
+        </field>
+        <field name="phone" position="attributes">
+            <attribute name="filter_domain">[('phone_ids.phone', 'ilike', self)]</attribute>
+        </field>
     </field>
 </record>
 

base_usability/__manifest__.py

@@ -10,7 +10,7 @@
     'summary': 'Better usability in base module',
     'author': 'Akretion',
     'website': 'http://www.akretion.com',
-    'depends': ['base'],
+    'depends': ['web'],
     'data': [
         'security/group.xml',
         'security/ir.model.access.csv',
@@ -21,6 +21,7 @@
         'views/ir_module.xml',
         'views/ir_sequence.xml',
         'views/ir_property.xml',
+        'views/assets.xml',
         ],
     'installable': True,
 }

base_usability/i18n/base_usability.pot

@@ -6,8 +6,8 @@ msgid ""
 msgstr ""
 "Project-Id-Version: Odoo Server 14.0\n"
 "Report-Msgid-Bugs-To: \n"
-"POT-Creation-Date: 2021-07-01 10:02+0000\n"
-"PO-Revision-Date: 2021-07-01 10:02+0000\n"
+"POT-Creation-Date: 2024-03-26 21:27+0000\n"
+"PO-Revision-Date: 2024-03-26 21:27+0000\n"
 "Last-Translator: \n"
 "Language-Team: \n"
 "MIME-Version: 1.0\n"
@@ -15,6 +15,18 @@ msgstr ""
 "Content-Transfer-Encoding: \n"
 "Plural-Forms: \n"
 
+#. module: base_usability
+#: code:addons/base_usability/models/res_company.py:0
+#, python-format
+msgid "%s with a capital of"
+msgstr ""
+
+#. module: base_usability
+#: code:addons/base_usability/models/res_company.py:0
+#, python-format
+msgid "APE:"
+msgstr ""
+
 #. module: base_usability
 #: model:ir.model,name:base_usability.model_res_partner_bank
 msgid "Bank Accounts"
@@ -25,11 +37,22 @@ msgstr ""
 msgid "Bank Name"
 msgstr ""
 
+#. module: base_usability
+#: code:addons/base_usability/models/res_company.py:0
+#, python-format
+msgid "Capital:"
+msgstr ""
+
 #. module: base_usability
 #: model:ir.model,name:base_usability.model_res_company
 msgid "Companies"
 msgstr ""
 
+#. module: base_usability
+#: model_terms:ir.ui.view,arch_db:base_usability.ir_property_view_search
+msgid "Company"
+msgstr ""
+
 #. module: base_usability
 #: model:ir.model,name:base_usability.model_res_partner
 msgid "Contact"
@@ -48,6 +71,7 @@ msgid "Customer Number:"
 msgstr ""
 
 #. module: base_usability
+#: model:ir.model.fields,field_description:base_usability.field_ir_actions_report__display_name
 #: model:ir.model.fields,field_description:base_usability.field_ir_mail_server__display_name
 #: model:ir.model.fields,field_description:base_usability.field_ir_model__display_name
 #: model:ir.model.fields,field_description:base_usability.field_res_company__display_name
@@ -65,12 +89,24 @@ msgstr ""
 msgid "E-mail:"
 msgstr ""
 
+#. module: base_usability
+#: code:addons/base_usability/models/res_company.py:0
+#, python-format
+msgid "EORI:"
+msgstr ""
+
+#. module: base_usability
+#: model_terms:ir.ui.view,arch_db:base_usability.ir_property_view_search
+msgid "Field"
+msgstr ""
+
 #. module: base_usability
 #: model_terms:ir.ui.view,arch_db:base_usability.res_country_search
 msgid "Group By"
 msgstr ""
 
 #. module: base_usability
+#: model:ir.model.fields,field_description:base_usability.field_ir_actions_report__id
 #: model:ir.model.fields,field_description:base_usability.field_ir_mail_server__id
 #: model:ir.model.fields,field_description:base_usability.field_ir_model__id
 #: model:ir.model.fields,field_description:base_usability.field_res_company__id
@@ -87,6 +123,7 @@ msgid "Installable"
 msgstr ""
 
 #. module: base_usability
+#: model:ir.model.fields,field_description:base_usability.field_ir_actions_report____last_update
 #: model:ir.model.fields,field_description:base_usability.field_ir_mail_server____last_update
 #: model:ir.model.fields,field_description:base_usability.field_ir_model____last_update
 #: model:ir.model.fields,field_description:base_usability.field_res_company____last_update
@@ -139,6 +176,11 @@ msgstr ""
 msgid "Partner Tags"
 msgstr ""
 
+#. module: base_usability
+#: model:ir.model.fields,field_description:base_usability.field_ir_actions_report__print_report_name
+msgid "Printed Report Name"
+msgstr ""
+
 #. module: base_usability
 #: model:ir.model.fields,field_description:base_usability.field_res_partner__ref
 #: model:ir.model.fields,field_description:base_usability.field_res_users__ref
@@ -146,8 +188,14 @@ msgid "Reference"
 msgstr ""
 
 #. module: base_usability
-#: model_terms:ir.ui.view,arch_db:base_usability.res_country_search
-msgid "Search Countries"
+#: model:ir.model,name:base_usability.model_ir_actions_report
+msgid "Report Action"
+msgstr ""
+
+#. module: base_usability
+#: code:addons/base_usability/models/res_company.py:0
+#, python-format
+msgid "SIRET:"
 msgstr ""
 
 #. module: base_usability
@@ -169,6 +217,14 @@ msgstr ""
 msgid "Tel:"
 msgstr ""
 
+#. module: base_usability
+#: model:ir.model.fields,help:base_usability.field_ir_actions_report__print_report_name
+msgid ""
+"This is the filename of the report going to download. Keep empty to not "
+"change the report filename. You can use a python expression with the "
+"'object' and 'time' variables."
+msgstr ""
+
 #. module: base_usability
 #: model:ir.model,name:base_usability.model_res_users
 msgid "Users"

base_usability/i18n/fr.po

@@ -6,15 +6,27 @@ msgid ""
 msgstr ""
 "Project-Id-Version: Odoo Server 14.0\n"
 "Report-Msgid-Bugs-To: \n"
-"POT-Creation-Date: 2021-07-01 10:02+0000\n"
-"PO-Revision-Date: 2021-07-01 12:15+0200\n"
-"Last-Translator: Alexis de Lattre <alexis@via.ecp.fr>\n"
+"POT-Creation-Date: 2024-03-26 21:27+0000\n"
+"PO-Revision-Date: 2024-03-26 21:27+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: base_usability
+#: code:addons/base_usability/models/res_company.py:0
+#, python-format
+msgid "%s with a capital of"
+msgstr "%s au capital de"
+
+#. module: base_usability
+#: code:addons/base_usability/models/res_company.py:0
+#, python-format
+msgid "APE:"
+msgstr "APE :"
+
 #. module: base_usability
 #: model:ir.model,name:base_usability.model_res_partner_bank
 msgid "Bank Accounts"
@@ -25,11 +37,22 @@ msgstr "Comptes bancaires"
 msgid "Bank Name"
 msgstr "Nom de la banque"
 
+#. module: base_usability
+#: code:addons/base_usability/models/res_company.py:0
+#, python-format
+msgid "Capital:"
+msgstr "Capital : "
+
 #. module: base_usability
 #: model:ir.model,name:base_usability.model_res_company
 msgid "Companies"
 msgstr "Sociétés"
 
+#. module: base_usability
+#: model_terms:ir.ui.view,arch_db:base_usability.ir_property_view_search
+msgid "Company"
+msgstr "Société"
+
 #. module: base_usability
 #: model:ir.model,name:base_usability.model_res_partner
 msgid "Contact"
@@ -42,11 +65,13 @@ msgstr "Devise"
 
 #. module: base_usability
 #: code:addons/base_usability/models/res_partner.py:0
+#: code:addons/base_usability/models/res_partner.py:0
 #, python-format
 msgid "Customer Number:"
 msgstr "N° client :"
 
 #. module: base_usability
+#: model:ir.model.fields,field_description:base_usability.field_ir_actions_report__display_name
 #: model:ir.model.fields,field_description:base_usability.field_ir_mail_server__display_name
 #: model:ir.model.fields,field_description:base_usability.field_ir_model__display_name
 #: model:ir.model.fields,field_description:base_usability.field_res_company__display_name
@@ -64,12 +89,24 @@ msgstr "Nom affiché"
 msgid "E-mail:"
 msgstr "E-mail :"
 
+#. module: base_usability
+#: code:addons/base_usability/models/res_company.py:0
+#, python-format
+msgid "EORI:"
+msgstr "EORI :"
+
+#. module: base_usability
+#: model_terms:ir.ui.view,arch_db:base_usability.ir_property_view_search
+msgid "Field"
+msgstr "Champ"
+
 #. module: base_usability
 #: model_terms:ir.ui.view,arch_db:base_usability.res_country_search
 msgid "Group By"
 msgstr "Grouper par"
 
 #. module: base_usability
+#: model:ir.model.fields,field_description:base_usability.field_ir_actions_report__id
 #: model:ir.model.fields,field_description:base_usability.field_ir_mail_server__id
 #: model:ir.model.fields,field_description:base_usability.field_ir_model__id
 #: model:ir.model.fields,field_description:base_usability.field_res_company__id
@@ -78,7 +115,7 @@ msgstr "Grouper par"
 #: model:ir.model.fields,field_description:base_usability.field_res_partner_category__id
 #: model:ir.model.fields,field_description:base_usability.field_res_users__id
 msgid "ID"
-msgstr "ID"
+msgstr ""
 
 #. module: base_usability
 #: model_terms:ir.ui.view,arch_db:base_usability.view_module_filter
@@ -86,6 +123,7 @@ msgid "Installable"
 msgstr "Installable"
 
 #. module: base_usability
+#: model:ir.model.fields,field_description:base_usability.field_ir_actions_report____last_update
 #: model:ir.model.fields,field_description:base_usability.field_ir_mail_server____last_update
 #: model:ir.model.fields,field_description:base_usability.field_ir_model____last_update
 #: model:ir.model.fields,field_description:base_usability.field_res_company____last_update
@@ -136,7 +174,12 @@ msgstr "Personne (utilisé pour cacher des entrées de menu natifs)"
 #. module: base_usability
 #: model:ir.model,name:base_usability.model_res_partner_category
 msgid "Partner Tags"
-msgstr "Étiquettes du partenaire"
+msgstr "Étiquettes contact"
+
+#. module: base_usability
+#: model:ir.model.fields,field_description:base_usability.field_ir_actions_report__print_report_name
+msgid "Printed Report Name"
+msgstr "Nom du rapport imprimé"
 
 #. module: base_usability
 #: model:ir.model.fields,field_description:base_usability.field_res_partner__ref
@@ -145,11 +188,18 @@ msgid "Reference"
 msgstr "Référence"
 
 #. module: base_usability
-#: model_terms:ir.ui.view,arch_db:base_usability.res_country_search
-msgid "Search Countries"
+#: model:ir.model,name:base_usability.model_ir_actions_report
+msgid "Report Action"
 msgstr ""
 
 #. module: base_usability
+#: code:addons/base_usability/models/res_company.py:0
+#, python-format
+msgid "SIRET:"
+msgstr "SIRET :"
+
+#. module: base_usability
+#: code:addons/base_usability/models/res_partner.py:0
 #: code:addons/base_usability/models/res_partner.py:0
 #, python-format
 msgid "Supplier Number:"
@@ -158,7 +208,7 @@ msgstr "N° fournisseur :"
 #. module: base_usability
 #: model:ir.model.fields,field_description:base_usability.field_res_partner_category__name
 msgid "Tag Name"
-msgstr "Nom de l'étiquette"
+msgstr "Libellé de l'étiquette"
 
 #. module: base_usability
 #: code:addons/base_usability/models/res_company.py:0
@@ -167,6 +217,14 @@ msgstr "Nom de l'étiquette"
 msgid "Tel:"
 msgstr "Tél :"
 
+#. module: base_usability
+#: model:ir.model.fields,help:base_usability.field_ir_actions_report__print_report_name
+msgid ""
+"This is the filename of the report going to download. Keep empty to not "
+"change the report filename. You can use a python expression with the "
+"'object' and 'time' variables."
+msgstr ""
+
 #. module: base_usability
 #: model:ir.model,name:base_usability.model_res_users
 msgid "Users"

base_usability/models/res_company.py

@@ -3,6 +3,7 @@
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 
 from odoo import api, models, _
+from odoo.tools.misc import format_amount
 
 
 class ResCompany(models.Model):
@@ -39,32 +40,78 @@ class ResCompany(models.Model):
                 'value': self.phone,
                 # http://www.fileformat.info/info/unicode/char/1f4de/index.htm
                 'icon': '\U0001F4DE',
-                'label': _('Tel:')},
+                'label': _('Tel:'),
+                },
             'email': {
                 'value': self.email,
                 # http://www.fileformat.info/info/unicode/char/2709/index.htm
                 'icon': '\u2709',
-                'label': _('E-mail:')},
+                'label': _('E-mail:'),
+                },
             'website': {
                 'value': self.website,
                 'icon': '\U0001f310',
-                'label': _('Website:')},
+                'label': _('Website:'),
+                },
             'vat': {
                 'value': self.vat,
-                'label': _('VAT:')},
+                'label': _('VAT:'),
+                },
+            'ape': {
+                'value': hasattr(self, 'ape') and self.ape or False,
+                'label': _('APE:'),
+                },
+            'siret': {
+                'value': hasattr(self, 'siret') and self.siret or False,
+                'label': _('SIRET:'),
+                },
+            'siren': {
+                'value': hasattr(self, 'siren') and self.siren or False,
+                'label': _('SIREN:'),
+                },
+            'eori': {
+                'value': self._get_eori(),
+                'label': _('EORI:'),
+                },
+            'capital': {
+                # 'capital_amount' added by base_company_extension
+                'value': hasattr(self, 'capital_amount') and self.capital_amount and format_amount(self.env, self.capital_amount, self.currency_id) or False,
+                'label': _('Capital:'),
                 }
+            }
+        # 'legal_type' added by base_company_extension
+        if hasattr(self, 'legal_type') and self.legal_type:
+            options['capital']['label'] = _('%s with a capital of') % self.legal_type
         return options
 
+    def _get_eori(self):
+        eori = False
+        if self.partner_id.country_id.code == 'FR' and hasattr(self, 'siret') and self.siret:
+            # Currently migrating from EORI-SIRET to EORI-SIREN :
+            # https://www.pwcavocats.com/fr/ealertes/ealertes-france/2023/avril/reforme-numero-eori-siren-siret.html
+            # But, for the moment, we continue to use EORI-SIRET
+            eori = f'FR{self.siret}'
+        return eori
+
     def _report_company_legal_name(self):
         '''Method inherited in the module base_company_extension'''
         self.ensure_one()
         return self.name
 
+    def _report_header_line_details(self):
+        """This method is designed to be inherited"""
+        # I decided not to put email in the default header because only a few very small
+        # companies have a generic company email address
+        line_details = [['phone', 'website', 'capital'], ['vat', 'siret', 'eori', 'ape']]
+        return line_details
+
     # for reports
     def _display_report_header(
-            self, line_details=[['phone', 'website'], ['vat']],
-            icon=True, line_separator=' - '):
+            self, line_details=None, icon=True, line_separator=' - '):
         self.ensure_one()
+        if line_details is None:
+            line_details = self._report_header_line_details()
+
         res = ''
         address = self.partner_id._display_address(without_company=True)
         address = address.replace('\n', ' - ')

base_usability/models/res_partner.py

@@ -125,6 +125,20 @@ class ResPartner(models.Model):
                 'label': _('Supplier Number:'),
                 },
             }
+        if hasattr(self, 'siren'):
+            options['siren'] = {
+                'value': self.siren,
+                'label': _("SIREN:"),
+                }
+        if hasattr(self, 'siret'):
+            if hasattr(self, 'siren'):  # l10n_fr_siret is installed
+                siret = self.siren and self.nic and self.siret or False
+            else:
+                siret = self.siret
+            options['siret'] = {
+                'value': siret,
+                'label': _("SIRET:"),
+                }
         res = []
         for detail in details:
             if options.get(detail) and options[detail]['value']:

base_usability/static/src/scss/form_view.scss

@@ -0,0 +1,10 @@
+.o_form_view {
+    .o_address_format {
+        .o_address_state {
+            margin-right: 0;
+        }
+        .o_address_zip {
+            margin-right: 2%;
+        }
+    }
+}

base_usability/views/assets.xml

@@ -0,0 +1,15 @@
+<?xml version="1.0" encoding="UTF-8" ?>
+<odoo>
+    <template
+        id="assets_backend"
+        name="web_sheet_full_width"
+        inherit_id="web.assets_backend"
+    >
+        <xpath expr="." position="inside">
+            <link
+                rel="stylesheet"
+                href="/base_usability/static/src/scss/form_view.scss"
+            />
+        </xpath>
+    </template>
+</odoo>

base_usability/views/res_partner.xml

@@ -20,6 +20,21 @@
         <div attrs="{'invisible': [('same_vat_partner_id', '=', False)]}" position="attributes">
             <attribute name="class">alert alert-warning</attribute>
         </div>
+        <!-- Native order: city / state_id / zip. New order: zip / city / state_id -->
+        <xpath expr="//div[hasclass('o_address_format')]/field[@name='city']" position="before">
+            <field name="zip" position="move"/>
+        </xpath>
+    </field>
+</record>
+
+<record id="res_partner_view_form_private" model="ir.ui.view">
+    <field name="model">res.partner</field>
+    <field name="inherit_id" ref="base.res_partner_view_form_private"/>
+    <field name="arch" type="xml">
+        <xpath expr="//field[@name='bank_ids']/tree/field[@name='acc_holder_name']" position="attributes">
+            <attribute name="invisible">0</attribute>
+            <attribute name="optional">hide</attribute>
+        </xpath>
     </field>
 </record>
 

base_usability/views/res_partner_bank.xml

@@ -16,6 +16,10 @@
         <field name="bank_name" position="after">
             <field name="bank_id"/>
         </field>
+        <field name="acc_holder_name" position="attributes">
+            <attribute name="invisible">0</attribute>
+            <attribute name="optional">hide</attribute>
+        </field>
     </field>
 </record>
 </odoo>

base_usability/web-advanced_search_startswith.diff

@@ -0,0 +1,28 @@
+diff --git a/addons/web/static/src/js/control_panel/custom_filter_item.js b/addons/web/static/src/js/control_panel/custom_filter_item.js
+index 6682b660b78..b2ff23344d9 100644
+--- a/addons/web/static/src/js/control_panel/custom_filter_item.js
++++ b/addons/web/static/src/js/control_panel/custom_filter_item.js
+@@ -180,6 +180,10 @@ odoo.define('web.CustomFilterItem', function (require) {
+                         [field.name, '>=', domainValue[0]],
+                         [field.name, '<=', domainValue[1]]
+                     );
++                } else if (operator.symbol === 'startswith') {
++                    domainArray.push([field.name, '=ilike', domainValue[0] + '%']);
++                } else if (operator.symbol === 'endswith') {
++                    domainArray.push([field.name, '=ilike', '%' + domainValue[0]]);
+                 } else {
+                     domainArray.push([field.name, operator.symbol, domainValue[0]]);
+                 }
+diff --git a/addons/web/static/src/js/control_panel/search_utils.js b/addons/web/static/src/js/control_panel/search_utils.js
+index 8fce5b23ef6..d240d2e1fb2 100644
+--- a/addons/web/static/src/js/control_panel/search_utils.js
++++ b/addons/web/static/src/js/control_panel/search_utils.js
+@@ -18,6 +18,8 @@ odoo.define('web.searchUtils', function (require) {
+         char: [
+             { symbol: "ilike", description: _lt("contains") },
+             { symbol: "not ilike", description: _lt("doesn't contain") },
++            { symbol: "startswith", description: _lt("starts with") },
++            { symbol: "endswith", description: _lt("ends with") },
+             { symbol: "=", description: _lt("is equal to") },
+             { symbol: "!=", description: _lt("is not equal to") },
+             { symbol: "!=", description: _lt("is set"), value: false },

crm_usability/views/crm_lead.xml

@@ -19,4 +19,10 @@
     </field>
 </record>
 
+<!-- By default, the menu entry "CRM > Sales > My Activities" is limited to
+the sales Manager. The Sale user should be able to see it -->
+<record id="crm.crm_lead_menu_my_activities" model="ir.ui.menu">
+    <field name="groups_id" eval="[(6, 0, [])]"/>
+</record>
+
 </odoo>

delivery_usability/__manifest__.py

@@ -22,6 +22,7 @@ This module has been written by Alexis de Lattre from Akretion <alexis.delattre@
     'depends': ['delivery'],
     'data': [
         'views/stock_picking.xml',
+        'views/product_packaging.xml',
         ],
     'installable': True,
 }

delivery_usability/models/__init__.py

@@ -1 +1,4 @@
+from . import product_packaging
 from . import stock_picking
+from . import stock_move
+from . import stock_quant_package

delivery_usability/models/product_packaging.py

@@ -0,0 +1,31 @@
+# Copyright 2018-2021 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 odoo import fields, models
+
+
+class ProductPackaging(models.Model):
+    _inherit = 'product.packaging'
+
+    # product.packaging is defined in the 'product' module and enhanced in the 'delivery' module
+    # I used to make the improvements on the datamodel of product.packaging in the OCA module
+    # 'stock_packaging_usability_pp' from OCA/stock-logistics-tracking,
+    # but I eventually figured out that the feature provided by 'stock_packaging_usability_pp'
+    # was native in the 'delivery' module via the wizard choose.delivery.package.
+    # So I stopped using 'stock_packaging_usability_pp' and I moved the datamodel changes
+    # here in the module 'delivery_usability'
+    name = fields.Char(translate=True)
+    weight = fields.Float(digits="Stock Weight", string="Empty Package Weight")
+    active = fields.Boolean(default=True)
+    # packaging_type is important, in particular for pallets for which
+    # we need a special implementation to enter the height
+    packaging_type = fields.Selection(
+        [
+            ("unit", "Unit"),
+            ("pack", "Pack"),
+            ("box", "Box"),
+            ("pallet", "Pallet"),
+        ],
+        string="Type",
+    )

delivery_usability/models/stock_move.py

@@ -0,0 +1,23 @@
+# Copyright 2019 Akretion France (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 odoo import api, models
+
+
+class StockMove(models.Model):
+    _inherit = "stock.move"
+
+    # Fixing bug https://github.com/odoo/odoo/issues/34702
+    @api.depends("product_id", "product_uom_qty", "product_uom")
+    def _cal_move_weight(self):
+        weight_uom_categ = self.env.ref("uom.product_uom_categ_kgm")
+        kg_uom = self.env.ref("uom.product_uom_kgm")
+        for move in self:
+            if move.product_id.uom_id.category_id == weight_uom_categ:
+                move.weight = move.product_id.uom_id._compute_quantity(
+                    move.product_qty, kg_uom
+                )
+            else:
+                move.weight = move.product_qty * move.product_id.weight

delivery_usability/models/stock_quant_package.py

@@ -0,0 +1,68 @@
+# Copyright 2019-2024 Akretion France (https://www.akretion.com)
+# @author Alexis de Lattre <alexis.delattre@akretion.com>
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+
+
+from odoo import fields, models
+from odoo.tools import float_is_zero
+
+
+class StockQuantPackage(models.Model):
+    _inherit = "stock.quant.package"
+
+    # These 2 fields are defined in the 'delivery' module but they forgot
+    # the decimal precision
+    shipping_weight = fields.Float(digits="Stock Weight")
+    weight = fields.Float(digits="Stock Weight")
+
+    # Fixing bug https://github.com/odoo/odoo/issues/34702
+    # and take into account the weight of the packaging
+    # WARNING: this method _compute_weight() is also inherited by the OCA module
+    # base_delivery_carrier_label so if you use that module, you should copy
+    # that piece of code in a custom module that depend on delivery_usability
+    # and base_delivery_carrier_label
+    def _compute_weight(self):
+        smlo = self.env["stock.move.line"]
+        weight_uom_categ = self.env.ref("uom.product_uom_categ_kgm")
+        kg_uom = self.env.ref("uom.product_uom_kgm")
+        weight_prec = self.env['decimal.precision'].precision_get('Stock Weight')
+        for package in self:
+            # if the weight of the package has been measured,
+            # it is written in shipping_weight
+            if not float_is_zero(package.shipping_weight, precision_digits=weight_prec):
+                weight = package.shipping_weight
+            # otherwise, we compute the theorical weight from the weight of the products
+            # and the weight of the packaging
+            # Since Odoo v11, consu products don't create quants, so I can't loop
+            # on pack.quant_ids to get all the items inside a package: I have to
+            # get the picking, then loop on the stock.move.line of that picking
+            # linked to that package
+            else:
+                weight = 0.0
+                # the package can be seen in a return
+                # So I get the picking of it's first appearance
+                domain = [
+                    ("result_package_id", "=", package.id),
+                    ("product_id", "!=", False),
+                    ]
+                first_move_line = smlo.search(
+                    domain + [('picking_id', '!=', False)], limit=1, order='id')
+                if first_move_line:
+                    picking_id = first_move_line.picking_id.id
+                    current_picking_move_line_ids = smlo.search(
+                        domain + [("picking_id", "=", picking_id)])
+                    for ml in current_picking_move_line_ids:
+                        if ml.product_uom_id.category_id == weight_uom_categ:
+                            weight += ml.product_uom_id._compute_quantity(
+                                ml.qty_done, kg_uom
+                            )
+                        else:
+                            weight += (
+                                ml.product_uom_id._compute_quantity(
+                                    ml.qty_done, ml.product_id.uom_id
+                                )
+                                * ml.product_id.weight
+                            )
+                if package.packaging_id:
+                    weight += package.packaging_id.weight
+            package.weight = weight

delivery_usability/views/product_packaging.xml

@@ -0,0 +1,79 @@
+<?xml version="1.0" encoding="utf-8" ?>
+<!--
+  Copyright 2019-2024 Akretion France (http://www.akretion.com/)
+  @author Alexis de Lattre <alexis.delattre@akretion.com>
+  License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+-->
+<odoo>
+
+
+<!-- I don't know why the form view of product.packaging in the delivery
+     module has "<field name="inherit_id" eval="False"/>"
+    instead of a standard inherit of product.product_packaging_form_view -->
+<record id="product_packaging_delivery_form" model="ir.ui.view">
+    <field name="name">stock_packaging_usability_pp.product.packaging.form</field>
+    <field name="model">product.packaging</field>
+    <field name="inherit_id" ref="delivery.product_packaging_delivery_form" />
+    <field name="arch" type="xml">
+        <field name="package_carrier_type" position="after">
+            <field name="packaging_type" />
+            <field name="active" invisible="1" />
+        </field>
+        <label for="max_weight" position="before">
+            <label for="weight" />
+            <div class="o_row" name="weight">
+                <field name="weight" />
+                <span><field name="weight_uom_name" /></span>
+            </div>
+        </label>
+        <label for="name" position="before">
+            <widget
+                    name="web_ribbon"
+                    title="Archived"
+                    bg_color="bg-danger"
+                    attrs="{'invisible': [('active', '=', True)]}"
+                />
+        </label>
+    </field>
+</record>
+
+<record id="product_packaging_delivery_tree" model="ir.ui.view">
+    <field name="name">stock_packaging_usability_pp.product.packaging.tree</field>
+    <field name="model">product.packaging</field>
+    <field name="inherit_id" ref="delivery.product_packaging_delivery_tree" />
+    <field name="arch" type="xml">
+        <field name="name" position="after">
+            <field name="packaging_type" optional="show" />
+        </field>
+        <field name="max_weight" position="before">
+            <field name="weight" optional="show" />
+        </field>
+    </field>
+</record>
+
+<!-- There is no native serch view for product.packaging -->
+<record id="product_packaging_search" model="ir.ui.view">
+    <field name="name">product.packaging.search</field>
+    <field name="model">product.packaging</field>
+    <field name="arch" type="xml">
+        <search>
+            <field name="name" />
+            <separator />
+            <filter
+                    string="Archived"
+                    name="inactive"
+                    domain="[('active', '=', False)]"
+                />
+            <group name="groupby">
+                <filter
+                        name="packaging_type_groupby"
+                        string="Packaging Type"
+                        context="{'group_by': 'packaging_type'}"
+                    />
+            </group>
+        </search>
+    </field>
+</record>
+
+
+</odoo>

developer_menu/__manifest__.py

@@ -1,12 +1,12 @@
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 
 {
-    'name': 'Developer Menu',
-    'version': '12.0.0.0.0',
-    'category': 'Tools',
-    'license': 'AGPL-3',
-    'summary': "Menu Shortcut for developer usage",
-    'description': """
+    "name": "Developer Menu",
+    "version": "14.0.0.0.0",
+    "category": "Tools",
+    "license": "AGPL-3",
+    "summary": "Menu Shortcut for developer usage",
+    "description": """
 Developer menu
 ==============
 
@@ -21,11 +21,13 @@ near `Technical` menu
 This module has been written by David Béal
 from Akretion <david.beal@akretion.com>.
     """,
-    'author': 'Akretion',
-    'website': 'http://www.akretion.com',
-    'depends': ['mail'],
-    'data': [
-        'menu_view.xml'
+    "author": "Akretion",
+    "website": "https://www.akretion.com",
+    "depends": [
+        "mail",
     ],
-    'installable': False,
+    "data": [
+        "menu_view.xml",
+    ],
+    "installable": True,
 }

developer_menu/menu_view.xml

@@ -2,7 +2,7 @@
 
 <odoo>
 
-<menuitem id="conf_tech" parent="base.menu_administration" name="🧰" groups="base.group_erp_manager" sequence="100"/>
+<menuitem id="conf_tech" parent="base.menu_administration" name="🧰" groups="base.group_erp_manager" sequence="1"/>
 <menuitem id="model" name="Model" parent="conf_tech" action="base.action_model_model" sequence="10"/>
 <menuitem id="view" name="View" parent="conf_tech" action="base.action_ui_view" sequence="20" />
 <menuitem id="rec_rule" name="Record Rule" parent="conf_tech" action="base.action_rule" sequence="30" />

eradicate_quick_create/hooks.py

@@ -10,7 +10,7 @@ def web_m2x_options_create(cr, registry):
     env = Environment(cr, SUPERUSER_ID, {})
     config_parameter = env['ir.config_parameter'].search(
         [('key', '=', 'web_m2x_options.create')])
-    if config_parameter and config_parameter.value != 'False':
+    if config_parameter:
         config_parameter.write({'value': 'False'})
     else:
         env['ir.config_parameter'].create({

intrastat_product_type/__init__.py

@@ -1,4 +1 @@
-# -*- coding: utf-8 -*-
-
 from . import intrastat_product_type
-from .post_install import set_intrastat_type_on_products

intrastat_product_type/__manifest__.py

@@ -4,7 +4,7 @@
 
 {
     'name': 'Intrastat Product Type',
-    'version': '12.0.1.0.0',
+    'version': '14.0.1.0.0',
     'category': 'Accounting',
     'license': 'AGPL-3',
     'summary': 'Adds a special field Intrastat Type on Products',
@@ -19,9 +19,8 @@ This module adds a field *Intrastat Type* on the Product Form with 2 possible op
 This module has been written by Alexis de Lattre from Akretion <alexis.delattre@akretion.com>.
     """,
     'author': 'Akretion',
-    'website': 'http://www.akretion.com',
+    'website': 'https://github.com/akretion/odoo-usability',
     'depends': ['intrastat_product', 'l10n_fr_intrastat_service'],
     'data': ['product_view.xml'],
-    'post_init_hook': 'set_intrastat_type_on_products',
-    'installable': False,
+    'installable': True,
 }

intrastat_product_type/intrastat_product_type.py

@@ -1,5 +1,4 @@
-# -*- coding: utf-8 -*-
-# Copyright 2016-2019 Akretion (http://www.akretion.com)
+# Copyright 2016-2023 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>
 
@@ -14,50 +13,36 @@ class ProductTemplate(models.Model):
     intrastat_type = fields.Selection([
         ('product', 'Product'),
         ('service', 'Service'),
-        ], string='Intrastat Type', default='product', required=True,
+        ],
+        compute='_compute_intrastat_type', readonly=False, store=True,
+        required=True, string='Intrastat Type',
         help="Type of product used for the intrastat declarations. "
         "For example, you can configure a product with "
         "'Product Type' = 'Consumable' and 'Intrastat Type' = 'Service'.")
 
-    @api.multi
     @api.constrains('type', 'intrastat_type')
     def check_intrastat_type(self):
         for pt in self:
             if pt.intrastat_type == 'product' and pt.type == 'service':
                 raise ValidationError(_(
-                    "On the product %s, you cannot set Product Type to "
-                    "'Service' and Intrastat Type to 'Product'.") % pt.name)
+                    "On the product '%s', you cannot set Product Type to "
+                    "'Service' and Intrastat Type to 'Product'.")
+                    % pt.display_name)
             if pt.intrastat_type == 'service' and pt.type == 'product':
                 raise ValidationError(_(
-                    "On the product %s, you cannot set Intrastat Type to "
+                    "On the product '%s', you cannot set Intrastat Type to "
                     "'Service' and Product Type to 'Stockable product' "
                     "(but you can set Product Type to 'Consumable' or "
-                    "'Service').") % pt.name)
+                    "'Service').") % pt.display_name)
 
-    @api.onchange('type')
-    def intrastat_type_onchange(self):
-        if self.type in ('product', 'consu'):
-            self.intrastat_type = 'product'
-        elif self.type == 'service':
-            self.intrastat_type = 'service'
-
-    @api.model
-    def create(self, vals):
-        if vals.get('type'):
-            if not vals.get('intrastat_type'):
-                if vals['type'] in ('product', 'consu'):
-                    vals['intrastat_type'] = 'product'
-                elif vals['type'] == 'service':
-                    vals['intrastat_type'] = 'service'
-            elif (
-                    vals.get('intrastat_type') == 'product' and
-                    vals['type'] == 'service'):
-                # usefull because intrastat_type = 'product' by default and
-                # wizards in other modules that don't depend on this module
-                # (e.g. sale_rental) may create a product with only
-                # {'type': 'service'} and no 'intrastat_type'
-                vals['intrastat_type'] = 'service'
-        return super(ProductTemplate, self).create(vals)
+    @api.depends('type')
+    def _compute_intrastat_type(self):
+        for pt in self:
+            if pt.type in ('product', 'consu'):
+                intrastat_type = 'product'
+            else:
+                intrastat_type = 'service'
+            pt.intrastat_type = intrastat_type
 
 
 class L10nFrIntrastatServiceDeclaration(models.Model):

intrastat_product_type/post_install.py

@@ -1,10 +0,0 @@
-# -*- coding: utf-8 -*-
-# Copyright 2016-2019 Akretion (Alexis de Lattre <alexis.delattre@akretion.com>)
-# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
-
-
-def set_intrastat_type_on_products(cr, registry):
-    cr.execute(
-        "UPDATE product_template SET intrastat_type='service' "
-        "WHERE type='service'")
-    return

intrastat_product_type/product_view.xml

@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="utf-8"?>
 <!--
-  Copyright 2016-2019 Akretion (http://www.akretion.com/)
+  Copyright 2016-2023 Akretion France (http://www.akretion.com/)
   @author Alexis de Lattre <alexis.delattre@akretion.com>
   License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 -->

mail_usability/__manifest__.py

@@ -23,7 +23,7 @@ Small usability improvements on mails:
     'website': 'http://www.akretion.com',
     'depends': ['mail'],
     'data': [
-        #'views/mail_view.xml',
+        'views/mail_activity.xml',
         #'data/mail_data.xml',
         #'wizard/email_template_preview_view.xml',
         #'wizard/mail_compose_message_view.xml',

mail_usability/models/__init__.py

@@ -1,2 +1,3 @@
 from . import res_partner
+from . import mail_activity
 from . import mail_template

mail_usability/models/mail_activity.py

@@ -0,0 +1,24 @@
+# Copyright 2023 Akretion France (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 odoo import fields, models
+
+
+class MailActivity(models.Model):
+    _inherit = 'mail.activity'
+
+    res_model_name = fields.Char(related='res_model_id.name', string='Document Type')
+
+    def jump_to_record(self):
+        self.ensure_one()
+        action = {}
+        if self.res_id and self.res_model and self.res_name:
+            action.update({
+                'type': 'ir.actions.act_window',
+                'name': self.res_name,
+                'res_model': self.res_model,
+                'view_mode': 'form',
+                'res_id': self.res_id,
+                })
+        return action

mail_usability/views/mail_activity.xml

@@ -0,0 +1,52 @@
+<?xml version="1.0" encoding="utf-8"?>
+<!--
+  Copyright 2023 Akretion France (http://www.akretion.com/)
+  @author: Alexis de Lattre <alexis.delattre@akretion.com>
+  License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+-->
+
+<odoo>
+
+
+<record id="mail_activity_view_search" model="ir.ui.view">
+    <field name="model">mail.activity</field>
+    <field name="inherit_id" ref="mail.mail_activity_view_search"/>
+    <field name="arch" type="xml">
+        <filter name="activities_upcoming_all" position="after">
+            <separator/>
+            <filter string="My Activities" domain="[('user_id', '=', uid)]" name="my_activities"/>
+        </filter>
+    </field>
+</record>
+
+<record id="mail_activity_view_tree" model="ir.ui.view">
+    <field name="model">mail.activity</field>
+    <field name="inherit_id" ref="mail.mail_activity_view_tree"/>
+    <field name="arch" type="xml">
+        <field name="summary" position="after">
+            <field name="user_id" optional="show" widget="many2one_avatar_user"/>
+        </field>
+        <field name="res_name" position="before">
+            <button name="jump_to_record" type="object" attrs="{'invisible': [('res_name', '=', False)]}" icon="fa-binoculars" string="Go to source" class="text-warning" invisible="not context.get('mail_activity_main_view')"/>
+        </field>
+        <field name="res_name" position="after">
+            <field name="res_model_name" invisible="not context.get('mail_activity_main_view')"/>
+        </field>
+    </field>
+</record>
+
+<record id="mail_activity_view_form_popup" model="ir.ui.view">
+    <field name="model">mail.activity</field>
+    <field name="inherit_id" ref="mail.mail_activity_view_form_popup"/>
+    <field name="arch" type="xml">
+        <field name="activity_type_id" position="before">
+            <field name="res_name" invisible="not context.get('mail_activity_main_view')"/>
+            <field name="res_model_name" invisible="not context.get('mail_activity_main_view')"/>
+        </field>
+        <button name="action_close_dialog" position="before">
+            <button name="jump_to_record" type="object" attrs="{'invisible': [('res_id', '=', False)]}" class="btn-primary" string="Go to source" invisible="not context.get('mail_activity_main_view')"/>
+        </button>
+    </field>
+</record>
+
+</odoo>

mrp_average_cost/__manifest__.py

@@ -1,10 +1,10 @@
-# Copyright (C) 2016-2019 Akretion (http://www.akretion.com)
+# Copyright (C) 2016-2024 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': 'MRP Average Cost',
-    'version': '12.0.1.0.0',  # WARNING: we'll probably not port this module to v14, because part of its feature is now provided by the module mrp_account
+    'version': '14.0.1.0.0',
     'category': 'Manufactuing',
     'license': 'AGPL-3',
     'summary': 'Update standard_price upon validation of a manufacturing order',
@@ -12,16 +12,16 @@
 MRP Average Cost
 ================
 
-By default, the official stock module updates the standard_price of a product that has costing_method = 'average' when validating an incoming picking. But the official 'mrp' module doesn't do that when you validate a manufactuging order.
+I initially developped this module for Odoo 12.0, when the module mrp_account didn't exist, so Odoo didn't support the update of the standard cost of a manufactured product.
 
-This module adds this feature : when you validate a manufacturing order of a product that has costing method = 'average', the standard_price of the product will be updated by taking into account the standard_price of each raw material and also a number of work hours defined on the BOM.
+In the mrp_account module, you must use workcenters to take the labor costs into account. This module aims at encoding theorical labor costs on the BOM and using it to compute the cost of the finished product.
 
-Together with this module, I recommend the use of my module product_usability, available in the same branch, which contains a backport of the model product.price.history from v8 to v7.
+With this module, when you validate a manufacturing order of a product that has costing method = 'average', the standard_price of the product will be updated by taking into account the standard_price of each raw material and also a number of work hours defined on the BOM plus the extra cost defined of the BOM.
 
 This module has been written by Alexis de Lattre from Akretion <alexis.delattre@akretion.com>.
     """,
     'author': 'Akretion',
-    'website': 'http://www.akretion.com',
+    'website': 'https://github.com/akretion/odoo-usability',
     'depends': ['mrp'],
     'data': [
         'security/mrp_average_cost_security.xml',
@@ -29,5 +29,5 @@ This module has been written by Alexis de Lattre from Akretion <alexis.delattre@
         'data/mrp_data.xml',
         'views/mrp_view.xml',
         ],
-    'installable': False,
+    'installable': True,
 }

mrp_average_cost/models/mrp.py

@@ -1,11 +1,9 @@
-# Copyright (C) 2016-2019 Akretion (http://www.akretion.com)
+# Copyright (C) 2016-2024 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 odoo import models, fields, api, _
-import odoo.addons.decimal_precision as dp
-from odoo.exceptions import UserError
-from odoo.tools import float_compare, float_is_zero
+from odoo.tools import float_compare
 import logging
 
 logger = logging.getLogger(__name__)
@@ -17,23 +15,19 @@ class MrpBomLabourLine(models.Model):
 
     bom_id = fields.Many2one(
         comodel_name='mrp.bom',
-        string='Labour Lines',
+        string='Bill of Material',
         ondelete='cascade')
-
     labour_time = fields.Float(
         string='Labour Time',
         required=True,
-        digits=dp.get_precision('Labour Hours'),
+        digits='Labour Hours',
         help="Average labour time for the production of "
              "items of the BOM, in hours.")
-
     labour_cost_profile_id = fields.Many2one(
         comodel_name='labour.cost.profile',
         string='Labour Cost Profile',
         required=True)
-
-    note = fields.Text(
-        string='Note')
+    note = fields.Text()
 
     _sql_constraints = [(
         'labour_time_positive',
@@ -44,6 +38,26 @@ class MrpBomLabourLine(models.Model):
 class MrpBom(models.Model):
     _inherit = 'mrp.bom'
 
+    labour_line_ids = fields.One2many(
+        'mrp.bom.labour.line', 'bom_id', string='Labour Lines')
+    total_labour_cost = fields.Float(
+        compute='_compute_total_labour_cost', digits='Product Price', store=True)
+    extra_cost = fields.Float(
+        tracking=True, digits='Product Price',
+        help="Extra cost for the production of the quantity of "
+        "items of the BOM, in company currency. "
+        "You can use this field to enter the cost of the consumables "
+        "that are used to produce the product but are not listed in "
+        "the BOM")
+    total_components_cost = fields.Float(
+        compute='_compute_total_cost', digits='Product Price')
+    total_cost = fields.Float(
+        compute='_compute_total_cost', digits='Product Price',
+        help="Total cost for the quantity and unit of measure of the bill of material. "
+        "Total Cost = Total Components Cost + Total Labour Cost + Extra Cost")
+    company_currency_id = fields.Many2one(
+        related='company_id.currency_id', string='Company Currency')
+
     @api.depends(
         'labour_line_ids.labour_time',
         'labour_line_ids.labour_cost_profile_id.hour_cost')
@@ -70,107 +84,77 @@ class MrpBom(models.Model):
             bom.total_components_cost = comp_cost
             bom.total_cost = total_cost
 
-    labour_line_ids = fields.One2many(
-        'mrp.bom.labour.line', 'bom_id', string='Labour Lines')
-    total_labour_cost = fields.Float(
-        compute='_compute_total_labour_cost', readonly=True,
-        digits=dp.get_precision('Product Price'),
-        string="Total Labour Cost", store=True)
-    extra_cost = fields.Float(
-        string='Extra Cost', track_visibility='onchange',
-        digits=dp.get_precision('Product Price'),
-        help="Extra cost for the production of the quantity of "
-        "items of the BOM, in company currency. "
-        "You can use this field to enter the cost of the consumables "
-        "that are used to produce the product but are not listed in "
-        "the BOM")
-    total_components_cost = fields.Float(
-        compute='_compute_total_cost', readonly=True,
-        digits=dp.get_precision('Product Price'),
-        string='Total Components Cost')
-    total_cost = fields.Float(
-        compute='_compute_total_cost', readonly=True,
-        string='Total Cost',
-        digits=dp.get_precision('Product Price'),
-        help="Total Cost = Total Components Cost + "
-        "Total Labour Cost + Extra Cost")
-    company_currency_id = fields.Many2one(
-        related='company_id.currency_id', string='Company Currency')
-
     @api.model
     def _phantom_update_product_standard_price(self):
-        logger.info('Start to auto-update cost price from phantom bom')
+        logger.info('Start to auto-update cost price from phantom boms')
         boms = self.search([('type', '=', 'phantom')])
-        boms.with_context(
-            product_price_history_origin='Automatic update of Phantom BOMs')\
-            .manual_update_product_standard_price()
-        logger.info('End of the auto-update cost price from phantom bom')
-        return True
+        boms.manual_update_product_standard_price()
+        logger.info('End of the auto-update cost price from phantom boms')
 
     def manual_update_product_standard_price(self):
-        if 'product_price_history_origin' not in self._context:
-            self = self.with_context(
-                product_price_history_origin='Manual update from BOM')
-        precision = self.env['decimal.precision'].precision_get(
+        prec = self.env['decimal.precision'].precision_get(
             'Product Price')
         for bom in self:
-            wproduct = bom.product_id
-            if not wproduct:
-                wproduct = bom.product_tmpl_id
-            if float_compare(
-                    wproduct.standard_price, bom.total_cost,
-                    precision_digits=precision):
-                wproduct.with_context().write(
-                    {'standard_price': bom.total_cost})
+            if bom.product_id:
+                products = bom.product_id
+            else:
+                products = bom.product_tmpl_id.product_variant_ids
+            for product in products:
+                standard_price = product._compute_bom_price(bom)
+                if float_compare(product.standard_price, standard_price, precision_digits=prec):
+                    product.write({'standard_price': standard_price})
                     logger.info(
                         'Cost price updated to %s on product %s',
-                    bom.total_cost, wproduct.display_name)
-        return True
+                        standard_price, product.display_name)
 
 
 class MrpBomLine(models.Model):
     _inherit = 'mrp.bom.line'
 
-    standard_price = fields.Float(
-        related='product_id.standard_price',
-        readonly=True,
-        string='Standard Price')
+    standard_price = fields.Float(related='product_id.standard_price')
+
+
+class ProductProduct(models.Model):
+    _inherit = 'product.product'
+
+    def _compute_bom_price(self, bom, boms_to_recompute=False):
+        # Native method of mrp_account
+        # WARNING dirty hack ; I hope it doesn't break too many things
+        self.ensure_one()
+        bom_cost_per_unit_in_product_uom = 0
+        qty_product_uom = bom.product_uom_id._compute_quantity(bom.product_qty, self.uom_id)
+        if qty_product_uom:
+            bom_cost_per_unit_in_product_uom = bom.total_cost / qty_product_uom
+        return bom_cost_per_unit_in_product_uom
 
 
 class LabourCostProfile(models.Model):
     _name = 'labour.cost.profile'
-    _inherit = ['mail.thread']
+    _inherit = ['mail.thread', 'mail.activity.mixin']
     _description = 'Labour Cost Profile'
 
     name = fields.Char(
-        string='Name',
         required=True,
-        track_visibility='onchange')
-
+        tracking=True)
     hour_cost = fields.Float(
         string='Cost per Hour',
         required=True,
-        digits=dp.get_precision('Product Price'),
-        track_visibility='onchange',
+        digits='Product Price',
+        tracking=True,
         help="Labour cost per hour per person in company currency")
 
     company_id = fields.Many2one(
-        comodel_name='res.company',
-        string='Company',
-        required=True,
-        default=lambda self: self.env['res.company']._company_default_get())
+        comodel_name='res.company', required=True,
+        default=lambda self: self.env.company)
 
     company_currency_id = fields.Many2one(
-        related='company_id.currency_id',
-        readonly=True,
-        store=True,
-        string='Company Currency')
+        related='company_id.currency_id', store=True, string='Company Currency')
 
     @api.depends('name', 'hour_cost', 'company_currency_id.symbol')
     def name_get(self):
         res = []
         for record in self:
-            res.append((record.id, u'%s (%s %s)' % (
+            res.append((record.id, '%s (%s %s)' % (
                 record.name, record.hour_cost,
                 record.company_currency_id.symbol)))
         return res
@@ -179,93 +163,25 @@ class LabourCostProfile(models.Model):
 class MrpProduction(models.Model):
     _inherit = 'mrp.production'
 
-    unit_cost = fields.Float(
-        string='Unit Cost', readonly=True,
-        digits=dp.get_precision('Product Price'),
-        help="This cost per unit in the unit of measure of the product "
-        "in company currency takes into account "
-        "the cost of the raw materials and the labour cost defined on"
-        "the BOM.")
-
     company_currency_id = fields.Many2one(
-        related='company_id.currency_id', readonly=True,
-        string='Company Currency')
+        related='company_id.currency_id', string='Company Currency')
+    # extra_cost is per unit in the UoM of the mrp.production (product_uom_id)
+    extra_cost = fields.Float(
+        compute='_compute_extra_cost', store=True, readonly=False,
+        help="For a regular production order, it takes into account the labor cost "
+        "and the extra cost defined on the bill of material.")
 
-    def compute_order_unit_cost(self):
-        self.ensure_one()
-        mo_total_price = 0.0  # In the UoM of the M0
-        labor_cost_per_unit = 0.0  # In the UoM of the product
-        extra_cost_per_unit = 0.0  # In the UoM of the product
-        subcontract_cost_per_unit = 0.0
-        # I read the raw materials MO, not on BOM, in order to make
-        # it work with the "dynamic" BOMs (few raw material are auto-added
-        # on the fly on MO)
-        prec = self.env['decimal.precision'].precision_get(
-            'Product Unit of Measure')
-        for raw_smove in self.move_raw_ids:
-            # I don't filter on state, in order to make it work with
-            # partial productions
-            # For partial productions, mo.product_qty is not updated
-            # so we compute with fully qty and we compute with all raw
-            # materials (consumed or not), so it gives a good price
-            # per unit at the end
-            raw_price = raw_smove.product_id.standard_price
-            raw_material_cost = raw_price * raw_smove.product_qty
-            logger.info(
-                'MO %s product %s: raw_material_cost=%s',
-                self.name, raw_smove.product_id.display_name,
-                raw_material_cost)
-            mo_total_price += raw_material_cost
-        if self.bom_id:
-            bom = self.bom_id
-            # if not bom.total_labour_cost:
-            #    raise orm.except_orm(
-            #        _('Error:'),
-            #        _("Total Labor Cost is 0 on bill of material '%s'.")
-            #        % bom.name)
-            if float_is_zero(bom.product_qty, precision_digits=prec):
-                raise UserError(_(
-                    "Missing Product Quantity on bill of material '%s'.")
-                    % bom.display_name)
-            bom_qty_product_uom = bom.product_uom_id._compute_quantity(
-                bom.product_qty, bom.product_tmpl_id.uom_id)
-            assert bom_qty_product_uom > 0, 'BoM qty should be positive'
-            labor_cost_per_unit = bom.total_labour_cost / bom_qty_product_uom
-            extra_cost_per_unit = bom.extra_cost / bom_qty_product_uom
-            if bom.type == 'subcontract':
-                one_finished_move = self.env['stock.move'].search([
-                    ('production_id', '=', self.id),
-                    ('product_id', '=', self.product_id.id),
-                    ('move_dest_ids', '!=', False)], limit=1)
-                if one_finished_move:
-                    subcontract_cost_per_unit = one_finished_move.move_dest_ids[0].price_unit
-        # mo_standard_price and labor_cost_per_unit are
-        # in the UoM of the product (not of the MO/BOM)
-        mo_qty_product_uom = self.product_uom_id._compute_quantity(
-            self.product_qty, self.product_id.uom_id)
-        assert mo_qty_product_uom > 0, 'MO qty should be positive'
-        mo_standard_price = mo_total_price / mo_qty_product_uom
-        logger.info(
-            'MO %s: labor_cost_per_unit=%s extra_cost_per_unit=%s '
-            'subcontract_cost_per_unit=%s',
-            self.name, labor_cost_per_unit, extra_cost_per_unit,
-            subcontract_cost_per_unit)
-        mo_standard_price += labor_cost_per_unit
-        mo_standard_price += extra_cost_per_unit
-        mo_standard_price += subcontract_cost_per_unit
-        return mo_standard_price
+    # Strategy for v14 : we write labor costs and bom's extra cost on the native field extra_cost
+    # of mrp.production => it is automatically added by the code of mrp_account
 
-    def post_inventory(self):
-        '''This is the method where _action_done() is called on finished move
-        So we write on 'price_unit' of the finished move and THEN we call
-        super() which will call _action_done() which itself calls
-        product_price_update_before_done()'''
-        for order in self:
-            if order.product_id.cost_method == 'average':
-                unit_cost = order.compute_order_unit_cost()
-                order.write({'unit_cost': unit_cost})
-                logger.info('MO %s: unit_cost=%s', order.name, unit_cost)
-                order.move_finished_ids.filtered(
-                    lambda x: x.product_id == order.product_id).write({
-                        'price_unit': unit_cost})
-        return super(MrpProduction, self).post_inventory()
+    @api.depends('bom_id', 'product_id')
+    def _compute_extra_cost(self):
+        for prod in self:
+            bom = prod.bom_id
+            if bom and bom.type == 'normal':
+                extra_cost_bom_qty_uom = bom.extra_cost + bom.total_labour_cost
+                extra_cost_per_unit_in_prod_uom = 0
+                qty_prod_uom = bom.product_uom_id._compute_quantity(bom.product_qty, prod.product_uom_id)
+                if qty_prod_uom:
+                    extra_cost_per_unit_in_prod_uom = extra_cost_bom_qty_uom / qty_prod_uom
+                prod.extra_cost = extra_cost_per_unit_in_prod_uom

mrp_average_cost/security/mrp_average_cost_security.xml

@@ -4,7 +4,7 @@
 <record id="labour_cost_profile_rule" model="ir.rule">
     <field name="name">Labour Cost Profile multi-company</field>
     <field name="model_id" ref="model_labour_cost_profile"/>
-    <field name="domain_force">['|', ('company_id', '=', False), ('company_id', 'child_of', [user.company_id.id])]</field>
+    <field name="domain_force">[('company_id', 'in', company_ids + [False])]</field>
 </record>
 
 </odoo>

mrp_average_cost/views/mrp_view.xml

@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="utf-8"?>
 <!--
-  Copyright (C) 2016-2019 Akretion (http://www.akretion.com/)
+  Copyright (C) 2016-2024 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).
 -->
@@ -13,7 +13,8 @@
     <field name="model">mrp.bom</field>
     <field name="inherit_id" ref="mrp.mrp_bom_form_view"/>
     <field name="arch" type="xml">
-        <field name="picking_type_id" position="after">
+        <xpath expr="//page[@name='miscellaneous']/group" position="inside">
+            <group name="costs">
                 <field name="total_components_cost" widget="monetary"
                     options="{'currency_field': 'company_currency_id'}"/>
                 <field name="total_labour_cost" widget="monetary"
@@ -29,7 +30,8 @@
                         string="Update Cost Price of Product" class="oe_link"/>
                 </div>
                 <field name="company_currency_id" invisible="1"/>
-        </field>
+            </group>
+        </xpath>
         <notebook position="inside">
             <page string="Labour" name="labour_lines">
                 <group name="labour_lines_grp">
@@ -117,10 +119,10 @@
     <field name="inherit_id" ref="mrp.mrp_production_form_view"/>
     <field name="model">mrp.production</field>
     <field name="arch" type="xml">
-        <field name="availability" position="after">
-            <field name="unit_cost" widget="monetary" options="{'currency_field': 'company_currency_id'}" attrs="{'invisible': [('state', '!=', 'done')]}"/>
+        <xpath expr="//page[@name='miscellaneous']//field[@name='origin']/.." position="inside">
+            <field name="extra_cost" widget="monetary" options="{'currency_field': 'company_currency_id'}"/>
             <field name="company_currency_id" invisible="1"/>
-        </field>
+        </xpath>
     </field>
 </record>
 

mrp_no_product_template_menu/__manifest__.py

@@ -4,7 +4,7 @@
 
 {
     'name': 'MRP No Product Template Menu',
-    'version': '12.0.1.0.0',
+    'version': '14.0.1.0.0',
     'category': 'Manufacturing',
     'license': 'AGPL-3',
     'summary': "Replace product.template menu entries by product.product menu",
@@ -22,9 +22,9 @@ This module has been written by Alexis de Lattre
 from Akretion <alexis.delattre@akretion.com>.
     """,
     'author': 'Akretion',
-    'website': 'http://www.akretion.com',
+    'website': 'https://github.com/akretion/odoo-usability',
     'depends': ['mrp', 'sale_purchase_no_product_template_menu'],
     'auto_install': True,
     'data': ['mrp_view.xml'],
-    'installable': False,
+    'installable': True,
 }

mrp_no_product_template_menu/mrp_view.xml

@@ -1,21 +1,18 @@
 <?xml version="1.0" encoding="utf-8"?>
 <!--
-  Copyright 2016-2019 Akretion France (http://www.akretion.com/)
+  Copyright 2016-2024 Akretion France (http://www.akretion.com/)
   @author: Alexis de Lattre <alexis.delattre@akretion.com>
   License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 -->
 
 <odoo>
 
-<record id="product_product_action_mrp" model="ir.actions.act_window">
-    <field name="name">Products</field>
-    <field name="res_model">product.product</field>
-    <field name="view_mode">tree,form,kanban</field>
-    <field name="context">{'search_default_consumable': 1, 'default_type': 'product'}</field>
-</record>
-
 <record id="mrp.menu_mrp_product_form" model="ir.ui.menu">
-    <field name="action" ref="product_product_action_mrp"/>
+    <field name="action" ref="product.product_normal_action"/>
 </record>
+<!-- we don't care about:
+    "search_default_consumable": 1 => not useful
+    "default_type": 'product' : stock_usability make it the default... no need to bother
+-->
 
 </odoo>

mrp_subcontracting_usability/__manifest__.py

@@ -0,0 +1,18 @@
+# Copyright 2023 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': 'MRP Subcontracting Usability',
+    'version': '14.0.1.0.0',
+    'category': 'Manufacturing',
+    'license': 'AGPL-3',
+    'summary': 'Usability improvements on mrp_subcontracting',
+    'author': 'Akretion',
+    'website': 'https://github.com/akretion/odoo-usability',
+    'depends': ['mrp_subcontracting'],
+    'data': [
+        'views/mrp_bom.xml',
+    ],
+    'installable': True,
+}

mrp_subcontracting_usability/views/mrp_bom.xml

@@ -0,0 +1,22 @@
+<?xml version="1.0" encoding="utf-8"?>
+<!--
+  Copyright 2023 Akretion France (http://www.akretion.com/)
+  @author: Alexis de Lattre <alexis.delattre@akretion.com>
+  License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+-->
+
+<odoo>
+
+
+<record id="view_mrp_bom_filter" model="ir.ui.view">
+    <field name="model">mrp.bom</field>
+    <field name="inherit_id" ref="mrp.view_mrp_bom_filter"/>
+    <field name="arch" type="xml">
+        <filter name="phantom" position="after">
+            <filter name="subcontract" domain="[('type', '=', 'subcontract')]" string="Subcontracting"/>
+        </filter>
+    </field>
+</record>
+
+
+</odoo>

mrp_usability/__manifest__.py

@@ -14,6 +14,7 @@
     'data': [
         'views/mrp_production.xml',
         'views/product_template.xml',
+        'views/stock_move_line.xml',
     #    'report/mrp_report.xml'  # TODO
     ],
     'installable': True,

mrp_usability/models/mrp_production.py

@@ -3,12 +3,23 @@
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 
 
-from odoo import api, models
+from odoo import api, fields, models
 
 
 class MrpProduction(models.Model):
     _inherit = 'mrp.production'
 
+    # Allow to change the destination location until 'Mark as done'.
+    # Native behavior: it is only possible to change it in draft state.
+    location_dest_id = fields.Many2one(states={
+        'draft': [('readonly', False)],      # native
+        'confirmed': [('readonly', False)],  # added
+        'progress': [('readonly', False)],   # added
+        'to_close': [('readonly', False)],   # added
+        }, tracking=True)
+    # Add field product_categ_id for reporting only
+    product_categ_id = fields.Many2one(related='product_id.categ_id', store=True)
+
     # Method used by the report, inherited in this module
     @api.model
     def get_stock_move_sold_out_report(self, move):

mrp_usability/views/mrp_production.xml

@@ -16,8 +16,9 @@
         <xpath expr="//page[@name='miscellaneous']/group/group/field[@name='location_src_id']" position="replace"/>
         <xpath expr="//page[@name='miscellaneous']/group/group/field[@name='location_dest_id']" position="replace"/>
         <field name="bom_id" position="after">
-            <field name="location_src_id" groups="stock.group_stock_multi_locations" options="{'no_create': True}" attrs="{'readonly': [('state', '!=', 'draft')]}"/>
-            <field name="location_dest_id" groups="stock.group_stock_multi_locations" options="{'no_create': True}" attrs="{'readonly': [('state', '!=', 'draft')]}"/>
+            <!-- no need to set readonly via attrs, the readonly status is in the field definition -->
+            <field name="location_src_id" groups="stock.group_stock_multi_locations" options="{'no_create': True}" />
+            <field name="location_dest_id" groups="stock.group_stock_multi_locations" options="{'no_create': True}"/>
         </field>
         <xpath expr="//page[@name='miscellaneous']/group/group/field[@name='date_deadline']" position="after">
             <field name="date_start"/>
@@ -40,4 +41,20 @@
     </field>
 </record>
 
+<!-- Menu Manufacturing > Reporting > Manufacturing orders -->
+<record id="mrp.mrp_production_report" model="ir.actions.act_window">
+    <!-- Change order: pivot first instead of graph -->
+    <field name="view_mode">pivot,graph,form</field>
+</record>
+
+<record id="view_production_pivot" model="ir.ui.view">
+    <field name="model">mrp.production</field>
+    <field name="inherit_id" ref="mrp.view_production_pivot"/>
+    <field name="arch" type="xml">
+        <pivot position="inside">
+            <field name="product_uom_qty" type="measure"/>
+        </pivot>
+    </field>
+</record>
+
 </odoo>

mrp_usability/views/stock_move_line.xml

@@ -0,0 +1,21 @@
+<?xml version="1.0" encoding="utf-8"?>
+<!--
+  Copyright 2024 Akretion France (http://www.akretion.com/)
+  @author: Alexis de Lattre <alexis.delattre@akretion.com>
+  License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+-->
+
+<odoo>
+
+<record id="view_move_line_form" model="ir.ui.view">
+    <field name="name">mrp_usability.stock.move.line.form</field>
+    <field name="model">stock.move.line</field>
+    <field name="inherit_id" ref="stock.view_move_line_form" />
+    <field name="arch" type="xml">
+        <field name="reference" position="before">
+            <field name="production_id" attrs="{'invisible': [('production_id', '=', False)]}"/>
+        </field>
+    </field>
+</record>
+
+</odoo>

pos_usability/__manifest__.py

@@ -31,7 +31,11 @@ Akretion:
         "views/report_pos_order.xml",
         "views/pos_category.xml",
         "views/pos_session.xml",
+        "views/pos_order.xml",
+        "views/pos_config.xml",
         "views/product.xml",
+        "views/pos_payment_method.xml",
+        "views/stock_warehouse.xml",
         ],
     "installable": True,
 }

pos_usability/models/__init__.py

@@ -1,3 +1,4 @@
 from . import product
 from . import pos_category
+from . import pos_config
 from . import pos_payment_method

pos_usability/models/pos_config.py

@@ -0,0 +1,12 @@
+# Copyright 2023 Akretion France (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 odoo import fields, models
+
+
+class PosConfig(models.Model):
+    _inherit = 'pos.config'
+    _order = 'sequence, id'
+
+    sequence = fields.Integer(default=10)

pos_usability/models/pos_payment_method.py

@@ -8,5 +8,7 @@ from odoo import fields, models
 class PosPaymentMethod(models.Model):
     _inherit = 'pos.payment.method'
     _check_company_auto = True
+    _order = 'sequence, id'
 
     cash_journal_id = fields.Many2one(check_company=True)
+    sequence = fields.Integer(default=10)

pos_usability/pos-payment_terminal-auto_validate.diff

@@ -0,0 +1,35 @@
+diff --git a/addons/point_of_sale/static/src/js/Screens/PaymentScreen/PaymentScreen.js b/addons/point_of_sale/static/src/js/Screens/PaymentScreen/PaymentScreen.js
+index 55aa635aa10..d91ea933a71 100644
+--- a/addons/point_of_sale/static/src/js/Screens/PaymentScreen/PaymentScreen.js
++++ b/addons/point_of_sale/static/src/js/Screens/PaymentScreen/PaymentScreen.js
+@@ -8,6 +8,7 @@ odoo.define('point_of_sale.PaymentScreen', function (require) {
+     const { useListener } = require('web.custom_hooks');
+     const Registries = require('point_of_sale.Registries');
+     const { onChangeOrder } = require('point_of_sale.custom_hooks');
++    const utils = require('web.utils');
+ 
+     class PaymentScreen extends PosComponent {
+         constructor() {
+@@ -20,6 +21,7 @@ odoo.define('point_of_sale.PaymentScreen', function (require) {
+             useListener('send-payment-cancel', this._sendPaymentCancel);
+             useListener('send-payment-reverse', this._sendPaymentReverse);
+             useListener('send-force-done', this._sendForceDone);
++            useListener('validate-order', () => this.validateOrder(false));
+             this.lockedValidateOrder = useAsyncLockedMethod(this.validateOrder);
+             NumberBuffer.use(this._getNumberBufferConfig);
+             onChangeOrder(this._onPrevOrder, this._onNewOrder);
+@@ -333,6 +335,14 @@ odoo.define('point_of_sale.PaymentScreen', function (require) {
+             if (isPaymentSuccessful) {
+                 line.set_payment_status('done');
+                 line.can_be_reversed = payment_terminal.supports_reversals;
++                // Automatically validate the order if, after an electronic payment,
++                // the current order is fully paid (BACKPORT FROM v16)
++                if (
++                    this.currentOrder.is_paid() &&
++                    utils.float_is_zero(this.currentOrder.get_due(), this.env.pos.currency.decimals)
++                ) {
++                    this.trigger('validate-order');
++                }
+             } else {
+                 line.set_payment_status('retry');
+             }

pos_usability/pos_payment_method-sort_by_sequence.diff

@@ -0,0 +1,22 @@
+diff --git a/addons/point_of_sale/static/src/js/models.js b/addons/point_of_sale/static/src/js/models.js
+index 86a7b44bcfb..06b46ba5645 100644
+--- a/addons/point_of_sale/static/src/js/models.js
++++ b/addons/point_of_sale/static/src/js/models.js
+@@ -514,16 +514,7 @@ exports.PosModel = Backbone.Model.extend({
+         fields: ['name', 'is_cash_count', 'use_payment_terminal'],
+         domain: function(self){return ['|',['active', '=', false], ['active', '=', true]]; },
+         loaded: function(self, payment_methods) {
+-            self.payment_methods = payment_methods.sort(function(a,b){
+-                // prefer cash payment_method to be first in the list
+-                if (a.is_cash_count && !b.is_cash_count) {
+-                    return -1;
+-                } else if (!a.is_cash_count && b.is_cash_count) {
+-                    return 1;
+-                } else {
+-                    return a.id - b.id;
+-                }
+-            });
++            self.payment_methods = payment_methods;
+             self.payment_methods_by_id = {};
+             _.each(self.payment_methods, function(payment_method) {
+                 self.payment_methods_by_id[payment_method.id] = payment_method;

pos_usability/views/pos_config.xml

@@ -0,0 +1,21 @@
+<?xml version="1.0" encoding="utf-8"?>
+<!--
+  Copyright 2023 Akretion France (http://www.akretion.com/)
+  @author: Alexis de Lattre <alexis.delattre@akretion.com>
+  License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+-->
+
+<odoo>
+
+<record id="view_pos_config_tree" model="ir.ui.view">
+    <field name="model">pos.config</field>
+    <field name="inherit_id" ref="point_of_sale.view_pos_config_tree"/>
+    <field name="arch" type="xml">
+        <field name="name" position="before">
+            <field name="sequence" widget="handle"/>
+        </field>
+    </field>
+</record>
+
+</odoo>
+

pos_usability/views/pos_order.xml

@@ -0,0 +1,21 @@
+<?xml version="1.0" encoding="utf-8"?>
+
+<!--
+  Copyright 2023 Akretion France (http://www.akretion.com/)
+  @author: Alexis de Lattre <alexis.delattre@akretion.com>
+  License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+-->
+
+<odoo>
+
+<record id="view_pos_pos_form" model="ir.ui.view">
+    <field name="model">pos.order</field>
+    <field name="inherit_id" ref="point_of_sale.view_pos_pos_form"/>
+    <field name="arch" type="xml">
+        <xpath expr="//field[@name='lines']/form//field[@name='full_product_name']" position="before">
+            <field name="product_id"/>
+        </xpath>
+    </field>
+</record>
+
+</odoo>

pos_usability/views/pos_payment_method.xml

@@ -0,0 +1,20 @@
+<?xml version="1.0" encoding="utf-8"?>
+<!--
+  Copyright 2023 Akretion France (http://www.akretion.com/)
+  @author: Alexis de Lattre <alexis.delattre@akretion.com>
+  License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+-->
+
+<odoo>
+
+<record id="pos_payment_method_view_tree" model="ir.ui.view">
+    <field name="model">pos.payment.method</field>
+    <field name="inherit_id" ref="point_of_sale.pos_payment_method_view_tree"/>
+    <field name="arch" type="xml">
+        <field name="name" position="before">
+            <field name="sequence" widget="handle"/>
+        </field>
+    </field>
+</record>
+
+</odoo>

pos_usability/views/stock_warehouse.xml

@@ -0,0 +1,20 @@
+<?xml version="1.0" encoding="utf-8"?>
+<!--
+  Copyright 2024 Akretion France (http://www.akretion.com/)
+  @author: Alexis de Lattre <alexis.delattre@akretion.com>
+  License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+-->
+
+<odoo>
+
+<record id="view_warehouse" model="ir.ui.view">
+    <field name="model">stock.warehouse</field>
+    <field name="inherit_id" ref="stock.view_warehouse"/>
+    <field name="arch" type="xml">
+        <field name="out_type_id" position="after">
+            <field name="pos_type_id"/>
+        </field>
+    </field>
+</record>
+
+</odoo>

product_category_tax/i18n/fr.po

@@ -4,17 +4,47 @@
 #
 msgid ""
 msgstr ""
-"Project-Id-Version: Odoo Server 8.0-20150727\n"
+"Project-Id-Version: Odoo Server 14.0\n"
 "Report-Msgid-Bugs-To: \n"
-"POT-Creation-Date: 2015-12-04 17:16+0000\n"
-"PO-Revision-Date: 2015-12-04 17:16+0000\n"
-"Last-Translator: <>\n"
+"POT-Creation-Date: 2023-10-16 20:34+0000\n"
+"PO-Revision-Date: 2023-10-16 20:34+0000\n"
+"Last-Translator: Alexis de Lattre <alexis.delattre@akretion.com>\n"
 "Language-Team: \n"
+"Language: fr\n"
 "MIME-Version: 1.0\n"
 "Content-Type: text/plain; charset=UTF-8\n"
 "Content-Transfer-Encoding: \n"
 "Plural-Forms: \n"
 
+#. module: product_category_tax
+#: model:ir.model,name:product_category_tax.model_product_categ_tax_mixin
+msgid "Common code for taxes on product categories"
+msgstr "Code commun pour les taxes sur les catégories d'articles"
+
+#. module: product_category_tax
+#: model:ir.model.fields,field_description:product_category_tax.field_product_categ_tax_mixin__display_name
+#: model:ir.model.fields,field_description:product_category_tax.field_product_category__display_name
+#: model:ir.model.fields,field_description:product_category_tax.field_product_product__display_name
+#: model:ir.model.fields,field_description:product_category_tax.field_product_template__display_name
+msgid "Display Name"
+msgstr "Nom affiché"
+
+#. module: product_category_tax
+#: model:ir.model.fields,field_description:product_category_tax.field_product_categ_tax_mixin__id
+#: model:ir.model.fields,field_description:product_category_tax.field_product_category__id
+#: model:ir.model.fields,field_description:product_category_tax.field_product_product__id
+#: model:ir.model.fields,field_description:product_category_tax.field_product_template__id
+msgid "ID"
+msgstr "ID"
+
+#. module: product_category_tax
+#: model:ir.model.fields,field_description:product_category_tax.field_product_categ_tax_mixin____last_update
+#: model:ir.model.fields,field_description:product_category_tax.field_product_category____last_update
+#: model:ir.model.fields,field_description:product_category_tax.field_product_product____last_update
+#: model:ir.model.fields,field_description:product_category_tax.field_product_template____last_update
+msgid "Last Modified on"
+msgstr "Dernière modification le"
+
 #. module: product_category_tax
 #: model:ir.model,name:product_category_tax.model_product_product
 msgid "Product"
@@ -23,7 +53,7 @@ msgstr "Article"
 #. module: product_category_tax
 #: model:ir.model,name:product_category_tax.model_product_category
 msgid "Product Category"
-msgstr "Catégorie d'articles"
+msgstr "Catégorie d'article"
 
 #. module: product_category_tax
 #: model:ir.model,name:product_category_tax.model_product_template
@@ -31,23 +61,31 @@ msgid "Product Template"
 msgstr "Modèle d'article"
 
 #. module: product_category_tax
-#: field:product.category,purchase_tax_ids:0
+#: model:ir.model.fields,field_description:product_category_tax.field_product_category__purchase_tax_ids
 msgid "Purchase Taxes"
-msgstr "Taxes fournisseurs"
+msgstr "Taxes à l'achat"
 
 #. module: product_category_tax
-#: field:product.category,sale_tax_ids:0
+#: model:ir.model.fields,field_description:product_category_tax.field_product_category__sale_tax_ids
 msgid "Sale Taxes"
 msgstr "Taxes à la vente"
 
 #. module: product_category_tax
-#: code:addons/product_category_tax/product.py:57
+#: code:addons/product_category_tax/product.py:0
 #, python-format
-msgid "The purchase taxes configured on the product '%s' are not the same as the purchase taxes configured on it's related product category '%s'."
-msgstr "Les taxes fournisseurs paramétrées sur l'article '%s' ne sont pas les mêmes que les taxes fournisseurs paramétrées sur la catégorie interne '%s'."
+msgid ""
+"The purchase taxes configured on the product '%s' are not the same as the "
+"purchase taxes configured on it's related internal category '%s'."
+msgstr ""
+"Les taxes à l'achat configurées sur l'article '%s' ne sont pas les mêmes que "
+"les taxes à l'achat configurées sur sa catégorie interne '%s'."
 
 #. module: product_category_tax
-#: code:addons/product_category_tax/product.py:49
+#: code:addons/product_category_tax/product.py:0
 #, python-format
-msgid "The sale taxes configured on the product '%s' are not the same as the sale taxes configured on it's related product category '%s'."
-msgstr "Les taxes à la vente paramétrées sur l'article '%s' ne sont pas les mêmes que les taxes à la vente paramétrées sur la catégorie interne '%s'."
+msgid ""
+"The sale taxes configured on the product '%s' are not the same as the sale "
+"taxes configured on it's related internal category '%s'."
+msgstr ""
+"Les taxes à la vente configurées sur l'article '%s' ne sont pas les mêmes que "
+"les taxes à la vente configurées sur sa catégorie interne '%s'."

product_category_tax/product.py

@@ -48,7 +48,7 @@ class ProductTemplate(models.Model):
     _inherit = ['product.template', 'product.categ.tax.mixin']
     _name = 'product.template'
 
-    @api.constrains('taxes_id', 'supplier_taxes_id')
+    @api.constrains('taxes_id', 'supplier_taxes_id', 'categ_id')
     def _check_tax_categ(self):
         # self.name != 'Pay Debt' is a stupid hack to avoid blocking the
         # installation of the module 'pos_debt_notebook'

product_detailed_type/__init__.py

@@ -0,0 +1,2 @@
+from . import models
+from .post_install import set_product_detailed_type

product_detailed_type/__manifest__.py

@@ -0,0 +1,19 @@
+# Copyright 2023 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': 'Product Detailed Type',
+    'version': '14.0.1.0.0',
+    'category': 'Product',
+    'license': 'LGPL-3',
+    'summary': 'Backport detailed_type from v16 to v14',
+    'author': 'Akretion',
+    'website': 'https://github.com/akretion/odoo-usability',
+    'depends': ['product'],
+    'data': [
+        'views/product.xml',
+        ],
+    'post_init_hook': 'set_product_detailed_type',
+    'installable': True,
+}

product_detailed_type/models/__init__.py

@@ -0,0 +1 @@
+from . import product_template

product_detailed_type/models/product_template.py

@@ -0,0 +1,25 @@
+# Copyright 2023 Akretion France (http://www.akretion.com/)
+# Copyright 2023 Odoo SA (contains code copy-pasted from Odoo v16)
+# @author: Alexis de Lattre <alexis.delattre@akretion.com>
+# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
+
+from odoo import api, fields, models
+
+
+class ProductTemplate(models.Model):
+    _inherit = 'product.template'
+
+    detailed_type = fields.Selection([
+        ('consu', 'Consumable'),
+        ('service', 'Service'),
+        ], string='Product Type', default='consu', required=True, tracking=True)
+    type = fields.Selection(compute='_compute_type', store=True, string="Type")
+
+    def _detailed_type_mapping(self):
+        return {}
+
+    @api.depends('detailed_type')
+    def _compute_type(self):
+        type_mapping = self._detailed_type_mapping()
+        for record in self:
+            record.type = type_mapping.get(record.detailed_type, record.detailed_type)

product_detailed_type/post_install.py

@@ -0,0 +1,7 @@
+# Copyright 2023 Akretion France (http://www.akretion.com/)
+# @author: Alexis de Lattre <alexis.delattre@akretion.com>
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+
+
+def set_product_detailed_type(cr, registry):
+    cr.execute('UPDATE product_template SET detailed_type=type')