Skip to content

[18.0] [FIX] l10n_it_riba_oca: revert and fix related documents computation#4984

Merged
OCA-git-bot merged 1 commit into
OCA:18.0from
odooNextev:18.0-fix-l10n_it_riba_oca-relateddocs
Nov 17, 2025
Merged

[18.0] [FIX] l10n_it_riba_oca: revert and fix related documents computation#4984
OCA-git-bot merged 1 commit into
OCA:18.0from
odooNextev:18.0-fix-l10n_it_riba_oca-relateddocs

Conversation

@odooNextev

Copy link
Copy Markdown
Contributor

Issue: odoo-italia/task989

Ripristina il collegamento calcolato con i documenti correlati per impostare CIG e CUP nelle Ri.Ba.

Inoltre ho cercato di migliorare l'assegnazione controllando separatamente i dati a livello:

  • Invoice-level related documents (account.move.related_document_ids)
  • Invoice line-level related documents (account.move.line.related_document_ids)

Invoice line-level deve prevalere ed in caso di più occorrenze per la stessa riga di distinta viene sollevata un'eccezione.

@odooNextev odooNextev changed the title [FIX] l10n_it_riba_oca: revert and fix related documents computation [18.0] [FIX] l10n_it_riba_oca: revert and fix related documents computation Nov 6, 2025
@odooNextev

Copy link
Copy Markdown
Contributor Author

@tafaRU

@eLBati eLBati 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.

LGTM

@stenext stenext force-pushed the 18.0-fix-l10n_it_riba_oca-relateddocs branch from ee49476 to 4f0b830 Compare November 7, 2025 14:45
Now it checks both:
        - Invoice-level related documents (account.move.related_document_ids)
        - Invoice line-level related documents (account.move.line.related_document_ids)
Priority: Invoice line-level values always take precedence over invoice-level values.
@stenext stenext force-pushed the 18.0-fix-l10n_it_riba_oca-relateddocs branch from 6cf34b7 to 93eaf8a Compare November 7, 2025 15:38

@VBNext VBNext 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.

Funzionale ok

@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

@primes2h primes2h left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@primes2h

Copy link
Copy Markdown
Contributor

/ocabot merge minor

@OCA-git-bot

Copy link
Copy Markdown
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 18.0-ocabot-merge-pr-4984-by-primes2h-bump-minor, awaiting test results.

@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

@OCA-git-bot OCA-git-bot merged commit bf27206 into OCA:18.0 Nov 17, 2025
6 of 7 checks passed
@odooNextev

Copy link
Copy Markdown
Contributor Author

/ocabot merge minor

Grazie

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.

6 participants