Skip to content

Syncing from upstream OCA/reporting-engine (19.0) - #868

Merged
bt-admin merged 103 commits into
brain-tec:19.0from
OCA:19.0
Nov 1, 2025
Merged

Syncing from upstream OCA/reporting-engine (19.0)#868
bt-admin merged 103 commits into
brain-tec:19.0from
OCA:19.0

Conversation

@bt-admin

@bt-admin bt-admin commented Nov 1, 2025

Copy link
Copy Markdown

bt_gitbot

Adrien Peiffer (ACSONE) and others added 30 commits October 12, 2025 19:47
…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/
OCA-git-bot and others added 27 commits October 12, 2025 19:47
This implements behavior expected when configuring "Save as Attachment Prefix"
within xlsx reports.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

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/
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/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

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/
* 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/
Signed-off-by HviorForgeFlow
@bt-admin bt-admin added the 19.0 label Nov 1, 2025
@bt-admin
bt-admin merged commit 6eb5cb6 into brain-tec:19.0 Nov 1, 2025
4 checks passed
@codecov

codecov Bot commented Nov 1, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 64.67890% with 77 lines in your changes missing coverage. Please review.
✅ Project coverage is 70.75%. Comparing base (b663dd4) to head (fb11848).
⚠️ Report is 105 commits behind head on 19.0.

Files with missing lines Patch % Lines
report_xlsx/controllers/main.py 23.33% 46 Missing ⚠️
report_xlsx/report/report_abstract_xlsx.py 71.15% 15 Missing ⚠️
report_xlsx/models/ir_report.py 67.44% 10 Missing and 4 partials ⚠️
report_xlsx/tests/test_report.py 95.55% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             19.0     #868      +/-   ##
==========================================
- Coverage   78.78%   70.75%   -8.04%     
==========================================
  Files          10       20      +10     
  Lines         165      383     +218     
  Branches       11       33      +22     
==========================================
+ Hits          130      271     +141     
- Misses         25       98      +73     
- Partials       10       14       +4     

☔ View full report in Codecov by Sentry.
📢 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.