Skip to content

[16.0] [FIX] queue_job: fix exception msg handling#824

Merged
OCA-git-bot merged 1 commit into
OCA:16.0from
acsone:16.0-fix_exception_name_to_args0-bwi
Sep 18, 2025
Merged

[16.0] [FIX] queue_job: fix exception msg handling#824
OCA-git-bot merged 1 commit into
OCA:16.0from
acsone:16.0-fix_exception_name_to_args0-bwi

Conversation

@ThomasBinsfeld
Copy link
Copy Markdown
Contributor

@ThomasBinsfeld ThomasBinsfeld commented Sep 16, 2025

Use args[0] instead of name to get the exception message as the name attribute is deprecated

17.0 forward-port PR : #825
18.0 forward-port PR : #826

@OCA-git-bot
Copy link
Copy Markdown
Contributor

Hi @guewen,
some modules you are maintaining are being modified, check this out!

@ThomasBinsfeld ThomasBinsfeld force-pushed the 16.0-fix_exception_name_to_args0-bwi branch from e353b5a to 6341d0d Compare September 16, 2025 09:23
@ThomasBinsfeld
Copy link
Copy Markdown
Contributor Author

Reopening of the automatically closed PR #644

@simahawk
@amh-mw
@FrancoMaxime
@benwillig

Could we move forward with this one as the first PR seemed ready to be merged?

Copy link
Copy Markdown
Member

@FrancoMaxime FrancoMaxime left a comment

Choose a reason for hiding this comment

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

LGTM: code review

Copy link
Copy Markdown
Member

@amh-mw amh-mw left a comment

Choose a reason for hiding this comment

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

Test coverage?

@ThomasBinsfeld
Copy link
Copy Markdown
Contributor Author

The modified file doesn't seem covered at all, we won't write its coverage in this PR. I suggest to merge as it. ping @guewen

'name' attributes for odoo's exception has been deprecated and produces a warning message. This commit fixes the behavior
@ThomasBinsfeld ThomasBinsfeld force-pushed the 16.0-fix_exception_name_to_args0-bwi branch from 6341d0d to 3a2f075 Compare September 17, 2025 08:17
@guewen
Copy link
Copy Markdown
Member

guewen commented Sep 18, 2025

/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-824-by-guewen-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit b1dd4d9 into OCA:16.0 Sep 18, 2025
7 checks passed
@OCA-git-bot
Copy link
Copy Markdown
Contributor

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

@sbidoul sbidoul deleted the 16.0-fix_exception_name_to_args0-bwi branch January 4, 2026 11:35
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.

7 participants