[14.0][IMP]l10n_it_fatturapa_in: copy changing values only

[PicchiSeba]

There's no need to overwrite values when they don't change. Moreover a user might not have permissions to write certain fields (in my case it was stock_valuation_layer_ids, which requires the Stock / Administrator group)

Solves: #4242

[aleuffre]

Code review

[aleuffre]

Code review, LGTM

[francesco-ooops]

Funzionale ok!

[OCA-git-bot]

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). 🤖

[francesco-ooops]

@OCA/local-italy-maintainers si può mergiare?

[francesco-ooops]

@SirAionTech buona per te?

[SirAionTech]

@SirAionTech buona per te?

A me già non convinceva questa parte di codice che prende tutto quello che c'è nella cache e lo scrive.
Questa PR non mi sembra migliorare questo aspetto, ma già non mi spiegavo il codice di prima quindi non saprei cosa dire di questa modifica.
Forse sarebbe meglio lasciar stare la cache.

[francesco-ooops]

@SirAionTech riusciamo a risalire a chi/quando era stata aggiunta questa funzione?

[SirAionTech]

@SirAionTech riusciamo a risalire a chi/quando era stata aggiunta questa funzione?

Fu aggiunta in 773e1c3 (il commit originale è cb063e9) da @eLBati

[francesco-ooops]

Ok, dunque è lì da un bel po'

@eLBati hai idea dell'utilità di questa funzione?

[eLBati]

_convert_to_write era usato per replicare nell'importazione quello che avviene nell'interfaccia all'esecuzione dell'onchange e scriverlo sul record (altrimenti le modifiche dell'onchange non venivano salvate). Quindi in questo caso serviva a salvare le modifiche fatte da _onchange_invoice_line_wt_ids.

[francesco-ooops]

@eLBati grazie, secondo te come possiamo intervenire per risolvere il problema indicato nella issue?

[francesco-ooops]

@eLBati ping! Che ne pensi di questa PR? Andiamo a fare danni?