Spostare menu RiBa in modo da renderlo visibile anche nella versione Enterprise

[tafaRU]

Module

• 12.0 chiusa in quanto non supportata
• 14.0 Spostare menu RiBa in modo da renderlo visibile anche nella versione Enterprise #2623
• 16.0 [MIG] l10n_it_ricevute_bancarie -> l10n_it_riba #3615

Vedi #2139 e relativo commento qui sotto:

Riporto quanto detto stamattina in chiamata.
Questa PR non è mergiabile perché dipende dal modulo account_accountant che è enterprise, però il problema descritto va in qualche modo risolto.

In generale bisogna fare in modo che il modulo account_accountant sposti anche questo menu.
Visto che il modulo sposta i menu in modo statico semplicemente cambiando il parent dei menu che conosce (https://github.com/odoo/enterprise/blob/693ea0e9f35738a09465f0b628f6678ca8f4e601/account_accountant/data/account_accountant_data.xml#L10-L28), secondo me l'unica soluzione è mettere il menu delle RiBa sotto uno dei menù esistenti e verrà così spostato automaticamente.

Può comunque valere la pena indagare se ci sono altre soluzioni.

Originally posted by @SimoRubi in #2139 (review)

[primes2h]

Una possibile soluzione potrebbe essere #2623 (comment).

[francesco-ooops]

@odooNextev questa è da fare anche per la 16?

[odooNextev]

@odooNextev questa è da fare anche per la 16?

Il codice aggiunto dalla PR è già integrato nel modulo l10n_it_riba, ma non funziona

[francesco-ooops]

allora direi di sì che va aggiunta una checkbox per la 16 @tafaRU

[odooNextev]

A me comunque neanche quella della 14 funziona

[francesco-ooops]

@odooNextev perchè mi sembra ci siano diverse PR ancora aperte, vedi #2623 (comment)

[SirAionTech]

Il commit f1f9e4c di #2623 è stato portato alla 16.0 da #3615 (ora è b028d5b).

Per sicurezza ho anche verificato che la problematica in oggetto non succeda in 16.0, quindi direi che si può chiudere.

[SirAionTech]

@odooNextev questa è da fare anche per la 16?

Il codice aggiunto dalla PR è già integrato nel modulo l10n_it_riba, ma non funziona

Scusa @odooNextev vedo ora questo commento, puoi riportare i passi che hai fatto?
Come scrivevo poco fa ho appena provato e mi sembra funzionare correttamente, i passi che ho fatto sono:

1. Creo nuovo DB in italiano
2. Installo l10n_it_edi
3. Disinstallo l10n_it_edi (perché incompatibile con l10n_it_fatturapa)
4. Installo l10n_it_riba
   -> Il menu delle RiBa è sotto Fatturazione
5. Installo account_accountant
   -> Il menu delle RiBa è sotto Contabilità

[odooNextev]

@SirAionTech se installi account_accountant prima di l10n_it_riba ti genera il menù Accounting arancione

[SirAionTech]

@SirAionTech se installi account_accountant prima di l10n_it_riba ti genera il menù Accounting arancione

Capito, quindi in pratica inverti 4. e 5. di #2526 (comment), io l'ho sempre provato come descritto in #2526 (comment) e non trovavo il problema.

La fix attuale fa in modo che il menu RiBa segua i suoi menu sibling quando vengono spostati, se però viene installato dopo non può farlo.
Per risolvere si potrebbe usare un post_init_hook per spostare il menu RiBa nel caso venga installato quando i menu sono già stati spostati da account_accountant.

[francesco-ooops]

risolta su tutte le versioni

[SirAionTech]

@francesco-ooops il commento #2526 (comment) è un problema aperto, quindi è da risolvere in una nuova issue?

[francesco-ooops]

@SirAionTech il problema riportato da @odooNextev è su entrambe le versioni?

se sì puoi aprire una nuova issue?

[SirAionTech]

@SirAionTech il problema riportato da @odooNextev è su entrambe le versioni?

Penso di sì

se sì puoi aprire una nuova issue?

#3799