[14.0][IMP] l10n_it_ricevute_bancarie: Add exposure/risk and improve the management of unsolved riba #4457
Conversation
|
@SirAionTech va bene che teniamo tutto in un solo commit o va gestito come una migrazione? |
toita86
force-pushed
the
14.0-MIG-l10n_it_ricevute_bancarie-toita86
branch
from
6f99361
to
7a71ef9
Compare
toita86
force-pushed
the
14.0-MIG-l10n_it_ricevute_bancarie-toita86
branch
from
7a71ef9
to
15bcfb8
Compare
francesco-ooops
added
14.0
is porting
toita86
force-pushed
the
14.0-MIG-l10n_it_ricevute_bancarie-toita86
branch
from
15bcfb8
to
1514290
Compare
Secondo me devono essere portati i singoli commit come in una migrazione perché ognuno ha il suo scopo. @odooNextev ti chiedo di revisionare perché la PR e i commit portati sono tuoi, come il nuovo commit in questa PR. |
toita86
force-pushed
the
14.0-MIG-l10n_it_ricevute_bancarie-toita86
branch
from
1514290
to
28b4471
Compare
|
@SirAionTech giusto un dubbio, la descrizione dei commit "l10n_it_riba" va modificata con il nome v14 del modulo o lasciamo così come da commit originale? |
Si va nelle aree grigie 😄 secondo me ci andrebbe il nome del modulo che si sta effettivamente modificando nel commit |
…k as past due" wizard
toita86
force-pushed
the
14.0-MIG-l10n_it_ricevute_bancarie-toita86
branch
from
28b4471
to
6d9c241
Compare
| esposizione, cioè l'importo dovuto dal cliente a fronte dell'emissione | ||
| della RiBa non ancora scaduta. | ||
|
|
||
| In maniera predefinita la data delle registrazioni dei pagamenti viene |
There was a problem hiding this comment.
Qua nella 16.0 avevo messo questa descrizione per indicare tutte le modalità di modifica della data di scadenza:
In maniera predefinita la data delle registrazioni dei pagamenti viene
impostata con la data di scadenza della RiBa, ma è possibile modificarla in due momenti:
- durante la creazione del pagamento,
cliccando su "Segna righe come pagate" o su "Segna coma pagata"
o usando l'azione "Registrazione Riba a data di scadenza"
e indicando una data nel campo `Data pagamento`,
- successivamente a pagamento effettivamente avvenuto selezionando la
registrazione dalla vista ed elenco ed eseguendo l'azione "Imposta data
di pagamento RiBa".
There was a problem hiding this comment.
@odooNextev dal blame risulta che quella parte è aggiunta in #4383 (stiamo backportando anche quella)
| @@ -216,6 +226,11 @@ def action_cancel_draft(self): | |||
| for line in riba_list.line_ids: | |||
| line.state = "draft" | |||
|
|
|||
| def action_open_lines(self): | |||
| action = self.env.ref("l10n_it_riba.detail_riba_action").read()[0] | |||
There was a problem hiding this comment.
Va modificato l'ID esterno in l10n_it_ricevute_bancarie.detail_riba_action
| @@ -34,6 +33,12 @@ def _compute_payment_ids(self): | |||
| move_lines |= line.payment_ids | |||
| riba.payment_ids = move_lines | |||
|
|
|||
| def _compute_total_amount(self): | |||
There was a problem hiding this comment.
C'è qualcosa che non torna nel compute perchè i valori dello scoperto rimangono sempre a 0:
There was a problem hiding this comment.
@odooNextev il problema risiede in questa linea
dato che l'uso di
display_type non è lo stesso che se ne fa nella 16.0 non fa partire il calcolo.
io ho pensato a and line.payment_term_id ma non credo sia la stessa cosa.
Hai qualche suggerimento con cosa si possa sostituire?
There was a problem hiding this comment.
display_type = "payment_term" deriva da questa condizione: https://github.com/odoo/odoo/blob/16.0/addons/account/models/account_move_line.py#L440
potresti replicarla
There was a problem hiding this comment.
@odooNextev grazie per il consiglio, credo che la versione riadattata sia
and line.account_id.internal_type in ['receivable', 'payable']
solo che cosi facendo se una riba è insoluta il campo open_amount viene calcolato a 0
toita86
force-pushed
the
14.0-MIG-l10n_it_ricevute_bancarie-toita86
branch
from
6d9c241
to
38f1d86
Compare
Back port di tutti i cambiamenti fatti nella PR #3984. Con piccoli riadattamenti per la 14.0