Skip to content

Commit

Permalink
[FIX] sale_order_type: Incompatibility with l10n_latam_check module
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
maq-adhoc committed Jan 28, 2025
1 parent 3607581 commit cf37158
Showing 1 changed file with 0 additions and 6 deletions.
6 changes: 0 additions & 6 deletions sale_order_type/models/account_move.py
Original file line number Diff line number Diff line change
Expand Up @@ -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

0 comments on commit cf37158

Please sign in to comment.