Skip to content

[19.0][MIG] l10n_it_intrastat_oca: Migration to 19.0#5089

Open
Borruso wants to merge 88 commits intoOCA:19.0from
Borruso:19.0-mig-l10n_it_intrastat
Open

[19.0][MIG] l10n_it_intrastat_oca: Migration to 19.0#5089
Borruso wants to merge 88 commits intoOCA:19.0from
Borruso:19.0-mig-l10n_it_intrastat

Conversation

@Borruso
Copy link
Copy Markdown
Contributor

@Borruso Borruso commented Feb 6, 2026

Lara Baggio and others added 30 commits January 29, 2026 09:39
Currently translated at 0.4% (42 of 10990 strings)

Translation: l10n-italy-12.0/l10n-italy-12.0-l10n_it_intrastat
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-l10n_it_intrastat/it/
Currently translated at 14.5% (1595 of 10982 strings)

Translation: l10n-italy-12.0/l10n-italy-12.0-l10n_it_intrastat
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-l10n_it_intrastat/it/
Currently translated at 14.5% (1595 of 10982 strings)

Translation: l10n-italy-12.0/l10n-italy-12.0-l10n_it_intrastat
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-l10n_it_intrastat/it/
Investigating an issue reported by Sergio Zanchetta.
Currently translated at 14.5% (1593 of 10990 strings)

Translation: l10n-italy-12.0/l10n-italy-12.0-l10n_it_intrastat
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-l10n_it_intrastat/it/
Currently translated at 14.6% (1601 of 10988 strings)

Translation: l10n-italy-12.0/l10n-italy-12.0-l10n_it_intrastat
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-l10n_it_intrastat/it/
Currently translated at 0.5% (56 of 10988 strings)

Translation: l10n-italy-12.0/l10n-italy-12.0-l10n_it_intrastat
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-l10n_it_intrastat/es/
…n't adapt type 'account.intrastat.transaction.nature' + test
Pre-commit issues:
l10n_it_intrastat/models/account.py:304:5: C901 'AccountInvoice.compute_intrastat_lines' is too complex (17)
l10n_it_intrastat/models/account.py:540: [C8108(method-compute), AccountInvoiceIntrastat] Name of compute method should start with "_compute_"
l10n_it_intrastat/models/account.py:170:96: B950 line too long (95 > 80 characters)
l10n_it_intrastat/models/account.py:176:100: B950 line too long (99 > 80 characters)
l10n_it_intrastat/models/account.py:189:92: B950 line too long (91 > 80 characters)
l10n_it_intrastat/models/account.py:303:43: E741 ambiguous variable name 'l'
l10n_it_intrastat/models/account.py:373:21: B007 Loop control variable 'key' not used within the loop body. If this is intended, start the name with an underscore.
l10n_it_intrastat/models/account.py:380:25: B007 Loop control variable 'key' not used within the loop body. If this is intended, start the name with an underscore.
l10n_it_intrastat/models/account.py:397:17: B007 Loop control variable 'key' not used within the loop body. If this is intended, start the name with an underscore.
l10n_it_intrastat/models/account.py:409:13: E741 ambiguous variable name 'l'
l10n_it_intrastat/tests/test_intrastat.py:74:35: E741 ambiguous variable name 'l'

Migrate old invoice fields and fix tests
- views do no more inherit from anything, otherwise following error might occur:
  Element '<tree string="Codes">' cannot be located in parent view

  View name: views.tree.report.intrastat.code
  Error context:
   view: ir.ui.view(2275,)
   xmlid: view_tree_report_intrastat_code
   view.model: report.intrastat.code
   view.parent: ir.ui.view(2269,)
   file: /path/to/odoo/addons/l10n_it_intrastat/views/intrastat.xml

- intrastat codes might have been moved by enterprise module account_intrastat
Otherwise the following warning is raised:
unknown parameter 'size', if this is an actual parameter you may want to override the method _valid_field_parameter on the relevant model in order to allow it
Currently translated at 14.5% (1601 of 10988 strings)

Translation: l10n-italy-14.0/l10n-italy-14.0-l10n_it_intrastat
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-14-0/l10n-italy-14-0-l10n_it_intrastat/it/
Currently translated at 14.5% (1603 of 10988 strings)

Translation: l10n-italy-14.0/l10n-italy-14.0-l10n_it_intrastat
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-14-0/l10n-italy-14-0-l10n_it_intrastat/it/
@tafaRU
Copy link
Copy Markdown
Member

tafaRU commented Feb 6, 2026

@Borruso ci sarebbe già #5031 ma io personalmente preferirei avere PR separate e quindi mantenere la tua. @eLBati cosa ne pensi?

@Borruso
Copy link
Copy Markdown
Contributor Author

Borruso commented Feb 6, 2026

@Borruso ci sarebba già #5031 ma io personalmente preferirei avere PR separate e quindi mantenere la tua. @eLBati cosa ne pensi?

idem in realtà

@Borruso Borruso force-pushed the 19.0-mig-l10n_it_intrastat branch 7 times, most recently from fc49b38 to 31d8c77 Compare February 7, 2026 16:12
@micheledic
Copy link
Copy Markdown
Contributor

micheledic commented Feb 18, 2026

da circa 2 settimane sulla enterprise è uscito l10n_it_intrastat . Non so se il modulo nativo copre già tutte le casistiche gestite da l10n_it_intrastat OCA, nel caso non le coprisse bisognerebbe fare un rename del modulo al solito l10n_it_intrastat**_oca** con tanto di migration rename

@Borruso
Copy link
Copy Markdown
Contributor Author

Borruso commented Feb 18, 2026

da circa 2 settimane sulla enterprise è uscito l10n_it_intrastat . Non so se il modulo nativo copre già tutte le casistiche gestite da l10n_it_intrastat OCA, nel caso non le coprisse bisognerebbe fare un rename del modulo al solito l10n_it_intrastat**_oca** con tanto di migration rename

grazie mille comunque anche se coprisse per chi usa solo non community non ci sarà nulla quindi sapere da migrare forse fare attenzione ai nomi dei campi, ecc..

@Borruso
Copy link
Copy Markdown
Contributor Author

Borruso commented Feb 18, 2026

da circa 2 settimane sulla enterprise è uscito l10n_it_intrastat . Non so se il modulo nativo copre già tutte le casistiche gestite da l10n_it_intrastat OCA, nel caso non le coprisse bisognerebbe fare un rename del modulo al solito l10n_it_intrastat**_oca** con tanto di migration rename

per lo statement dell'intrastat va aggiunto pure lì _oca?

@micheledic
Copy link
Copy Markdown
Contributor

da circa 2 settimane sulla enterprise è uscito l10n_it_intrastat . Non so se il modulo nativo copre già tutte le casistiche gestite da l10n_it_intrastat OCA, nel caso non le coprisse bisognerebbe fare un rename del modulo al solito l10n_it_intrastat**_oca** con tanto di migration rename

per lo statement dell'intrastat va aggiunto pure lì _oca?

Attualmente non esiste sull Enterprise, ma aggiungerei _oca cosi da rendere coerenti i nomi dei moduli oca

@Borruso Borruso force-pushed the 19.0-mig-l10n_it_intrastat branch from 31d8c77 to 71b33cc Compare February 18, 2026 17:33
@Borruso Borruso changed the title [19.0][MIG] l10n_it_intrastat: Migration to 19.0 [19.0][MIG] l10n_it_intrastat_oca: Migration to 19.0 Feb 18, 2026
@Borruso Borruso force-pushed the 19.0-mig-l10n_it_intrastat branch 4 times, most recently from bbb3003 to 0301185 Compare February 18, 2026 17:55
@Borruso Borruso force-pushed the 19.0-mig-l10n_it_intrastat branch from 0301185 to fd4f33b Compare February 22, 2026 12:38
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.