OCA / l10n-netherlands

Odoo Dutch Localization
GNU Affero General Public License v3.0
21 stars 55 forks source link

[14.0][FIX] *+xaf_auditfile_export: use correct company in name #393

Closed NL66278 closed 1 year ago

NL66278 commented 1 year ago

Currently the export record is created with the current company in the environment (self.env.company), but the default company for the user (self.env.user.company_id) is applied to the name generated. This can and will lead to mismatches.

This fix will consistently use self.env.company.

ddejong-therp commented 1 year ago

Tested it, and with this fix, the name is now based on the actually selected company.

OCA-git-bot commented 1 year ago

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

NL66278 commented 1 year ago

/ocabot merge minor

OCA-git-bot commented 1 year ago

On my way to merge this fine PR! Prepared branch 14.0-ocabot-merge-pr-393-by-NL66278-bump-minor, awaiting test results.

OCA-git-bot commented 1 year ago

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