[FIX] commission module installable

This speeds-up the opening of commission results

account_invoice_update_wizard/__manifest__.py

@@ -4,7 +4,7 @@
 
 {
     'name': 'Account Invoice Update Wizard',
-    'version': '14.0.1.0.0',
+    'version': '18.0.1.0.0',
     'category': 'Accounting & Finance',
     'license': 'AGPL-3',
     'summary': 'Wizard to update non-legal fields of an open/paid invoice',
@@ -18,5 +18,5 @@
         'wizard/account_move_update_view.xml',
         'views/account_move.xml',
     ],
-    'installable': False,
+    'installable': True,
 }

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/pyproject.toml

@@ -0,0 +1,3 @@
+[build-system]
+requires = ["whool"]
+build-backend = "whool.buildapi"

account_invoice_update_wizard/tests/test_account_move_update_wizard.py

@@ -1,10 +1,10 @@
 # Copyright 2018-2022 Camptocamp
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 
-from odoo.tests.common import SavepointCase
+from odoo.tests.common import TransactionCase
 
 
-class TestAccountInvoiceUpdateWizard(SavepointCase):
+class TestAccountInvoiceUpdateWizard(TransactionCase):
 
     @classmethod
     def setUpClass(cls):
@@ -13,6 +13,17 @@ class TestAccountInvoiceUpdateWizard(SavepointCase):
         cls.product16 = cls.env.ref('product.product_product_16')
         uom_unit = cls.env.ref('uom.product_uom_categ_unit')
 
+        cls.plan = cls.env['account.analytic.plan'].create({'name': 'Test Plan'})
+        cls.analytic_account_1 = cls.env['account.analytic.account'].create({
+            'name': 'analytic 1 test plan',
+            'plan_id': cls.plan.id,
+            'company_id': False,
+        })
+        cls.analytic_account_2 = cls.env['account.analytic.account'].create({
+            'name': 'analytic 2 test plan',
+            'plan_id': cls.plan.id,
+            'company_id': False,
+        })
         cls.move1 = cls.env['account.move'].create({
             'name': 'Test invoice',
             'partner_id': cls.customer12.id,
@@ -30,13 +41,6 @@ class TestAccountInvoiceUpdateWizard(SavepointCase):
             ],
         })
 
-        cls.aa1 = cls.env.ref('analytic.analytic_partners_camp_to_camp')
-        cls.aa2 = cls.env.ref('analytic.analytic_nebula')
-        cls.atag1 = cls.env.ref('analytic.tag_contract')
-        cls.atag2 = cls.env['account.analytic.tag'].create({
-            'name': 'の',
-        })
-
     def create_wizard(self, move):
         res = move.prepare_update_wizard()
         self.wiz = self.env['account.move.update'].browse(res['res_id'])
@@ -54,13 +58,14 @@ class TestAccountInvoiceUpdateWizard(SavepointCase):
 
         wiz_line = self.wiz.line_ids.filtered(
             lambda rec: rec.invoice_line_id.product_id.id == self.product16.id)
-        wiz_line.analytic_account_id = self.aa1
+        wiz_line.analytic_distribution = {self.analytic_account_1.id: 50, self.analytic_account_2.id: 50}
         self.wiz.run()
 
         related_ml = self.move1.invoice_line_ids.filtered(
             lambda rec: rec.product_id == self.product16)
-        self.assertEqual(related_ml.analytic_account_id, self.aa1)
-        self.assertEqual(related_ml.analytic_line_ids.account_id, self.aa1)
+        self.assertEqual(related_ml.analytic_distribution, {str(self.analytic_account_1.id): 50.0, str(self.analytic_account_2.id): 50.0})
+        self.assertEqual(len(related_ml.analytic_line_ids), 2)
+        self.assertEqual(related_ml.analytic_line_ids[0].amount, 21.0)
 
     def test_change_analytic_account_line1(self):
         """ Change analytic account on a move line
@@ -70,86 +75,21 @@ class TestAccountInvoiceUpdateWizard(SavepointCase):
             - update the move line
             - update the existing analytic line."""
         move_line1 = self.move1.invoice_line_ids.filtered(lambda rec: rec.product_id == self.product16)
-        move_line1.analytic_account_id = self.aa2
+        move_line1.analytic_distribution = {self.analytic_account_1.id: 100}
 
         self.move1._post()
         self.create_wizard(self.move1)
 
         wiz_line = self.wiz.line_ids.filtered(
             lambda rec: rec.invoice_line_id.product_id.id == self.product16.id)
-        wiz_line.analytic_account_id = self.aa1
+        wiz_line.analytic_distribution = {self.analytic_account_1.id: 50, self.analytic_account_2.id: 50}
         self.wiz.run()
 
         related_ml = self.move1.invoice_line_ids.filtered(
             lambda rec: rec.product_id == self.product16)
-        self.assertEqual(related_ml.analytic_account_id, self.aa1)
-        self.assertEqual(related_ml.analytic_line_ids.account_id, self.aa1)
-
-    def test_add_analytic_tags_line1(self):
-        """ Add analytic tags on a move line
-        after the move has been approved.
-
-        This will update move line.
-        """
-        self.move1._post()
-        self.create_wizard(self.move1)
-
-        wiz_line = self.wiz.line_ids.filtered(
-            lambda rec: rec.invoice_line_id.product_id.id == self.product16.id)
-        wiz_line.analytic_tag_ids = self.atag2
-        self.wiz.run()
-
-        related_ml = self.move1.invoice_line_ids.filtered(
-            lambda rec: rec.product_id == self.product16)
-        self.assertEqual(related_ml.analytic_tag_ids, self.atag2)
-        self.assertFalse(related_ml.analytic_line_ids)
-
-    def test_change_analytic_tags_line1(self):
-        """ Change analytic tags on a move line
-        after the move has been approved.
-
-        It will update move line and analytic line
-        """
-        move_line1 = self.move1.invoice_line_ids.filtered(lambda rec: rec.product_id == self.product16)
-        move_line1.analytic_account_id = self.aa2
-        move_line1.analytic_tag_ids = self.atag1
-
-        self.move1._post()
-        self.create_wizard(self.move1)
-
-        wiz_line = self.wiz.line_ids.filtered(
-            lambda rec: rec.invoice_line_id.product_id.id == self.product16.id)
-        wiz_line.analytic_tag_ids = self.atag2
-        self.wiz.run()
-
-        related_ml = self.move1.invoice_line_ids.filtered(
-            lambda rec: rec.product_id == self.product16)
-        self.assertEqual(related_ml.analytic_tag_ids, self.atag2)
-        self.assertEqual(related_ml.analytic_line_ids.tag_ids, self.atag2)
-
-    def test_add_analytic_info_line1(self):
-        """ Add analytic account and tags on a move line
-        after the move has been approved.
-
-        This will:
-            - update move line
-            - create an analytic line
-        """
-        self.move1._post()
-        self.create_wizard(self.move1)
-
-        wiz_line = self.wiz.line_ids.filtered(
-            lambda rec: rec.invoice_line_id.product_id.id == self.product16.id)
-        wiz_line.analytic_account_id = self.aa1
-        wiz_line.analytic_tag_ids = self.atag2
-        self.wiz.run()
-
-        related_ml = self.move1.invoice_line_ids.filtered(
-            lambda rec: rec.product_id == self.product16)
-        self.assertEqual(related_ml.analytic_account_id, self.aa1)
-        self.assertEqual(related_ml.analytic_tag_ids, self.atag2)
-        self.assertEqual(related_ml.analytic_line_ids.account_id, self.aa1)
-        self.assertEqual(related_ml.analytic_line_ids.tag_ids, self.atag2)
+        self.assertEqual(related_ml.analytic_distribution, {str(self.analytic_account_1.id): 50.0, str(self.analytic_account_2.id): 50.0})
+        self.assertEqual(len(related_ml.analytic_line_ids), 2)
+        self.assertEqual(related_ml.analytic_line_ids[0].amount, 21.0)
 
     def test_empty_analytic_account_line1(self):
         """ Remove analytic account
@@ -158,16 +98,16 @@ class TestAccountInvoiceUpdateWizard(SavepointCase):
         This will raise an error as it is not implemented.
         """
         move_line1 = self.move1.invoice_line_ids.filtered(lambda rec: rec.product_id == self.product16)
-        move_line1.analytic_account_id = self.aa2
+        move_line1.analytic_distribution = {self.analytic_account_1.id: 100}
 
         self.move1._post()
         self.create_wizard(self.move1)
 
         wiz_line = self.wiz.line_ids.filtered(
             lambda rec: rec.invoice_line_id.product_id.id == self.product16.id)
-        wiz_line.analytic_account_id = False
+        wiz_line.analytic_distribution = False
         self.wiz.run()
         related_ml = self.move1.invoice_line_ids.filtered(
             lambda rec: rec.product_id == self.product16)
-        self.assertFalse(related_ml.analytic_account_id)
+        self.assertFalse(related_ml.analytic_distribution)
         self.assertFalse(related_ml.analytic_line_ids)

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" invisible="state != 'posted' or move_type == 'entry'" groups="account.group_account_invoice"/>
+            <button name="prepare_update_wizard" type="object" string="Update Entry" invisible="state != 'posted' or move_type != 'entry'" groups="account.group_account_invoice"/>
         </button>
     </field>
 </record>

account_invoice_update_wizard/wizard/account_move_update.py

@@ -2,7 +2,7 @@
 # Copyright 2018-2022 Camptocamp
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 
-from odoo import models, fields, api, _
+from odoo import Command, models, fields, api
 from odoo.exceptions import UserError
 import odoo.addons.decimal_precision as dp
 
@@ -17,10 +17,11 @@ class AccountMoveUpdate(models.TransientModel):
     move_type = fields.Selection(related='invoice_id.move_type')
     company_id = fields.Many2one(related='invoice_id.company_id')
     partner_id = fields.Many2one(related='invoice_id.partner_id')
-    user_id = fields.Many2one('res.users', string='Salesperson')
+    invoice_user_id = fields.Many2one('res.users', string='Salesperson')
     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,11 +31,11 @@ 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):
-        return ['invoice_payment_term_id', 'user_id', 'partner_bank_id']
+        return ['invoice_payment_term_id', 'invoice_user_id', 'partner_bank_id']
 
     @api.model
     def _prepare_default_get(self, invoice):
@@ -44,17 +45,14 @@ class AccountMoveUpdate(models.TransientModel):
         for m2ofield in self._m2o_fields2update():
             res[m2ofield] = invoice[m2ofield].id or False
         for line in invoice.invoice_line_ids:
-            aa_tags = line.analytic_tag_ids
-            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,
-                'analytic_account_id': line.analytic_account_id.id,
+                'analytic_distribution': line.analytic_distribution,
                 'currency_id': line.currency_id.id,
-                'analytic_tag_ids': aa_tags,
-                'display_type': line.display_type,
             }])
         return res
 
@@ -87,15 +85,15 @@ class AccountMoveUpdate(models.TransientModel):
 
     @api.model
     def _line_simple_fields2update(self):
-        return ["name"]
+        return ["name", "analytic_distribution"]
 
     @api.model
     def _line_m2o_fields2update(self):
-        return ["analytic_account_id"]
+        return []
 
     @api.model
     def _line_m2m_fields2update(self):
-        return ["analytic_tag_ids"]
+        return []
 
     @api.model
     def _prepare_invoice_line(self, line):
@@ -108,30 +106,9 @@ class AccountMoveUpdate(models.TransientModel):
                 vals[field] = line[field].id
         for field in self._line_m2m_fields2update():
             if line[field] != line.invoice_line_id[field]:
-                vals[field] = [(6, 0, line[field].ids)]
+                vals[field] = [Command.set(line[field].ids)]
         return vals
 
-    def _prepare_move_line_and_analytic_line(self, inv_line):
-        mlvals = {}
-        alvals = {}
-        inv_line_upd = self.line_ids.filtered(
-            lambda rec: rec.invoice_line_id == inv_line)
-
-        ini_aa = inv_line.analytic_account_id
-        new_aa = inv_line_upd.analytic_account_id
-
-        if ini_aa != new_aa:
-            mlvals['analytic_account_id'] = new_aa.id
-            alvals['account_id'] = new_aa.id
-
-        ini_aa_tags = inv_line.analytic_tag_ids
-        new_aa_tags = inv_line_upd.analytic_tag_ids
-
-        if ini_aa_tags != new_aa_tags:
-            mlvals['analytic_tag_ids'] = [(6, None, new_aa_tags.ids)]
-            alvals['tag_ids'] = [(6, None, new_aa_tags.ids)]
-        return mlvals, alvals
-
     def _update_payment_term_move(self):
         self.ensure_one()
         inv = self.invoice_id
@@ -144,7 +121,7 @@ class AccountMoveUpdate(models.TransientModel):
             # the reconcile marks to put the new maturity date on the right
             # lines
             if inv.payment_id:
-                raise UserError(_(
+                raise UserError(self.env._(
                     "This wizard doesn't support the update of payment "
                     "terms on an invoice which is partially or fully "
                     "paid."))
@@ -168,7 +145,7 @@ class AccountMoveUpdate(models.TransientModel):
                         mlines[amount] = [line]
             for iamount, lines in mlines.items():
                 if len(lines) != len(new_pterm.get(iamount, [])):
-                    raise UserError(_(
+                    raise UserError(self.env._(
                         "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 "
@@ -188,64 +165,55 @@ class AccountMoveUpdate(models.TransientModel):
         if ivals:
             updated = True
             inv.write(ivals)
-        if inv:
-            for ml in inv.line_ids.filtered(
-                    # we are only interested in invoice lines, not tax lines
-                    lambda rec: bool(rec.product_id)
-            ):
-                if ml.credit == 0.0:
-                    continue
-                analytic_account = ml.analytic_account_id
-                mlvals, alvals = self._prepare_move_line_and_analytic_line(ml)
-                if mlvals:
-                    updated = True
-                    ml.write(mlvals)
-                aalines = ml.analytic_line_ids
-                if aalines and alvals:
-                    updated = True
-                    if ('account_id' in alvals and
-                            alvals['account_id'] is False):
-                        former_aa = analytic_account
-                        to_remove_aalines = aalines.filtered(
-                            lambda rec: rec.account_id == former_aa)
-                        # remove existing analytic line
-                        to_remove_aalines.unlink()
-                    else:
-                        aalines.write(alvals)
-                elif 'account_id' in alvals:
-                    # Create analytic lines if analytic account
-                    # is added later
-                    ml.create_analytic_lines()
         for line in self.line_ids:
             ilvals = self._prepare_invoice_line(line)
             if ilvals:
                 updated = True
+                # note that updating analytic_distribution will delete/re-create
+                # the analytic line with inverse method, we do not need additional
+                # logic about that.
                 line.invoice_line_id.write(ilvals)
         if updated:
-            inv.message_post(body=_(
+            inv.message_post(body=self.env._(
                 'Non-legal fields of invoice updated via the Invoice Update '
                 'wizard.'))
+        # Purge existing PDF
+        report = self.env.ref("account.account_invoices")
+        attachment = report.retrieve_attachment(inv)
+        # attachment may be None
+        if attachment:
+            attachment.unlink()
         return True
 
 
 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.move.line', string='Invoice Line', readonly=True)
     name = fields.Text(string='Description', required=True)
-    display_type = fields.Selection([
-        ('line_section', "Section"),
-        ('line_note', "Note")], default=False, help="Technical field for UX purpose.")
+    display_type = fields.Selection(
+        related="invoice_line_id.display_type",
+        help="Technical field for UX purpose.")
     quantity = fields.Float(
         string='Quantity', digits='Product Unit of Measure', readonly=True)
     price_subtotal = fields.Monetary(
         string='Amount', readonly=True)
-    analytic_account_id = fields.Many2one(
-        'account.analytic.account', string='Analytic Account')
-    analytic_tag_ids = fields.Many2many(
-        'account.analytic.tag', string='Analytic Tags')
     currency_id = fields.Many2one('res.currency', readonly=True)
+    analytic_distribution = fields.Json(
+        string="Analytic",
+#        compute="_compute_writeoff_analytic_distribution",
+#        readonly=False,
+#        store=True,
+#        precompute=True,
+    )
+    analytic_precision = fields.Integer(
+        default=lambda self: self.env["decimal.precision"].precision_get(
+            "Percentage Analytic"
+        ),
+    )

account_invoice_update_wizard/wizard/account_move_update_view.xml

@@ -15,25 +15,33 @@
                     <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="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="Bill Date" invisible="move_type not in ('in_invoice', 'in_refund')" name="invoice_date"/>
+                    <field string="Bill Reference" invisible="move_type not in ('in_invoice', 'in_refund')" name="ref"/>
+                    <field string="Customer Reference" invisible="move_type not in ('out_invoice', 'out_refund')" name="ref"/>
+                    <field string="Ref" invisible="move_type != 'entry'" name="ref"/>
+                    <field name="invoice_origin" invisible="move_type == 'entry'"/>
+                    <!-- update of payment term is broken -->
+                    <!-- <field name="invoice_payment_term_id" widget="selection"/>-->
+                    <field name="partner_bank_id" invisible="move_type == 'entry'"/>
+                    <field name="invoice_user_id" options="{'no_open': True, 'no_create': True, 'no_create_edit': True}" invisible="move_type == 'entry'"/>
                 </group>
                 <group name="lines">
-                    <field name="line_ids" nolabel="1">
-                        <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="line_ids" nolabel="1" widget="section_and_note_one2many">
+                        <list editable="bottom" create="false" delete="false" edit="true">
+                            <field name="invoice_line_id" column_invisible="1"/>
+                            <field name="display_type" column_invisible="1"/>
+                            <field name="currency_id" column_invisible="1"/>
                             <field name="name"/>
-                            <field name="quantity" attrs="{'invisible': [('display_type', '!=', False)]}"/>
-                            <field name="price_subtotal" attrs="{'invisible': [('display_type', '!=', False)]}"/>
-                            <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>
+                            <field name="quantity" invisible="display_type != 'product'" column_invisible="parent.move_type == 'entry'"/>
+                            <field name="price_subtotal" invisible="display_type != 'product'" column_invisible="parent.move_type == 'entry'"/>
+                            <field
+                                name="analytic_distribution"
+                                widget="analytic_distribution"
+                                groups="analytic.group_analytic_accounting"
+                                options="{'account_field': 'account_id', 'business_domain': 'general'}"
+                                invisible="display_type != 'product'"
+                            />
+                        </list>
                     </field>
                 </group>
                 <footer>

account_menu_usability/__manifest__.py

@@ -1,18 +0,0 @@
-# Copyright 2022 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': 'Account Menu Usability',
-    'version': '14.0.1.0.0',
-    'category': 'Accounting & Finance',
-    'license': 'AGPL-3',
-    'summary': 'Small usability enhancements in account_menu module',
-    'author': 'Akretion',
-    'website': 'https://github.com/akretion/odoo-usability',
-    'depends': ['account_menu'],
-    'data': [
-        'views/account_menu.xml',
-        ],
-    'installable': False,
-}

account_menu_usability/views/account_menu.xml

@@ -1,20 +0,0 @@
-<?xml version="1.0" encoding="utf-8"?>
-<!--
-  Copyright 2022 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 Templates are useful only when creating a new company/loading
-a chart of accounts, so we put it under 'Settings > Technical' and not
-under 'Invoicing > Configuration' which already has a lot of menu entries -->
-<record id="account_menu.menu_account_coa_settings" model="ir.ui.menu">
-    <field name="name">Accounting Templates</field>
-    <field name="parent_id" ref="base.menu_custom"/>
-</record>
-
-
-</odoo>

account_product_fiscal_classification_pos_sale/__manifest__.py

@@ -29,4 +29,5 @@ This module has been written by Alexis de Lattre from Akretion
     'depends': ['pos_sale', 'account_product_fiscal_classification'],
     "data": ['report/sale_report_view.xml'],
     'auto_install': True,
+    'installable': False,
 }

account_usability_akretion/README.rst

@@ -2,10 +2,13 @@
 Account Usability
 =================
 
-.. !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+.. 
+   !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
    !! This file is generated by oca-gen-addon-readme !!
    !! changes will be overwritten.                   !!
    !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+   !! source digest: sha256:dbf367e7cf8330c803622d0c171cd10e128344d318466d6e37ad7b0fc812e152
+   !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
 
 .. |badge1| image:: https://img.shields.io/badge/maturity-Beta-yellow.png
     :target: https://odoo-community.org/page/development-status
@@ -14,23 +17,23 @@ Account Usability
     :target: http://www.gnu.org/licenses/agpl-3.0-standalone.html
     :alt: License: AGPL-3
 .. |badge3| image:: https://img.shields.io/badge/github-akretion%2Fodoo--usability-lightgray.png?logo=github
-    :target: https://github.com/akretion/odoo-usability/tree/12.0/account_usability
+    :target: https://github.com/akretion/odoo-usability/tree/18.0/account_usability_akretion
     :alt: akretion/odoo-usability
 
-|badge1| |badge2| |badge3| 
+|badge1| |badge2| |badge3|
 
 This modules adds the following functions:
 
 * Add an *Overdue* filter on invoice search view (this feature was previously 
   located in te module *account_invoice_overdue_filter*)
-* Increase the default limit of 80 lines in journal entries and journal items views.
+* Increase the default limit of 80 lines in account move and account move line view.
 * disable reconciliation "guessing"
-* fast search on *Reconcile Ref* for journal items.
+* fast search on *Reconcile Ref* for account move line.
 * add sale dates to invoice report to be compliant with
   https://www.service-public.fr/professionnels-entreprises/vosdroits/F31808
 * Sale date on qweb invoices
 * A wizard to mark several invoices as sent at once (forward from v8)
-* Default date for reversal journal entry is now D+1 instead of today
+* Default date for Account Move Reversal is now D+1 instead of today
 * Track more fields on invoice (see details in account.py)
 * Add boolean fields `has_discount` and `has_attachment` on invoice
 * Add button "Delete line qty = 0" on supplier invoice
@@ -41,8 +44,8 @@ This modules adds the following functions:
 * add direct search of journal using code
 * add copy=False on some fields
 * Add unicity constraint on analytic codes per company
-* Better default values on journal entry
-* Add link from journal items to to invoice
+* Better default values on account move
+* Add link from account move line to invoice
 * Add start_date and end_date on bank statements
 * Add transfer_account_id to invoicing config page
 * Improve domain reconciliation widget
@@ -52,7 +55,6 @@ This modules adds the following functions:
 * don't attach PDF upon invoice report generation on supplier invoices/refunds
 * Add filter on debit and credit amount for Move Lines
 * Add supplier invoice number in invoice tree view
-* Add date in outstanding payment widget on invoice form view (requires `odoo PR 84180 <https://github.com/odoo/odoo/pull/84180>`_)
 
 Together with this module, I recommend the use of the following modules:
 
@@ -70,8 +72,8 @@ Bug Tracker
 
 Bugs are tracked on `GitHub Issues <https://github.com/akretion/odoo-usability/issues>`_.
 In case of trouble, please check there if your issue has already been reported.
-If you spotted it first, help us smashing it by providing a detailed and welcomed
-`feedback <https://github.com/akretion/odoo-usability/issues/new?body=module:%20account_usability%0Aversion:%2012.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
+If you spotted it first, help us to smash it by providing a detailed and welcomed
+`feedback <https://github.com/akretion/odoo-usability/issues/new?body=module:%20account_usability_akretion%0Aversion:%2018.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
 
 Do not contact contributors directly about support or help with technical issues.
 
@@ -92,6 +94,6 @@ Contributors
 Maintainers
 ~~~~~~~~~~~
 
-This module is part of the `akretion/odoo-usability <https://github.com/akretion/odoo-usability/tree/12.0/account_usability>`_ project on GitHub.
+This module is part of the `akretion/odoo-usability <https://github.com/akretion/odoo-usability/tree/18.0/account_usability_akretion>`_ project on GitHub.
 
 You are welcome to contribute.

account_usability_akretion/__init__.py

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

account_usability_akretion/__manifest__.py

@@ -1,10 +1,10 @@
-# Copyright 2015-2022 Akretion (http://www.akretion.com)
+# Copyright 2015-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).
 
 {
     'name': 'Account Usability',
-    'version': '16.0.1.0.0',
+    'version': '18.0.1.0.0',
     'category': 'Accounting & Finance',
     'license': 'AGPL-3',
     'summary': 'Small usability enhancements in account module',
@@ -12,7 +12,7 @@
     'website': 'https://github.com/akretion/odoo-usability',
     'depends': [
         'account',
-        'base_usability',  # needed only to access base_usability.group_nobody
+        'base_usability_akretion',  # needed only to access base_usability.group_nobody
         ],
     'data': [
         'views/account_account.xml',
@@ -21,16 +21,17 @@
         'views/account_invoice_report.xml',
         'views/account_journal.xml',
         'views/account_move.xml',
+        'views/account_move_line.xml',
+        'views/account_payment.xml',
         'views/account_analytic_line.xml',
         'views/account_menu.xml',
         'views/account_tax.xml',
 #        'views/product.xml',  # TODO
         'views/res_company.xml',
-        'views/account_report.xml',
-        'wizard/account_invoice_mark_sent_view.xml',
-        'wizard/account_group_generate_view.xml',
-        'wizard/account_payment_register_views.xml',
-        'wizard/account_move_reversal.xml',
+#        'views/account_report.xml',
+        'wizards/account_invoice_mark_sent_view.xml',
+#        'wizards/account_group_generate_view.xml',
+        'wizards/account_move_reversal.xml',
         'security/ir.model.access.csv',
 #        'report/invoice_report.xml',  # TODO
         "views/res_partner.xml",

account_usability_akretion/hooks.py

@@ -2,8 +2,8 @@
 # @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
+# 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()
+# def post_init_hook(cr, registry):
+#    env = api.Environment(cr, SUPERUSER_ID, {})
+#    env["account.move.line"].update_matching_number()

account_usability_akretion/migrations/14.0.1.1.0/post-migrate.py

@@ -1,9 +0,0 @@
-# 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_akretion/models/__init__.py

@@ -1,12 +1,10 @@
-from . import account_account
 from . import account_analytic_account
-#from . import account_bank_statement
 from . import account_incoterms
 from . import account_journal
 from . import account_move
+from . import account_move_line
 from . import account_partial_reconcile
 from . import res_partner
 from . import res_company
 #from . import product
 from . import account_invoice_report
-from . import res_partner_bank

account_usability_akretion/models/account_account.py

@@ -1,19 +0,0 @@
-# Copyright 2015-2022 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 api, models
-
-
-class AccountAccount(models.Model):
-    _inherit = 'account.account'
-
-    @api.depends('name', 'code')
-    def name_get(self):
-        if self._context.get('account_account_show_code_only'):
-            res = []
-            for record in self:
-                res.append((record.id, record.code))
-            return res
-        else:
-            return super().name_get()

account_usability_akretion/models/account_analytic_account.py

@@ -1,21 +1,20 @@
-# Copyright 2015-2022 Akretion (http://www.akretion.com)
+# Copyright 2015-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 models
+from odoo import api, models
 
 
 class AccountAnalyticAccount(models.Model):
     _inherit = 'account.analytic.account'
 
-    def name_get(self):
+    @api.depends_context('analytic_account_show_code_only')
+    def _compute_display_name(self):
         if self._context.get('analytic_account_show_code_only'):
-            res = []
-            for record in self:
-                res.append((record.id, record.code or record.name))
-            return res
+            for rec in self:
+                rec.display_name = rec.code or rec.name
         else:
-            return super().name_get()
+            return super()._compute_display_name()
 
     _sql_constraints = [(
         'code_company_unique',

account_usability_akretion/models/account_bank_statement.py

@@ -1,104 +0,0 @@
-# Copyright 2015-2020 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 api, fields, models
-from odoo.tools.misc import format_date
-
-
-class AccountBankStatement(models.Model):
-    _inherit = 'account.bank.statement'
-
-    start_date = fields.Date(
-        compute='_compute_dates', string='Start Date', store=True)
-    end_date = fields.Date(
-        compute='_compute_dates', string='End Date', store=True)
-    line_count = fields.Integer(
-        compute='_compute_dates', string='# of Lines', store=True)
-    hide_bank_statement_balance = fields.Boolean(
-        related='journal_id.hide_bank_statement_balance', readonly=True)
-
-    @api.depends('line_ids.date')
-    def _compute_dates(self):
-        for st in self:
-            dates = [line.date for line in st.line_ids]
-            st.start_date = dates and min(dates) or False
-            st.end_date = dates and max(dates) or False
-            st.line_count = len(dates)
-
-    def _check_balance_end_real_same_as_computed(self):
-        for stmt in self:
-            if not stmt.hide_bank_statement_balance:
-                super(AccountBankStatement, stmt)._check_balance_end_real_same_as_computed()
-        return True
-
-    @api.depends('name', 'start_date', 'end_date')
-    def name_get(self):
-        res = []
-        for statement in self:
-            name = "%s (%s => %s)" % (
-                statement.name,
-                statement.start_date and format_date(self.env, statement.start_date) or '',
-                statement.end_date and format_date(self.env, statement.end_date) or '')
-            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'
-    # Native order is:
-    # _order = 'statement_id desc, sequence, id desc'
-    _order = 'statement_id desc, date desc, sequence, id desc'
-
-    # Disable guessing for reconciliation
-    # because my experience with several customers shows that it is a problem
-    # in the following scenario : move line 'x' has been "guessed" by OpenERP
-    # to be reconciled with a statement line 'Y' at the end of the bank
-    # statement, but it is a mistake because it should be reconciled with
-    # statement line 'B' at the beginning of the bank statement
-    # When the user is on statement line 'B', he tries to select
-    # move line 'x', but it can't find it... because it is already "reserved"
-    # by the guess of OpenERP for statement line 'Y' ! To solve this problem,
-    # the user must go to statement line 'Y' and unselect move line 'x'
-    # and then come back on statement line 'B' and select move line 'A'...
-    # but non super-expert users can't do that because it is impossible to
-    # figure out that the fact that the user can't find move line 'x'
-    # is caused by this.
-    # Set search_reconciliation_proposition to False by default
-    # TODO: re-write in v10
-    # def get_data_for_reconciliations(
-    #        self, cr, uid, ids, excluded_ids=None,
-    #        search_reconciliation_proposition=False, context=None):
-    #    # Make variable name shorted for PEP8 !
-    #    search_rec_prop = search_reconciliation_proposition
-    #    return super().\
-    #        get_data_for_reconciliations(
-    #            cr, uid, ids, excluded_ids=excluded_ids,
-    #            search_reconciliation_proposition=search_rec_prop,
-    #            context=context)
-
-    def show_account_move(self):
-        self.ensure_one()
-        action = self.env["ir.actions.actions"]._for_xml_id(
-            'account.action_move_line_form')
-        # Note: this action is on account.move, not account.move.line !
-        action.update({
-            'views': False,
-            'view_id': False,
-            'view_mode': 'form,tree',
-            '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_akretion/models/account_incoterms.py

@@ -1,31 +1,15 @@
-# Copyright 2015-2022 Akretion (http://www.akretion.com)
+# Copyright 2015-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 api, models
+from odoo import models
 
 
 class AccountIncoterms(models.Model):
     _inherit = 'account.incoterms'
+    _rec_names_search = ["name", "code"]
 
     _sql_constraints = [(
         'code_unique',
         'unique(code)',
         'This incoterm code already exists.')]
-
-    @api.depends('code', 'name')
-    def name_get(self):
-        res = []
-        for rec in self:
-            res.append((rec.id, '[%s] %s' % (rec.code, rec.name)))
-        return res
-
-    @api.model
-    def name_search(self, name='', args=None, operator='ilike', limit=100):
-        if args is None:
-            args = []
-        if name and operator == 'ilike':
-            recs = self.search([('code', '=ilike', name + '%')] + args, limit=limit)
-            if recs:
-                return recs.name_get()
-        return super().name_search(name=name, args=args, operator=operator, limit=limit)

account_usability_akretion/models/account_invoice_report.py

@@ -1,8 +1,9 @@
-# Copyright 2022 Akretion (http://www.akretion.com)
+# Copyright 2022-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 api, fields, models
+from odoo.tools import SQL
 
 
 class AccountInvoiceReport(models.Model):
@@ -12,6 +13,6 @@ class AccountInvoiceReport(models.Model):
 
     @api.model
     def _select(self):
-        res = super()._select()
-        res += ", COALESCE(partner.industry_id, commercial_partner.industry_id) AS industry_id"
-        return res
+        return SQL(
+            "%s, COALESCE(partner.industry_id, commercial_partner.industry_id) AS industry_id",
+            super()._select())

account_usability_akretion/models/account_journal.py

@@ -1,4 +1,4 @@
-# Copyright 2015-2022 Akretion (http://www.akretion.com)
+# Copyright 2015-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).
 
@@ -17,41 +17,34 @@ class AccountJournal(models.Model):
         "the start/end balance and you regularly check the accounting balance "
         "of the bank account vs the amount of your bank account."
         )
-    # Used to set default user_type_id on account fields via context
-#    account_type_current_assets_id = fields.Many2one(
-#        'account.account.type',
-#        default=lambda self: self.env.ref('account.data_account_type_current_assets').id)
 
-    @api.depends(
-        'name', 'currency_id', 'company_id', 'company_id.currency_id', 'code')
-    def name_get(self):
-        res = []
+    @api.depends('name', 'currency_id', 'company_id', 'code')
+    @api.depends_context('journal_show_code_only')
+    def _compute_display_name(self):
         if self._context.get('journal_show_code_only'):
             for journal in self:
-                res.append((journal.id, journal.code))
-            return res
+                journal.display_name = journal.code
         else:
             for journal in self:
-                name = "[%s] %s" % (journal.code, journal.name)
+                name = f"[{journal.code}] {journal.name}"
                 if (
                         journal.currency_id and
                         journal.currency_id != journal.company_id.currency_id):
-                    name = "%s (%s)" % (name, journal.currency_id.name)
-                res.append((journal.id, name))
-            return res
+                    name = f"{name} ({journal.currency_id.name})"
+                journal.display_name = name
 
-    def open_outstanding_payments(self):
-        self.ensure_one()
-        action = self.env["ir.actions.actions"]._for_xml_id(
-            "account.action_account_moves_all")
-        action['domain'] = [
-            ('account_id', 'in', (self.payment_debit_account_id.id, self.payment_credit_account_id.id)),
-            ('journal_id', '=', self.id),
-            ('display_type', 'not in', ('line_section', 'line_note')),
-            ('parent_state', '!=', 'cancel'),
-            ]
-        action['context'] = {
-            'search_default_unreconciled': True,
-            'search_default_posted': True,
-            }
-        return action
+#    def open_outstanding_payments(self):
+#        self.ensure_one()
+#        action = self.env["ir.actions.actions"]._for_xml_id(
+#            "account.action_account_moves_all")
+#        action['domain'] = [
+#            ('account_id', 'in', (self.payment_debit_account_id.id, self.payment_credit_account_id.id)),
+#            ('journal_id', '=', self.id),
+#            ('display_type', 'not in', ('line_section', 'line_note')),
+#            ('parent_state', '!=', 'cancel'),
+#            ]
+#        action['context'] = {
+#            'search_default_unreconciled': True,
+#            'search_default_posted': True,
+#            }
+#        return action

account_usability_akretion/models/account_move.py

@@ -1,17 +1,14 @@
-# Copyright 2015-2022 Akretion (http://www.akretion.com)
+# Copyright 2015-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 datetime import timedelta
-from collections import defaultdict
 import logging
 
 from odoo import api, fields, models, _
-from odoo.exceptions import UserError
 from odoo.osv import expression
 from odoo.tools import float_is_zero
 from odoo.tools.misc import format_date
-from odoo.tools.safe_eval import safe_eval, time
 
 _logger = logging.getLogger(__name__)
 
@@ -19,9 +16,6 @@ _logger = logging.getLogger(__name__)
 class AccountMove(models.Model):
     _inherit = 'account.move'
 
-    # By default, we can still modify "ref" when account move is posted
-    # which seems a bit lazy for me...
-    ref = fields.Char(states={'posted': [('readonly', True)]})
     invoice_date_due = fields.Date(tracking=True)
     invoice_payment_term_id = fields.Many2one(tracking=True)
     journal_id = fields.Many2one(tracking=True)
@@ -38,36 +32,34 @@ class AccountMove(models.Model):
         compute="_compute_sales_dates",
         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,
-    )
+    # The native "blocked" field (bool) on account.move.line has been removed in v18
+#    blocked = fields.Boolean(
+#        compute="_compute_blocked",
+#        inverse="_inverse_blocked",
+#        store=True,
+#        string="Dispute",
+#        tracking=True,
+#    )
     # Field search_account_id is just for search view
     search_account_id = fields.Many2one(related='line_ids.account_id')
 
-    @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.account_type in ("liability_payable", "asset_receivable")
-                ]
-            )
+#    @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.account_type in ("liability_payable", "asset_receivable")
+#                ]
+#            )
 
-    def _inverse_blocked(self):
-        for move in self:
-            for line in move.line_ids.filtered(
-                lambda l: l.account_id.account_type in ("liability_payable", "asset_receivable")
-            ):
-                line.blocked = move.blocked
+#    def _inverse_blocked(self):
+#        for move in self:
+#            for line in move.line_ids.filtered(
+#                lambda l: l.account_id.account_type in ("liability_payable", "asset_receivable")
+#            ):
+#                line.blocked = move.blocked
 
     def _compute_has_discount(self):
         prec = self.env['decimal.precision'].precision_get('Discount')
@@ -103,35 +95,6 @@ class AccountMove(models.Model):
         res = [('id', value and 'in' or 'not in', list(att_inv_ids))]
         return res
 
-    # when you have an invoice created from a lot of sale orders, the 'name'
-    # field is very large, which makes the name_get() of that invoice very big
-    # which screws-up the form view of that invoice because of the link at the
-    # top of the screen
-    # That's why we have to cut the name_get() when it's too long
-    def name_get(self):
-        old_res = super().name_get()
-        res = []
-        for old_re in old_res:
-            name = old_re[1]
-            if name and len(name) > 100:
-                # nice cut
-                name = '%s ...' % ', '.join(name.split(', ')[:3])
-                # if not enough, hard cut
-                if len(name) > 120:
-                    name = '%s ...' % old_re[1][:120]
-            res.append((old_re[0], name))
-        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', '=', 'product'),
@@ -204,94 +167,27 @@ class AccountMove(models.Model):
                         ])
                 move.suitable_journal_ids = self.env['account.journal'].search(domain)
 
-    def button_draft(self):
+# There is no more attachment by default on invoice reports...
+# TODO check what's the editor strategy on this
+#    def button_draft(self):
         # Delete attached pdf invoice
-        for move in self.filtered(lambda x: x.move_type in ('out_invoice', 'out_refund')):
-            for report_xmlid in ('account.account_invoices', 'account.account_invoices_without_payment'):
-                report = self.env.ref(report_xmlid)
-                attach = report.retrieve_attachment(move)
-                if attach:
-                    attach.unlink()
-        super().button_draft()
+#        for move in self.filtered(lambda x: x.move_type in ('out_invoice', 'out_refund')):
+#            for report_xmlid in ('account.account_invoices', 'account.account_invoices_without_payment'):
+#                report = self.env.ref(report_xmlid)
+#                attach = report.retrieve_attachment(move)
+#                if attach:
+#                    attach.unlink()
+#        super().button_draft()
 
-    def _get_accounting_date(self, invoice_date, has_tax):
+    def _get_accounting_date(self, invoice_date, has_tax, lock_dates=None):
         # On vendor bills/refunds, we want date = invoice_date unless
         # we have a company tax_lock_date and the invoice has taxes
         # and invoice_date <= tax_lock_date
-        date = super()._get_accounting_date(invoice_date, has_tax)
-        if self.is_purchase_document(include_receipts=True):
-            tax_lock_date = self.company_id.tax_lock_date
-            if invoice_date and tax_lock_date and has_tax and invoice_date <= tax_lock_date:
-                invoice_date = tax_lock_date + timedelta(days=1)
-            date = invoice_date
+        date = super()._get_accounting_date(invoice_date, has_tax, lock_dates=lock_dates)
+        lock_dates = lock_dates or self._get_violated_lock_dates(invoice_date, has_tax)
+        if self.is_purchase_document(include_receipts=True) and invoice_date:
+            if lock_dates:
+                date = max([entry[0] for entry in lock_dates]) + timedelta(1)
+            else:
+               date = invoice_date
         return date
-
-
-class AccountMoveLine(models.Model):
-    _inherit = 'account.move.line'
-    # Native order:
-    # _order = "date desc, move_name desc, id"
-    # Problem: when you manually create a journal entry, the
-    # order of the lines is inverted when you save ! It is quite annoying for
-    # the user...
-    _order = "date desc, id asc"
-
-    # In the 'account' module, we have related stored field for:
-    # name (move_name), date, ref, state (parent_state),
-    # journal_id, company_id, payment_id, statement_line_id,
-    account_reconcile = fields.Boolean(related='account_id.reconcile')
-    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')
-    # for optional display in tree view
-    product_barcode = fields.Char(related='product_id.barcode', string="Product Barcode")
-
-    def show_account_move_form(self):
-        self.ensure_one()
-        action = self.env["ir.actions.actions"]._for_xml_id(
-            'account.action_move_line_form')
-        action.update({
-            'res_id': self.move_id.id,
-            'view_id': False,
-            'views': False,
-            'view_mode': 'form,tree',
-        })
-        return action
-
-    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 _compute_name(self):
-        # This is useful when you want to have the product code in a dedicated
-        # column in your customer invoice report
-        # The same ir.config_parameter is used in sale_usability,
-        # purchase_usability and account_usability
-        no_product_code_param = self.env['ir.config_parameter'].sudo().get_param(
-            'usability.line_name_no_product_code')
-        if no_product_code_param and no_product_code_param == 'True':
-            self = self.with_context(display_default_code=False)
-        return super()._compute_name()
-
-    def reconcile(self):
-        """Explicit error message if unposted lines"""
-        unposted_ids = self.filtered(lambda l: l.move_id.state != "posted")
-        if unposted_ids:
-            m = _("Please post the following entries before reconciliation :")
-            sep = "\n - "
-            unpost = sep.join([am.display_name for am in unposted_ids.move_id])
-            raise UserError(m + sep + unpost)
-
-        return super().reconcile()

account_usability_akretion/models/account_move_line.py

@@ -0,0 +1,70 @@
+# Copyright 2015-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
+import logging
+
+_logger = logging.getLogger(__name__)
+
+
+class AccountMoveLine(models.Model):
+    _inherit = 'account.move.line'
+    # Native order:
+    # _order = "date desc, move_name desc, id"
+    # Problem: when you manually create a journal entry, the
+    # order of the lines is inverted when you save ! It is quite annoying for
+    # the user...
+    _order = "date desc, id asc"
+
+    # for optional display in list view
+    product_barcode = fields.Char(related='product_id.barcode', string="Product Barcode")
+
+    def show_account_move_form(self):
+        self.ensure_one()
+        action = self.env["ir.actions.actions"]._for_xml_id(
+            'account.action_move_line_form')
+        action.update({
+            'res_id': self.move_id.id,
+            'view_id': False,
+            'views': False,
+            'view_mode': 'form,list',
+        })
+        return action
+
+#    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 _compute_name(self):
+        # This is useful when you want to have the product code in a dedicated
+        # column in your customer invoice report
+        # The same ir.config_parameter is used in sale_usability,
+        # purchase_usability and account_usability
+        no_product_code_param = self.env['ir.config_parameter'].sudo().get_param(
+            'usability.line_name_no_product_code')
+        if no_product_code_param and no_product_code_param == 'True':
+            self = self.with_context(display_default_code=False)
+        return super()._compute_name()
+
+#    def reconcile(self):
+#        """Explicit error message if unposted lines"""
+#        unposted_ids = self.filtered(lambda l: l.move_id.state != "posted")
+#        if unposted_ids:
+#            m = _("Please post the following entries before reconciliation :")
+#            sep = "\n - "
+#            unpost = sep.join([am.display_name for am in unposted_ids.move_id])
+#            raise UserError(m + sep + unpost)
+#        return super().reconcile()

account_usability_akretion/models/account_partial_reconcile.py

@@ -3,21 +3,21 @@
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 
 from odoo import models
-from odoo.tools.misc import formatLang
+# from odoo.tools.misc import formatLang
 
 
 class AccountPartialReconcile(models.Model):
     _inherit = "account.partial.reconcile"
-    _rec_name = "id"
+#    _rec_name = "id"
 
-    def name_get(self):
-        res = []
-        for rec in self:
+#    def name_get(self):
+#        res = []
+#        for rec in self:
             # There is no seq for partial rec, so I simulate one with the ID
             # Prefix for full rec: 'A' (upper case)
             # Prefix for partial rec: 'a' (lower case)
-            amount_fmt = formatLang(
-                self.env, rec.amount, currency_obj=rec.company_currency_id)
-            name = 'a%d (%s)' % (rec.id, amount_fmt)
-            res.append((rec.id, name))
-        return res
+#            amount_fmt = formatLang(
+#                self.env, rec.amount, currency_obj=rec.company_currency_id)
+#            name = 'a%d (%s)' % (rec.id, amount_fmt)
+#            res.append((rec.id, name))
+#        return res

account_usability_akretion/models/product.py

@@ -4,6 +4,9 @@
 
 from odoo import api, fields, models, _
 
+# CODE DISABLED FOR THE MOMENT. I need to decide if I want to drop it
+# because native code is enough and if I want to re-enable it
+
 
 class ProductTemplate(models.Model):
     _inherit = 'product.template'

account_usability_akretion/models/res_company.py

@@ -13,7 +13,7 @@ class ResCompany(models.Model):
     # But there are several problems with this native field:
     # - it is copied on the 'narration' field of account.move => we don't want that
     # - the text block is very small on the form view of res.config.settings
-    # So I decided to have our own field "fixed_invoice_terms"
+    # So I decided to have our own field "static_invoice_terms"
     # The native field can still be used when you need to customise some
     # terms and conditions on each invoice (not very common, but...)
     # To underline this different with the native field, I prefix it with 'static_'

account_usability_akretion/models/res_partner.py

@@ -1,4 +1,4 @@
-# Copyright 2017-2022 Akretion France (https://akretion.com/)
+# Copyright 2017-2024 Akretion France (https://akretion.com/)
 # @author: Alexis de Lattre <alexis.delattre@akretion.com>
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 

account_usability_akretion/models/res_partner_bank.py

@@ -1,21 +0,0 @@
-# Copyright 2015-2022 Akretion France (http://www.akretion.com/)
-# @author: Mourad EL HADJ MIMOUNE <mourad.elhadj.mimoune@akretion.com>
-# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
-
-from odoo import models
-
-
-class ResPartnerBank(models.Model):
-    _inherit = 'res.partner.bank'
-
-    def name_get(self):
-        res = []
-        for acc in self:
-            name = acc.acc_number
-            if acc.currency_id:
-                name = "%s (%s)" % (name, acc.currency_id.name)
-            if acc.bank_id.name:
-                name = "%s - %s" % (name, acc.bank_id.name)
-            res += [(acc.id, name)]
-        return res
-

account_usability_akretion/pyproject.toml

@@ -0,0 +1,3 @@
+[build-system]
+requires = ["whool"]
+build-backend = "whool.buildapi"

account_usability_akretion/security/ir.model.access.csv

@@ -1,3 +1,2 @@
 id,name,model_id:id,group_id:id,perm_read,perm_write,perm_create,perm_unlink
-access_account_group_generate_full,Full access on account.group.generate,model_account_group_generate,account.group_account_manager,1,1,1,1
 access_account_invoice_mark_sent_full,Full access on account.invoice.mark.sent,model_account_invoice_mark_sent,account.group_account_invoice,1,1,1,1

account_usability_akretion/static/description/index.html

@@ -1,20 +1,20 @@
-<?xml version="1.0" encoding="utf-8" ?>
 <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
 <html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">
 <head>
 <meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
-<meta name="generator" content="Docutils 0.15.1: http://docutils.sourceforge.net/" />
+<meta name="generator" content="Docutils: https://docutils.sourceforge.io/" />
 <title>Account Usability</title>
 <style type="text/css">
 
 /*
 :Author: David Goodger (goodger@python.org)
-:Id: $Id: html4css1.css 7952 2016-07-26 18:15:59Z milde $
+:Id: $Id: html4css1.css 9511 2024-01-13 09:50:07Z milde $
 :Copyright: This stylesheet has been placed in the public domain.
 
 Default cascading style sheet for the HTML output of Docutils.
+Despite the name, some widely supported CSS2 features are used.
 
-See http://docutils.sf.net/docs/howto/html-stylesheets.html for how to
+See https://docutils.sourceforge.io/docs/howto/html-stylesheets.html for how to
 customize this style sheet.
 */
 
@@ -275,7 +275,7 @@ pre.literal-block, pre.doctest-block, pre.math, pre.code {
   margin-left: 2em ;
   margin-right: 2em }
 
-pre.code .ln { color: grey; } /* line numbers */
+pre.code .ln { color: gray; } /* line numbers */
 pre.code, code { background-color: #eeeeee }
 pre.code .comment, code .comment { color: #5C6576 }
 pre.code .keyword, code .keyword { color: #3B0D06; font-weight: bold }
@@ -301,7 +301,7 @@ span.option {
 span.pre {
   white-space: pre }
 
-span.problematic {
+span.problematic, pre.problematic {
   color: red }
 
 span.section-subtitle {
@@ -366,8 +366,10 @@ ul.auto-toc {
 <!-- !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
 !! This file is generated by oca-gen-addon-readme !!
 !! changes will be overwritten.                   !!
+!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+!! source digest: sha256:dbf367e7cf8330c803622d0c171cd10e128344d318466d6e37ad7b0fc812e152
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
-<p><a class="reference external" href="https://odoo-community.org/page/development-status"><img alt="Beta" src="https://img.shields.io/badge/maturity-Beta-yellow.png" /></a> <a class="reference external" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/licence-AGPL--3-blue.png" /></a> <a class="reference external" href="https://github.com/akretion/odoo-usability/tree/12.0/account_usability"><img alt="akretion/odoo-usability" src="https://img.shields.io/badge/github-akretion%2Fodoo--usability-lightgray.png?logo=github" /></a></p>
+<p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Beta" src="https://img.shields.io/badge/maturity-Beta-yellow.png" /></a> <a class="reference external image-reference" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/licence-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/akretion/odoo-usability/tree/18.0/account_usability_akretion"><img alt="akretion/odoo-usability" src="https://img.shields.io/badge/github-akretion%2Fodoo--usability-lightgray.png?logo=github" /></a></p>
 <p>This modules adds the following functions:</p>
 <ul class="simple">
 <li>Add an <em>Overdue</em> filter on invoice search view (this feature was previously
@@ -411,41 +413,41 @@ located in te module <em>account_invoice_overdue_filter</em>)</li>
 <p><strong>Table of contents</strong></p>
 <div class="contents local topic" id="contents">
 <ul class="simple">
-<li><a class="reference internal" href="#bug-tracker" id="id1">Bug Tracker</a></li>
-<li><a class="reference internal" href="#credits" id="id2">Credits</a><ul>
-<li><a class="reference internal" href="#authors" id="id3">Authors</a></li>
-<li><a class="reference internal" href="#contributors" id="id4">Contributors</a></li>
-<li><a class="reference internal" href="#maintainers" id="id5">Maintainers</a></li>
+<li><a class="reference internal" href="#bug-tracker" id="toc-entry-1">Bug Tracker</a></li>
+<li><a class="reference internal" href="#credits" id="toc-entry-2">Credits</a><ul>
+<li><a class="reference internal" href="#authors" id="toc-entry-3">Authors</a></li>
+<li><a class="reference internal" href="#contributors" id="toc-entry-4">Contributors</a></li>
+<li><a class="reference internal" href="#maintainers" id="toc-entry-5">Maintainers</a></li>
 </ul>
 </li>
 </ul>
 </div>
 <div class="section" id="bug-tracker">
-<h1><a class="toc-backref" href="#id1">Bug Tracker</a></h1>
+<h1><a class="toc-backref" href="#toc-entry-1">Bug Tracker</a></h1>
 <p>Bugs are tracked on <a class="reference external" href="https://github.com/akretion/odoo-usability/issues">GitHub Issues</a>.
 In case of trouble, please check there if your issue has already been reported.
-If you spotted it first, help us smashing it by providing a detailed and welcomed
-<a class="reference external" href="https://github.com/akretion/odoo-usability/issues/new?body=module:%20account_usability%0Aversion:%2012.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**">feedback</a>.</p>
+If you spotted it first, help us to smash it by providing a detailed and welcomed
+<a class="reference external" href="https://github.com/akretion/odoo-usability/issues/new?body=module:%20account_usability_akretion%0Aversion:%2018.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**">feedback</a>.</p>
 <p>Do not contact contributors directly about support or help with technical issues.</p>
 </div>
 <div class="section" id="credits">
-<h1><a class="toc-backref" href="#id2">Credits</a></h1>
+<h1><a class="toc-backref" href="#toc-entry-2">Credits</a></h1>
 <div class="section" id="authors">
-<h2><a class="toc-backref" href="#id3">Authors</a></h2>
+<h2><a class="toc-backref" href="#toc-entry-3">Authors</a></h2>
 <ul class="simple">
 <li>Akretion</li>
 </ul>
 </div>
 <div class="section" id="contributors">
-<h2><a class="toc-backref" href="#id4">Contributors</a></h2>
+<h2><a class="toc-backref" href="#toc-entry-4">Contributors</a></h2>
 <ul class="simple">
 <li>Alexis de Lattre &lt;<a class="reference external" href="mailto:alexis.delattre&#64;akretion.com">alexis.delattre&#64;akretion.com</a>&gt;</li>
 <li>David Beal &lt;<a class="reference external" href="mailto:david.beal&#64;akretion.com">david.beal&#64;akretion.com</a>&gt;</li>
 </ul>
 </div>
 <div class="section" id="maintainers">
-<h2><a class="toc-backref" href="#id5">Maintainers</a></h2>
-<p>This module is part of the <a class="reference external" href="https://github.com/akretion/odoo-usability/tree/12.0/account_usability">akretion/odoo-usability</a> project on GitHub.</p>
+<h2><a class="toc-backref" href="#toc-entry-5">Maintainers</a></h2>
+<p>This module is part of the <a class="reference external" href="https://github.com/akretion/odoo-usability/tree/18.0/account_usability_akretion">akretion/odoo-usability</a> project on GitHub.</p>
 <p>You are welcome to contribute.</p>
 </div>
 </div>

account_usability_akretion/views/account_account.xml

@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="utf-8"?>
 <!--
-  Copyright 2015-2022 Akretion France (http://www.akretion.com/)
+  Copyright 2015-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).
 -->
@@ -14,7 +14,7 @@
     <field name="inherit_id" ref="account.view_account_form"/>
     <field name="arch" type="xml">
         <field name="deprecated" position="before">
-            <field name="reconcile" attrs="{'invisible': ['|', ('account_type', 'in', ('asset_cash', 'liability_credit_card')), ('internal_group', '=', 'off_balance')]}" widget="boolean_toggle"/>
+            <field name="reconcile" invisible="account_type in ('asset_cash', 'liability_credit_card', 'off_balance')" widget="boolean_toggle"/>
         </field>
     </field>
 </record>
@@ -28,9 +28,6 @@
         <field name="name" position="after">
             <field name="code" filter_domain="[('code', '=like', self + '%')]" string="Code"/>
         </field>
-        <filter name="accounttype" position="after">
-            <filter name="group_groupby" string="Group" context="{'group_by': 'group_id'}"/>
-        </filter>
     </field>
 </record>
 

account_usability_akretion/views/account_analytic_line.xml

@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="utf-8"?>
 <!--
-  Copyright 2024 Akretion France (http://www.akretion.com/)
+  Copyright 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).
 -->
@@ -12,8 +12,8 @@
     <field name="model">account.analytic.line</field>
     <field name="inherit_id" ref="account.view_account_analytic_line_tree_inherit_account"/>
     <field name="arch" type="xml">
-	<field name="general_account_id" position="attributes">
-	    <attribute name="optional">show</attribute>
+        <field name="general_account_id" position="attributes">
+            <attribute name="optional">show</attribute>
         </field>
     </field>
 </record>

account_usability_akretion/views/account_group.xml

@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="utf-8"?>
 <!--
-  Copyright 2021-2022 Akretion France (http://www.akretion.com/)
+  Copyright 2021-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).
 -->

account_usability_akretion/views/account_invoice_report.xml

@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="utf-8"?>
 <!--
-  Copyright 2018-2024 Akretion (http://www.akretion.com/)
+  Copyright 2018-2024 Akretion (https://www.akretion.com/)
   @author: Alexis de Lattre <alexis.delattre@akretion.com>
   License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 -->
@@ -14,14 +14,13 @@
     <field name="inherit_id" ref="account.account_invoice_report_view_tree"/>
     <field name="arch" type="xml">
         <field name="partner_id" position="after">
-	   <field name="commercial_partner_id" optional="hide"/>
-            	<field name="country_id" optional="hide"/>
-            <field name="industry_id" optional="hide"/>
-            <field name="fiscal_position_id" optional="hide"/>
-	</field>
+           <field name="commercial_partner_id" optional="hide"/>
+           <field name="industry_id" optional="hide"/>
+           <field name="fiscal_position_id" optional="hide"/>
+        </field>
         <field name="quantity" position="after">
-		<field name="product_uom_id" groups="uom.group_uom" optional="hide"/>
-	</field>
+            <field name="product_uom_id" groups="uom.group_uom" optional="hide"/>
+        </field>
     </field>
 </record>
 
@@ -29,12 +28,19 @@
     <field name="model">account.invoice.report</field>
     <field name="inherit_id" ref="account.view_account_invoice_report_search"/>
     <field name="arch" type="xml">
-	<filter name="category_product" position="after">
-	    <filter string="Product" name="product_groupby" context="{'group_by': 'product_id', 'residual_invisible':True}"/>
-       </filter>
+        <field name="partner_id" position="after">
+            <field name="industry_id"/>
+        </field>
+        <filter name="category_product" position="after">
+            <filter string="Product" name="product_groupby" context="{'group_by': 'product_id', 'residual_invisible':True}"/>
+        </filter>
+        <filter name="partner_id" position="after">
+            <filter name="industry_groupby" string="Partner Industry" context="{'group_by': 'industry_id'}"/>
+        </filter>
    </field>
 </record>
 
+<!-- pivot in first position instead of graph -->
 <record id="account.action_account_invoice_report_all_supp" model="ir.actions.act_window">
     <field name="view_mode">pivot,graph</field>
 </record>

account_usability_akretion/views/account_journal.xml

@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="utf-8"?>
 <!--
-  Copyright 2015-2022 Akretion France (http://www.akretion.com/)
+  Copyright 2015-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).
 -->
@@ -13,30 +13,13 @@
     <field name="inherit_id" ref="account.view_account_journal_form"/>
     <field name="arch" type="xml">
         <xpath expr="//field[@name='account_control_ids']/.." position="after">
-            <group name="usability" string="Misc" attrs="{'invisible': [('type', '!=', 'bank')]}">
+            <group name="usability" string="Misc" invisible="type != 'bank'">
                 <field name="hide_bank_statement_balance" groups="account.group_account_readonly"/>
             </group>
         </xpath>
-        <xpath expr="//field[@name='inbound_payment_method_line_ids']/tree/field[@name='payment_account_id']" position="attributes">
-            <attribute name="optional">show</attribute>
-        </xpath>
-        <xpath expr="//field[@name='outbound_payment_method_line_ids']/tree/field[@name='payment_account_id']" position="attributes">
-            <attribute name="optional">show</attribute>
-        </xpath>
-        <!--
-        <field name="suspense_account_id" position="attributes">
-            <attribute name="context">{'default_user_type_id': account_type_current_assets_id, 'default_reconcile': False}</attribute>
-        </field>
-        <field name="payment_debit_account_id" position="attributes">
-            <attribute name="context">{'default_user_type_id': account_type_current_assets_id, 'default_reconcile': True}</attribute>
-        </field>
-        <field name="payment_credit_account_id" position="attributes">
-            <attribute name="context">{'default_user_type_id': account_type_current_assets_id, 'default_reconcile': True}</attribute>
-        </field> -->
     </field>
 </record>
 
-<!-- TODO
 <record id="account_journal_dashboard_kanban_view" model="ir.ui.view">
     <field name="name">usability.account.journal.dashboard</field>
     <field name="model">account.journal</field>
@@ -48,9 +31,10 @@
         <xpath expr="//div[@name='latest_statement']/.." position="attributes">
             <attribute name="t-if">dashboard.has_at_least_one_statement and dashboard.account_balance != dashboard.last_balance and !record.hide_bank_statement_balance.raw_value</attribute>
         </xpath>
+        <!--
         <t t-esc="dashboard.outstanding_pay_account_balance" position="replace">
             <a name="open_outstanding_payments" type="object" title="Outstanding Payments/Receipts"><t t-esc="dashboard.outstanding_pay_account_balance"/></a>
-        </t>
+        </t> -->
     </field>
 </record>
 -->

account_usability_akretion/views/account_menu.xml

@@ -1,13 +1,13 @@
 <?xml version="1.0" encoding="utf-8"?>
 <!--
-  Copyright 2015-2022 Akretion France (http://www.akretion.com/)
+  Copyright 2015-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).
 -->
 
 <odoo>
 
-<!-- Duplicate the menu "Sales > Configuration > Contacts > Bank Accounts"
+<!-- Duplicate the menu "Bank Accounts"
 under "Accounting > Configuration", because most users will try to find it there -->
 <menuitem id="res_bank_account_config_menu" action="base.action_res_bank_form" parent="account.account_banks_menu" sequence="10"/>
 

account_usability_akretion/views/account_move.xml

@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="utf-8"?>
 <!--
-  Copyright 2015-2022 Akretion France (http://www.akretion.com/)
+  Copyright 2015-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).
 -->
@@ -12,37 +12,30 @@
     <field name="model">account.move</field>
     <field name="inherit_id" ref="account.view_move_form"/>
     <field name="arch" type="xml">
-        <!--
-        <field name="fiscal_position_id" position="attributes">
-            <attribute name="widget">selection</attribute>
-        </field>
-        <field name="invoice_incoterm_id" position="attributes">
-            <attribute name="widget">selection</attribute>
-        </field> -->
         <button id="account_invoice_payment_btn" position="attributes">
             <attribute name="class">btn-default</attribute>
         </button>
         <button name="action_register_payment" position="before">
-            <button name="%(account.account_invoices)d" type="action" string="Print" attrs="{'invisible': [('move_type', 'not in', ('out_invoice', 'out_refund'))]}"/>
+            <button name="action_print_pdf" type="object" string="Print" invisible="move_type not in ('out_invoice', 'out_refund') or state != 'draft'"/>
         </button>
         <button name="preview_invoice" position="attributes">
-            <attribute name="attrs">{}</attribute>
             <attribute name="invisible">1</attribute>
         </button>
-        <!-- move sent field and make it visible -->
+        <!-- move field is_move_sent and make it visible -->
         <field name="is_move_sent" position="replace"/>
         <field name="invoice_origin" position="attributes">
             <attribute name="invisible">0</attribute>
         </field>
         <field name="invoice_origin" position="after">
-            <field name="is_move_sent" attrs="{'invisible': [('move_type', 'not in', ('out_invoice', 'out_refund'))]}"/>
+            <field name="is_move_sent" invisible="move_type not in ('out_invoice', 'out_refund')"/>
         </field>
-        <xpath expr="//field[@name='line_ids']/tree/field[@name='tax_tag_ids']" position="after">
+        <xpath expr="//field[@name='line_ids']/list/field[@name='tax_tag_ids']" position="after">
             <field name="matching_number" optional="show"/>
         </xpath>
-        <xpath expr="//field[@name='invoice_line_ids']/tree/field[@name='product_id']" position="after">
+        <xpath expr="//field[@name='invoice_line_ids']/list/field[@name='product_id']" position="after">
             <field name="product_barcode" optional="hide"/>
         </xpath>
+        <!--
         <field name="invoice_source_email" position="after">
             <field name="blocked"/>
         </field>
@@ -52,15 +45,15 @@
                     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>
-        <xpath expr="//button[@name='open_duplicated_ref_bill_view']/.." position="attributes">
+        </div> -->
+        <xpath expr="//field[@name='duplicated_ref_ids']/.." position="attributes">
             <!-- show duplicate warning not only in draft state, but also in posted state -->
-            <attribute name="attrs">{'invisible': ['|', ('state', '=', 'cancel'), ('duplicated_ref_ids', '=', [])]}</attribute>
+            <attribute name="invisible">not duplicated_ref_ids</attribute>
         </xpath>
-        <button name="button_cancel" attrs="{'invisible' : ['|', '|', ('id', '=', False), ('state', '!=', 'draft'),('move_type', '!=', 'entry')]}" position="attributes">
+        <button name="button_cancel" invisible="not id or state != 'draft' or move_type != 'entry'" position="attributes">
             <attribute name="confirm">Are you sure you want to cancel this journal entry?</attribute>
         </button>
-        <button name="button_cancel" attrs="{'invisible' : ['|', '|', ('id', '=', False), ('state', '!=', 'draft'),('move_type', '==', 'entry')]}" position="attributes">
+        <button name="button_cancel" invisible="not id or state != 'draft' or move_type == 'entry'" position="attributes">
             <attribute name="confirm">Are you sure you want to cancel this invoice?</attribute>
         </button>
     </field>
@@ -71,6 +64,7 @@
     <field name="inherit_id" ref="account.view_invoice_tree"/>
     <field name="arch" type="xml">
         <field name="amount_residual_signed" position="attributes">
+            <!-- switch from hide to show -->
             <attribute name="optional">show</attribute>
         </field>
     </field>
@@ -82,15 +76,11 @@
     <field name="inherit_id" ref="account.view_account_invoice_filter"/>
     <field name="arch" type="xml">
         <filter name="due_date" position="after">
-            <separator/>
-             <filter name="to_send" string="To Send" domain="[('is_move_sent', '=', False), ('state', '=', 'posted'), ('move_type', 'in', ('out_invoice', 'out_refund'))]"/>
-             <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 name="dispute" string="Dispute" domain="[('blocked', '=', True)]"/> -->
         </filter>
-        <filter name="salesperson" position="before">
+        <filter name="partner" position="before">
             <filter name="commercial_partner_groupby" string="Commercial Partner" context="{'group_by': 'commercial_partner_id'}"/>
         </filter>
         <filter name="status" position="after">
@@ -109,70 +99,5 @@
     </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_type', 'not in', ('liability_payable', 'asset_receivable'))]}</attribute>
-        </xpath>
-        <field name="account_id" position="after">
-            <field name="account_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"/>
-    <field name="arch" type="xml">
-        <field name="analytic_distribution" position="after">
-            <button title="View Journal Entry Form" type="object" name="show_account_move_form" icon="fa-arrow-right"/>
-        </field>
-        <!-- balance is already present
-        <field name="credit" position="after">
-            <field name="balance" sum="Balance" optional="show"/>
-        </field>  -->
-    </field>
-</record>
-
-<record id="view_account_move_line_filter" model="ir.ui.view">
-    <field name="name">account_usability.account_move_line_search</field>
-    <field name="model">account.move.line</field>
-    <field name="inherit_id" ref="account.view_account_move_line_filter"/>
-    <field name="arch" type="xml">
-        <field name="partner_id" position="after">
-            <field name="matching_number" />
-            <field name="debit" filter_domain="['|', ('debit', '=', self), ('credit', '=', self)]" string="Debit or Credit"/>
-        </field>
-        <filter name="unreconciled" position="before">
-            <filter name="reconciled" string="Fully Reconciled" domain="[('account_id.reconcile', '=', True), ('full_reconcile_id', '!=', False)]"/>
-        </filter>
-        <filter name="unreconciled" position="attributes">
-            <attribute name="string">Unreconciled or Partially Reconciled</attribute>
-        </filter>
-        <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>
-
-<!-- Inherit action called from button of account.account form view
-Make it similar to standard "Journal Items" menu account.action_account_moves_all_a -->
-<record id="account.action_move_line_select" model="ir.actions.act_window">
-    <field name="domain">[('display_type', 'not in', ('line_section', 'line_note'))]</field>
-    <field name="view_id" ref="account.view_move_line_tree"/>
-    <field name="view_mode">tree,pivot,graph,kanban</field>
-</record>
 
 </odoo>

account_usability_akretion/views/account_move_line.xml

@@ -0,0 +1,52 @@
+<?xml version="1.0" encoding="utf-8"?>
+<!--
+  Copyright 2015-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).
+-->
+
+<odoo>
+
+<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"/>
+    <field name="arch" type="xml">
+        <list position="inside">
+            <button title="View Journal Entry Form" type="object" name="show_account_move_form" icon="fa-arrow-right"/>
+        </list>
+    </field>
+</record>
+
+<record id="view_account_move_line_filter" model="ir.ui.view">
+    <field name="name">account_usability.account_move_line_search</field>
+    <field name="model">account.move.line</field>
+    <field name="inherit_id" ref="account.view_account_move_line_filter"/>
+    <field name="arch" type="xml">
+        <field name="partner_id" position="after">
+            <field name="matching_number" />
+            <field name="debit" filter_domain="['|', ('debit', '=', self), ('credit', '=', self)]" string="Debit or Credit"/>
+        </field>
+        <filter name="reconcilable_account" position="before">
+            <filter name="reconciled" string="Fully Reconciled" domain="[('account_id.reconcile', '=', True), ('full_reconcile_id', '!=', False)]"/>
+        </filter>
+        <filter name="reconcilable_account" position="attributes">
+            <attribute name="string">Unreconciled or Partially Reconciled</attribute>
+        </filter>
+        <field name="name" position="attributes">
+            <attribute name="string">Label, Reference, Account or Partner</attribute>
+        </field>
+        <field name="partner_id" position="attributes">
+            <attribute name="domain">['|', ('parent_id', '=', False), ('is_company', '=', True)]</attribute>
+        </field>
+    </field>
+</record>
+
+<!-- Inherit action called from button of account.account form view
+Make it similar to standard "Journal Items" menu account.action_account_moves_all_a -->
+<record id="account.action_move_line_select" model="ir.actions.act_window">
+    <field name="domain">[('display_type', 'not in', ('line_section', 'line_note'))]</field>
+    <field name="view_id" ref="account.view_move_line_tree"/>
+    <field name="view_mode">list,pivot,graph,kanban</field>
+</record>
+
+</odoo>

account_usability_akretion/views/account_payment.xml

@@ -0,0 +1,20 @@
+<?xml version="1.0" encoding="utf-8"?>
+<!--
+  Copyright 2025 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).
+-->
+
+<odoo>
+
+<record id="view_account_payment_form" model="ir.ui.view">
+    <field name="model">account.payment</field>
+    <field name="inherit_id" ref="account.view_account_payment_form"/>
+    <field name="arch" type="xml">
+        <group name="group2" position="inside">
+            <field name="move_id" readonly="1"/>
+        </group>
+    </field>
+</record>
+
+</odoo>

account_usability_akretion/views/account_report.xml

@@ -7,6 +7,10 @@
 
 <odoo>
 
+        <!-- in v18, attachments are disabled by default
+                TODO  see what's the editor idea about it
+        in the meantime, this field is disabled in manifest -->
+
 <record id="account.account_invoices" model="ir.actions.report">
     <!-- Attach only on customer invoices/refunds -->
     <field name="attachment">(object.move_type in ('out_invoice', 'out_refund')) and (object.state == 'posted') and ((object.name or 'INV').replace('/','_')+'.pdf')</field>

account_usability_akretion/views/account_tax.xml

@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="utf-8"?>
 <!--
-  Copyright 2015-2022 Akretion France (http://www.akretion.com/)
+  Copyright 2015-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).
 -->

account_usability_akretion/views/res_company.xml

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

account_usability_akretion/views/res_partner.xml

@@ -1,20 +1,30 @@
 <?xml version="1.0" encoding="utf-8"?>
 <!--
-  Copyright 2014-2024 Akretion (http://www.akretion.com/)
+  Copyright 2014-2024 Akretion (https://www.akretion.com/)
   @author: Mourad EL HADJ MIMOUNE <mourad.elhadj.mimoune@akretion.com>
   License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 -->
 
 <odoo>
 
-<record id="view_partner_simple_form" model="ir.ui.view">
-    <field name="name">base_usability.title.on.partner.simplified.form</field>
+<record id="view_partner_property_form" model="ir.ui.view">
     <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']//field[@name='acc_number']" position="after">
+        <xpath expr="//field[@name='bank_ids']/list/field[@name='acc_number']" position="after">
             <field name="currency_id" optional="hide"/>
         </xpath>
     </field>
 </record>
+
+<record id="res_partner_view_tree" model="ir.ui.view">
+    <field name="model">res.partner</field>
+    <field name="inherit_id" ref="account.res_partner_view_tree"/>
+    <field name="arch" type="xml">
+	 <field name="invoice_edi_format" position="after">
+		 <field name="property_account_position_id" optional="hide"/>
+	 </field>
+    </field>
+</record>
+
 </odoo>

account_usability_akretion/wizard/account_payment_register_views.xml

@@ -1,22 +0,0 @@
-<?xml version="1.0" encoding="utf-8"?>
-<!--
-  Copyright 2021-2022 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>
-
-<!-- When you change the date, it resets the amount via the onchange
-So, in the view, the date should be BEFORE the amount -->
-<record id="view_account_payment_register_form" model="ir.ui.view">
-    <field name="model">account.payment.register</field>
-    <field name="inherit_id" ref="account.view_account_payment_register_form"/>
-    <field name="arch" type="xml">
-        <label for="amount" position="before">
-            <field name="payment_date" position="move"/>
-        </label>
-    </field>
-</record>
-
-</odoo>

account_usability_akretion/wizards/__init__.py

@@ -1,3 +1,3 @@
 from . import account_invoice_mark_sent
 from . import account_move_reversal
-from . import account_group_generate
+#from . import account_group_generate

account_usability_akretion/wizards/account_group_generate.py

@@ -1,4 +1,4 @@
-# Copyright 2015-2022 Akretion (http://www.akretion.com)
+# Copyright 2015-2024 Akretion (https://www.akretion.com)
 # @author Alexis de Lattre <alexis.delattre@akretion.com>
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 
@@ -56,7 +56,7 @@ class AccountGroupGenerate(models.TransientModel):
         action = {
             'type': 'ir.actions.act_window',
             'name': _('Account Groups'),
-            'view_mode': 'tree,form',
+            'view_mode': 'list,form',
             'res_model': 'account.group',
             }
         return action

account_usability_akretion/wizards/account_invoice_mark_sent.py

@@ -1,4 +1,4 @@
-# Copyright 2017-2022 Akretion France (https://akretion.com/en)
+# Copyright 2017-2024 Akretion France (https://akretion.com/en)
 # @author: Alexis de Lattre <alexis.delattre@akretion.com>
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 
@@ -12,7 +12,7 @@ class AccountInvoiceMarkSent(models.TransientModel):
     _description = 'Mark invoices as sent'
 
     def run(self):
-        assert self.env.context.get('active_model') == 'account.move',\
+        assert self.env.context.get('active_model') == 'account.move', \
             'Source model must be invoices'
         assert self.env.context.get('active_ids'), 'No invoices selected'
         invoices = self.env['account.move'].search([

account_usability_akretion/wizards/account_invoice_mark_sent_view.xml

@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="utf-8"?>
 <!--
-  Copyright 2017-2022 Akretion France
+  Copyright 2017-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).
 -->
@@ -11,7 +11,7 @@
     <field name="name">account.invoice.mark.sent.form</field>
     <field name="model">account.invoice.mark.sent</field>
     <field name="arch"  type="xml">
-        <form string="Mark invoices as sent">
+        <form>
             <p>
                 This wizard will mark as <i>sent</i> all the selected posted invoices.
             </p>

account_usability_akretion/wizards/account_move_reversal.py

@@ -1,10 +1,9 @@
-# Copyright 2018-2022 Akretion France (https://akretion.com/)
+# Copyright 2018-2024 Akretion France (https://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, _
-from dateutil.relativedelta import relativedelta
-from odoo.exceptions import UserError
+from datetime import timedelta
 
 
 class AccountMoveReversal(models.TransientModel):
@@ -17,15 +16,9 @@ class AccountMoveReversal(models.TransientModel):
         for wizard in self:
             moves = wizard.move_ids or self.env["account.move"].browse(self._context['active_ids'])
             reversed_moves = self.env["account.move"].search([('reversed_entry_id', 'in', moves.ids)])
-            warning = ""
-            for already_reversed_move in reversed_moves.reversed_entry_id:
-                if warning:
-                    warning += "\n"
-                reversed_by = " ; ".join(already_reversed_move.reversal_move_id.mapped("display_name"))
-                move_detail = _("%s reversed by %s") % (already_reversed_move.display_name, reversed_by)
-                warning += move_detail
-            wizard.already_reversed_warning = warning or False
-
+            # in v18, display_name contains "MISC/2024/0008 (Reversal of: MISC/2024/0007)"
+            warning = "\n".join([m.display_name for m in reversed_moves])
+            wizard.already_reversed_warning = warning
 
     # Set default reversal date to original move + 1 day
     # and raise error if original move has already been reversed
@@ -36,5 +29,5 @@ class AccountMoveReversal(models.TransientModel):
         amo = self.env['account.move']
         moves = amo.browse(self._context['active_ids'])
         if len(moves) == 1:
-            res['date'] = moves.date + relativedelta(days=1)
+            res['date'] = moves.date + timedelta(1)
         return res

account_usability_akretion/wizards/account_move_reversal.xml

@@ -9,10 +9,9 @@
             <div
                 class="alert alert-warning"
                 role="alert"
-                attrs="{'invisible': [('already_reversed_warning', '=', False)]}"
+                invisible="not already_reversed_warning"
             >
-                You are about to reverse entries that have already been reversed or partially reversed (refund). Make sure it is intented.
-                Already reversed entries are the following : 
+                You are about to reverse entries that have already been reversed:
                 <field
                     name="already_reversed_warning"
                 /> 

base_company_extension/README.rst

@@ -2,10 +2,13 @@
 Base Company Extension
 ======================
 
-.. !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+.. 
+   !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
    !! This file is generated by oca-gen-addon-readme !!
    !! changes will be overwritten.                   !!
    !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+   !! source digest: sha256:075f72950033a5c6f57ecfa5c2d101bd874dd2ae29adcecdc4905b378e89a3a2
+   !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
 
 .. |badge1| image:: https://img.shields.io/badge/maturity-Beta-yellow.png
     :target: https://odoo-community.org/page/development-status
@@ -14,25 +17,29 @@ Base Company Extension
     :target: http://www.gnu.org/licenses/agpl-3.0-standalone.html
     :alt: License: AGPL-3
 .. |badge3| image:: https://img.shields.io/badge/github-akretion%2Fodoo--usability-lightgray.png?logo=github
-    :target: https://github.com/akretion/odoo-usability/tree/12.0/base_company_extension
+    :target: https://github.com/akretion/odoo-usability/tree/18.0/base_company_extension
     :alt: akretion/odoo-usability
 
-|badge1| |badge2| |badge3| 
-
-This module adds the following fields on the company:
+|badge1| |badge2| |badge3|
 
+This module adds the following fields to the ResCompany model:
 * Capital Amount
 * Legal Type
 
-This is useful to display the legal name of the company in reports.
+This is useful to display the legal name of the company in reports
+
+**Table of contents**
+
+.. contents::
+   :local:
 
 Bug Tracker
 ===========
 
 Bugs are tracked on `GitHub Issues <https://github.com/akretion/odoo-usability/issues>`_.
 In case of trouble, please check there if your issue has already been reported.
-If you spotted it first, help us smashing it by providing a detailed and welcomed
-`feedback <https://github.com/akretion/odoo-usability/issues/new?body=module:%20base_company_extension%0Aversion:%2014.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
+If you spotted it first, help us to smash it by providing a detailed and welcomed
+`feedback <https://github.com/akretion/odoo-usability/issues/new?body=module:%20base_company_extension%0Aversion:%2018.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
 
 Do not contact contributors directly about support or help with technical issues.
 
@@ -52,6 +59,6 @@ Contributors
 Maintainers
 ~~~~~~~~~~~
 
-This module is part of the `akretion/odoo-usability <https://github.com/akretion/odoo-usability/tree/12.0/base_company_extension>`_ project on GitHub.
+This module is part of the `akretion/odoo-usability <https://github.com/akretion/odoo-usability/tree/18.0/base_company_extension>`_ project on GitHub.
 
 You are welcome to contribute.

base_company_extension/__manifest__.py

@@ -4,14 +4,14 @@
 
 {
     'name': 'Base Company Extension',
-    'version': '16.0.1.0.0',
+    'version': '18.0.1.0.0',
     'category': 'Partner',
     'license': 'AGPL-3',
     'summary': 'Adds capital and title on company',
     'author': 'Akretion',
     'website': 'https://github.com/akretion/odoo-usability',
     # I depend on base_usability only for _report_company_legal_name()
-    'depends': ['base_usability'],
+    'depends': ['base_usability_akretion'],
     'data': ['views/res_company.xml'],
     'installable': True,
 }

base_company_extension/pyproject.toml

@@ -0,0 +1,3 @@
+[build-system]
+requires = ["whool"]
+build-backend = "whool.buildapi"

base_company_extension/static/description/index.html

@@ -1,20 +1,20 @@
-<?xml version="1.0" encoding="utf-8" ?>
 <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
 <html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">
 <head>
 <meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
-<meta name="generator" content="Docutils 0.15.1: http://docutils.sourceforge.net/" />
+<meta name="generator" content="Docutils: https://docutils.sourceforge.io/" />
 <title>Base Company Extension</title>
 <style type="text/css">
 
 /*
 :Author: David Goodger (goodger@python.org)
-:Id: $Id: html4css1.css 7952 2016-07-26 18:15:59Z milde $
+:Id: $Id: html4css1.css 9511 2024-01-13 09:50:07Z milde $
 :Copyright: This stylesheet has been placed in the public domain.
 
 Default cascading style sheet for the HTML output of Docutils.
+Despite the name, some widely supported CSS2 features are used.
 
-See http://docutils.sf.net/docs/howto/html-stylesheets.html for how to
+See https://docutils.sourceforge.io/docs/howto/html-stylesheets.html for how to
 customize this style sheet.
 */
 
@@ -275,7 +275,7 @@ pre.literal-block, pre.doctest-block, pre.math, pre.code {
   margin-left: 2em ;
   margin-right: 2em }
 
-pre.code .ln { color: grey; } /* line numbers */
+pre.code .ln { color: gray; } /* line numbers */
 pre.code, code { background-color: #eeeeee }
 pre.code .comment, code .comment { color: #5C6576 }
 pre.code .keyword, code .keyword { color: #3B0D06; font-weight: bold }
@@ -301,7 +301,7 @@ span.option {
 span.pre {
   white-space: pre }
 
-span.problematic {
+span.problematic, pre.problematic {
   color: red }
 
 span.section-subtitle {
@@ -366,8 +366,10 @@ ul.auto-toc {
 <!-- !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
 !! This file is generated by oca-gen-addon-readme !!
 !! changes will be overwritten.                   !!
+!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+!! source digest: sha256:075f72950033a5c6f57ecfa5c2d101bd874dd2ae29adcecdc4905b378e89a3a2
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
-<p><a class="reference external" href="https://odoo-community.org/page/development-status"><img alt="Beta" src="https://img.shields.io/badge/maturity-Beta-yellow.png" /></a> <a class="reference external" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/licence-AGPL--3-blue.png" /></a> <a class="reference external" href="https://github.com/akretion/odoo-usability/tree/12.0/base_company_extension"><img alt="akretion/odoo-usability" src="https://img.shields.io/badge/github-akretion%2Fodoo--usability-lightgray.png?logo=github" /></a></p>
+<p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Beta" src="https://img.shields.io/badge/maturity-Beta-yellow.png" /></a> <a class="reference external image-reference" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/licence-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/akretion/odoo-usability/tree/18.0/base_company_extension"><img alt="akretion/odoo-usability" src="https://img.shields.io/badge/github-akretion%2Fodoo--usability-lightgray.png?logo=github" /></a></p>
 <p>This module adds the following fields to the ResCompany model:
 * Capital Amount
 * Legal Type</p>
@@ -375,40 +377,40 @@ ul.auto-toc {
 <p><strong>Table of contents</strong></p>
 <div class="contents local topic" id="contents">
 <ul class="simple">
-<li><a class="reference internal" href="#bug-tracker" id="id1">Bug Tracker</a></li>
-<li><a class="reference internal" href="#credits" id="id2">Credits</a><ul>
-<li><a class="reference internal" href="#authors" id="id3">Authors</a></li>
-<li><a class="reference internal" href="#contributors" id="id4">Contributors</a></li>
-<li><a class="reference internal" href="#maintainers" id="id5">Maintainers</a></li>
+<li><a class="reference internal" href="#bug-tracker" id="toc-entry-1">Bug Tracker</a></li>
+<li><a class="reference internal" href="#credits" id="toc-entry-2">Credits</a><ul>
+<li><a class="reference internal" href="#authors" id="toc-entry-3">Authors</a></li>
+<li><a class="reference internal" href="#contributors" id="toc-entry-4">Contributors</a></li>
+<li><a class="reference internal" href="#maintainers" id="toc-entry-5">Maintainers</a></li>
 </ul>
 </li>
 </ul>
 </div>
 <div class="section" id="bug-tracker">
-<h1><a class="toc-backref" href="#id1">Bug Tracker</a></h1>
+<h1><a class="toc-backref" href="#toc-entry-1">Bug Tracker</a></h1>
 <p>Bugs are tracked on <a class="reference external" href="https://github.com/akretion/odoo-usability/issues">GitHub Issues</a>.
 In case of trouble, please check there if your issue has already been reported.
-If you spotted it first, help us smashing it by providing a detailed and welcomed
-<a class="reference external" href="https://github.com/akretion/odoo-usability/issues/new?body=module:%20base_company_extension%0Aversion:%2012.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**">feedback</a>.</p>
+If you spotted it first, help us to smash it by providing a detailed and welcomed
+<a class="reference external" href="https://github.com/akretion/odoo-usability/issues/new?body=module:%20base_company_extension%0Aversion:%2018.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**">feedback</a>.</p>
 <p>Do not contact contributors directly about support or help with technical issues.</p>
 </div>
 <div class="section" id="credits">
-<h1><a class="toc-backref" href="#id2">Credits</a></h1>
+<h1><a class="toc-backref" href="#toc-entry-2">Credits</a></h1>
 <div class="section" id="authors">
-<h2><a class="toc-backref" href="#id3">Authors</a></h2>
+<h2><a class="toc-backref" href="#toc-entry-3">Authors</a></h2>
 <ul class="simple">
 <li>Akretion</li>
 </ul>
 </div>
 <div class="section" id="contributors">
-<h2><a class="toc-backref" href="#id4">Contributors</a></h2>
+<h2><a class="toc-backref" href="#toc-entry-4">Contributors</a></h2>
 <ul class="simple">
 <li>Alexis de Lattre &lt;<a class="reference external" href="mailto:alexis.delattre&#64;akretion.com">alexis.delattre&#64;akretion.com</a>&gt;</li>
 </ul>
 </div>
 <div class="section" id="maintainers">
-<h2><a class="toc-backref" href="#id5">Maintainers</a></h2>
-<p>This module is part of the <a class="reference external" href="https://github.com/akretion/odoo-usability/tree/12.0/base_company_extension">akretion/odoo-usability</a> project on GitHub.</p>
+<h2><a class="toc-backref" href="#toc-entry-5">Maintainers</a></h2>
+<p>This module is part of the <a class="reference external" href="https://github.com/akretion/odoo-usability/tree/18.0/base_company_extension">akretion/odoo-usability</a> project on GitHub.</p>
 <p>You are welcome to contribute.</p>
 </div>
 </div>

base_dynamic_list/__manifest__.py

@@ -27,13 +27,13 @@ To use it, you need to do 2 or 3 things :
 
 1) Add an entry in the domain field and the object you selected:
 
-domain = fields.Selection(selection_add=[('risk.type', "Risk Type")], ondelete={"risk.type": "cascade"})
+domain = fields.Selection(selection_add=[('risk_type', "Risk Type")], ondelete={"risk_type": "cascade"})
 
 2) Add the many2one field on your object:
 
 risk_type_id = fields.Many2one(
     'dynamic.list', string="Risk Type",
-    ondelete='restrict', domain=[('domain', '=', 'risk.type')])
+    ondelete='restrict', domain=[('domain', '=', 'risk_type')])
 
 
 3) Optionally, you can add a dedicated action and a menu entry (otherwize, you can use the generic menu entry under *Settings > Technical > Dynamic Lists*:
@@ -42,8 +42,8 @@ risk_type_id = fields.Many2one(
     <field name="name">Risk Type</field>
     <field name="res_model">dynamic.list</field>
     <field name="view_mode">tree,form</field>
-    <field name="domain">[('domain', '=', 'risk.type')]</field>
-    <field name="context">{'default_domain': 'risk.type'}</field>
+    <field name="domain">[('domain', '=', 'risk_type')]</field>
+    <field name="context">{'default_domain': 'risk_type'}</field>
 </record>
 
 <menuitem id="dynamic_list_risk_type_menu" action="dynamic_list_risk_type_action"
@@ -58,5 +58,5 @@ Limitation: when you want to have different access rights on these lists dependi
         'security/ir.model.access.csv',
         'views/dynamic_list.xml',
         ],
-    'installable': True,
+    'installable': False,
 }

base_mail_sender_bcc/__manifest__.py

@@ -4,7 +4,7 @@
 
 {
     'name': 'Mail Sender Bcc',
-    'version': '16.0.1.0.0',
+    'version': '18.0.1.0.0',
     'category': 'Mail',
     'license': 'AGPL-3',
     'summary': "Always send a copy of the mail to the sender",

base_mail_sender_bcc/pyproject.toml

@@ -0,0 +1,3 @@
+[build-system]
+requires = ["whool"]
+build-backend = "whool.buildapi"

base_partner_one2many_phone/__manifest__.py

@@ -27,6 +27,6 @@ It has been developped by brother Bernard from Barroux Abbey and Alexis de Lattr
         'views/res_partner.xml',
         'security/ir.model.access.csv',
         ],
-    'installable': True,
+    'installable': False,
     'post_init_hook': 'migrate_to_partner_phone',
 }

base_partner_ref/__manifest__.py

@@ -1,10 +1,10 @@
-# Copyright 2017-2023 Akretion France (https://www.akretion.com)
+# Copyright 2017-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).
 
 {
     'name': 'Base Partner Reference',
-    'version': '16.0.1.0.0',
+    'version': '18.0.1.0.0',
     'category': 'Partner',
     'license': 'AGPL-3',
     'summary': "Improve usage of partner's Internal Reference",

base_partner_ref/models/res_partner.py

@@ -23,51 +23,24 @@ class ResPartner(models.Model):
     def _compute_display_name(self):
         super()._compute_display_name()
 
-    def _get_name(self):
-        partner = self
-        name = partner.name or ''
-
+    def _get_complete_name(self):
+        self.ensure_one() 
+        displayed_types = self._complete_name_displayed_types
+        type_description = dict(self._fields['type']._description_selection(self.env))
+        name = self.name or ''
         # START modif of native method
-        if partner.ref:
-            name = "[%s] %s" % (partner.ref, name)
+        if not self._context.get('show_address') and self.ref:
+            name = "[%s] %s" % (self.ref, name)
         # END modif of native method
-        if partner.company_name or partner.parent_id:
-            if not name and partner.type in ['invoice', 'delivery', 'other']:
-                name = dict(self.fields_get(
-                    ['type'])['type']['selection'])[partner.type]
-            if not partner.is_company:
-                # START modif of native name_get() method
-                company_name = partner.commercial_company_name or\
-                    partner.sudo().parent_id.name
-                if partner.parent_id.ref:
-                    company_name = "[%s] %s" % (partner.parent_id.ref, company_name)
-                name = "%s, %s" % (company_name, name)
-                # END modif of native name_get() method
-        if self._context.get('show_address_only'):
-            name = partner._display_address(without_company=True)
-        if self._context.get('show_address'):
-            name = name + "\n" + partner._display_address(without_company=True)
-        name = re.sub(r'\s+\n', '\n', name)
-        if self._context.get('partner_show_db_id'):
-            name = "%s (%s)" % (name, partner.id)
-        if self._context.get('address_inline'):
-            splitted_names = name.split("\n")
-            name = ", ".join([n for n in splitted_names if n.strip()])
-        if self._context.get('show_email') and partner.email:
-            name = "%s <%s>" % (name, partner.email)
-        if self._context.get('html_format'):
-            name = name.replace('\n', '<br/>')
-        if self._context.get('show_vat') and partner.vat:
-            name = "%s ‒ %s" % (name, partner.vat)
+        if self.company_name or self.parent_id:
+            if not name and self.type in displayed_types:
+                name = type_description[self.type]
+            if not self.is_company:
+                name = f"{self.commercial_company_name or self.sudo().parent_id.name}, {name}"
+                # START modif of native method
+                if not self._context.get('show_address') and self.parent_id.ref:
+                    name = f"[{self.parent_id.ref}] {name}"
+                # END modif of native method
         return name.strip()
 
-    @api.model
-    def name_search(self, name='', args=None, operator='ilike', limit=100):
-        if args is None:
-            args = []
-        if name and operator == 'ilike':
-            recs = self.search([('ref', '=', name)] + args, limit=limit)
-            if recs:
-                rec_childs = self.search([('id', 'child_of', recs.ids)])
-                return rec_childs.name_get()
-        return super().name_search(name=name, args=args, operator=operator, limit=limit)
+    # native _rec_names_search contains "ref", so no need to inherit name_search()

base_partner_ref/post_install.py

@@ -2,11 +2,8 @@
 # @author: Alexis de Lattre <alexis.delattre@akretion.com>
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 
-from odoo import api, SUPERUSER_ID
 
-
-def update_partner_display_name(cr, registry):
-    env = api.Environment(cr, SUPERUSER_ID, {})
+def update_partner_display_name(env):
     partners = env['res.partner'].with_context(active_test=False).search(
         [('ref', '!=', False)])
     partners.write({'invalidate_display_name': True})

base_partner_ref/pyproject.toml

@@ -0,0 +1,3 @@
+[build-system]
+requires = ["whool"]
+build-backend = "whool.buildapi"

base_partner_ref/views/res_partner.xml

@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="utf-8"?>
 <!--
-  Copyright 2017-2023 Akretion France (http://www.akretion.com/)
+  Copyright 2017-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).
 -->
@@ -13,9 +13,9 @@
     <field name="inherit_id" ref="base.view_partner_form"/>
     <field name="priority">1000</field>  <!-- inherit after l10n_fr -->
     <field name="arch" type="xml">
-        <field name="vat" position="before">
+        <div class="o_address_format" position="after">
             <field name="ref"/>
-        </field>
+        </div>
         <xpath expr="//page[@name='sales_purchases']//field[@name='ref']" position="attributes">
             <attribute name="invisible">1</attribute>
         </xpath>

base_profile_akretion/__manifest__.py

@@ -0,0 +1,43 @@
+# Copyright 2025 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).
+
+{
+    'name': 'Base Profile by Akretion',
+    'version': '18.0.1.0.0',
+    'category': 'Tools',
+    'license': 'AGPL-3',
+    'summary': 'Base module set selected by Alexis de Lattre',
+    'author': 'Akretion',
+    'website': 'https://github.com/akretion/odoo-usability',
+    'depends': [
+        # PARTNER
+        'partner_firstname',  # OCA/partner-contact
+        'partner_email_duplicate_warn',  # OCA/partner-contact
+        'partner_mobile_duplicate_warn',  # OCA/partner-contact
+        'contacts',  # official addons
+        # AUTH
+        'auth_admin_passkey',  # OCA/server-auth
+        # REMOVE or FIX BAD NATIVE STUFF
+        'disable_odoo_online',  # OCA/server-brand
+        'remove_odoo_enterprise',  # OCA/server-brand
+        'mail_debrand',   # OCA/mail
+        'partner_disable_gravatar',  # OCA/partner-contact
+        'base_technical_features',  # OCA/server-ux
+        ### WEB
+        'web_responsive',  # OCA/web
+        'web_environment_ribbon',  # OCA/web
+        'web_no_bubble',  # OCA/web
+        'web_dialog_size',  # OCA/web
+        'web_chatter_position',  # OCA/web
+         ### MISC
+        'base_usability_akretion',  # akretion/odoo-usability
+        'mail_usability_akretion',  # akretion/odoo-usability
+        'eradicate_quick_create',  # akretion/odoo-usability
+        'base_company_extension',  # akretion/odoo-usability
+        # password_security will be enabled when the move to ir.config_parameter
+        # will be backported
+        #'password_security',  # OCA/server-auth
+        ],
+    'installable': True,
+}

base_profile_akretion/pyproject.toml

@@ -0,0 +1,3 @@
+[build-system]
+requires = ["whool"]
+build-backend = "whool.buildapi"

base_usability/models/ir_model_fields.py

@@ -1,16 +0,0 @@
-# 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).
-
-from odoo import api, models
-
-
-class IrModelFields(models.Model):
-    _inherit = 'ir.model.fields'
-
-    @api.depends('name', 'field_description')
-    def name_get(self):
-        res = []
-        for rec in self:
-            res.append((rec.id, '%s (%s)' % (rec.field_description, rec.name)))
-        return res

base_usability/models/res_partner_bank.py

@@ -1,14 +0,0 @@
-# Copyright 2015-2022 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 ResPartnerBank(models.Model):
-    _inherit = 'res.partner.bank'
-
-    # In the 'base' module, they didn't put any string, so the bank name is
-    # displayed as 'Name', which the string of the related field it
-    # points to
-    bank_name = fields.Char(string='Bank Name')

base_usability/views/ir_property.xml

@@ -1,40 +0,0 @@
-<?xml version="1.0" encoding="utf-8"?>
-<!--
-  Copyright 2022 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="ir_property_view_tree" model="ir.ui.view">
-    <field name="model">ir.property</field>
-    <field name="inherit_id" ref="base.ir_property_view_tree"/>
-    <field name="arch" type="xml">
-        <field name="type" position="after">
-            <field name="value_reference" optional="show"/>
-            <field name="value_integer" optional="hide"/>
-            <field name="value_float" optional="hide"/>
-            <field name="value_datetime" optional="hide"/>
-            <field name="value_text" optional="hide"/>
-        </field>
-        <field name="type" position="attributes">
-            <attribute name="optional">hide</attribute>
-        </field>
-    </field>
-</record>
-
-<record id="ir_property_view_search" model="ir.ui.view">
-    <field name="model">ir.property</field>
-    <field name="inherit_id" ref="base.ir_property_view_search"/>
-    <field name="arch" type="xml">
-        <search position="inside">
-            <group name="groupby">
-                <filter name="field_groupby" string="Field" context="{'group_by': 'fields_id'}"/>
-                <filter name="company_groupby" string="Company" context="{'group_by': 'company_id'}"/>
-            </group>
-        </search>
-    </field>
-</record>
-
-</odoo>

base_usability/web-advanced_search_startswith.diff

@@ -1,24 +0,0 @@
-diff --git a/addons/web/static/src/search/filter_menu/custom_filter_item.js b/addons/web/static/src/search/filter_menu/custom_filter_item.js
-index f67f5fb40af..22525b7cbfd 100644
---- a/addons/web/static/src/search/filter_menu/custom_filter_item.js
-+++ b/addons/web/static/src/search/filter_menu/custom_filter_item.js
-@@ -46,6 +46,8 @@ const FIELD_OPERATORS = {
-     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 },
-@@ -257,6 +259,10 @@ export class CustomFilterItem extends Component {
-                     [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]]);
-             }

base_usability_akretion/README.rst

@@ -2,10 +2,13 @@
 Base Usability
 ==============
 
-.. !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+.. 
+   !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
    !! This file is generated by oca-gen-addon-readme !!
    !! changes will be overwritten.                   !!
    !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+   !! source digest: sha256:6998b819c2a5d0be20947d00a368d24aa2a1e1f2655e6463e450f2d7ad9acfcd
+   !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
 
 .. |badge1| image:: https://img.shields.io/badge/maturity-Beta-yellow.png
     :target: https://odoo-community.org/page/development-status
@@ -13,17 +16,11 @@ Base Usability
 .. |badge2| image:: https://img.shields.io/badge/licence-AGPL--3-blue.png
     :target: http://www.gnu.org/licenses/agpl-3.0-standalone.html
     :alt: License: AGPL-3
-.. |badge3| image:: https://img.shields.io/badge/github-OCA%2Fodoo--usability-lightgray.png?logo=github
-    :target: https://github.com/OCA/odoo-usability/tree/16.0/base_usability
-    :alt: OCA/odoo-usability
-.. |badge4| image:: https://img.shields.io/badge/weblate-Translate%20me-F47D42.png
-    :target: https://translation.odoo-community.org/projects/odoo-usability-16-0/odoo-usability-16-0-base_usability
-    :alt: Translate me on Weblate
-.. |badge5| image:: https://img.shields.io/badge/runboat-Try%20me-875A7B.png
-    :target: https://runboat.odoo-community.org/webui/builds.html?repo=OCA/odoo-usability&target_branch=16.0
-    :alt: Try me on Runboat
+.. |badge3| image:: https://img.shields.io/badge/github-akretion%2Fodoo--usability-lightgray.png?logo=github
+    :target: https://github.com/akretion/odoo-usability/tree/18.0/base_usability_akretion
+    :alt: akretion/odoo-usability
 
-|badge1| |badge2| |badge3| |badge4| |badge5| 
+|badge1| |badge2| |badge3|
 
 This module adds the following functions:
 
@@ -60,10 +57,10 @@ This module adds the following functions:
 Bug Tracker
 ===========
 
-Bugs are tracked on `GitHub Issues <https://github.com/OCA/odoo-usability/issues>`_.
+Bugs are tracked on `GitHub Issues <https://github.com/akretion/odoo-usability/issues>`_.
 In case of trouble, please check there if your issue has already been reported.
-If you spotted it first, help us smashing it by providing a detailed and welcomed
-`feedback <https://github.com/OCA/odoo-usability/issues/new?body=module:%20base_usability%0Aversion:%2016.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
+If you spotted it first, help us to smash it by providing a detailed and welcomed
+`feedback <https://github.com/akretion/odoo-usability/issues/new?body=module:%20base_usability_akretion%0Aversion:%2018.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
 
 Do not contact contributors directly about support or help with technical issues.
 
@@ -85,16 +82,6 @@ Contributors
 Maintainers
 ~~~~~~~~~~~
 
-This module is maintained by the OCA.
+This module is part of the `akretion/odoo-usability <https://github.com/akretion/odoo-usability/tree/18.0/base_usability_akretion>`_ project on GitHub.
 
-.. image:: https://odoo-community.org/logo.png
-   :alt: Odoo Community Association
-   :target: https://odoo-community.org
-
-OCA, or the Odoo Community Association, is a nonprofit organization whose
-mission is to support the collaborative development of Odoo features and
-promote its widespread use.
-
-This module is part of the `OCA/odoo-usability <https://github.com/OCA/odoo-usability/tree/16.0/base_usability>`_ project on GitHub.
-
-You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute.
+You are welcome to contribute.

base_usability_akretion/__manifest__.py

@@ -4,7 +4,7 @@
 
 {
     'name': 'Base Usability',
-    'version': '16.0.1.0.0',
+    'version': '18.0.1.0.0',
     'category': 'Partner',
     'license': 'AGPL-3',
     'summary': 'Better usability in base module',
@@ -21,7 +21,6 @@
         'views/ir_config_parameter.xml',
         'views/ir_module.xml',
         'views/ir_sequence.xml',
-        'views/ir_property.xml',
         ],
     'installable': True,
 }

base_usability_akretion/models/__init__.py

@@ -4,4 +4,3 @@ from . import res_partner_bank
 from . import res_company
 from . import ir_mail_server
 from . import ir_model
-from . import ir_model_fields

base_usability_akretion/models/ir_mail_server.py

@@ -1,4 +1,4 @@
-# Copyright 2015-2022 Akretion France (http://www.akretion.com/)
+# Copyright 2015-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).
 
@@ -18,8 +18,15 @@ class IrMailServer(models.Model):
             smtp_ssl_certificate=None, smtp_ssl_private_key=None,
             smtp_debug=False, smtp_session=None):
         # Start copy from native method
+        smtp = smtp_session
+        if not smtp:
+            smtp = self.connect(
+                smtp_server, smtp_port, smtp_user, smtp_password, smtp_encryption,
+                smtp_from=message['From'], ssl_certificate=smtp_ssl_certificate, ssl_private_key=smtp_ssl_private_key,
+                smtp_debug=smtp_debug, mail_server_id=mail_server_id,)
+
         smtp_from, smtp_to_list, message = self._prepare_email_message(
-            message, smtp_session)
+            message, smtp)
         # End copy from native method
         logger.info(
             "Sending email from '%s' to '%s' Cc '%s' Bcc '%s' "

base_usability_akretion/models/ir_model.py

@@ -1,4 +1,4 @@
-# Copyright 2019-2022 Akretion France (http://www.akretion.com/)
+# 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).
 
@@ -9,8 +9,6 @@ class IrModel(models.Model):
     _inherit = 'ir.model'
 
     @api.depends('name', 'model')
-    def name_get(self):
-        res = []
+    def _compute_display_name(self):
         for rec in self:
-            res.append((rec.id, '%s (%s)' % (rec.name, rec.model)))
-        return res
+            rec.display_name = f'{rec.name} ({rec.model})'

base_usability_akretion/models/res_company.py

@@ -1,4 +1,4 @@
-# Copyright 2015-2022 Akretion France (http://www.akretion.com/)
+# Copyright 2015-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).
 

base_usability_akretion/models/res_partner.py

@@ -1,4 +1,4 @@
-# Copyright 2015-2022 Akretion France (http://www.akretion.com/)
+# Copyright 2015-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).
 
@@ -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_akretion/models/res_partner_bank.py

@@ -0,0 +1,24 @@
+# Copyright 2015-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 api, fields, models
+
+
+class ResPartnerBank(models.Model):
+    _inherit = 'res.partner.bank'
+
+    # In the 'base' module, they didn't put any string, so the bank name is
+    # displayed as 'Name', which the string of the related field it
+    # points to
+    bank_name = fields.Char(string='Bank Name')
+
+    @api.depends('currency_id')
+    def _compute_display_name(self):
+        for acc in self:
+            name = acc.acc_number
+            if acc.currency_id:
+                 name = f"{name} ({acc.currency_id.name})"
+            if acc.bank_id:
+                 name = f"{name} - {acc.bank_id.name}"
+            acc.display_name = name

base_usability_akretion/models/res_users.py

@@ -2,7 +2,7 @@
 # @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, SUPERUSER_ID
+from odoo import api, fields, models
 import logging
 from odoo.tools.misc import format_datetime
 
@@ -14,11 +14,9 @@ class ResUsers(models.Model):
 
     @api.model
     def _script_partners_linked_to_users_no_company(self):
-        if self.env.user.id != SUPERUSER_ID:
-            self = self.sudo()
         logger.info(
             'START to set company_id=False on partners related to users')
-        users = self.with_context(active_test=False).search([])
+        users = self.sudo().with_context(active_test=False).search([])
         for user in users:
             if user.partner_id.company_id:
                 user.partner_id.write({'company_id': False})

base_usability_akretion/pyproject.toml

@@ -0,0 +1,3 @@
+[build-system]
+requires = ["whool"]
+build-backend = "whool.buildapi"

base_usability_akretion/security/group.xml

@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="utf-8"?>
 <!--
-  Copyright 2016-2022 Akretion France (http://www.akretion.com/)
+  Copyright 2016-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).
 -->

base_usability_akretion/static/description/index.html

@@ -1,20 +1,20 @@
-<?xml version="1.0" encoding="utf-8" ?>
 <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
 <html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">
 <head>
 <meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
-<meta name="generator" content="Docutils: http://docutils.sourceforge.net/" />
+<meta name="generator" content="Docutils: https://docutils.sourceforge.io/" />
 <title>Base Usability</title>
 <style type="text/css">
 
 /*
 :Author: David Goodger (goodger@python.org)
-:Id: $Id: html4css1.css 7952 2016-07-26 18:15:59Z milde $
+:Id: $Id: html4css1.css 9511 2024-01-13 09:50:07Z milde $
 :Copyright: This stylesheet has been placed in the public domain.
 
 Default cascading style sheet for the HTML output of Docutils.
+Despite the name, some widely supported CSS2 features are used.
 
-See http://docutils.sf.net/docs/howto/html-stylesheets.html for how to
+See https://docutils.sourceforge.io/docs/howto/html-stylesheets.html for how to
 customize this style sheet.
 */
 
@@ -275,7 +275,7 @@ pre.literal-block, pre.doctest-block, pre.math, pre.code {
   margin-left: 2em ;
   margin-right: 2em }
 
-pre.code .ln { color: grey; } /* line numbers */
+pre.code .ln { color: gray; } /* line numbers */
 pre.code, code { background-color: #eeeeee }
 pre.code .comment, code .comment { color: #5C6576 }
 pre.code .keyword, code .keyword { color: #3B0D06; font-weight: bold }
@@ -301,7 +301,7 @@ span.option {
 span.pre {
   white-space: pre }
 
-span.problematic {
+span.problematic, pre.problematic {
   color: red }
 
 span.section-subtitle {
@@ -366,8 +366,10 @@ ul.auto-toc {
 <!-- !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
 !! This file is generated by oca-gen-addon-readme !!
 !! changes will be overwritten.                   !!
+!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+!! source digest: sha256:6998b819c2a5d0be20947d00a368d24aa2a1e1f2655e6463e450f2d7ad9acfcd
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
-<p><a class="reference external" href="https://odoo-community.org/page/development-status"><img alt="Beta" src="https://img.shields.io/badge/maturity-Beta-yellow.png" /></a> <a class="reference external" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/licence-AGPL--3-blue.png" /></a> <a class="reference external" href="https://github.com/OCA/odoo-usability/tree/16.0/base_usability"><img alt="OCA/odoo-usability" src="https://img.shields.io/badge/github-OCA%2Fodoo--usability-lightgray.png?logo=github" /></a> <a class="reference external" href="https://translation.odoo-community.org/projects/odoo-usability-16-0/odoo-usability-16-0-base_usability"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external" href="https://runboat.odoo-community.org/webui/builds.html?repo=OCA/odoo-usability&amp;target_branch=16.0"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
+<p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Beta" src="https://img.shields.io/badge/maturity-Beta-yellow.png" /></a> <a class="reference external image-reference" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/licence-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/akretion/odoo-usability/tree/18.0/base_usability_akretion"><img alt="akretion/odoo-usability" src="https://img.shields.io/badge/github-akretion%2Fodoo--usability-lightgray.png?logo=github" /></a></p>
 <p>This module adds the following functions:</p>
 <ul class="simple">
 <li>Adds <em>track_visibility=’onchange’</em> on all the important fields of the Partner object</li>
@@ -398,33 +400,33 @@ ul.auto-toc {
 <p><strong>Table of contents</strong></p>
 <div class="contents local topic" id="contents">
 <ul class="simple">
-<li><a class="reference internal" href="#bug-tracker" id="id1">Bug Tracker</a></li>
-<li><a class="reference internal" href="#credits" id="id2">Credits</a><ul>
-<li><a class="reference internal" href="#authors" id="id3">Authors</a></li>
-<li><a class="reference internal" href="#contributors" id="id4">Contributors</a></li>
-<li><a class="reference internal" href="#maintainers" id="id5">Maintainers</a></li>
+<li><a class="reference internal" href="#bug-tracker" id="toc-entry-1">Bug Tracker</a></li>
+<li><a class="reference internal" href="#credits" id="toc-entry-2">Credits</a><ul>
+<li><a class="reference internal" href="#authors" id="toc-entry-3">Authors</a></li>
+<li><a class="reference internal" href="#contributors" id="toc-entry-4">Contributors</a></li>
+<li><a class="reference internal" href="#maintainers" id="toc-entry-5">Maintainers</a></li>
 </ul>
 </li>
 </ul>
 </div>
 <div class="section" id="bug-tracker">
-<h1><a class="toc-backref" href="#id1">Bug Tracker</a></h1>
-<p>Bugs are tracked on <a class="reference external" href="https://github.com/OCA/odoo-usability/issues">GitHub Issues</a>.
+<h1><a class="toc-backref" href="#toc-entry-1">Bug Tracker</a></h1>
+<p>Bugs are tracked on <a class="reference external" href="https://github.com/akretion/odoo-usability/issues">GitHub Issues</a>.
 In case of trouble, please check there if your issue has already been reported.
-If you spotted it first, help us smashing it by providing a detailed and welcomed
-<a class="reference external" href="https://github.com/OCA/odoo-usability/issues/new?body=module:%20base_usability%0Aversion:%2016.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**">feedback</a>.</p>
+If you spotted it first, help us to smash it by providing a detailed and welcomed
+<a class="reference external" href="https://github.com/akretion/odoo-usability/issues/new?body=module:%20base_usability_akretion%0Aversion:%2018.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**">feedback</a>.</p>
 <p>Do not contact contributors directly about support or help with technical issues.</p>
 </div>
 <div class="section" id="credits">
-<h1><a class="toc-backref" href="#id2">Credits</a></h1>
+<h1><a class="toc-backref" href="#toc-entry-2">Credits</a></h1>
 <div class="section" id="authors">
-<h2><a class="toc-backref" href="#id3">Authors</a></h2>
+<h2><a class="toc-backref" href="#toc-entry-3">Authors</a></h2>
 <ul class="simple">
 <li>Akretion</li>
 </ul>
 </div>
 <div class="section" id="contributors">
-<h2><a class="toc-backref" href="#id4">Contributors</a></h2>
+<h2><a class="toc-backref" href="#toc-entry-4">Contributors</a></h2>
 <ul class="simple">
 <li>Alexis de Lattre &lt;<a class="reference external" href="mailto:alexis.delattre&#64;akretion.com">alexis.delattre&#64;akretion.com</a>&gt;</li>
 <li>Raphaël Valyi &lt;<a class="reference external" href="mailto:rvalyi&#64;akretion.com">rvalyi&#64;akretion.com</a>&gt;</li>
@@ -432,14 +434,9 @@ If you spotted it first, help us smashing it by providing a detailed and welcome
 </ul>
 </div>
 <div class="section" id="maintainers">
-<h2><a class="toc-backref" href="#id5">Maintainers</a></h2>
-<p>This module is maintained by the OCA.</p>
-<a class="reference external image-reference" href="https://odoo-community.org"><img alt="Odoo Community Association" src="https://odoo-community.org/logo.png" /></a>
-<p>OCA, or the Odoo Community Association, is a nonprofit organization whose
-mission is to support the collaborative development of Odoo features and
-promote its widespread use.</p>
-<p>This module is part of the <a class="reference external" href="https://github.com/OCA/odoo-usability/tree/16.0/base_usability">OCA/odoo-usability</a> project on GitHub.</p>
-<p>You are welcome to contribute. To learn how please visit <a class="reference external" href="https://odoo-community.org/page/Contribute">https://odoo-community.org/page/Contribute</a>.</p>
+<h2><a class="toc-backref" href="#toc-entry-5">Maintainers</a></h2>
+<p>This module is part of the <a class="reference external" href="https://github.com/akretion/odoo-usability/tree/18.0/base_usability_akretion">akretion/odoo-usability</a> project on GitHub.</p>
+<p>You are welcome to contribute.</p>
 </div>
 </div>
 </div>

base_usability_akretion/views/ir_config_parameter.xml

@@ -6,9 +6,9 @@
     <field name="model">ir.config_parameter</field>
     <field name="inherit_id" ref="base.view_ir_config_list"/>
     <field name="arch" type="xml">
-        <xpath expr="//tree/field[last()]" position="after">
-            <field name="write_date"/>
-        </xpath>
+        <list position="inside">
+            <field name="write_date" optional="show"/>
+        </list>
     </field>
 </record>