From cf37158c4250a8563dc64fb1b1c32d3db1097ebb Mon Sep 17 00:00:00 2001 From: Martin Quinteros Date: Tue, 28 Jan 2025 17:18:01 -0300 Subject: [PATCH] [FIX] sale_order_type: Incompatibility with l10n_latam_check module This is a work-in-progress PR and is not yet finished. When adding a dependency to the _compute_journal_id method, it breaks the functionality of the l10n_latam_check module when calculating the l10n_latam_check_issuer_vat field during the creation of a third-party check payment. Steps to reproduce: Create a payment. Choose a third-party check journal. Modify the l10n_latam_check_issuer_vat issuer. Save the payment. The l10n_latam_check_issuer_vat field is recomputed. --- sale_order_type/models/account_move.py | 6 ------ 1 file changed, 6 deletions(-) diff --git a/sale_order_type/models/account_move.py b/sale_order_type/models/account_move.py index efc0de8bec9..785b66178a5 100644 --- a/sale_order_type/models/account_move.py +++ b/sale_order_type/models/account_move.py @@ -48,9 +48,3 @@ def _compute_invoice_payment_term_id(self): move.invoice_payment_term_id = move.sale_type_id.payment_term_id return res - @api.depends("sale_type_id") - def _compute_journal_id(self): - res = super()._compute_journal_id() - for move in self.filtered("sale_type_id.journal_id"): - move.journal_id = move.sale_type_id.journal_id - return res