[IMP] base, account, l10n_ch, website_sale, payment, web: make payment QR-code generation more generic

This new modelling makes it easier to add new QR-code formats, and allows using all of them in website_sale and account.payment's form view as well (so, Swiss QR codes are now available there, while they were restricted to only invoices in the past). All barcodes are now generated as reports, from a dedicated route. This was only partly the case before : Swiss QR added a cross on top of the QR-code directly in the template, it wasn't part of the image returned by the route; now it is.

[ADD] base_qr_code_sepa: new module decoupling SEPA QR-codes generation from the base module

Each new QR-code generation option should thus be done in a dedicated module (or added to a localization) in the future.

[IMP] base_qr_code_sepa: update the generated QR codes to version 2 of the specification

Version 1 is still supported, so no need to backport this.

[IMP] l10n_ch: make Swiss QR-codes compatible with the new version of the specification (the old one is deprecated)

This will be backported to 11.0 and 12.0, as these QR-codes will soon replace ISR.

[IMP] account: make it possible to mark manual payments as sent with a button on the form view

This way, when making them directly with a QR-code (or doing a more classical wire transfer), people can keep track of what they already have asked the bank to do, and what they still have to treat.

closes odoo/odoo#44839

Related: odoo/enterprise#8262
Related: odoo/upgrade#992
Signed-off-by: Laurent Smet <smetl@users.noreply.github.com>

2 files changed, 65 insertions, 0 deletions

diff --git a/addons/account_qr_code_sepa/tests/__init__.py b/addons/account_qr_code_sepa/tests/__init__.py
new file mode 100644
index 00000000000..b570597d7f4
--- /dev/null
+++ b/addons/account_qr_code_sepa/tests/__init__.py
@@ -0,0 +1,3 @@
+# *-* coding:utf-8 *-*
+
+from . import test_sepa_qr
\ No newline at end of file
diff --git a/addons/account_qr_code_sepa/tests/test_sepa_qr.py b/addons/account_qr_code_sepa/tests/test_sepa_qr.py
new file mode 100644
index 00000000000..27575632bc3
--- /dev/null
+++ b/addons/account_qr_code_sepa/tests/test_sepa_qr.py
@@ -0,0 +1,62 @@
+# -*- coding:utf-8 -*-
+
+from odoo.exceptions import UserError
+from odoo.addons.account.tests.common import AccountTestInvoicingCommon
+
+class TestSEPAQRCode(AccountTestInvoicingCommon):
+    """ Tests the generation of Swiss QR-codes on invoices
+    """
+
+    @classmethod
+    def setUpClass(cls):
+        super(TestSEPAQRCode, cls).setUpClass()
+
+        cls.company_data['company'].qr_code = True
+        cls.acc_sepa_iban = cls.env['res.partner.bank'].create({
+            'acc_number': 'BE15001559627230',
+            'partner_id': cls.company_data['company'].partner_id.id,
+        })
+
+        cls.acc_non_sepa_iban = cls.env['res.partner.bank'].create({
+            'acc_number': 'SA4420000001234567891234',
+            'partner_id': cls.company_data['company'].partner_id.id,
+        })
+
+        cls.sepa_qr_invoice = cls.env['account.move'].create({
+            'move_type': 'out_invoice',
+            'partner_id': cls.partner_a.id,
+            'currency_id': cls.env.ref('base.EUR').id,
+            'invoice_partner_bank_id': cls.acc_sepa_iban.id,
+            'company_id': cls.company_data['company'].id,
+            'invoice_line_ids': [
+                (0, 0, {'quantity': 1, 'price_unit': 100})
+            ],
+        })
+
+    def test_sepa_qr_code_generation(self):
+        """ Check different cases of SEPA QR-code generation, when qr_method is
+        specified beforehand.
+        """
+        self.sepa_qr_invoice.qr_code_method = 'sct_qr'
+
+        # Using a SEPA IBAN should work
+        self.sepa_qr_invoice.generate_qr_code()
+
+        # Using a non-SEPA IBAN shouldn't
+        self.sepa_qr_invoice.invoice_partner_bank_id = self.acc_non_sepa_iban
+        with self.assertRaises(UserError, msg="It shouldn't be possible to generate a SEPA QR-code for IBAN of countries outside SEPA zone."):
+            self.sepa_qr_invoice.generate_qr_code()
+
+        # Changing the currency should break it as well
+        self.sepa_qr_invoice.invoice_partner_bank_id = self.acc_sepa_iban
+        self.sepa_qr_invoice.currency_id = self.env.ref('base.USD').id
+        with self.assertRaises(UserError, msg="It shouldn't be possible to generate a SEPA QR-code for another currency as EUR."):
+            self.sepa_qr_invoice.generate_qr_code()
+
+    def test_sepa_qr_code_detection(self):
+        """ Checks SEPA QR-code auto-detection when no specific QR-method
+        is given to the invoice.
+        """
+        self.sepa_qr_invoice.generate_qr_code()
+        self.assertEqual(self.sepa_qr_invoice.qr_code_method, 'sct_qr', "SEPA QR-code generator should have been chosen for this invoice.")
+