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

[SimoRubi]

Risolve #2526 per 14.0.

Sostituisce #2139.

[primes2h]

@SimoRubi
Penso che questa PR(#2675) possa risolvere in modo più semplice.
È per la 12.0 ma penso basti portare la stessa logica nella 14.0.

Sarà poi necessaria un'altra PR che modifichi questa riga

l10n-italy/l10n_it_ricevute_bancarie/views/riba_view.xml

Line 4 in dbb00ab

<menuitem name="C/O" parent="account.menu_finance" id="menu_riba" />

parent dovrà puntare a l10n_it_account.menu_account

[SimoRubi]

@SimoRubi Penso che questa PR(#2675) possa risolvere in modo più semplice. È per la 12.0 ma penso basti portare la stessa logica nella 14.0.

Sarà poi necessaria un'altra PR che modifichi questa riga

l10n-italy/l10n_it_ricevute_bancarie/views/riba_view.xml

Line 4 in dbb00ab

<menuitem name="C/O" parent="account.menu_finance" id="menu_riba" />

parent dovrà puntare a l10n_it_account.menu_account

Quindi sposti tutti i menu di account sotto un nostro nuovo menu l10n_it_account.menu_account, in pratica è la stessa cosa che fa account_accountant.

Però non capisco come possa risolvere il problema in oggetto: installando account_accountant il menu RiBa rimarrà ancorato al nostro nuovo menu l10n_it_account.menu_account, mentre tutti gli altri verranno spostati sotto il menu creato da account_accountant no?
Forse mi sono perso qualche pezzo?

[primes2h]

@SimoRubi Penso che questa PR(#2675) possa risolvere in modo più semplice. È per la 12.0 ma penso basti portare la stessa logica nella 14.0.
Sarà poi necessaria un'altra PR che modifichi questa riga

l10n-italy/l10n_it_ricevute_bancarie/views/riba_view.xml

Line 4 in dbb00ab

<menuitem name="C/O" parent="account.menu_finance" id="menu_riba" />

parent dovrà puntare a l10n_it_account.menu_account

Quindi sposti tutti i menu di account sotto un nostro nuovo menu l10n_it_account.menu_account, in pratica è la stessa cosa che fa account_accountant.

Però non capisco come possa risolvere il problema in oggetto: installando account_accountant il menu RiBa rimarrà ancorato al nostro nuovo menu l10n_it_account.menu_account, mentre tutti gli altri verranno spostati sotto il menu creato da account_accountant no?

Ma è un'ipotesi o hai testato la cosa?
Da una prima verifica sembra funzionare, l'unica differenza è che con il modulo RiBa installato resta l'icona del modulo account e non quella di account_accountant.

L'alternativa sarebbe quella di usare lo stesso nome menù usato da account_accountant. il risultato è lo stesso ma non so se implichi eventuali problemi di licenza.

[primes2h]

@SimoRubi Penso che questa PR(#2675) possa risolvere in modo più semplice. È per la 12.0 ma penso basti portare la stessa logica nella 14.0.
Sarà poi necessaria un'altra PR che modifichi questa riga

l10n-italy/l10n_it_ricevute_bancarie/views/riba_view.xml

Line 4 in dbb00ab

<menuitem name="C/O" parent="account.menu_finance" id="menu_riba" />

parent dovrà puntare a l10n_it_account.menu_account

Quindi sposti tutti i menu di account sotto un nostro nuovo menu l10n_it_account.menu_account.

È l'unica maniera che ho trovato per rendere traducibile il campo.
Attualmente in inglese compare "Accounting", in italiano "Fatturazione".

A suo tempo ho provato anche a risolvere in altri modi, tutti miseramente falliti.

[SimoRubi]

@SimoRubi Penso che questa PR(#2675) possa risolvere in modo più semplice. È per la 12.0 ma penso basti portare la stessa logica nella 14.0.
Sarà poi necessaria un'altra PR che modifichi questa riga

l10n-italy/l10n_it_ricevute_bancarie/views/riba_view.xml

Line 4 in dbb00ab

<menuitem name="C/O" parent="account.menu_finance" id="menu_riba" />

parent dovrà puntare a l10n_it_account.menu_account

Quindi sposti tutti i menu di account sotto un nostro nuovo menu l10n_it_account.menu_account, in pratica è la stessa cosa che fa account_accountant.
Però non capisco come possa risolvere il problema in oggetto: installando account_accountant il menu RiBa rimarrà ancorato al nostro nuovo menu l10n_it_account.menu_account, mentre tutti gli altri verranno spostati sotto il menu creato da account_accountant no?

Ma è un'ipotesi o hai testato la cosa?

È un'ipotesi, forse mi potevi chiarire come farebbe a funzionare perché a prima vista non l'ho capito; quando ho tempo lo provo.

[primes2h]

@SimoRubi Penso che questa PR(#2675) possa risolvere in modo più semplice. È per la 12.0 ma penso basti portare la stessa logica nella 14.0.
Sarà poi necessaria un'altra PR che modifichi questa riga

l10n-italy/l10n_it_ricevute_bancarie/views/riba_view.xml

Line 4 in dbb00ab

<menuitem name="C/O" parent="account.menu_finance" id="menu_riba" />

parent dovrà puntare a l10n_it_account.menu_account

Quindi sposti tutti i menu di account sotto un nostro nuovo menu l10n_it_account.menu_account, in pratica è la stessa cosa che fa account_accountant.
Però non capisco come possa risolvere il problema in oggetto: installando account_accountant il menu RiBa rimarrà ancorato al nostro nuovo menu l10n_it_account.menu_account, mentre tutti gli altri verranno spostati sotto il menu creato da account_accountant no?

Ma è un'ipotesi o hai testato la cosa?

È un'ipotesi, forse mi potevi chiarire come farebbe a funzionare perché a prima vista non l'ho capito; quando ho tempo lo provo.

Anch'io inizialmente ho avuto lo stesso dubbio, ma sembra funzionare (anche se non ho avuto modo di approfondire il motivo).
Un'ulteriore test è molto ben accetto.

P.S: tra l'altro anche la modifica qui sotto, in teoria, avrebbe dovuto rendere traducibile il termine ma non funziona.
Con Odoo ho smesso di chiedermi molte cose. 😄

l10n-italy/l10n_it_account/views/account_menuitem.xml

Lines 6 to 8 in 8187107

	<record id="account.menu_finance" model="ir.ui.menu">
	<field name="name">Accounting</field>
	</record>

[SimoRubi]

@SimoRubi Penso che questa PR(#2675) possa risolvere in modo più semplice. È per la 12.0 ma penso basti portare la stessa logica nella 14.0.
Sarà poi necessaria un'altra PR che modifichi questa riga

l10n-italy/l10n_it_ricevute_bancarie/views/riba_view.xml

Line 4 in dbb00ab

<menuitem name="C/O" parent="account.menu_finance" id="menu_riba" />

parent dovrà puntare a l10n_it_account.menu_account

Quindi sposti tutti i menu di account sotto un nostro nuovo menu l10n_it_account.menu_account, in pratica è la stessa cosa che fa account_accountant.
Però non capisco come possa risolvere il problema in oggetto: installando account_accountant il menu RiBa rimarrà ancorato al nostro nuovo menu l10n_it_account.menu_account, mentre tutti gli altri verranno spostati sotto il menu creato da account_accountant no?

Ma è un'ipotesi o hai testato la cosa?

È un'ipotesi, forse mi potevi chiarire come farebbe a funzionare perché a prima vista non l'ho capito; quando ho tempo lo provo.

Ora ho testato #2675 con anche la modifica del menu delle RiBa che hai suggerito (il codice è in https://github.com/SimoRubi/l10n-italy/tree/12.0-fix-accounting-menu) e non mi pare risolvere il problema in oggetto.

Installando il modulo l10n_it_ricevute_bancarie i menu sono:

Dopo aver installato account_accountant i menu diventano:

dove:

• Accounting è il nuovo menu aggiunto da l10n_it_account e contiene solo il menu delle RiBa
• Contabilità è il menu aggiunto da account_accountant e contiene tutti i menu che prima erano in Accounting, tranne il menu delle RiBa

In pratica il vecchio Fatturazione è diventato Accounting, ma per il resto il comportamento descritto in #2526 non cambia quindi direi che #2675 non può sostituire questa PR.

[primes2h]

In pratica il vecchio Fatturazione è diventato Accounting, ma per il resto il comportamento descritto in #2526 non cambia quindi direi che #2675 non può sostituire questa PR.

Non avendo accesso diretto alla Enterprise mi mancava solo quest'ultimo test che hai fatto tu. 😞

Riesci per cortesia a fare un ultimo tentativo?
Prova a mettere menu_accounting al posto di menu_account sia in l10n_it_account che in l10n_it_ricevute_bancarie, potrebbe (forse) funzionare.

[SimoRubi]

In pratica il vecchio Fatturazione è diventato Accounting, ma per il resto il comportamento descritto in #2526 non cambia quindi direi che #2675 non può sostituire questa PR.

Non avendo accesso diretto alla Enterprise mi mancava solo quest'ultimo test che hai fatto tu. 😞

Riesci per cortesia a fare un ultimo tentativo? Prova a mettere menu_accounting al posto di menu_account sia in l10n_it_account che in l10n_it_ricevute_bancarie, potrebbe (forse) funzionare.

Ho aggiornato il codice in https://github.com/SimoRubi/l10n-italy/tree/12.0-fix-accounting-menu come hai suggerito, ma il risultato è lo stesso descritto in #2623 (comment)

quindi direi che #2675 non può sostituire questa PR.

[primes2h]

quindi direi che #2675 non può sostituire questa PR.

Ok, grazie mille per il test.

Al momento quindi l'unica soluzione per risolvere entrambi i problemi è quella di utilizzare la #2675 insieme alla tua PR, modificando anche il parent del menu riba. (l'ho appena fatto qui)

[SimoRubi]

@Borruso @primes2h potete verificare se questa modifica risolve il problema?
@OCA/local-italy-maintainers qualcuno può verificare se va bene?

[primes2h]

@Borruso @primes2h potete verificare se questa modifica risolve il problema? @OCA/local-italy-maintainers qualcuno può verificare se va bene?

@SimoRubi
L'ho provata in locale e funziona, ma non avendo accesso diretto a una Enterprise standard il mio test non è esaustivo.

[Borruso]

@Borruso @primes2h potete verificare se questa modifica risolve il problema? @OCA/local-italy-maintainers qualcuno può verificare se va bene?

@SimoRubi la modifica funziona
se installo solo ricevute bancarie installa account, mentre dopo l'installazione se installo account_accountant allora il menu viene spostato

[primes2h]

Al momento quindi l'unica soluzione per risolvere entrambi i problemi è quella di utilizzare la #2675 insieme alla tua PR, modificando anche il parent del menu riba. (l'ho appena fatto qui)

@SimoRubi @Borruso
Ho aperto #2881 per la v. 14.0 (che corrisponde alla #2675 della 12.0) che, abbinata alla presente PR dovrebbe risolvere in modo definitivo il problema.

[primes2h]

Al momento quindi l'unica soluzione per risolvere entrambi i problemi è quella di utilizzare la #2675 insieme alla tua PR, modificando anche il parent del menu riba. (l'ho appena fatto qui)

@SimoRubi @Borruso Ho aperto #2881 per la v. 14.0 (che corrisponde alla #2675 della 12.0) che, abbinata alla presente PR dovrebbe risolvere in modo definitivo il problema.

Sorry, @SirTakobi

[SirTakobi]

Al momento quindi l'unica soluzione per risolvere entrambi i problemi è quella di utilizzare la #2675 insieme alla tua PR, modificando anche il parent del menu riba. (l'ho appena fatto qui)

@SimoRubi @Borruso Ho aperto #2881 per la v. 14.0 (che corrisponde alla #2675 della 12.0) che, abbinata alla presente PR dovrebbe risolvere in modo definitivo il problema.

Sorry, @SirTakobi

Ho ripreso un po' le fila delle varie issue/PR, non è stato proprio banale, @primes2h fammi sapere se scrivo qualche cavolata.
In pratica ci sono 2 problemi:

1. Spostamento del menu RiBa all'installazione di Enterprise (più precisamente account_accountant): il problema è descritto in Spostare menu RiBa in modo da renderlo visibile anche nella versione Enterprise #2526 e questa PR lo risolve per 14.0 (è in descrizione della PR).
2. Traduzione del menu Accounting/Invoicing: il problema è descritto in [12.0][FIX] accounting menu not translatable #2675 e risolto in [12.0][FIX] accounting menu not translatable #2675 (per 12.0) e in [14.0][FIX] accounting menu not translatable #2881 (per 14.0).
   Per rendere il processo più simile a quanto descritto in https://github.com/OCA/l10n-italy/wiki/Team-di-sviluppo#apertura-issue, @primes2h potresti aprire una issue di tracciamento?

Mi pare che i problemi siano indipendenti e cercherei di lasciarli per quanto possibile come tali; perché mi sa che in questo e altri thread si possa fraintendere quale sia il problema.

Per quanto riguarda il tuo ultimo messaggio @primes2h, ti ho lasciato una review in #2881 (review)

[primes2h]

Al momento quindi l'unica soluzione per risolvere entrambi i problemi è quella di utilizzare la #2675 insieme alla tua PR, modificando anche il parent del menu riba. (l'ho appena fatto qui)

@SimoRubi @Borruso Ho aperto #2881 per la v. 14.0 (che corrisponde alla #2675 della 12.0) che, abbinata alla presente PR dovrebbe risolvere in modo definitivo il problema.

Sorry, @SirTakobi

Ho ripreso un po' le fila delle varie issue/PR, non è stato proprio banale, @primes2h fammi sapere se scrivo qualche cavolata. In pratica ci sono 2 problemi:

1. Spostamento del menu RiBa all'installazione di Enterprise (più precisamente account_accountant): il problema è descritto in Spostare menu RiBa in modo da renderlo visibile anche nella versione Enterprise #2526 e questa PR lo risolve per 12.0 (è in descrizione della PR).

Questa PR lo risolve per la 14.0 non per la 12.0

2. Traduzione del menu Accounting/Invoicing all'installazione di Enterprise: il problema è descritto in [12.0][FIX] accounting menu not translatable #2675 e risolto in [12.0][FIX] accounting menu not translatable #2675 (per 12.0) e in [14.0][FIX] accounting menu not translatable #2881 (per 14.0).

Non proprio, il problema non è relativo all'Enterprise.
Cerco di riassumere.

Comportamento standard di Odoo:
In una Odoo CE il menù è Invoicing → Fatturazione.
In una Enterprise, dopo l'installazione di account_accountant, il menù diventa Accounting → Contabilità.

L'obiettivo è fare in modo che in una Odoo CE il menù diventi Accounting → Contabilità dopo aver installato l10n_it_account (contabilità base di l10n-italy).
Attualmente questa parte di codice https://github.com/OCA/l10n-italy/blob/14.0/l10n_it_account/views/account_menuitem.xml#L5-L8 raggiunge parzialmente l'obiettivo.
In inglese il menù diventa correttamente Accounting ma in Italiano resta Fatturazione, perché non viene generata la relativa stringa nel file .pot #2675 (comment)

Dopo vari tentativi falliti sono arrivato alla conclusione che l'unico modo possibile per ottenere il risultato voluto sia di utilizzare la stessa tecnica presente in account_accountant( che è quella utilizzata nelle PR #2675 v12 e #2881 v14).

Per rendere il processo più simile a quanto descritto in https://github.com/OCA/l10n-italy/wiki/Team-di-sviluppo#apertura-issue, @primes2h potresti aprire una issue di tracciamento?

Fatto #2885

Mi pare che i problemi siano indipendenti e cercherei di lasciarli per quanto possibile come tali; perché mi sa che in questo e altri thread si possa fraintendere quale sia il problema.

I problemi sono indipendenti ed è corretto cha abbiano issue e PR separate.
La valutazione però deve comunque essere complessiva dal momento che entrambi coinvolgono il menù delle Riba.

Per quanto riguarda il tuo ultimo messaggio @primes2h, ti ho lasciato una review in #2881 (review)

Grazie.

[SirTakobi]

Al momento quindi l'unica soluzione per risolvere entrambi i problemi è quella di utilizzare la #2675 insieme alla tua PR, modificando anche il parent del menu riba. (l'ho appena fatto qui)

@SimoRubi @Borruso Ho aperto #2881 per la v. 14.0 (che corrisponde alla #2675 della 12.0) che, abbinata alla presente PR dovrebbe risolvere in modo definitivo il problema.

Sorry, @SirTakobi

Ho ripreso un po' le fila delle varie issue/PR, non è stato proprio banale, @primes2h fammi sapere se scrivo qualche cavolata. In pratica ci sono 2 problemi:

1. Spostamento del menu RiBa all'installazione di Enterprise (più precisamente account_accountant): il problema è descritto in Spostare menu RiBa in modo da renderlo visibile anche nella versione Enterprise #2526 e questa PR lo risolve per 12.0 (è in descrizione della PR).

Questa PR lo risolve per la 14.0 non per la 12.0

Grazie, ho corretto

2. Traduzione del menu Accounting/Invoicing all'installazione di Enterprise: il problema è descritto in [12.0][FIX] accounting menu not translatable #2675 e risolto in [12.0][FIX] accounting menu not translatable #2675 (per 12.0) e in [14.0][FIX] accounting menu not translatable #2881 (per 14.0).

Non proprio, il problema non è relativo all'Enterprise.

Corretto, grazie.

[TheMule71]

LGTM

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

[tafaRU]

/ocabot merge patch

[OCA-git-bot]

What a great day to merge this nice PR. Let's do it!
Prepared branch 14.0-ocabot-merge-pr-2623-by-tafaRU-bump-patch, awaiting test results.

[OCA-git-bot]

Congratulations, your PR was merged at a9d0396. Thanks a lot for contributing to OCA. ❤️