[20.0][MIG] report_xlsx: Migration to 20.0 by propublisher01 · Pull Request #1203 · OCA/reporting-engine

propublisher01 · 2026-09-28T13:11:37Z

Migration of report_xlsx to 20.0 (history preserved with git format-patch / git am).

Changes:

odoo.http is now a package: content_disposition is imported from odoo.http.stream and serialize_exception from odoo.http.dispatcher.
ir.actions.report no longer has a report_file field: removed from the demo data, the tests and the usage example (README regenerated).

Tested on Odoo 20.0 (nightly 20260928), with demo data:

module tests pass;
/report/xlsx/... and /report/download return a valid .xlsx file.

The JS report handler needs no change: the ir.actions.report handlers registry and its (action, options, env) signature are unchanged in 20.0.

AI assistance: this migration was prepared with Claude Opus 5.5 (Claude Code); I reviewed and tested every change. Commits carry the corresponding Assisted-by: trailer.

… in selection

…s to print to the 'create_xlsx_report' method + Instanciate the parser and make it available for the 'generate_xlsx_report' method

mileo · 2026-09-30T10:11:03Z

Thanks a lot for this migration! It installs and its tests pass locally on 20.0 with a clean log.

One thing blocks it for now: commit 5299985 carries a Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> trailer, and the OCA AI policy does not allow AI tools in authorship fields such as Co-authored-by:. The expected way to disclose AI assistance is an Assisted-by: trailer instead, and the policy also asks to mention it in the PR description:
https://github.com/OCA/.github/blob/master/AI_POLICY.md

Could you please amend that commit, replacing the trailer with Assisted-by: Claude Opus 5.5, and force-push? For example, with the [MIG] commit on top of the branch:

git commit --amend   # edit the message: Co-Authored-By -> Assisted-by
git push --force-with-lease

I will be glad to review again right after.

propublisher01 · 2026-09-30T12:04:38Z

Done, thanks for pointing this out!

mileo

LGTM

…t_xlsx PRs account_financial_report depends on date_range and report_xlsx. Their 20.0 migrations are open pull requests: OCA/server-ux#1341 and OCA/reporting-engine#1203. Install them from the pull request heads until they are merged.

micheledic · 2026-10-05T15:57:14Z

@propublisher01 why it is draft ?

OCA-git-bot · 2026-10-05T16:00:03Z

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

…t_xlsx PRs account_financial_report depends on date_range and report_xlsx. Their 20.0 migrations are open pull requests: OCA/server-ux#1341 and OCA/reporting-engine#1203. Install them from the pull request heads until they are merged.

micheledic · 2026-10-13T14:36:51Z

@propublisher01 can you mark the PR ready so @pedrobaeza can merge?

carlos-lopez-tecnativa

Non-blocking comment, but it would be great if you could apply this.

Additionally, please squash the administrative commits.
TT64867

carlos-lopez-tecnativa · 2026-10-13T18:55:59Z

+    _logger.debug("Can not import openpyxl`.")
+
+
+class TestReport(common.TransactionCase):


Could you please inherit from BaseCommon and change setUp to setUpClass?

Currently translated at 23.0% (3 of 13 strings) Translation: reporting-engine-16.0/reporting-engine-16.0-report_xlsx Translate-URL: https://translation.odoo-community.org/projects/reporting-engine-16-0/reporting-engine-16-0-report_xlsx/hr/

Currently translated at 100.0% (13 of 13 strings) Translation: reporting-engine-16.0/reporting-engine-16.0-report_xlsx Translate-URL: https://translation.odoo-community.org/projects/reporting-engine-16-0/reporting-engine-16-0-report_xlsx/es/

This implements behavior expected when configuring "Save as Attachment Prefix" within xlsx reports.

Currently translated at 100.0% (7 of 7 strings) Translation: reporting-engine-17.0/reporting-engine-17.0-report_xlsx Translate-URL: https://translation.odoo-community.org/projects/reporting-engine-17-0/reporting-engine-17-0-report_xlsx/it/

Currently translated at 100.0% (7 of 7 strings) Translation: reporting-engine-17.0/reporting-engine-17.0-report_xlsx Translate-URL: https://translation.odoo-community.org/projects/reporting-engine-17-0/reporting-engine-17-0-report_xlsx/sv/

* report_py3o, report_xlsx, report_xml odoo core introduced an additional argument in [1] to the method inherited by report_py3o, report_xlsx, report_xml. When the same method is inherited in other addons, it raises a "TypeError: ... takes X positional arguments but Y were given" due to a mismatch in the argument count. This commit updates the mentioned method signature to match the latest odoo core definition, ensuring compatibility and preventing runtime errors when multiple addons override the same function. No functional changes are introduced. Only method signatures have been synchronized with upstream Odoo

Currently translated at 85.7% (6 of 7 strings) Translation: reporting-engine-18.0/reporting-engine-18.0-report_xlsx Translate-URL: https://translation.odoo-community.org/projects/reporting-engine-18-0/reporting-engine-18-0-report_xlsx/tr/

Currently translated at 100.0% (7 of 7 strings) Translation: reporting-engine-18.0/reporting-engine-18.0-report_xlsx Translate-URL: https://translation.odoo-community.org/projects/reporting-engine-18-0/reporting-engine-18-0-report_xlsx/tr/

…rt instead of user context

…r on OCA Financial Reports.

- odoo.http is now a package: import content_disposition from odoo.http.stream and serialize_exception from odoo.http.dispatcher. - ir.actions.report no longer has a report_file field: removed from the demo data, the tests and the usage example. Assisted-by: Claude Opus 5.5

propublisher01 · 2026-10-16T15:26:24Z

@carlos-lopez-tecnativa Done: tests now inherit from BaseCommon with setUpClass (test user in base.group_system, since report actions require it), and the administrative commits are squashed (110 → 58 commits, translations from different translators kept separate). Tests pass locally on 20.0. @micheledic ready for merge on my side, thanks!

pedrobaeza

In a test in the runboat with the demo report, I see that the report is not automatically added to the print menu, so I think you need to add this for getting it (not tested though) and ease the demo testing.

pedrobaeza · 2026-10-19T06:41:20Z

+        <field name="name">Print to XLSX</field>
+        <field name="model">res.partner</field>
+        <field name="report_type">xlsx</field>
+        <field name="report_name">report_xlsx.partner_xlsx</field>


Suggested change

<field name="report_name">report_xlsx.partner_xlsx</field>

<field name="report_name">report_xlsx.partner_xlsx</field>

<field name="binding_model_id" ref="base.model_res_partner"/>

<field name="binding_type">report</field>

Adrien Peiffer (ACSONE) and others added 3 commits September 28, 2026 14:23

[CHG] Rename base_report_xlsx to report_xlsx and add xlsx report type…

6b4cee9

… in selection

[IMP] report_xlsx - Pass the report instance as well as ids of record…

fc73124

…s to print to the 'create_xlsx_report' method + Instanciate the parser and make it available for the 'generate_xlsx_report' method

[MIG] Migration of report_xlsx to 10.0 (OCA#76)

1a68ac0

OCA-git-bot added mod:report_xlsx Module report_xlsx series:20.0 labels Sep 28, 2026

legalsylvain marked this pull request as draft September 28, 2026 15:07

This was referenced Sep 30, 2026

[20.0][MIG] mis_builder: Migration to 20.0 OCA/mis-builder#842

Open

[20.0][MIG] l10n_br_mis_report: Migration to 20.0 OCA/l10n-brazil#5268

Draft

propublisher01 force-pushed the 20.0-mig-report_xlsx branch from 5299985 to b07289a Compare September 30, 2026 11:45

mileo approved these changes Sep 30, 2026

View reviewed changes

This was referenced Sep 30, 2026

[20.0][MIG] account_asset_management: Migration to 20.0 ledoent/account-financial-tools#13

Open

[20.0][MIG] mis_builder: Migration to 20.0 ledoent/mis-builder#8

Open

mileo mentioned this pull request Oct 3, 2026

[20.0][MIG] l10n_ar_ledger OCA/l10n-argentina#111

Draft

micheledic approved these changes Oct 5, 2026

View reviewed changes

OCA-git-bot added approved ready to merge labels Oct 5, 2026

julien2364 mentioned this pull request Oct 8, 2026

[20.0][MIG] l10n_be_mis_reports: Migration to 20.0 OCA/l10n-belgium#290

Open

alan196 mentioned this pull request Oct 10, 2026

[20.0][MIG] report_xlsx: Migration to 20.0 Jarsa/reporting-engine#4

Merged

propublisher01 marked this pull request as ready for review October 13, 2026 15:21

carlos-lopez-tecnativa approved these changes Oct 13, 2026

View reviewed changes

carlos-lopez-tecnativa mentioned this pull request Oct 13, 2026

[20.0][MIG] report_xlsx_helper: Migration to version 20.0 #1214

Open

1 task

mileo mentioned this pull request Oct 15, 2026

[20.0][ADD] mis_builder_analytic_source OCA/mis-builder-contrib#78

Open

alexis-via and others added 23 commits October 16, 2026 17:17

[FIX] report_xlsx: add token in report_download() controller

7061188

Translated using Weblate (Croatian)

0b61a0b

Currently translated at 23.0% (3 of 13 strings) Translation: reporting-engine-16.0/reporting-engine-16.0-report_xlsx Translate-URL: https://translation.odoo-community.org/projects/reporting-engine-16-0/reporting-engine-16-0-report_xlsx/hr/

[FIX] report_xlsx: initialize reportname

4129a4f

Translated using Weblate (Spanish)

2f2f593

Currently translated at 100.0% (13 of 13 strings) Translation: reporting-engine-16.0/reporting-engine-16.0-report_xlsx Translate-URL: https://translation.odoo-community.org/projects/reporting-engine-16-0/reporting-engine-16-0-report_xlsx/es/

[FIX] report_xlsx: Save as attachment

607c055

This implements behavior expected when configuring "Save as Attachment Prefix" within xlsx reports.

[IMP] report_xlsx: pre-commit auto fixes

7970cf3

[MIG] report_xlsx: Migration to 17.0

0ab2482

[FIX] migrated version number in __manifest__.py

775fae0

Translated using Weblate (Italian)

916f30d

Currently translated at 100.0% (7 of 7 strings) Translation: reporting-engine-17.0/reporting-engine-17.0-report_xlsx Translate-URL: https://translation.odoo-community.org/projects/reporting-engine-17-0/reporting-engine-17-0-report_xlsx/it/

Translated using Weblate (Swedish)

a1999c8

Currently translated at 100.0% (7 of 7 strings) Translation: reporting-engine-17.0/reporting-engine-17.0-report_xlsx Translate-URL: https://translation.odoo-community.org/projects/reporting-engine-17-0/reporting-engine-17-0-report_xlsx/sv/

[FIX] report_xlsx: Remove XML header in HTML

cbefb0c

[DOC] report_xlsx: Update USAGE.rst

1f670b2

[MIG] report_xlsx: Migration to 18.0

e95431f

Translated using Weblate (Turkish)

3b9b50c

Currently translated at 85.7% (6 of 7 strings) Translation: reporting-engine-18.0/reporting-engine-18.0-report_xlsx Translate-URL: https://translation.odoo-community.org/projects/reporting-engine-18-0/reporting-engine-18-0-report_xlsx/tr/

Translated using Weblate (Turkish)

9d00c35

Currently translated at 100.0% (7 of 7 strings) Translation: reporting-engine-18.0/reporting-engine-18.0-report_xlsx Translate-URL: https://translation.odoo-community.org/projects/reporting-engine-18-0/reporting-engine-18-0-report_xlsx/tr/

[IMP] report_xlsx: pre-commit auto fixes

bba01f6

[MIG] report_xlsx: Migration to 19.0

07ed269

[FIX] report_xlsx: Preserve custom context by using context from repo…

aa56c2d

…rt instead of user context

[FIX] report_xslx: Revert action_manager_report.esm.js to fix KeyErro…

b1d9e10

…r on OCA Financial Reports.

Added translation using Weblate (Danish)

0ec8348

[IMP] report_xlsx: pre-commit auto fixes

9e0f992

propublisher01 force-pushed the 20.0-mig-report_xlsx branch from b07289a to 42922c2 Compare October 16, 2026 15:21

stridebs mentioned this pull request Oct 19, 2026

[20.0][MIG] account_asset_management: Migration to 20.0 OCA/account-financial-tools#2408

Open

pedrobaeza requested changes Oct 19, 2026

View reviewed changes

OCA-git-bot removed approved ready to merge labels Oct 19, 2026

marchar1 added a commit to SuperbirdOdoo/reporting-engine that referenced this pull request Oct 23, 2026

Merge OCA#1203: report_xlsx for Odoo 20

98bab19

Navigation Menu

Repository navigation

[20.0][MIG] report_xlsx: Migration to 20.0 - #1203

[20.0][MIG] report_xlsx: Migration to 20.0#1203
propublisher01 wants to merge 58 commits into
OCA:20.0from
propublisher01:20.0-mig-report_xlsx

propublisher01 commented Sep 28, 2026 •

edited

Loading

Uh oh!

mileo commented Sep 30, 2026

Uh oh!

propublisher01 commented Sep 30, 2026

Uh oh!

mileo left a comment

Uh oh!

micheledic commented Oct 5, 2026

Uh oh!

OCA-git-bot commented Oct 5, 2026

Uh oh!

micheledic commented Oct 13, 2026

Uh oh!

carlos-lopez-tecnativa left a comment

Uh oh!

carlos-lopez-tecnativa Oct 13, 2026

Uh oh!

propublisher01 commented Oct 16, 2026

Uh oh!

pedrobaeza left a comment

Uh oh!

pedrobaeza Oct 19, 2026

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

20 participants

		_logger.debug("Can not import openpyxl`.")


		class TestReport(common.TransactionCase):

Conversation

propublisher01 commented Sep 28, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

mileo commented Sep 30, 2026

Uh oh!

propublisher01 commented Sep 30, 2026

Uh oh!

mileo left a comment

Choose a reason for hiding this comment

Uh oh!

micheledic commented Oct 5, 2026

Uh oh!

OCA-git-bot commented Oct 5, 2026

Uh oh!

micheledic commented Oct 13, 2026

Uh oh!

carlos-lopez-tecnativa left a comment

Choose a reason for hiding this comment

Uh oh!

carlos-lopez-tecnativa Oct 13, 2026

Choose a reason for hiding this comment

Uh oh!

propublisher01 commented Oct 16, 2026

Uh oh!

pedrobaeza left a comment

Choose a reason for hiding this comment

Uh oh!

pedrobaeza Oct 19, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

20 participants

propublisher01 commented Sep 28, 2026 •

edited

Loading