[12.0][FIX] accounting menu not translatable

[primes2h]

Risolve #2885 per la v.12.0

[SimoRubi]

Il problema è solo che la v12 non è più presente in Transifex?
Perché per v13 ho trovato entrambe le stringhe tradotte (grazie per le traduzioni):

• Invoicing -> Fatturazione: https://www.transifex.com/odoo/odoo-13/translate/#it/account/180726208
• Accounting -> Contabilità: https://www.transifex.com/odoo/odoo-13/translate/#it/account_accountant/180750195

Giusto per capire se è una fix che poi andrà portata anche nelle altre versioni

[primes2h]

Il problema è solo che la v12 non è più presente in Transifex? Perché per v13 ho trovato entrambe le stringhe tradotte (grazie per le traduzioni):

* Invoicing -> Fatturazione: https://www.transifex.com/odoo/odoo-13/translate/#it/account/180726208

* Accounting -> Contabilità: https://www.transifex.com/odoo/odoo-13/translate/#it/account_accountant/180750195

Il problema nasce nella versione Community e non dipende dal fatto che la v12 non è più presente in Transifex.
Nella Enterprise funziona proprio perché in account_accountant viene cambiato il nome del menu e viene quindi generata la corrispondente stringa nel .pot.

La modifica che avevo fatto a suo tempo su l10n_it_account

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>

cambia il termine inglese ma non viene generato il riferimento alla stringa nel .pot.

l10n-italy/l10n_it_account/i18n/l10n_it_account.pot

Lines 38 to 42 in 8187107

	#. module: l10n_it_account
	#: model_terms:ir.ui.view,arch_db:l10n_it_account.product_template_form_view
	#: model_terms:ir.ui.view,arch_db:l10n_it_account.res_config_settings_view_form
	#: model_terms:ir.ui.view,arch_db:l10n_it_account.view_partner_property_form
	msgid "Accounting"

In pratica significa che nella CE in inglese compare correttamente "Accounting", in italiano resta "Fatturazione".

Giusto per capire se è una fix che poi andrà portata anche nelle altre versioni

Direi di si.

[SimoRubi]

Giusto per capire se è una fix che poi andrà portata anche nelle altre versioni

Direi di si.

Ho provato ora nell'ultimo runboat di v14 (http://oca-l10n-italy-14-0-a473de9e38c3.runboat.odoo-community.org/) e il menu è tradotto:

quindi direi che non è da portare in v14.. O forse mi sono perso qualche passaggio?

[primes2h]

Giusto per capire se è una fix che poi andrà portata anche nelle altre versioni

Direi di si.

Ho provato ora nell'ultimo runboat di v14 (http://oca-l10n-italy-14-0-a473de9e38c3.runboat.odoo-community.org/) e il menu è tradotto: quindi direi che non è da portare in v14.. O forse mi sono perso qualche passaggio?

Ma è tradotto con Fatturazione, non con Contabilità come dovrebbe essere dopo l'installazione di l10n_it_account, è proprio quello il problema.

[SimoRubi]

Ma è tradotto con Fatturazione, non con Contabilità come dovrebbe essere dopo l'installazione di l10n_it_account, è proprio quello il problema.

Ah ecco, sospettavo fosse questo il passaggio che mi ero perso :) puoi aggiungerlo come Comportamento atteso in descrizione?

Fatto.

[SimoRubi]

Quindi la root cause in pratica è che Odoo non riporta nei .po* di l10n_it_account la parte per tradurre il nuovo record account.menu_finance modificato (c'è la stringa "Accounting" ma non per il menu).

Esiste un riferimento tra le problematiche di Odoo (https://github.com/odoo/odoo/issues)?

[primes2h]

Quindi la root cause in pratica è che Odoo non riporta nei .po* di l10n_it_account la parte per tradurre il nuovo record account.menu_finance modificato (c'è la stringa "Accounting" ma non per il menu).

Esatto.

Esiste un riferimento tra le problematiche di Odoo (https://github.com/odoo/odoo/issues)?

Non mi sembra.
All'epoca non ho aperto una issue perché pensavo che non avrebbe ricevuto molta attenzione.
Nella Enterprise il nome viene convertito correttamente (in altro modo) quindi è probabile che non abbiano tanto tempo da perderci dietro.

[SimoRubi]

Quindi la root cause in pratica è che Odoo non riporta nei .po* di l10n_it_account la parte per tradurre il nuovo record account.menu_finance modificato (c'è la stringa "Accounting" ma non per il menu).

Esatto.

Esiste un riferimento tra le problematiche di Odoo (https://github.com/odoo/odoo/issues)?

Non mi sembra. All'epoca non ho aperto una issue perché pensavo che non avrebbe ricevuto molta attenzione. Nella Enterprise il nome viene convertito correttamente (in altro modo) quindi è probabile che non abbiano tanto tempo da perderci dietro.

Guardando nel codice, mi sembra che vengano volutamente esportati solo i termini del modulo corrente in https://github.com/odoo/odoo/blob/6b6b11ea2f8391f1f195bd1650531f77b6bed729/odoo/tools/translate.py#L841 quindi penso che il comportamento attuale sia quello atteso da Odoo; è anche vero però che se definisco qualcosa nei dati di un modulo, vorrei che quel modulo potesse tradurlo.

Secondo me il problema è più generico di account vs l10n_it_account vs account_accountant quindi si potrebbe aprire una issue.

A livello di codice questa PR, per quanto sia un work-around stile copia/incolla, mi sembra una soluzione accettabile per il problema che vuoi risolvere.

Sarebbe migliorabile prendendo in modo dinamico gli attributi del menu account.menu_finance, in questo modo elimineresti il copia/incolla, ma non vedo problemi con l'approccio attuale soprattutto perché il menu in v12 non cambierà più visto che non è più mantenuta.

Consiglio di aprire la issue a Odoo almeno per renderli partecipi del comportamento inatteso che hai trovato.

[primes2h]

Guardando nel codice, mi sembra che vengano volutamente esportati solo i termini del modulo corrente in https://github.com/odoo/odoo/blob/6b6b11ea2f8391f1f195bd1650531f77b6bed729/odoo/tools/translate.py#L841 quindi penso che il comportamento attuale sia quello atteso da Odoo; è anche vero però che se definisco qualcosa nei dati di un modulo, vorrei che quel modulo potesse tradurlo.

Secondo me il problema è più generico di account vs l10n_it_account vs account_accountant quindi si potrebbe aprire una issue.

Anche questo è vero, anche se ho poca fiducia nel fatto che la prendano in considerazione. Ma tentar non nuoce.

A livello di codice questa PR, per quanto sia un work-around stile copia/incolla, mi sembra una soluzione accettabile per il problema che vuoi risolvere.

Più che un wordaround mi sembra sia il modo standard che usa Odoo per il cambio di nome dei menu principali, e sembra essere l'unico possibile.

Sarebbe migliorabile prendendo in modo dinamico gli attributi del menu account.menu_finance, in questo modo elimineresti il copia/incolla,

Ricordo di averci provato ma non funzionava.

ma non vedo problemi con l'approccio attuale soprattutto perché il menu in v12 non cambierà più visto che non è più mantenuta.

Neanche nelle altre versioni in teoria, dovrebbe essere un menù piuttosto statico nel codice Odoo standard.

[github-actions]

There hasn't been any activity on this pull request in the past 4 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 PR to never become stale, please ask a PSC member to apply the "no stale" label.