Skip to content
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.

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

[15.0][FIX] base_comment_template: Fix access error #774

Merged
merged 1 commit into from
Jul 20, 2023

Conversation

BT-pgaiser
Copy link

When using the base comment template feature on contacts (res_partner), when a user without admin rights tries to open up the form view of an affected contact, he gets an access error because he doesn't have the necessary rights to read the fields from 'ir_model'.

This adjustment fixes that.

@pedrobaeza pedrobaeza added this to the 15.0 milestone Jun 30, 2023
@pedrobaeza
Copy link
Member

Fix an access error in the base_comment_template module in
the BaseCommentTemplate.name_get() method.
Add sudo() before reading ir_model.name fields to fix access error for users
without admin rights when opening contact form views.
@BT-pgaiser
Copy link
Author

@pedrobaeza Fixed commit message.

@pedrobaeza pedrobaeza changed the title Fix Access Errors When Opening Contact Form Views [15.0][FIX] base_comment_template: Fix access error Jul 4, 2023
@pedrobaeza
Copy link
Member

/ocabot merge patch

Please fw-port it to v16 if applicable

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 15.0-ocabot-merge-pr-774-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit a287d9e into OCA:15.0 Jul 20, 2023
5 of 6 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 7bb4e8d. 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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants