Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[14.0][MIG] Migration: l10n_it_fatturapa #1984

Merged
merged 216 commits into from
Apr 30, 2021

Conversation

TheMule71
Copy link
Contributor

Porting del modulo l10n_it_fattura dalla versione 12.0 alla 14.0

ovviamente per provarlo occorre mergiare le dipendenze

da vedere l'impatto della rimozione di res_partner.customer e res_partner.supplier

--
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

@TheMule71 TheMule71 changed the title 14.0 mig l10n it fatturapa [14.0] Migration: l10n_it_fatturapa Dec 15, 2020
@TheMule71 TheMule71 mentioned this pull request Dec 15, 2020
74 tasks
@TheMule71 TheMule71 force-pushed the 14.0-mig-l10n_it_fatturapa branch 2 times, most recently from b21d9ba to a00249c Compare December 17, 2020 13:46
@TheMule71 TheMule71 changed the title [14.0] Migration: l10n_it_fatturapa [14.0][MIG] Migration: l10n_it_fatturapa Dec 18, 2020
@TheMule71 TheMule71 force-pushed the 14.0-mig-l10n_it_fatturapa branch 2 times, most recently from a87e75b to 7873938 Compare February 24, 2021 18:43
@TheMule71 TheMule71 force-pushed the 14.0-mig-l10n_it_fatturapa branch 2 times, most recently from 2df70a2 to 5aac46c Compare February 27, 2021 16:29
@dcorio
Copy link
Contributor

dcorio commented Mar 4, 2021

Ciao @TheMule71,
a quanto pare i documenti correlati non vengono salvati e/o spariscono.
Non ho ancora investigato sulla causa.

@TheMule71 TheMule71 force-pushed the 14.0-mig-l10n_it_fatturapa branch 6 times, most recently from 1d762d0 to 3148b19 Compare March 19, 2021 17:41
l10n_it_fatturapa/__manifest__.py Outdated Show resolved Hide resolved
l10n_it_fatturapa/models/account.py Outdated Show resolved Hide resolved
l10n_it_fatturapa/migrations/14.0.2.0.3/pre-migrate.py Outdated Show resolved Hide resolved
l10n_it_fatturapa/models/account.py Show resolved Hide resolved
l10n_it_fatturapa/models/ir_attachment.py Outdated Show resolved Hide resolved
l10n_it_fatturapa/views/account_view.xml Show resolved Hide resolved
l10n_it_fatturapa/views/account_view.xml Show resolved Hide resolved
@@ -4,7 +4,7 @@
#
msgid ""
msgstr ""
"Project-Id-Version: Odoo Server 12.0\n"
"Project-Id-Version: Odoo Server 14.0\n"
"Report-Msgid-Bugs-To: \n"
"Last-Translator: <>\n"
"Language-Team: \n"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Installando le traduzioni italiane ottengo una 50ina di log del tipo:

2021-04-02 13:27:45,371 73947 INFO odoo14-l10n_it_fatturapa odoo.tools.translate: Skipped deprecated occurrence selection:fatturapa.related_document_type,type

Direi che è dovuto a odoo/odoo@7593b88, puoi correggere o rigenerare le traduzioni?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mi è stato detto di non toccarle la traduzioni, che vengono rigenerate poi dal bot. In questo caso, il cambio 12.0 -> 14.0 era solo per via del grep che trova tutte le occorrenze di 12.0 credo, che ho modificato. Se ho capito male e il .pot non viene rigenerato dal bot, me lo rifo io.

Copy link
Member

@SimoRubi SimoRubi Apr 6, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

È vero che è meglio evitare di modificare manualmente le traduzioni quando si modifica un modulo, ma questo in pratica è un modulo nuovo perché in v14 non esiste quindi secondo me si potrebbe fare.

Quando te l'ho scritto pensavo fosse un WARN quindi pensavo potesse far fallire il job di runbot ma vedo ora che è un INFO quindi non dovrebbe dare problemi.

Temo che la rigenerazione delle traduzioni da parte di ocabot possa farne saltare un po' mentre la rigenerazione manuale ti permetterebbe di ripristinarle; ad ogni modo, ok anche lasciarle così, vedremo se il bot è così bravo da rimuoverci questi log :)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

, ma questo in pratica è un modulo nuovo perché in v14 non esiste quindi secondo me si potrebbe fare

Confermo

l10n_it_fatturapa/views/partner_view.xml Outdated Show resolved Hide resolved
l10n_it_fatturapa/views/account_view.xml Show resolved Hide resolved
@SimoRubi
Copy link
Member

SimoRubi commented Apr 2, 2021

Ciao @TheMule71,
a quanto pare i documenti correlati non vengono salvati e/o spariscono.
Non ho ancora investigato sulla causa.

Il motivo per cui i related_documents (e admin_ref) non vengono salvati è che il core esclude le modifiche a invoice_line_ids in https://github.com/odoo/odoo/blob/d44dfff647939022b8fe8d091bc1f6562cc47d86/addons/account/models/account_move.py#L1872.
Per qualche motivo i related_documents vengono passati come modifica a invoice_line_ids mentre ad esempio le imposte sono passate comunque come modifiche a line_ids:
image
Abbiamo fatto un po' di indagini su questo problema senza arrivare a una soluzione.
Sembra ci sia qualche meccanismo oscuro per cui dei campi vengono copiati da invoice_line_ids a line_ids (vedi ad esempio i metodi create, write e _onchange_invoice_line_ids di account.move.line).

@TheMule71 TheMule71 force-pushed the 14.0-mig-l10n_it_fatturapa branch 3 times, most recently from 3105989 to d3429e3 Compare April 2, 2021 22:32
@TheMule71
Copy link
Contributor Author

Abbiamo fatto un po' di indagini su questo problema senza arrivare a una soluzione.
Sembra ci sia qualche meccanismo oscuro per cui dei campi vengono copiati da invoice_line_ids a line_ids (vedi ad esempio i metodi create, write e _onchange_invoice_line_ids di account.move.line).

In vista di questo, ho aggiunto il tag development_level a alpha, che da come leggo io dovrebbe indicare proprio i moduli che hanno problemi/incompleti. Ciò dovrebbe consentire il merge anche se c'è un bug. Ovviamente l'ideale sarebbe trovare la soluzione :)

@TheMule71 TheMule71 force-pushed the 14.0-mig-l10n_it_fatturapa branch 2 times, most recently from 6dd6997 to caf68d2 Compare April 9, 2021 14:49
Copy link
Member

@SimoRubi SimoRubi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Grazie delle modifiche, per me ok 👍🏻

@TheMule71 TheMule71 deleted the 14.0-mig-l10n_it_fatturapa branch April 30, 2021 16:17
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Apr 30, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jun 6, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jun 18, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jun 18, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 2, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 2, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 17, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 20, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 30, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 30, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Aug 6, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Aug 20, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Sep 3, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Sep 17, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Oct 1, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Oct 8, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Oct 15, 2021
TheMule71 added a commit to TheMule71/l10n-italy that referenced this pull request Jan 21, 2022
TheMule71 added a commit to TheMule71/l10n-italy that referenced this pull request Jan 21, 2022
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jun 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.