Skip to content
Snippets Groups Projects
Commit 4769ed4e authored by Denis Ledoux's avatar Denis Ledoux
Browse files

[FIX] base: unique bank account number constraint too restrictive

There is a multi-company record rule on `res.partner.bank`,
meaning you cannot see the bank accounts of other companies.

Making a constraint on `sanitized_acc_number` alone prevent
to create a same bank account into different companies,
while you cannot see this bank account from the other
companies.

In this case, the user tries to import its bank statement,
and it contains a bank account which already exists
in the database but for another company than the one
he is currently importing the statement, and,
 - it cannot find it because of the multi-company rule,
 - it cannot create it because of the constraint

The constraint must take into account the company

opw-690969
parent 8a767512
No related branches found
No related tags found
No related merge requests found
......@@ -67,7 +67,7 @@ class ResPartnerBank(models.Model):
company_id = fields.Many2one('res.company', 'Company', default=lambda self: self.env.user.company_id, ondelete='cascade')
_sql_constraints = [
('unique_number', 'unique(sanitized_acc_number)', 'Account Number must be unique'),
('unique_number', 'unique(sanitized_acc_number, company_id)', 'Account Number must be unique'),
]
@api.one
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment