[12.0] [FIX] translation source terms and readme

[primes2h]

Descrizione del problema o della funzionalità:

Moduli l10n_it_intrastat e l10n_it_intrastat_statement

1. Alcune stringhe in inglese non erano uniformi nel contesto (e con [11.0] ADD intrastat modules #1283).
2. I README presentavano alcuni problemi di formattazione e leggibilità, es. questo elenco puntato tutto in una riga:

P.S.: nel README di l10n_it_intrastat_statement (USAGE.rst) ci sono due link rotti perché mancano le immagini corrispondenti.
https://github.com/OCA/l10n-italy/tree/12.0/l10n_it_intrastat_statement
@labaggio @glauco70 se me le fornite le posso aggiungere alla PR.

P.S. 2: nel .pot compaiono diverse stringhe che non sono presenti né nel codice di questi moduli, né nel codice di altri moduli OCA o Odoo.
Sembrano vecchie stringhe che per qualche ragione non sono state eliminate durante la rigenerazione del .pot.

Es.
Purchase Section1 Operation Amount
Purchase Section1 Operation Number
Purchase Section2 Operation Amount
Purchase Section2 Operation Number ecc...

Sale Section1 Operation Amount
Sale Section1 Operation Number
Sale Section2 Operation Amount
Sale Section2 Operation Number ecc...

@OCA/local-italy-developers Idee?

--
Confermo di aver firmato il CLA https://odoo-community.org/page/cla e di aver letto le linee guida su https://odoo-community.org/page/contributing

[glauco70]

P.S.: nel README di l10n_it_intrastat_statement (USAGE.rst) ci sono due link rotti perché mancano le immagini corrispondenti.
https://github.com/OCA/l10n-italy/tree/12.0/l10n_it_intrastat_statement
@labaggio @glauco70 se me le fornite le posso aggiungere alla PR.

@primes2h non ho idea di che immagini siano, secondo me si possono togliere

[primes2h]

P.S.: nel README di l10n_it_intrastat_statement (USAGE.rst) ci sono due link rotti perché mancano le immagini corrispondenti.
https://github.com/OCA/l10n-italy/tree/12.0/l10n_it_intrastat_statement
@labaggio @glauco70 se me le fornite le posso aggiungere alla PR.

@primes2h non ho idea di che immagini siano, secondo me si possono togliere

Ottimo, fatto!

[primes2h]

P.S. 2: nel .pot compaiono diverse stringhe che non sono presenti né nel codice di questi moduli, né nel codice di altri moduli OCA o Odoo.
Sembrano vecchie stringhe che per qualche ragione non sono state eliminate durante la rigenerazione del .pot.

Es.
Purchase Section1 Operation Amount
Purchase Section1 Operation Number
Purchase Section2 Operation Amount
Purchase Section2 Operation Number ecc...

Sale Section1 Operation Amount
Sale Section1 Operation Number
Sale Section2 Operation Amount
Sale Section2 Operation Number ecc...

@OCA/local-italy-developers Idee?

Ho risolto, tutte quelle stringhe comparivano perché ai relativi campi (es. sale_section1_operation_amount) mancava l'attributo string.

Adesso dovrebbe essere ok.

[glauco70]

P.S. 2: nel .pot compaiono diverse stringhe che non sono presenti né nel codice di questi moduli, né nel codice di altri moduli OCA o Odoo.
Sembrano vecchie stringhe che per qualche ragione non sono state eliminate durante la rigenerazione del .pot.
Es.
Purchase Section1 Operation Amount
Purchase Section1 Operation Number
Purchase Section2 Operation Amount
Purchase Section2 Operation Number ecc...
Sale Section1 Operation Amount
Sale Section1 Operation Number
Sale Section2 Operation Amount
Sale Section2 Operation Number ecc...
@OCA/local-italy-developers Idee?

Ho risolto, tutte quelle stringhe comparivano perché ai relativi campi (es. sale_section1_operation_amount) mancava l'attributo string.

Adesso dovrebbe essere ok.

Avevo tolto la string dal python e inserita nel file XML per evitare questi warning della 12 :
2019-12-17 09:08:21,746 172 WARNING openerp_test odoo.addons.base.models.ir_model: Two fields (sale_section2_operation_amount, sale_section1_operation_amount) of account.intrastat.statement() have the same label: Operation Amount.

[primes2h]

Avevo tolto la string dal python e inserita nel file XML per evitare questi warning della 12 :
2019-12-17 09:08:21,746 172 WARNING openerp_test odoo.addons.base.models.ir_model: Two fields (sale_section2_operation_amount, sale_section1_operation_amount) of account.intrastat.statement() have the same label: Operation Amount.

Ah ok, infatti ora vedo che sono ricomparsi, ma non sono bloccanti direi.
IMHO, sempre meglio di avere delle stringhe di traduzione inutili "inventate" dal sistema :-D

[primes2h]

Più che altro trovo strano che venga segnato ❌ su runbot per questo tipo di warning, anche perché viene comunque creato correttamente.

[GSLabIt]

LGTM

[tafaRU]

@primes2h, gli stessi WARNING sono presenti anche sulle build di Travis. Non è possibile differenziare quelle label?

[primes2h]

@primes2h, gli stessi WARNING sono presenti anche sulle build di Travis. Non è possibile differenziare quelle label?

Ho tolto le lettere maiuscole nei termini successivi al primo.

Adesso forse ho capito perché in Transifex ogni tanto trovo le stesse stringhe sia nella versione "tutte prime lettere maiuscole" che nella versione "solo prima lettera maiuscola". 😁

[primes2h]

@primes2h, gli stessi WARNING sono presenti anche sulle build di Travis. Non è possibile differenziare quelle label?

Ho tolto le lettere maiuscole nei termini successivi al primo.

Adesso forse ho capito perché in Transifex ogni tanto trovo le stesse stringhe sia nella versione "tutte prime lettere maiuscole" che nella versione "solo prima lettera maiuscola". grin

...anche se trovo questi warning piuttosto bizzarri.

[primes2h]

@primes2h, gli stessi WARNING sono presenti anche sulle build di Travis. Non è possibile differenziare quelle label?

Ho tolto le lettere maiuscole nei termini successivi al primo.

Adesso forse ho capito perché in Transifex ogni tanto trovo le stesse stringhe sia nella versione "tutte prime lettere maiuscole" che nella versione "solo prima lettera maiuscola". grin

Come non detto, vedo di risolvere in altro modo.

[primes2h]

Adesso è ok, potete procedere con il merge. Grazie!

[tafaRU]

/ocabot merge minor

[OCA-git-bot]

On my way to merge this fine PR!
Prepared branch 12.0-ocabot-merge-pr-1546-by-tafaRU-bump-minor, awaiting test results.

[OCA-git-bot]

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