Skip to content

[16.0][FIX] report_xml: Remove useless and faulty post init hook#1151

Merged
OCA-git-bot merged 1 commit intoOCA:16.0from
acsone:16.0-report-xml-useless-hook
Apr 14, 2026
Merged

[16.0][FIX] report_xml: Remove useless and faulty post init hook#1151
OCA-git-bot merged 1 commit intoOCA:16.0from
acsone:16.0-report-xml-useless-hook

Conversation

@lmignon
Copy link
Copy Markdown
Contributor

@lmignon lmignon commented Apr 14, 2026

The post init hook created an ir attachment using content read as a string as value for the 'datas' field which expects a b64 encoded binary value.... This code comes from previous odoo version and should have been removed when migrating this addon

image

The post init hook created an ir attachment using content read as a string as value for the 'datas' field which expects a b64 encoded binary value....
This code comes from previous odoo version and should have been removed when migrating this addon
Copy link
Copy Markdown

@nicolas-delbovier-acsone nicolas-delbovier-acsone left a comment

Choose a reason for hiding this comment

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

Code review

@HviorForgeFlow
Copy link
Copy Markdown
Member

/ocabot merge patch

@OCA-git-bot
Copy link
Copy Markdown
Contributor

On my way to merge this fine PR!
Prepared branch 16.0-ocabot-merge-pr-1151-by-HviorForgeFlow-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit c51cebf into OCA:16.0 Apr 14, 2026
6 of 7 checks passed
@OCA-git-bot
Copy link
Copy Markdown
Contributor

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

@lmignon lmignon deleted the 16.0-report-xml-useless-hook branch April 15, 2026 05:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants