Skip to content

[20.0][MIG] report_xlsx_helper: Migration to 20.0 - #5

Merged
alan196 merged 190 commits into
Jarsa:20.0from
Jarsa-dev:20.0-mig-report_xlsx_helper
Oct 10, 2026
Merged

alan196 merged 190 commits into
Jarsa:20.0from
Jarsa-dev:20.0-mig-report_xlsx_helper

Conversation

@alan196

@alan196 alan196 commented Oct 10, 2026

Copy link
Copy Markdown

Migration of report_xlsx_helper from 19.0 to 20.0 (history ported with oca-port).

Depends on #4 (report_xlsx): this branch is rebased on top of 20.0-mig-report_xlsx, so the diff shows both modules until #4 is merged. Do not merge before #4.

Changes for 20.0

  • content_disposition is no longer exported by odoo.http; it is imported from odoo.http.stream in the controller.
  • datetime.now() replaced by fields.Datetime.now() in the XLSX footer helper (naive UTC value, as context_timestamp expects; flagged by ruff DTZ005 in the 20.0 pre-commit config).
  • Version bumped to 20.0.1.0.0; README regenerated by pre-commit.

No change was needed for the ir.access migration (the module defines no new models), icons, OWL or UoM.

Tests

Installed on a fresh 20.0 database with demo data and ran -i report_xlsx_helper --test-enable --test-tags /report_xlsx_helper: 0 failed, 0 error(s), no ERROR/CRITICAL in the log. pre-commit (repo config for 20.0) passes.

Assisted-by: Claude Fable 5.1

Adrien Peiffer (ACSONE) and others added 30 commits September 28, 2026 14:23
…s to print to the 'create_xlsx_report' method + Instanciate the parser and make it available for the 'generate_xlsx_report' method
* refactor: On new ActionManager override _makeReportUrls and triggerDownload to support XLSX format
* pass action to custom _downloadReportXLSX: Need action to get report data
* dict constructor to remove context lang
Updated by Update PO files to match POT (msgmerge) hook in Weblate.
Currently translated at 88.2% (15 of 17 strings)

Translation: reporting-engine-12.0/reporting-engine-12.0-report_xlsx
Translate-URL: https://translation.odoo-community.org/projects/reporting-engine-12-0/reporting-engine-12-0-report_xlsx/de/
Currently translated at 94.4% (17 of 18 strings)

Translation: reporting-engine-12.0/reporting-engine-12.0-report_xlsx
Translate-URL: https://translation.odoo-community.org/projects/reporting-engine-12-0/reporting-engine-12-0-report_xlsx/pt/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: reporting-engine-13.0/reporting-engine-13.0-report_xlsx
Translate-URL: https://translation.odoo-community.org/projects/reporting-engine-13-0/reporting-engine-13-0-report_xlsx/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: reporting-engine-13.0/reporting-engine-13.0-report_xlsx
Translate-URL: https://translation.odoo-community.org/projects/reporting-engine-13-0/reporting-engine-13-0-report_xlsx/
weblate and others added 26 commits October 10, 2026 07:38
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: reporting-engine-16.0/reporting-engine-16.0-report_xlsx_helper
Translate-URL: https://translation.odoo-community.org/projects/reporting-engine-16-0/reporting-engine-16-0-report_xlsx_helper/
Currently translated at 14.2% (2 of 14 strings)

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

Translation: reporting-engine-16.0/reporting-engine-16.0-report_xlsx_helper
Translate-URL: https://translation.odoo-community.org/projects/reporting-engine-16-0/reporting-engine-16-0-report_xlsx_helper/es/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

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

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

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

Translation: reporting-engine-18.0/reporting-engine-18.0-report_xlsx_helper
Translate-URL: https://translation.odoo-community.org/projects/reporting-engine-18-0/reporting-engine-18-0-report_xlsx_helper/tr/
- Bump version to 20.0.1.0.0.
- content_disposition is no longer exported by odoo.http: import it from
  odoo.http.stream.
- Use fields.Datetime.now() (naive UTC, as context_timestamp expects)
  instead of datetime.now() flagged by ruff DTZ005.

Assisted-by: Claude Fable 5.1
@jarsabot

Copy link
Copy Markdown
    **WARNING**: Could not link to Odoo task. Record ID not found in the title of the MR, supported formats:
{ record type }#{ number }

Available types: 'task#', 't#' (case insensitive)

Examples:

  • task#123

  • i#123

@codecov-commenter

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 75.61779% with 148 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (20.0@6521747). Learn more about missing BASE report.

Files with missing lines Patch % Lines
report_xlsx_helper/report/report_xlsx_abstract.py 84.15% 34 Missing and 14 partials ⚠️
report_xlsx/controllers/main.py 24.59% 46 Missing ⚠️
report_xlsx_helper/controllers/main.py 26.92% 19 Missing ⚠️
report_xlsx/models/ir_report.py 62.79% 13 Missing and 3 partials ⚠️
report_xlsx/report/report_abstract_xlsx.py 71.15% 15 Missing ⚠️
report_xlsx/tests/test_report.py 95.55% 2 Missing ⚠️
report_xlsx_helper/models/ir_actions_report.py 84.61% 1 Missing and 1 partial ⚠️
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             20.0       #5   +/-   ##
=======================================
  Coverage        ?   75.61%           
=======================================
  Files           ?       21           
  Lines           ?      607           
  Branches        ?       58           
=======================================
  Hits            ?      459           
  Misses          ?      130           
  Partials        ?       18           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@alan196
alan196 merged commit b50f2fe into Jarsa:20.0 Oct 10, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.