[16.0][IMP]l10n_it_riba: Forward port of #3437 by matteotognini · Pull Request #4655 · OCA/l10n-italy

matteotognini · 2025-03-18T13:59:11Z

Solves #2118 #4637

francesco-ooops · 2025-04-07T13:02:05Z

@CiroBoxHub puoi fare review?

CiroBoxHub · 2025-05-14T09:21:20Z

Buongiorno a tutti. Questa PR è mergiabile?

andreampiovesana

LGTM

odooNextev

LGTM

CiroBoxHub · 2025-06-11T13:38:37Z

Buongiorno, ci sono problemi con il merge di questo fix?

francesco-ooops · 2025-06-11T13:42:52Z

/ocabot rebase

OCA-git-bot · 2025-06-11T13:43:03Z

Congratulations, PR rebased to 16.0.

francesco-ooops · 2025-06-11T15:55:49Z

@OCA/local-italy-maintainers buona?

OCA-git-bot · 2025-06-11T16:00:04Z

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

monen17

Grazie della PR!

@OCA/local-italy-maintainers buona?

Per me no perché i commit non seguono https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#71commit-message.
Ad esempio, il primo punto è:

Commit messages are in English

mentre al momento i commit sono in italiano:

matteotognini · 2025-06-27T11:37:24Z

@monen17 Grazie della review, va bene andare a modificare il messaggio di commit anche se non sono io l'autore? Se si li traduco

monen17 · 2025-06-27T11:57:33Z

@monen17 Grazie della review, va bene andare a modificare il messaggio di commit anche se non sono io l'autore? Se si li traduco

I messaggi dei commit sono già diversi da quelli di #3437 che sono

Anche nel contenuto qualcosa è cambiato, credo per adattarlo a 16.0.
Quindi perché porsi ora il problema? 🤔

Secondo me in generale se l'autore (in questo caso @alessandrocamilli) è d'accordo o le modifiche sono minime puoi farle e lasciare l'autore originale, altrimenti ti metti come co-autore e fai le modifiche che devi.
Includerei la traduzione tra le modifiche minime visto che non cambia il significato del messaggio, e in più va fatta per seguire le linee guida.

TL;DR: Per me sì

matteotognini · 2025-06-27T12:10:20Z

Grazie della spiegazione. Ho modificato i commit messages.

CiroBoxHub · 2025-07-15T07:38:36Z

Buongiorno, ci sono novità? Ci sono azioni da fare?

monen17 · 2025-07-18T07:18:33Z

Buongiorno, ci sono novità? Ci sono azioni da fare?

Solo quello che vedi qui nella PR.

La prossima cosa da fare è per l'autore @matteonext: quando devo aggiornare la revisione segnalo qui in github

Fino ad allora per me ci stai ancora lavorando.

CiroBoxHub · 2025-09-15T13:46:58Z

Buon pomeriggio, abbiamo estrapolato la PR per poterla mettere su un nostro cliente e abbiamo fatto dei test.
A parte l'aggiunta dei dati nle partner, il comportamento è il medesimo.
Abbiamo due fatture con scadenza ottobre e una con tre scadenze (gennaio, febbraio, marzo)
Se confermo quella di ottobre mi crea correttamente la riga d'incasso. Quando poi confermo quella con tre scadenze non crea righe con spese d'incasso.
Stessa cosa se confermo prima quella con 3 date crea correttamente le righe, quando poi confermo la fattura con scadenza ottobre non aggiunge nulla

CiroBoxHub · 2025-09-15T13:50:27Z

Aggiungo anche che se la modalità è una spesa ogni scadenza vengono addirittura doppiate le righe per quante sono le scadenze

monen17

Grazie della PR!

@OCA/local-italy-maintainers buona?

Per me no perché i commit non seguono https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#71commit-message. Ad esempio, il primo punto è:

Commit messages are in English

mentre al momento i commit sono in italiano:

Grazie di aver tradotto i commit in inglese 🇬🇧
Ora i commit sono

e ancora non seguono https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#71commit-message, puoi verificare?

Ho dato solo un'occhiata al codice e ho notato questo commit:

(369419b)
il titolo è "usa una tupla invece di una lista", ma il contenuto non ha nulla a che fare con il titolo, come mai?

VBNext · 2025-11-07T16:50:17Z

riba_policy_expenses = one_a_month
Tiene conto della data fattura

Da correggere il seguente comportamento
Comportamento osservato:
Emetto 1° fattura nel mese di Ottobre tipo pagamento bonifico
Emetto 2° fattura nel mese di Ottobre tipo pagamento Riba
Non addebita le spese Riba sulla 2°
Comportamento desiderato:
Addebita le spese Riba sulla 2°
Perchè non deve tenere conto delle fattura con tipo di pagamento diverso da Riba

Comportamento osservato:
Non addebita le spese Riba su fatture con data successiva ad oggi
Comportamento desiderato:
Anche se la data fattura è successiva ad oggi devono essere addebitate le spese per la prima fattura del mese

Comportamento osservato:
Addebita 2 volte le spese se la scadenza riba ha due date (Riba 30/60)
Comportamento desiderato:
Deve addebitare 1 volta sola le spese

riba_policy_expenses = unlimited
cambiare la descrizione da "One expenses per maturity" in "One expenses per invoice"

Aggiungere riba_policy_expenses = one_a_maturity
con funzionamento simile a one_a_month ma che tenga conto della data scadenza
Quindi :
Addebitare 2 volte le spese se la scadenza riba ha due date (Riba 30/60)
Non addebitare la spese se esiste una fattura con scadenza stesso mese es:
Fattura 1 - data fattura Ottobre - Scadenza Dicembre (Riba 60GG) --> spese SI
Fattura 2- data fattura Novembre - Scadenza Dicembre (Riba 30GG) --> spese NO

andreampiovesana

One expenses?

andreampiovesana

LGTM

monen17 · 2026-03-13T17:46:03Z

-        for d in all_date_due:
-            if invoice_date_due.month == d.month and invoice_date_due.year == d.year:
-                return True
+        self.ensure_one()
+        if self.partner_id.riba_policy_expenses != "unlimited":
+            for d in all_date_due:
+                if invoice_date_due.strftime("%Y-%m") == str(d.strftime("%Y-%m")):
+                    return True


Prima di questa modifica il controllo era su month e year, possiamo mantenerlo com'era?
La modifica del commit originale 6cc3d21 aggiungeva praticamente solo l'if.

Fatto, puoi rivedere la revisione?

monen17 · 2026-03-13T17:50:56Z

-                untaxed_amount_currency=0,
-                sign=1,
+                sign=1 if invoice.is_inbound(include_receipts=True) else -1,
+                untaxed_amount_currency=invoice.amount_untaxed,


Queste modifiche non c'erano nel commit originale 6cc3d21, come mai servono?

Era una correzione perlopiù semantica:

untaxed_amount_currency: _compute_terms usa gli importi per decidere quante rate generare e come distribuirle, con 0, in certi casi, la lista di rate ottenuta potrebbe essere diversa (o vuota) rispetto a passare l'importo reale e quindi date diverse o mancanti.

sign: passare sign=1 su una nota credito non produce date sbagliate nel caso standard, ma è concettualmente scorretto

Non cambia il risultato nel 99% dei casi pratici delle RIBA, ma è più robusto anche in vista di future modifiche.

monen17 · 2026-03-13T17:56:48Z

                    return True
        return False

+    def maturity_check(self, invoice_date_due, all_date_due):


Queste modifiche non vengono dalla PR originale, probabilmente sono per risolvere #4655 (comment), hai mica verificato se dipendono dalle alte modifiche di questa PR? Se non è così penso sia meglio farle in una PR separata così si può chiudere questo porting.
Sarebbe anche utile sapere se in 14.0 succede.

Sì, porto il commit in un'altra PR

…fees

…tion fee policy on partner

tafaRU · 2026-06-12T06:36:03Z

@monen17 potresti aggiornare la tua review?

matteotognini mentioned this pull request Mar 18, 2025

[16.0][IMP]l10n_it_riba: Exclude current invoice lines when computing RIBA fees #4636

Closed

matteotognini force-pushed the 16.0-imp-l10n_italy_riba branch from 6bb79e1 to 38707ea Compare March 18, 2025 14:49

francesco-ooops mentioned this pull request Mar 17, 2025

l10n_it_riba: generazione riga costi di incasso #4637

Open

3 tasks

francesco-ooops linked an issue Mar 19, 2025 that may be closed by this pull request

l10n_it_riba: generazione riga costi di incasso #4637

Open

3 tasks

francesco-ooops mentioned this pull request Mar 19, 2025

Miglioria RIBA - Politica di addebito spese d'incasso #2118

Open

4 tasks

francesco-ooops linked an issue Mar 19, 2025 that may be closed by this pull request

Miglioria RIBA - Politica di addebito spese d'incasso #2118

Open

4 tasks

CiroBoxHub approved these changes May 9, 2025

View reviewed changes

andreampiovesana approved these changes May 14, 2025

View reviewed changes

OCA-git-bot added the approved label May 14, 2025

odooNextev approved these changes Jun 5, 2025

View reviewed changes

OCA-git-bot force-pushed the 16.0-imp-l10n_italy_riba branch from 38707ea to 75db01b Compare June 11, 2025 13:43

OCA-git-bot added the ready to merge label Jun 11, 2025

monen17 requested changes Jun 27, 2025

View reviewed changes

OCA-git-bot removed approved ready to merge labels Jun 27, 2025

matteotognini force-pushed the 16.0-imp-l10n_italy_riba branch from 75db01b to 369419b Compare June 27, 2025 12:08

monen17 self-requested a review July 18, 2025 14:43

monen17 requested changes Nov 7, 2025

View reviewed changes

matteotognini force-pushed the 16.0-imp-l10n_italy_riba branch from 49af11b to c71655c Compare November 7, 2025 15:37

matteotognini requested a review from monen17 November 11, 2025 07:38

andreampiovesana suggested changes Dec 17, 2025

View reviewed changes

matteotognini force-pushed the 16.0-imp-l10n_italy_riba branch from 39d34df to 1116809 Compare December 17, 2025 15:50

andreampiovesana approved these changes Dec 17, 2025

View reviewed changes

matteotognini force-pushed the 16.0-imp-l10n_italy_riba branch 2 times, most recently from 31bae74 to 66a36a4 Compare December 22, 2025 14:44

monen17 added the is porting This pull request is porting a change from another version label Feb 20, 2026

monen17 requested changes Mar 13, 2026

View reviewed changes

alessandrocamilli added 2 commits May 25, 2026 11:15

[IMP]l10n_it_riba_oca:Flag to exclude a partner from RIBA collection …

3b6af7e

…fees

[IMP]l10n_it_riba_oca:added the possibility to choose the RIBA collec…

bbad8b1

…tion fee policy on partner

stenext force-pushed the 16.0-imp-l10n_italy_riba branch from 66a36a4 to bbad8b1 Compare May 25, 2026 09:19

OCA-git-bot added series:16.0 mod:l10n_it_riba_oca Module l10n_it_riba_oca labels May 25, 2026

Uh oh!

Conversation

matteotognini commented Mar 18, 2025 • edited by francesco-ooops Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

francesco-ooops commented Apr 7, 2025

Uh oh!

CiroBoxHub commented May 14, 2025

Uh oh!

andreampiovesana left a comment

Choose a reason for hiding this comment

Uh oh!

odooNextev left a comment

Choose a reason for hiding this comment

Uh oh!

CiroBoxHub commented Jun 11, 2025

Uh oh!

francesco-ooops commented Jun 11, 2025

Uh oh!

OCA-git-bot commented Jun 11, 2025

Uh oh!

francesco-ooops commented Jun 11, 2025

Uh oh!

OCA-git-bot commented Jun 11, 2025

Uh oh!

monen17 left a comment

Choose a reason for hiding this comment

Uh oh!

matteotognini commented Jun 27, 2025

Uh oh!

monen17 commented Jun 27, 2025

Uh oh!

matteotognini commented Jun 27, 2025

Uh oh!

CiroBoxHub commented Jul 15, 2025

Uh oh!

monen17 commented Jul 18, 2025

Uh oh!

CiroBoxHub commented Sep 15, 2025

Uh oh!

CiroBoxHub commented Sep 15, 2025

Uh oh!

monen17 left a comment

Choose a reason for hiding this comment

Uh oh!

VBNext commented Nov 7, 2025

Uh oh!

andreampiovesana left a comment

Choose a reason for hiding this comment

Uh oh!

andreampiovesana left a comment

Choose a reason for hiding this comment

Uh oh!

monen17 Mar 13, 2026

Choose a reason for hiding this comment

Uh oh!

stenext May 25, 2026

Choose a reason for hiding this comment

Uh oh!

monen17 Mar 13, 2026

Choose a reason for hiding this comment

Uh oh!

stenext May 25, 2026

Choose a reason for hiding this comment

Uh oh!

monen17 Mar 13, 2026

Choose a reason for hiding this comment

Uh oh!

stenext May 25, 2026

Choose a reason for hiding this comment

Uh oh!

tafaRU commented Jun 12, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

matteotognini commented Mar 18, 2025 •

edited by francesco-ooops

Loading