Skip to content

[16.0] [IMP] l10n_it_account_stamp: avoid use distinct write to create account move line(s) for account stamp#5219

Merged
OCA-git-bot merged 1 commit into
OCA:16.0from
patrickt-oforce:16.0-IMP-l10n_it_account_stamp
Jun 12, 2026
Merged

[16.0] [IMP] l10n_it_account_stamp: avoid use distinct write to create account move line(s) for account stamp#5219
OCA-git-bot merged 1 commit into
OCA:16.0from
patrickt-oforce:16.0-IMP-l10n_it_account_stamp

Conversation

@patrickt-oforce

Copy link
Copy Markdown
Contributor

Sometimes can happen when reset to draft an invoice or confirm it, that operation raise an exception for unbalanced move. Due to creation of account stamp move line(s) is executed in two different step even if context key check_move_validity is present with value False.

This PR, execute the create command in a single step, avoid calling multiple times methods to recompute dynamic lines

@MaurizioPellegrinet MaurizioPellegrinet left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test funzionale: OK

@tafaRU

tafaRU commented Jun 12, 2026

Copy link
Copy Markdown
Member

/ocabot merge nobump

@OCA-git-bot

Copy link
Copy Markdown
Contributor

On my way to merge this fine PR!
Prepared branch 16.0-ocabot-merge-pr-5219-by-tafaRU-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit bf187dc into OCA:16.0 Jun 12, 2026
7 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at bf187dc. Thanks a lot for contributing to OCA. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[16.0] l10n_it_account_stamp: write account stamp journal items result in unbalaced move

4 participants