Refactoring cespiti per correzioni vari bug (assets_management)

[sergiocorato]

Module

assets_management

Describe the bug

Riassumo in questa issue le varie problematiche risolte in varie PR:

• bottone 'Vedi' su report cespiti e report cespiti previsionale non funzionante:
  • 14.0: [14.0][FIX] assets_management view html report #3348
  • 16.0: integrata in [MIG] assets_management -> l10n_it_asset_management: Migration to 16.0 #3776
• rimozione dalle stampe dei cespiti venduti, con opzione per includerli:
  • 14.0: [FIX][14.0] assets_management Aggiunta possibilità di non stampare i cespiti venduti prima della data di stampa #3330
  • 16.0: integrata in [MIG] assets_management -> l10n_it_asset_management: Migration to 16.0 #3776
• dismissione di un cespite (distruzione) senza vendita:
  • 14.0: [14.0][IMP] assets_management: add ability to dismiss directly an asset #3239
  • 16.0: integrata in [MIG] assets_management -> l10n_it_asset_management: Migration to 16.0 #3776
• rimozione diretta delle righe contabili collegate a fatture senza dover risalire alla fattura:
  • 14.0: [14.0][IMP][assets_management] Rimosso errore inutile quando si cancella un ammortamento #3238
  • 16.0: integrata in [MIG] assets_management -> l10n_it_asset_management: Migration to 16.0 #3776
• corretta funzione di verifica data dismissione:
  • 14.0: [14.0][FIX] assets_management get only not null dates #3346 (issue Errore per date 'False' nella stampa del registro cespiti #3170)
  • 16.0: [MIG] assets_management -> l10n_it_asset_management: Migration to 16.0 #3776
• corretta funzione di generazione ammortamenti in data:
  • 14.0: [FIX][14.0] assets_management depreciable amount computation by date #3316 (issue Calcolo di ammortamenti su cespiti completamente ammortizzati con incremento valore successivo #3318)
  • 16.0: integrata in [MIG] assets_management -> l10n_it_asset_management: Migration to 16.0 #3776
• rimossa la creazione di righe ammortamento a 0:
  • 14.0: [14.0][FIX] assets_management do not create depreciation lines with zero amount #3125
  • 16.0: [MIG] assets_management -> l10n_it_asset_management: Migration to 16.0 #3776
• corretta funzione di conversione valuta:
  • 14.0: [14.0][FIX] assets_management currency convert function #3349
  • 16.0: integrata in [MIG] assets_management -> l10n_it_asset_management: Migration to 16.0 #3776
• aggiunte funzioni di estrazione movimenti in e out:
  • 14.0: [14.0][FIX] assets_management get in-out values and minor fixes #3350 non so l'utilità di queste funzioni, da riprendere in mano se servisse
  • 16.0: non so l'utilità di queste funzioni, da riprendere in mano se servisse
• correct permission management:
  • 14.0: [FIX] assets_management: correct permission management #3281
  • 16.0: integrata in [MIG] assets_management -> l10n_it_asset_management: Migration to 16.0 #3776
• includere nella stampa cespiti non ancora ammortizzati:
  • 14.0: [14.0][FIX] assets_management: includere nella stampa cespiti non ancora ammortizzati #3393
  • 16.0: [MIG] assets_management -> l10n_it_asset_management: Migration to 16.0 #3776

To Reproduce

Affected versions:

• 16.0: [MIG] assets_management -> l10n_it_asset_management: Migration to 16.0 #3776

Expected behavior

Additional context
Ho caricato anche una PR stile 'SUPERMERGE' per facilitare i test qui: #3345

[aleuffre]

Posso aggiungere questa tra le problematiche? I model sono aperti a tutti senza permessi di sicurezza

#3281

[matteoopenf]

@sergiocorato per caso per quelle approvate si potrebbe procedere al merge?

[odooNextev]

@sergiocorato puoi linkare la PR nella casella della 16: #3271
Ho già integrato le modifiche della #3348 e #3349

[odooNextev]

Ho integrato nella PR della migrazione alla 16 anche questa funzionalità: #3330

[sergiocorato]

Posso aggiungere questa tra le problematiche? I model sono aperti a tutti senza permessi di sicurezza

#3281

Fatto, grazie.

[sergiocorato]

@sergiocorato puoi linkare la PR nella casella della 16: #3271 Ho già integrato le modifiche della #3348 e #3349

Fatto (compreso la #3330 ), grazie.

[sergiocorato]

@sergiocorato per caso per quelle approvate si potrebbe procedere al merge?

Fatto

[stenext]

Ho integrato nella migrazione alla 16 anche #3238

[stenext]

Ho integrato nella migrazione alla 16 anche #3316
Ora faccio un test dello script di migrazione del DB e poi pubblico questi commit

[francesco-ooops]

@sergiocorato se ci sono nuovi comportamenti importanti introdotti da queste PR, potresti aggiungerle alla descrizione del modulo? giusto per non lasciare al buio i funzionali. Grazie!

[odooNextev]

Ho integrato nella migrazione alla 16 anche questa #3239

[odooNextev]

@sergiocorato vuoi aggiungere qui il tracciamento di questo fix a .format(line_type.name) o devo creare una nuova issue?
#3271 (comment)

[odooNextev]

Ho integrato nella migrazione anche #3281

[francesco-ooops]

@sergiocorato #3393 dovrebbe essere aggiunta a questa lista?

[sergiocorato]

@sergiocorato #3393 dovrebbe essere aggiunta a questa lista?

giusto

[sergiocorato]

probabilmente si possono rimuovere i link alla v. 12.0

[francesco-ooops]

probabilmente si possono rimuovere i link alla v. 12.0

fatto!

@SirAionTech confermi che tutte le PR mergiate per la 14 sono presenti nella tua PR di migrazione alla 16?
o anche alcune non mergiate?

[SirAionTech]

probabilmente si possono rimuovere i link alla v. 12.0

fatto!

@SirAionTech confermi che tutte le PR mergiate per la 14 sono presenti nella tua PR di migrazione alla 16?

Tutto ciò che è mergiato in 14.0 è anche nella migrazione alla 16.0 #3776.

o anche alcune non mergiate?

Nella migrazione ci sono anche le fix per #3009 e #3775 (che non sono in questa issue).

[SirAionTech]

Incluse anche #3346 #3393 #3125 in #3776