[l10n_it_RIBA_OCA] Mancato inserimento spese riba in fattura

[AlSacco]

Module

[l10n_it_RIBA_OCA]

Describe the bug

Se valido una fattura in bozza tramite la funzione standard di Odoo di CONFERMA REGISTRAZIONI presente nella tree delle fatture, il modulo OCA non aggiunge le righe di addebito delle spese riba

To Reproduce

Affected Version:

• v.16
• v.18 [FIX]l10n_it_riba_oca fix add riba cost: add collection fees on batch post #5119

Steps to reproduce the behavior:

1. Creo fattura in bozza con tipo pagamento riba che prevede addebito di spesa
2. Dalla tree delle fatture seleziono una o piu fatture in bozza
3. Valido le selezioni precedenti tramite l'azione CONFERMA REGISTRAZIONI e il sistema conferma le fatture ma non aggiunge le righe delle spese di incasso riba

Expected behavior
Aggiunga le spese come avviene se confermo la fattura in bozza tramite il tasto CONFERMA presente dentro la fattura

Additional context
Add any other context about the problem here. (e.g. OS, Python version, ...)

[francesco-ooops]

@AlSacco hai verificato se sulla v16 funziona correttamente?

[AlSacco]

@francesco-ooops ho verificato anche sulla 16 e non funziona. Fa esattamente come sulla 18

[SirPyTech]

Ho dato un'occhiata veloce e secondo me serve una soluzione analoga a #4862:

-    def action_post(self):
-        ret = super().action_post()
+    def _post(self, soft=True):
+        ret = super()._post(soft=soft)

tutte le operazioni da fare alla conferma vanno in _post invece che action_post.

[mmircoli-nexapp]

#5103 PR per risolvere il problema come suggerito da @SirPyTech

[mmircoli-nexapp]

Ho dato un'occhiata veloce e secondo me serve una soluzione analoga a #4862:

• def action_post(self):

•    ret = super().action_post()
  

• def _post(self, soft=True):

•    ret = super()._post(soft=soft)
  

tutte le operazioni da fare alla conferma vanno in _post invece che action_post.

abbiamo verificato che non basta fare in questo modo qua, perchè si fixa da una parte il problema e si ripresenta dall'altra, ossia quando poi si valida la fattura dal form a quel punto non prende più le spese.
In effetti poi falliscono tutti i test.
Va pensato in modo diverso il fix credo.

[enigmista91]

Ho dato un'occhiata veloce e secondo me serve una soluzione analoga a #4862:

• def action_post(self):

•    ret = super().action_post()
  

• def _post(self, soft=True):

•    ret = super()._post(soft=soft)
  

tutte le operazioni da fare alla conferma vanno in _post invece che action_post.

Ho riprovato qui #5106 mi fallisce un solo test, per caso hai idee ulteriori? perchè con varie prove, mi sono avvicinato alla soluzione.
Ho estratto il codice e l'ho portato fuori su due metodi.
Ora rimane il problema su un test

[SirPyTech]

Ho riprovato qui #5106 mi fallisce un solo test, per caso hai idee ulteriori?

No mi spiace, non riesco a guardarci meglio per ora.

[mmircoli-nexapp]

Ho riprovato qui #5106 mi fallisce un solo test, per caso hai idee ulteriori?

No mi spiace, non riesco a guardarci meglio per ora.

ok provo a rifare qualche prova appena posso.
Grazie intanto