Skip to content

[FIX] l10n_it_edi_related_document: fix singleton error in inverse method#5065

Merged
OCA-git-bot merged 1 commit into
OCA:18.0from
k0dev:18.0-fix-related-document-singleton
Feb 6, 2026
Merged

[FIX] l10n_it_edi_related_document: fix singleton error in inverse method#5065
OCA-git-bot merged 1 commit into
OCA:18.0from
k0dev:18.0-fix-related-document-singleton

Conversation

@k0dev

@k0dev k0dev commented Feb 6, 2026

Copy link
Copy Markdown

Riepilogo

Corregge un errore singleton in _inverse_original_related_document_fields quando vengono create più fatture contemporaneamente.

Il metodo itera con for record in self: ma poi usa self invece di record per accedere ai campi, causando:

ValueError: Expected singleton: account.move(...)

Introdotto nella PR #5046.

Modifiche

Sostituzione di self con record in 8 punti all'interno del loop (righe 177-192).

…thod

The _inverse_original_related_document_fields method uses `self` instead
of `record` inside the for loop, causing a "Expected singleton" error
when multiple invoices are created at once.

This was introduced in PR OCA#5046.

@tafaRU tafaRU left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Grazie 👍

@tafaRU

tafaRU commented Feb 6, 2026

Copy link
Copy Markdown
Member

/ocabot merge patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

On my way to merge this fine PR!
Prepared branch 18.0-ocabot-merge-pr-5065-by-tafaRU-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 76bb326 into OCA:18.0 Feb 6, 2026
7 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at 9d9aa65. 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.

5 participants