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

OCA-git-bot merged 1 commit into
OCA:16.0from
acsone:16.0-fix_exception_name_to_args0-bwi

Conversation

@ThomasBinsfeld

@ThomasBinsfeld ThomasBinsfeld commented Sep 16, 2025 •

Copy link
Copy Markdown
Contributor

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?

@FrancoMaxime FrancoMaxime left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM: code review

@amh-mw amh-mw left a comment

Copy link
Copy Markdown
Member

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

guewen commented Sep 18, 2025

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-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