[16.0] [IMP] l10n_it_riba: aggiungere esposizione/rischio e migliorare la gestione degli insoluti #3984

odooNextev · 2024-02-22T13:46:52Z

Ammontare scoperto/esposizione

Per risolvere questa issue #3983 abbiamo assegnato all'account.move.line del pagamento che viene creata alla conferma della riba.slip.line la data effettiva della scadenza della riba.slip.line e non quella della riba.slip.

Nel modello account.move abbiamo aggiunto un campo che calcola lo scoperto/esposizione della fattura, ovvero quello che attualmente si suppone solamente che sia pagato, ma effettivamente no come indicato nella vista qui sotto:

Insoluti

Per risolvere questa issue #3983 abbiamo eliminato le 2 righe di registrazione sezionale con nome "Bills" e conto effects_account_id in credito e quella con nome "RiBa" e conto riba_bank_account_id in debito e quando si marca come insoluta una riba.slip.line abbiamo annullato la riconciliazione con la account.move.line del pagamento correlata.
Facendo ciò la fattura tornerebbe "non pagata" o "parzialmente pagata" (oltre ad avere il flag "past due").

Ho dovuto rimuovere una parte dei test con questo commit: af22ff9
Suppongo che riba_list.line_ids[0].past_due_move_id.line_ids non si riferisca più alle 2 righe di registrazione contabile "Bills" e "RiBa" che non vengono più create e quindi è inutile fare dei check, però l'ho messo in un commitseparato per conferma.

MaurizioPellegrinet

LGTM

MarcoCalcagni

LGTM

Go !!

odooNextev · 2024-03-07T13:27:48Z

@OCA/local-italy-maintainers abbiamo un paio di review funzionali positive, c'è qualcuno che può farne una tecnica?

tafaRU · 2024-03-08T08:52:00Z

l10n_it_riba/models/account.py

    riba_credited_ids = fields.One2many(
        "riba.slip", "credit_move_id", "Credited RiBa Slips", readonly=True
    )
+
+    exposition = fields.Float(


Trattandosi di un nuovo campo documenterei la sua funzionalità sul README. Inoltre potrebbe aver senso renderlo store in modo da ordinare la tree per il nuovo campo. Cosa ne pensi?
Ultima cosa: documenterei anche lato codice la nuova funzionalità ad esempio aggiungendo l'attributo help al campo.

@odooNextev riesci a fare le modifiche?

domani dovrei riuscire

@tafaRU ho fatto quanto chiedevi
@andreampiovesana più tardi cerco di aggiungere quello che mi hai chiesto

grazie @odooNextev

mancherebbe ancora questo punto 😉

Trattandosi di un nuovo campo documenterei la sua funzionalità sul README

Hai modo di farlo?

Ho aggiunto un messaggio nello USAGE, non so se va bene.
In ogni caso, precommit mi ha generato e modificato i file .rst dagli .md, è corretto o bisogna lasciarlo fare a Github?

@tafaRU va bene?

primes2h · 2024-03-28T14:09:17Z

l10n_it_riba/models/account.py

    riba_credited_ids = fields.One2many(
        "riba.slip", "credit_move_id", "Credited RiBa Slips", readonly=True
    )
+
+    exposition = fields.Float(


Solo un appunto al volo, exposition in inglese indica esposizione nel senso di mostra (es. evento pubblico come una fiera), non ha a che fare con l'ambito contabile.

E' vero, infatti in chiamata un venerdì avevo chiesto che termine era più appropriato per il campo, ma era passato questo.
Sì potrebbe chiamare "risk" o "overdraft"
Dimmi tu

E' vero, infatti in chiamata un venerdì avevo chiesto che termine era più appropriato per il campo, ma era passato questo. Sì potrebbe chiamare "risk" o "overdraft" Dimmi tu

Un "overdraft" si verifica quando viene prelevata una somma superiore a quella presente in un conto.
In questo caso mi pare di capire che il termine si riferisca alle fatture non pagate ma "non" ancora scadute, giusto?
In tal caso potrebbe andar bene un semplice "open amount".

@primes2h ho fatto

@primes2h ho fatto

Grazie 👍

@primes2h ho fatto

@odooNextev Ah, visto ora.
Andrebbe modificato anche il messaggio di commit 1aa2e40

MaurizioPellegrinet

Riscontro un'anomalia e a tal proposito allego 2 video:

riba insoluta errata --> quando registro l'insoluto con "salta e conferma insoluto" non evidenzia l'insoluto, nè riapre la posizione pertanto non posso riemettere la riba a saldo
riba insoluta corretta --> quando registro l'insoluto con "crea" tutto procede correttamente e posso riemettere la riba sulla fattura.

Sarebbe inoltre interessante prendere in esame l'idea di emettere riba scaglionate a fronte di un unico insoluto
Video.zip

odooNextev · 2024-04-03T06:40:44Z

Riscontro un'anomalia e a tal proposito allego 2 video:

riba insoluta errata --> quando registro l'insoluto con "salta e conferma insoluto" non evidenzia l'insoluto, nè riapre la posizione pertanto non posso riemettere la riba a saldo

riba insoluta corretta --> quando registro l'insoluto con "crea" tutto procede correttamente e posso riemettere la riba sulla fattura.

Sarebbe inoltre interessante prendere in esame l'idea di emettere riba scaglionate a fronte di un unico insoluto Video.zip

Grazie della segnalazione, provo a cercare l'errore.

odooNextev · 2024-04-12T12:41:00Z

E' stato aggiornato qualcosa nel pre-commit?
Prima di oggi non ricevevo questo messaggio ed in locale non fallisce ancora adesso:
l10n_it_riba/models/account.py:69:46: B023 Function definition does not bind loop variable today``

In alcuni thread ho trovato che dovrei fare qualcosa del genere:

lambda line, today=today: line.riba

Così passa, ma non ne capisco il senso

SirAionTech · 2024-04-12T13:35:38Z

E' stato aggiornato qualcosa nel pre-commit?
Prima di oggi non ricevevo questo messaggio ed in locale non fallisce ancora adesso:
l10n_it_riba/models/account.py:69:46: B023 Function definition does not bind loop variable today``

Non credo, ho dovuto fare un bel po' di correzioni per quell'errore già in #4059.

odooNextev · 2024-04-19T14:48:58Z

Riscontro un'anomalia e a tal proposito allego 2 video:

riba insoluta errata --> quando registro l'insoluto con "salta e conferma insoluto" non evidenzia l'insoluto, nè riapre la posizione pertanto non posso riemettere la riba a saldo

riba insoluta corretta --> quando registro l'insoluto con "crea" tutto procede correttamente e posso riemettere la riba sulla fattura.

Sarebbe inoltre interessante prendere in esame l'idea di emettere riba scaglionate a fronte di un unico insoluto Video.zip

Abbiamo provato a risolvere il problema con questo commit: e1f5850
Il senso è "se si salta il wizard per marcare come insoluta una riga della distinta, significa che non voglio creare ulteriori righe in contabilità" perciò abbiamo pensato di annullare e poi eliminare la registrazione relativa alla riga che viene creata all'accredito della distinta (acceptance_move_id).
In questa maniera la fattura torna "parzialmente pagata" (se ci sono altri pagamenti) o "non pagata" ed è possibile riemettere la riba (togliendo i filtri dalla lista "emissione".

@MaurizioPellegrinet fammi sapere cosa ne pensi.

MaurizioPellegrinet · 2024-05-02T08:44:49Z

Riscontro un'anomalia e a tal proposito allego 2 video:

riba insoluta errata --> quando registro l'insoluto con "salta e conferma insoluto" non evidenzia l'insoluto, nè riapre la posizione pertanto non posso riemettere la riba a saldo

riba insoluta corretta --> quando registro l'insoluto con "crea" tutto procede correttamente e posso riemettere la riba sulla fattura.

Sarebbe inoltre interessante prendere in esame l'idea di emettere riba scaglionate a fronte di un unico insoluto Video.zip

Abbiamo provato a risolvere il problema con questo commit: e1f5850 Il senso è "se si salta il wizard per marcare come insoluta una riga della distinta, significa che non voglio creare ulteriori righe in contabilità" perciò abbiamo pensato di annullare e poi eliminare la registrazione relativa alla riga che viene creata all'accredito della distinta (acceptance_move_id). In questa maniera la fattura torna "parzialmente pagata" (se ci sono altri pagamenti) o "non pagata" ed è possibile riemettere la riba (togliendo i filtri dalla lista "emissione".

@MaurizioPellegrinet fammi sapere cosa ne pensi.

Ciao, secondo me la soluzione migliore rimane quella di fare i vari passaggi esclusivamente con il tasto "crea", sia nell'operazione di accredito che in quella di pagamento o insoluto.
In questo modo contabilmente ho traccia dei vari passaggi e riesco a far chiudere i conti transitori degli effetti

andreampiovesana

LGTM
@primes2h puoi verificare se adesso va bene?

odooNextev · 2024-05-06T13:18:45Z

@tafaRU puoi aggiornare la review?

MaurizioPellegrinet

LGTM

andreampiovesana · 2024-05-10T06:57:38Z

merge?

primes2h · 2024-05-10T07:50:43Z

@odooNextev riesci a fare un rebase per risolvere i confilitti?

tafaRU · 2024-08-08T15:45:21Z

alcuni parti di codice riportano il copyright di altri autori ma non vedo l'autorship del commit: è voluto?

Immagino intendi questo: https://github.com/OCA/l10n-italy/pull/3984/files#diff-14f17eafd7915abe5ba350f6628d6b1b7850500fff1816f21c532091cfaa22d5L16 Non so perchè mi riporti questa differenza, devo controllare gli spazi

Mi riferisco a questi due commit:

a4227cd [1]
75d93fd [2]

In [1] aggiungi a4227cd#diff-04a8a365a32c3d6bceb153a18902bea732d9146b2328388fda2901a10f3b3d3bR147 come mai?
In [2] aggiungi 75d93fd#diff-40a5fed0388ac924cc0dfaa472e3c8873a1186f102c57eb7a267585c48d931f8R3 è corretto?

odooNextev · 2024-08-08T15:50:01Z

alcuni parti di codice riportano il copyright di altri autori ma non vedo l'autorship del commit: è voluto?

Immagino intendi questo: https://github.com/OCA/l10n-italy/pull/3984/files#diff-14f17eafd7915abe5ba350f6628d6b1b7850500fff1816f21c532091cfaa22d5L16 Non so perchè mi riporti questa differenza, devo controllare gli spazi

Mi riferisco a questi due commit:

a4227cd [1]

75d93fd [2]

In [1] aggiungi a4227cd#diff-04a8a365a32c3d6bceb153a18902bea732d9146b2328388fda2901a10f3b3d3bR147 come mai? In [2] aggiungi 75d93fd#diff-40a5fed0388ac924cc0dfaa472e3c8873a1186f102c57eb7a267585c48d931f8R3 è corretto?

Penso sia dovuto ad un rebase con fixup...
Devo guardarci meglio

odooNextev · 2024-08-09T14:28:20Z

@tafaRU dovrei aver fatto tutto quello che c'era da fare:

aggiungere descrizioni in usage e configuration
togliere copyright
aggiungere test con entrambe le configurazioni di sbf

odooNextev · 2024-08-26T09:49:15Z

@tafaRU gentile reminder 🙏

l10n_it_riba/models/riba.py

l10n_it_riba/tests/test_riba.py

…" wizard

tafaRU

Un'ultimo nitpick e poi dovremmo esserci 👍
Grazie 1K per il tuo lavoro 🙏

l10n_it_riba/tests/riba_common.py

tafaRU · 2024-08-28T09:05:11Z

/ocabot merge minor

OCA-git-bot · 2024-08-28T09:05:18Z

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

OCA-git-bot · 2024-08-28T09:57:22Z

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

odooNextev mentioned this pull request Feb 22, 2024

[IMP] l10n_it_riba: refactoring #3983

Open

2 tasks

francesco-ooops linked an issue Feb 22, 2024 that may be closed by this pull request

[IMP] l10n_it_riba: refactoring #3983

Open

2 tasks

odooNextev changed the title ~~[16.0] [IMP] l10n_it_riba: add exposition amount field~~ [16.0] [IMP] l10n_it_riba: aggiungere esposizione/rischio e migliorare la gestione degli insoluti Feb 23, 2024

stenext force-pushed the 16.0-imp-l10n_it_riba-expo branch from a6dbfc3 to 23aded2 Compare February 23, 2024 09:30

MaurizioPellegrinet approved these changes Feb 29, 2024

View reviewed changes

MarcoCalcagni approved these changes Mar 1, 2024

View reviewed changes

OCA-git-bot added the approved label Mar 1, 2024

tafaRU requested changes Mar 8, 2024

View reviewed changes

OCA-git-bot removed the approved label Mar 8, 2024

primes2h requested changes Mar 28, 2024

View reviewed changes

MaurizioPellegrinet suggested changes Apr 2, 2024

View reviewed changes

stenext force-pushed the 16.0-imp-l10n_it_riba-expo branch 2 times, most recently from 243518f to 4e6cd9d Compare April 12, 2024 10:31

stenext force-pushed the 16.0-imp-l10n_it_riba-expo branch from 4e6cd9d to dde3ba4 Compare April 12, 2024 12:43

odooNextev requested review from tafaRU and primes2h April 12, 2024 12:47

odooNextev requested a review from MaurizioPellegrinet April 19, 2024 14:50

andreampiovesana approved these changes May 6, 2024

View reviewed changes

MaurizioPellegrinet approved these changes May 7, 2024

View reviewed changes

primes2h approved these changes May 10, 2024

View reviewed changes

stenext force-pushed the 16.0-imp-l10n_it_riba-expo branch from e1f5850 to c6a2f10 Compare May 10, 2024 08:13

stenext force-pushed the 16.0-imp-l10n_it_riba-expo branch 3 times, most recently from 2e87feb to e1b616c Compare August 9, 2024 14:01

[IMP] l10n_it_riba: add open amount field

ee7c8c4

stenext force-pushed the 16.0-imp-l10n_it_riba-expo branch from e1b616c to 5803821 Compare August 9, 2024 14:23

odooNextev requested a review from tafaRU August 9, 2024 14:24

tafaRU requested changes Aug 26, 2024

View reviewed changes

l10n_it_riba/models/riba.py Outdated Show resolved Hide resolved

l10n_it_riba/tests/test_riba.py Show resolved Hide resolved

stenext force-pushed the 16.0-imp-l10n_it_riba-expo branch from 5803821 to f88bbf9 Compare August 27, 2024 07:43

odooNextev added 2 commits August 27, 2024 09:51

[IMP] l10n_it_riba: add slip line view and slip total amount

40fc87c

[FIX] l10n_it_riba: reopen payment when skipping on "mark as past due…

483fbc3

…" wizard

stenext force-pushed the 16.0-imp-l10n_it_riba-expo branch from f88bbf9 to 5025714 Compare August 27, 2024 08:49

odooNextev requested a review from tafaRU August 27, 2024 12:03

tafaRU requested changes Aug 27, 2024

View reviewed changes

l10n_it_riba/tests/riba_common.py Outdated Show resolved Hide resolved

odooNextev added 2 commits August 27, 2024 16:28

[IMP] l10n_it_riba: add sbf_collection_type

58e0345

[IMP] l10n_it_riba: add possibility to change riba payments date

b75e8aa

stenext force-pushed the 16.0-imp-l10n_it_riba-expo branch from 402696c to b75e8aa Compare August 27, 2024 14:28

tafaRU approved these changes Aug 28, 2024

View reviewed changes

OCA-git-bot added the approved label Aug 28, 2024

OCA-git-bot added the bot is merging ⏳ label Aug 28, 2024

OCA-git-bot merged commit 9f5e270 into OCA:16.0 Aug 28, 2024
5 of 7 checks passed

OCA-git-bot added merged 🎉 and removed bot is merging ⏳ labels Aug 28, 2024

toita86 mentioned this pull request Nov 25, 2024

[14.0][IMP] l10n_it_ricevute_bancarie: Add exposure/risk and improve the management of unsolved riba #4457

Open

[16.0] [IMP] l10n_it_riba: aggiungere esposizione/rischio e migliorare la gestione degli insoluti #3984

[16.0] [IMP] l10n_it_riba: aggiungere esposizione/rischio e migliorare la gestione degli insoluti #3984

Conversation

odooNextev commented Feb 22, 2024 • edited Loading

Ammontare scoperto/esposizione

Insoluti

MaurizioPellegrinet left a comment

Choose a reason for hiding this comment

MarcoCalcagni left a comment

Choose a reason for hiding this comment

odooNextev commented Mar 7, 2024

tafaRU Mar 8, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

primes2h Mar 28, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

MaurizioPellegrinet left a comment

Choose a reason for hiding this comment

odooNextev commented Apr 3, 2024

odooNextev commented Apr 12, 2024 • edited Loading

SirAionTech commented Apr 12, 2024

odooNextev commented Apr 19, 2024 • edited Loading

MaurizioPellegrinet commented May 2, 2024

andreampiovesana left a comment • edited Loading

Choose a reason for hiding this comment

odooNextev commented May 6, 2024

MaurizioPellegrinet left a comment

Choose a reason for hiding this comment

andreampiovesana commented May 10, 2024

primes2h commented May 10, 2024

tafaRU commented Aug 8, 2024

odooNextev commented Aug 8, 2024

odooNextev commented Aug 9, 2024

odooNextev commented Aug 26, 2024

tafaRU left a comment

Choose a reason for hiding this comment

tafaRU commented Aug 28, 2024

OCA-git-bot commented Aug 28, 2024

OCA-git-bot commented Aug 28, 2024

odooNextev commented Feb 22, 2024 •

edited

Loading

tafaRU Mar 8, 2024 •

edited

Loading

primes2h Mar 28, 2024 •

edited

Loading

odooNextev commented Apr 12, 2024 •

edited

Loading

odooNextev commented Apr 19, 2024 •

edited

Loading

andreampiovesana left a comment •

edited

Loading