[IMP] Use the partner to find the bank account (for reconciliation) #59
Conversation
| [('company_id', '=', self.company_id.id), ('acc_number', '=', self.account_number)]) | ||
| [('company_id', '=', self.company_id.id), | ||
| ('acc_number', '=', self.account_number), | ||
| ('partner_id', '=', self.partner_id.id)]) |
There was a problem hiding this comment.
Is self.partner_id always known at this stage ? I've not looked in details but I know scenarios (e.g. CODA files import) where the account number is used to find the partner.
There was a problem hiding this comment.
Yes the partner_id is always known at this stage.
The only call to that method is this:
if self.account_number and self.partner_id and not self.partner_bank_id: self.partner_bank_id = self._find_or_create_bank_account()
There was a problem hiding this comment.
There are maybe other places where Odoo is looking for bank account without using the partner but I only fix the issue on the reconciliation widget provided by Odoo.
|
Note to our future selves: this is necessary when we lift the constraints that allows a bank account to be linked to only one partner. |
No description provided.