diff --git a/account_banking_mandate/models/account_banking_mandate.py b/account_banking_mandate/models/account_banking_mandate.py index 4cf2f2098..cbb05b04c 100644 --- a/account_banking_mandate/models/account_banking_mandate.py +++ b/account_banking_mandate/models/account_banking_mandate.py @@ -61,7 +61,7 @@ class AccountBankingMandate(models.Model): default=lambda self: self.env.company, ) unique_mandate_reference = fields.Char( - string="Unique Mandate Reference", tracking=10, copy=False + string="Unique Mandate Reference", tracking=10, copy=False, default="/" ) signature_date = fields.Date( string="Date of Signature of the Mandate", @@ -177,8 +177,8 @@ class AccountBankingMandate(models.Model): @api.model def create(self, vals=None): - unique_mandate_reference = vals.get("unique_mandate_reference") - if not unique_mandate_reference or unique_mandate_reference == "New": + unique_mandate_reference = vals.get("unique_mandate_reference", "/") + if unique_mandate_reference == "/": vals["unique_mandate_reference"] = ( self.env["ir.sequence"].next_by_code("account.banking.mandate") or "New" ) diff --git a/account_banking_mandate/tests/test_mandate.py b/account_banking_mandate/tests/test_mandate.py index b6d6ca796..61ec6e8fd 100644 --- a/account_banking_mandate/tests/test_mandate.py +++ b/account_banking_mandate/tests/test_mandate.py @@ -175,9 +175,8 @@ class TestMandate(TransactionCase): def test_mandate_reference_03(self): """ - Test case: create a mandate with "New" as reference - Expected result: the reference of the created mandate is not empty and - is not "New" + Test case: create a mandate with "TEST" as reference + Expected result: the reference of the created mandate is "TEST" """ bank_account = self.env.ref("account_payment_mode.res_partner_12_iban") mandate = self.env["account.banking.mandate"].create( @@ -185,15 +184,32 @@ class TestMandate(TransactionCase): "partner_bank_id": bank_account.id, "signature_date": "2015-01-01", "company_id": self.company.id, - "unique_mandate_reference": "New", + "unique_mandate_reference": "TEST", } ) self.assertTrue(mandate.unique_mandate_reference) - self.assertNotEqual(mandate.unique_mandate_reference, "New") + self.assertEqual(mandate.unique_mandate_reference, "TEST") + + def test_mandate_reference_04(self): + """ + Test case: create a mandate with "/" as reference + Expected result: the reference of the created mandate is not "/" + """ + bank_account = self.env.ref("account_payment_mode.res_partner_12_iban") + mandate = self.env["account.banking.mandate"].create( + { + "partner_bank_id": bank_account.id, + "signature_date": "2015-01-01", + "company_id": self.company.id, + "unique_mandate_reference": "/", + } + ) + self.assertTrue(mandate.unique_mandate_reference) + self.assertNotEqual(mandate.unique_mandate_reference, "/") def test_mandate_reference_05(self): """ - Test case: create a mandate with False as reference + Test case: create a mandate without reference Expected result: the reference of the created mandate is not empty """ bank_account = self.env.ref("account_payment_mode.res_partner_12_iban") @@ -202,23 +218,6 @@ class TestMandate(TransactionCase): "partner_bank_id": bank_account.id, "signature_date": "2015-01-01", "company_id": self.company.id, - "unique_mandate_reference": False, - } - ) - self.assertTrue(mandate.unique_mandate_reference) - - def test_mandate_reference_06(self): - """ - Test case: create a mandate with a empty string as reference - Expected result: the reference of the created mandate is not empty - """ - bank_account = self.env.ref("account_payment_mode.res_partner_12_iban") - mandate = self.env["account.banking.mandate"].create( - { - "partner_bank_id": bank_account.id, - "signature_date": "2015-01-01", - "company_id": self.company.id, - "unique_mandate_reference": "", } ) self.assertTrue(mandate.unique_mandate_reference) diff --git a/account_banking_mandate/views/account_banking_mandate_view.xml b/account_banking_mandate/views/account_banking_mandate_view.xml index c31374da4..400a84546 100644 --- a/account_banking_mandate/views/account_banking_mandate_view.xml +++ b/account_banking_mandate/views/account_banking_mandate_view.xml @@ -50,7 +50,7 @@