Skip to content

[19.0][MIG] report_xml #1072

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 75 commits into from
Oct 29, 2025
Merged

[19.0][MIG] report_xml #1072

merged 75 commits into from
Oct 29, 2025

Conversation

@etobella
Copy link
Member

@etobella etobella commented Oct 5, 2025

Migration to 19.0.

Added some code on the tests in order to load demo data.

Jairo Llopis and others added 30 commits October 5, 2025 15:02
@etobella
Add module report_xml.
6056c1d
@etobella
Add template utf8_header.
5e05153 
Almost any XML must start with this. Let's make it easier.
@etobella
Easier XSD checking reports.
fb729cb 
Any report inheriting this AbstractModel can check its XML results
against any XSD.
@etobella
Add sample module.
f4b32cd
@etobella
Sort links.
6cfa8df
@etobella
Fix XML tag mismatch.
739e910
@etobella
Allow docargs to be loaded from context.
56718b8
@etobella
Only replace the docs key if it is missing.
c89a557
@etobella
Clearer code comments, add logging.
9a86bc1
@etobella
Oops, this header belongs to other module.
9c56eba
@etobella
Add module report_xml_sample.
4adedde
@etobella
Clean AGPL garbage preserving copyright.
f73f159 
See odoo/odoo@0bd4545.
@etobella
Return supers.
e6b41bc
@etobella
Credit creators, using same name across modules to avoid split statis…
792cf05 
…tics.
@sbidoul @etobella
prefix versions with 8.0
06f2b35
@etobella
[MIG] report_xml: Migration to 10.0
43ca169
@oca-transbot @etobella
OCA Transbot updated translations from Transifex
2f0eb42
@etobella
[MIG] report_xml: Migration to 11.0
dbde390
@oca-transbot @etobella
OCA Transbot updated translations from Transifex
66b0f29
@naglis @etobella
[IMP] report_fillpdf, report_xlsx, report_xml: Use content_dispositio…
330cda0 
…n helper
@tarteo @etobella
[FIX] report_xml readme
87d00a4
@oca-travis @etobella
[UPD] Update report_xml.pot
c290973
@pedrobaeza @etobella
[IMP] report_xml: Clarification in README
78a861e 
Previous instructions were misleading
@ernestotejeda @etobella
report_xml: add custom filename to response headers
9ab53da
@ernestotejeda @etobella
[MIG] report_xml: Migration to 12.0
fe6452b
@oca-travis @etobella
[UPD] Update report_xml.pot
3a9c181
@oca-transbot @etobella
Update translation files
6d83129 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: reporting-engine-12.0/reporting-engine-12.0-report_xml
Translate-URL: https://translation.odoo-community.org/projects/reporting-engine-12-0/reporting-engine-12-0-report_xml/
@OCA-git-bot @etobella
[UPD] README.rst
a9bcd2a
@Tatider @etobella
[IMP] report_xml: black, isort
74ef6e3
@Tatider @etobella
[MIG][13.0] report_xml: Migration to 13.0
1fe7248
oca-ci and others added 6 commits October 5, 2025 15:02
@etobella
[UPD] Update report_xml.pot
ef64710
@OCA-git-bot @etobella
[BOT] post-merge updates
a8f5e24
@mymage @etobella
Translated using Weblate (Italian)
f3e0155 
Currently translated at 100.0% (16 of 16 strings)

Translation: reporting-engine-18.0/reporting-engine-18.0-report_xml
Translate-URL: https://translation.odoo-community.org/projects/reporting-engine-18-0/reporting-engine-18-0-report_xml/it/
@niyasraphy @etobella
[FIX] *: align inherited method signature with odoo update
c9239ca 
* 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
@OCA-git-bot @etobella
[BOT] post-merge updates
efe6b97
@tsezgin @etobella
Translated using Weblate (Turkish)
3b7b138 
Currently translated at 100.0% (16 of 16 strings)

Translation: reporting-engine-18.0/reporting-engine-18.0-report_xml
Translate-URL: https://translation.odoo-community.org/projects/reporting-engine-18-0/reporting-engine-18-0-report_xml/tr/
@etobella etobella mentioned this pull request Oct 5, 2025
Open
@pedrobaeza
Copy link
Member

/ocabot migration report_xml

Please remind me one more time what this layer provides against putting a QWeb template with the XML header and content like Odoo does for Factur-X and others (example: https://github.com/odoo/odoo/blob/19.0/addons/account_edi_ubl_cii/data/cii_22_templates.xml)

@OCA-git-bot OCA-git-bot added this to the 19.0 milestone Oct 6, 2025
@OCA-git-bot OCA-git-bot mentioned this pull request Oct 6, 2025
Open
20 tasks
@etobella
Copy link
Member Author

etobella commented Oct 6, 2025

It allows to generate a report that delivers an XML file directly. Otherwise, Odoo will send you back an HTML (with xml format, but an HTML) or PDF.

@pedrobaeza
Copy link
Member

Let's include that in CONTEXT.md for later references.

@etobella etobella force-pushed the 19.0-mig-report_xml branch from 494a6e1 to 9a081cd Compare October 6, 2025 17:01
pedrobaeza
pedrobaeza reviewed Oct 6, 2025
View reviewed changes
Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't put lxml as external_dependencies. It's already a dependency of Odoo core.

@etobella etobella force-pushed the 19.0-mig-report_xml branch from 9a081cd to acd45cc Compare October 6, 2025 18:18
luisDIXMIT
luisDIXMIT reviewed Oct 27, 2025
View reviewed changes
Copy link

@luisDIXMIT luisDIXMIT left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be "Demo xml report" listed? It doesn't appears.

image

@etobella
Copy link
Member Author

etobella commented Oct 27, 2025 •
edited
Loading

with the new default value of demo, demo is not imported by default.

luisDIXMIT
luisDIXMIT approved these changes Oct 29, 2025
View reviewed changes
Copy link

@luisDIXMIT luisDIXMIT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@OCA-git-bot
Copy link
Contributor

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). 🤖

@pedrobaeza
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 19.0-ocabot-merge-pr-1072-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit ee72b6d into OCA:19.0 Oct 29, 2025
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at d8c5d91. Thanks a lot for contributing to OCA. ❤️

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

Reviewers

@pedrobaeza pedrobaeza pedrobaeza approved these changes

+1 more reviewer

@luisDIXMIT luisDIXMIT luisDIXMIT approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

approved merged 🎉 ready to merge

Projects

None yet

Milestone

19.0

Development

Successfully merging this pull request may close these issues.