[18.0][MIG] l10n_it_financial_statements_report: Migration to 18.0 #4789

micheledic · 2025-06-03T06:48:15Z

No description provided.

micheledic · 2025-06-03T08:48:32Z

Ho introdotto con questa PR anche un metodo per il campo account_ids che sulla 18.0 restituisce "cannot be searchable" Issue OCA/account-financial-reporting#1306
Lo stesso l'ho proposto sul modulo originario, quindi nel caso venisse approvato tolgo e ripunto ad account_ids
OCA/account-financial-reporting#1327

eLBati

Grazie!

l10n_it_financial_statements_report/migrations/18.0.0.0.1/post-migrate.py

eLBati · 2025-06-04T09:50:39Z

l10n_it_financial_statements_report/report/templates/financial_statements_report.xml

                    <t t-set="company" t-value="o.company_id" />
                    <link
-                        href="/account_financial_report/static/src/css/report.css"
+                        href="/l10n_it_financial_statements_report/static/src/css/report.css"


Come mai non usiamo più quello di account_financial_report?

il file scss di account_financial_report viene incluso solo nel bundle assets_backend , stampando il pdf non si renderizza correttamente in quanto non è nel bundle report_assets_common e per procedere comunque alla migrazione ho incluso questo report nel modulo

Meglio continuare a usare i file scss di account_financial_report invece di duplicare anche qui lo stesso file.
Per farlo funzionare hanno incluso i bundle del backend nel report nella migrazione OCA/account-financial-reporting@01f22b8:

<template id="account_financial_report.html_container"> <t t-call-assets="web.assets_backend" /> <t t-set="body_classname" t-value="'container'" />

(da https://github.com/OCA/account-financial-reporting/blob/69ec629b8c70640b137a45d012bb2befa502d6bc/account_financial_report/report/templates/layouts.xml#L3-L5)

Non dico che mi piaccia come soluzione, ma è meglio che copiare quello stesso file in questo modulo.

Per il css avevo già provato a usare <t t-call="account_financial_report.html_container">

ma non viene proprio stampato il bundle asset backend. Ho verificato proprio l'html generato che viene poi passato a wkhtml e c'è solo report asset bundle

Prova a fare come è stato fatto nel commit di migrazione che ti ho linkato:

- <link - href="/account_financial_report/static/src/css/report_html.css" - rel="stylesheet" - /> + <t t-call-assets="web.assets_backend" />

@monen17 https://github.com/OCA/account-financial-reporting/pull/1366/files a quanto pare cambieranno tornando a com'era, effettivamente il css di financial_report rompe il css in giro per odoo

eLBati · 2025-06-04T09:51:35Z

l10n_it_financial_statements_report/wizard/wizard_financial_statements_report.xml

-                >
+                <div>
+                    <!--invisible="[('not_only_one_unaffected_earnings_account', '=', False)]" -->
+


Non serve più?

mia svista, ripristinato sia questo che gli altri. C'è il boolean nella nuova versione ma il valore è l'opposto dell'altro

l10n_it_financial_statements_report/wizard/wizard_financial_statements_report.xml

eLBati · 2025-06-04T10:04:48Z

l10n_it_financial_statements_report/models/account_group.py

+        "account.account", compute="_compute_current_account_ids"
+    )
+
+    def _compute_current_account_ids(self):


Secondo me possiamo lasciare le modifiche in OCA/account-financial-reporting#1327 e far dipendere questa (#4789) da quella (OCA/account-financial-reporting#1327)

Va bene, solo che non so quando verrà mergiata la PR su financial_report . Semmai una volta fatto il giro che tutto funziona, appena la mergiano punto a quel campo e tolgo da qui.
Alla fine la compute è la stessa , quindi funzionerà uguale la computazione dei conti

@eLBati approvatela in 2-3 così costringiamo i maintainers a rispondere

Secondo me possiamo lasciare le modifiche in OCA/account-financial-reporting#1327 e far dipendere questa (#4789) da quella (OCA/account-financial-reporting#1327)

Concordo

Va bene, solo che non so quando verrà mergiata la PR su financial_report . Semmai una volta fatto il giro che tutto funziona, appena la mergiano punto a quel campo e tolgo da qui. Alla fine la compute è la stessa , quindi funzionerà uguale la computazione dei conti

E se di là vengono fatte delle modifiche poi chi si ricorda di portarle di qua? E se anche qualcuno se lo ricordasse, poi ha voglia di farlo?
Meglio non duplicare il codice.

micheledic · 2025-06-05T08:44:50Z

/ocabot migration

OCA-git-bot · 2025-06-05T08:44:53Z

Hi @micheledic. Your command failed:

Invalid options for command migration: .

Ocabot commands

ocabot merge major|minor|patch|nobump
ocabot rebase
ocabot migration {MODULE_NAME}

More information

ocabot documentation
OCA guidelines, specially the "Version Numbers" section.

micheledic · 2025-06-05T08:45:22Z

/ocabot migration l10n_it_financial_statements_report

OCA-git-bot · 2025-06-05T08:45:35Z

Sorry @micheledic you are not allowed to mark the addon to be migrated.

To do so you must either have push permissions on the repository, or be a declared maintainer of all modified addons.

If you wish to adopt an addon and become it's maintainer, open a pull request to add your GitHub login to the maintainers key of its manifest.

micheledic · 2025-06-05T08:46:16Z

@eLBati puoi tu ?

francesco-ooops · 2025-06-05T08:46:21Z

/ocabot migration l10n_it_financial_statements_report

monen17

Grazie della PR!
Ci sono diverse cose che non mi tornano sul segno dei gruppi di conti ma sono uguali a com'è in 16.0 quindi non è questo il momento di cambiarle, anche se qui il codice viene proprio aggiunto e non migrato perché è stato rimosso da df6d0e5.

Puoi schiacciare i commit come descritto in https://github.com/OCA/maintainer-tools/wiki/Merge-commits-in-pull-requests#mergesquash-the-commits-generated-by-bots-or-weblate? Al momento diversi commit portati sono dei bot:

Potresti anche togliere [18.0] dal commit di migrazione? Al momento è

Ma i commit dovrebbero seguire https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#71commit-message, quindi iniziare con un tag che in questo caso è [MIG].
Occhio anche al typo nel nome del modulo 😉
Ti consiglio di usare https://github.com/OCA/odoo-module-migrator che automatizza la creazione del commit e diversi passaggi per la migrazione.

l10n_it_financial_statements_report/migrations/18.0.0.0.1/post-migrate.py

monen17 · 2025-07-04T14:52:59Z

l10n_it_financial_statements_report/models/account_group.py

+        "account.account", compute="_compute_current_account_ids"
+    )
+
+    def _compute_current_account_ids(self):


Secondo me possiamo lasciare le modifiche in OCA/account-financial-reporting#1327 e far dipendere questa (#4789) da quella (OCA/account-financial-reporting#1327)

Concordo

Va bene, solo che non so quando verrà mergiata la PR su financial_report . Semmai una volta fatto il giro che tutto funziona, appena la mergiano punto a quel campo e tolgo da qui. Alla fine la compute è la stessa , quindi funzionerà uguale la computazione dei conti

E se di là vengono fatte delle modifiche poi chi si ricorda di portarle di qua? E se anche qualcuno se lo ricordasse, poi ha voglia di farlo?
Meglio non duplicare il codice.

l10n_it_financial_statements_report/report/financial_statements_report_xlsx.py

monen17 · 2025-07-04T15:21:28Z

l10n_it_financial_statements_report/report/templates/financial_statements_report.xml

                    <t t-set="company" t-value="o.company_id" />
                    <link
-                        href="/account_financial_report/static/src/css/report.css"
+                        href="/l10n_it_financial_statements_report/static/src/css/report.css"


Meglio continuare a usare i file scss di account_financial_report invece di duplicare anche qui lo stesso file.
Per farlo funzionare hanno incluso i bundle del backend nel report nella migrazione OCA/account-financial-reporting@01f22b8:

<template id="account_financial_report.html_container"> <t t-call-assets="web.assets_backend" /> <t t-set="body_classname" t-value="'container'" />

(da https://github.com/OCA/account-financial-reporting/blob/69ec629b8c70640b137a45d012bb2befa502d6bc/account_financial_report/report/templates/layouts.xml#L3-L5)

Non dico che mi piaccia come soluzione, ma è meglio che copiare quello stesso file in questo modulo.

l10n_it_financial_statements_report/wizard/wizard_financial_statements_report.xml

micheledic · 2025-07-07T10:34:38Z

@monen17 ho squashato i commit del bot, fixato nome commit e alcuni attributi, aggiunto anche test-requirementst.txt per far dipendere dalla PR fix di account_financial_report (è corretto fare così giusto?
Per il css avevo già provato a usare
<t t-call="account_financial_report.html_container">

ma non viene proprio stampato il bundle asset backend.
Ho verificato proprio l'html generato che viene poi passato a wkhtml e c'è solo report asset bundle

micheledic · 2025-09-11T10:37:11Z

@OCA/local-italy-maintainers rimosso i test-requirements in quanto hanno mergiato il fix, ora è ok

mrcast · 2025-09-12T10:52:48Z

Ho testato e funzionalmente mi pare ok, tranne un punto. Scrivo i passi:

-Su un conto che vado a movimentare ho impostato la valuta secondaria

-Faccio una fattura in valuta secondaria

e verifico che fa il cambio

-Lancio la stampa del conto economico con l'opzione: "Show Ending Balance in Currency"

OSSERVATO
Non viene valorizzato l'ending balance in valuta secondaria

ATTESO
ottengo il cambio come in registrazione

micheledic · 2025-09-12T11:34:58Z

Ho testato e funzionalmente mi pare ok, tranne un punto. Scrivo i passi:

-Su un conto che vado a movimentare ho impostato la valuta secondaria

-Faccio una fattura in valuta secondaria
e verifico che fa il cambio -Lancio la stampa del conto economico con l'opzione: "Show Ending Balance in Currency" **OSSERVATO** Non viene valorizzato l'ending balance in valuta secondaria **ATTESO** ottengo il cambio come in registrazione

Potrebbe essere un bug di account_financial_report e non di financial_report(l'ennesimo bug).
Se provi ad esportare il trial balance di account_financial_report, non da anche li l'ending balance.

Potrei occuparmene io di fixarlo, ma a questo punto ritengo non sia più un bug che blocchi la migrazione di financial statements, visto che è da fixare sul modulo da cui dipende.
@sergiocorato che ne pensi ?

micheledic · 2025-09-12T13:00:34Z

OCA/account-financial-reporting#1387 Aperta PR per fix del problema di cui sopra

Balance is too generic, Financial statements is what this module is producing

Currently translated at 100.0% (74 of 74 strings) Translation: l10n-italy-14.0/l10n-italy-14.0-l10n_it_financial_statements_report Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-14-0/l10n-italy-14-0-l10n_it_financial_statements_report/it/

Currently translated at 100.0% (70 of 70 strings) Translation: l10n-italy-16.0/l10n-italy-16.0-l10n_it_financial_statements_report Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-16-0/l10n-italy-16-0-l10n_it_financial_statements_report/it/

… update

Report XLSX: the syntax "if isinstance(value, int | float)" can be used on Python 3.10 and above versions. 16.0 code should be compatible with Python>=3.7 tho.

Currently translated at 100.0% (70 of 70 strings) Translation: l10n-italy-16.0/l10n-italy-16.0-l10n_it_financial_statements_report Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-16-0/l10n-italy-16-0-l10n_it_financial_statements_report/it/

…key'

Currently translated at 100.0% (71 of 71 strings) Translation: l10n-italy-16.0/l10n-italy-16.0-l10n_it_financial_statements_report Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-16-0/l10n-italy-16-0-l10n_it_financial_statements_report/it/

micheledic · 2025-09-15T13:58:57Z

@mrcast potresti riprovare ora? fix su financial_report mergiato

@OCA/local-italy-developers ora dovrebbe essere tutto ok

MaurizioPellegrinet · 2025-09-15T15:39:10Z

Sembra cha ancora non funzioni:

ho fatto 3 fatture, di cui 2 in valuta secondaria
il bilancio non restituisce il valore in valuta

micheledic · 2025-09-16T12:06:24Z

Sembra cha ancora non funzioni:

ho fatto 3 fatture, di cui 2 in valuta secondaria

il bilancio non restituisce il valore in valuta

Ciao,
http://oca-l10n-italy-18-0-pr4789-585833ad664f.runboat.odoo-community.org/odoo/customer-invoices/50/1
Ho dato nuovamente un rebuild, ho riprovato e me lo mostra correttamente. Ma viene mostrato selezionando la currency sul conto, come dal tuo primo test.
Peer com'è fatto nativamente il modulo account_financial_report , l'ending balance in currency viene sommato SOLO se sul conto c'è una currency selezionata come da description del campo :
https://github.com/OCA/account-financial-reporting/blob/4c53984534d148743e27c92f7b546d7fd77f4c98/account_financial_report/wizard/trial_balance_wizard.py#L57-62

financial_statements_report si basa su account_financial_report, i dati li prende dal wizard di financial_report e quindi ne eredita anche le logiche

micheledic force-pushed the 18.0_mig_l10n_it_financial_statements_report branch 2 times, most recently from e1793e5 to 04d7fb0 Compare June 3, 2025 06:52

micheledic marked this pull request as draft June 3, 2025 07:10

micheledic force-pushed the 18.0_mig_l10n_it_financial_statements_report branch from 04d7fb0 to 78694fe Compare June 3, 2025 08:35

micheledic marked this pull request as ready for review June 3, 2025 08:47

micheledic force-pushed the 18.0_mig_l10n_it_financial_statements_report branch 3 times, most recently from addc917 to 2555cb4 Compare June 3, 2025 09:15

francesco-ooops added 18.0 migration labels Jun 4, 2025

eLBati reviewed Jun 4, 2025

View reviewed changes

micheledic force-pushed the 18.0_mig_l10n_it_financial_statements_report branch from 2555cb4 to b33173d Compare June 4, 2025 10:30

micheledic requested review from eLBati and francesco-ooops June 4, 2025 13:07

odooNextev mentioned this pull request Jun 4, 2025

Migration to version 18.0 #4392

Open

38 tasks

OCA-git-bot added this to the 18.0 milestone Jun 5, 2025

Borruso mentioned this pull request Jun 13, 2025

PR aggiunte v18 DinamicheAziendali/l10n-italy#2

Open

8 tasks

Borruso added a commit to DinamicheAziendali/l10n-italy that referenced this pull request Jun 13, 2025

[ADD] l10n_it_financial_statements_report: aggiunta PR OCA#4789

27e39b6

monen17 requested changes Jul 4, 2025

View reviewed changes

micheledic force-pushed the 18.0_mig_l10n_it_financial_statements_report branch 2 times, most recently from 1bdeda0 to 39c908e Compare July 7, 2025 10:31

micheledic force-pushed the 18.0_mig_l10n_it_financial_statements_report branch from 39c908e to 81ff7a5 Compare July 7, 2025 10:53

SirTakobi and others added 20 commits September 15, 2025 15:39

[REF] l10n_it_account_balance_report: Rename

31b36d2

Balance is too generic, Financial statements is what this module is producing

[IMP] l10n_it_financial_statements_report: pre-commit execution

321920c

[MIG] l10n_it_financial_statements_report: Migration to 16.0

c75a863

[REF] l10n_it_financial_statements_report: Remove test output file

62df338

Updated from template

80a835c

[REF] l10n_it_financial_statements_report: Update repo template

36f1278

[REF] l10n_it_financial_statements_report: Reformatted after template…

fb6de31

… update

Updated from template

8624d42

[FIX] l10n_it_financial_statements_report: Prepayments are assets

944656c

[FIX]l10n_it_financial_statements_report: Syntax compatibility

dabb564

Report XLSX: the syntax "if isinstance(value, int | float)" can be used on Python 3.10 and above versions. 16.0 code should be compatible with Python>=3.7 tho.

FIX l10n_it_financial_statements_report: use landscape format for PDF

d4e512f

[FIX] l10n_it_financial_statements_report: fix for KeyError 'grouped_…

0a59054

…key'

[IMP] l10n_it_financial_statements_report: add company header

89d12d7

[IMP] l10n_it_financial_statements_report: pre-commit stuff

f44a821

micheledic force-pushed the 18.0_mig_l10n_it_financial_statements_report branch from 389b5d6 to 22f0280 Compare September 15, 2025 13:40

[MIG] l10n_it_financial_statements_report: Migration to 18.0

585833a

micheledic force-pushed the 18.0_mig_l10n_it_financial_statements_report branch from 22f0280 to 585833a Compare September 16, 2025 10:34

Uh oh!

[18.0][MIG] l10n_it_financial_statements_report: Migration to 18.0 #4789

Are you sure you want to change the base?

[18.0][MIG] l10n_it_financial_statements_report: Migration to 18.0 #4789

Conversation

micheledic commented Jun 3, 2025

Uh oh!

micheledic commented Jun 3, 2025

Uh oh!

eLBati left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

micheledic Jun 4, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

micheledic commented Jun 5, 2025

Uh oh!

OCA-git-bot commented Jun 5, 2025

Uh oh!

micheledic commented Jun 5, 2025

Uh oh!

OCA-git-bot commented Jun 5, 2025

Uh oh!

micheledic commented Jun 5, 2025

Uh oh!

francesco-ooops commented Jun 5, 2025

Uh oh!

monen17 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

micheledic commented Jul 7, 2025

Uh oh!

micheledic commented Sep 11, 2025

Uh oh!

mrcast commented Sep 12, 2025

Uh oh!

micheledic commented Sep 12, 2025

Uh oh!

micheledic commented Sep 12, 2025

Uh oh!

micheledic commented Sep 15, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

MaurizioPellegrinet commented Sep 15, 2025

Uh oh!

micheledic Jun 4, 2025 •

edited

Loading

micheledic commented Sep 15, 2025 •

edited

Loading

micheledic commented Sep 16, 2025 •

edited

Loading