l10n-italy icon indicating copy to clipboard operation
l10n-italy copied to clipboard

14.0 l10n_it_fatturapa documenti correlati non vengono salvati

Open TheMule71 opened this issue 3 years ago • 6 comments

NOTA: dopo il merge di #1984

Ciao @TheMule71, a quanto pare i documenti correlati non vengono salvati e/o spariscono. Non ho ancora investigato sulla causa.

Il motivo per cui i related_documents (e admin_ref) non vengono salvati è che il core esclude le modifiche a invoice_line_ids in https://github.com/odoo/odoo/blob/d44dfff647939022b8fe8d091bc1f6562cc47d86/addons/account/models/account_move.py#L1872. Per qualche motivo i related_documents vengono passati come modifica a invoice_line_ids mentre ad esempio le imposte sono passate comunque come modifiche a line_ids: image Abbiamo fatto un po' di indagini su questo problema senza arrivare a una soluzione. Sembra ci sia qualche meccanismo oscuro per cui dei campi vengono copiati da invoice_line_ids a line_ids (vedi ad esempio i metodi create, write e _onchange_invoice_line_ids di account.move.line).

Originally posted by @SimoRubi in https://github.com/OCA/l10n-italy/issues/1984#issuecomment-812579508

TheMule71 avatar Apr 09 '21 14:04 TheMule71

@TheMule71 facendo prove in quest'area ho notato che inserendo il campo anche nella vista lista di line_ids questa issue non si presenta, esempio:

<xpath expr="//field[@name='invoice_line_ids']/form/sheet/field[@name='name']"
       position="after">
    <field name="nuovo_campo"/>
</xpath>
<xpath expr="//field[@name='line_ids']/tree" position="inside">
    <field name="nuovo_campo" invisible="1"/>
</xpath>

fredzamoabg avatar Dec 09 '21 09:12 fredzamoabg

Il problema evidenziato si verifica quando sono installati sia l10n_it_fatturapa_out sia l10n_it_fatturapa_in. Questo perché related_documents viene messo a readonly in questo punto e di conseguenza su di esso non viene chiamata la write. La migrazione della vista l10n_it_fatturapa_in.view_invoice_form_fatturapa_in è da rivedere perché ad esempio contiene ancora riferimenti a viste che sulla 14.0 non ci sono più:

  • account.invoice_supplier_form: punto 1 e punto 2
  • account.invoice_supplier_tree: qui

Oltre a capire come farla convivere quando è installato anche l10n_it_fatturapa_out.

tafaRU avatar Dec 31 '21 08:12 tafaRU

Il problema evidenziato si verifica quando sono installati sia l10n_it_fatturapa_out sia l10n_it_fatturapa_in. Questo perché related_documents viene messo a readonly in questo punto e di conseguenza su di esso non viene chiamata la write. La migrazione della vista l10n_it_fatturapa_in.view_invoice_form_fatturapa_in è da rivedere perché ad esempio contiene ancora riferimenti a viste che sulla 14.0 non ci sono più:

  • account.invoice_supplier_form: punto 1 e punto 2
  • account.invoice_supplier_tree: qui

Oltre a capire come farla convivere quando è installato anche l10n_it_fatturapa_out.

Potrebbe essere sufficiente aggiungere un force_save = "1" su related_documents quando si aggiunge readonly="1"?

GSLabIt avatar Jan 08 '22 13:01 GSLabIt

Potrebbe essere sufficiente aggiungere un force_save = "1" su related_documents quando si aggiunge readonly="1"?

@GSLabIt grazie del suggerimento. Ho provato però non ha funzionato. Ho invece risolto con https://github.com/OCA/l10n-italy/pull/2586

tafaRU avatar Jan 11 '22 11:01 tafaRU

Quindi questa si può chiudere?

eLBati avatar Jan 20 '22 15:01 eLBati

Quindi questa si può chiudere?

No perché abbiamo chiarito in https://github.com/OCA/l10n-italy/pull/2586#issuecomment-1009963618 (riporto qui sotto) che la PR #2586 non risolve questo problema:

And it closes #2211

La issue https://github.com/OCA/l10n-italy/issues/2211 parla dei documenti correlati assegnati alle righe invoice_line_ids mentre dalla descrizione mi pare che questa PR risolva un problema per i documenti correlati associati alla fattura qundi non credo risolva la issue indicata.

SimoRubi avatar Jan 20 '22 16:01 SimoRubi

There hasn't been any activity on this issue in the past 6 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. If you want this issue to never become stale, please ask a PSC member to apply the "no stale" label.

github-actions[bot] avatar Jan 07 '24 12:01 github-actions[bot]

@aleuffre puoi verificare se si presenta anche sulla 16?

francesco-ooops avatar Jan 07 '24 19:01 francesco-ooops

Sinceramente, non riesco nemmeno ad arrivarci sulla v16.

Ho trovato il campo ma riesco a vederlo solo se la fattura è già confermata, e quindi il campo è readonly (peraltro si vede male)

image

ma fintanto che la fattura è aperta e ancora modificabile non riesco ad arrivarci...

aleuffre avatar Jan 08 '24 09:01 aleuffre

@TheMule71 riesci a fare il punto su questa issue? grazie!

francesco-ooops avatar Jan 08 '24 09:01 francesco-ooops