[16.0][brand_external_report_layout][FIX] Add missing sudo() in template

OCA / brand

Manage brands for products and companies

GNU Affero General Public License v3.0

48 stars 164 forks source link

[16.0][brand_external_report_layout][FIX] Add missing sudo() in template #177

Closed Sibert-Aerts closed 1 year ago

Sibert-Aerts commented 1 year ago

The inherited external layout template attempts to read a field "key" from a Brand's external_report_layout_id, which is an ir.ui.view record, which by standard Odoo most users do not have read access on.

The result is users with limited permissions receive an unrelated access error when rendering external PDF reports.

OCA-git-bot commented 1 year ago

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

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

sbejaoui commented 1 year ago

/ocabot merge patch

OCA-git-bot commented 1 year ago

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 16.0-ocabot-merge-pr-177-by-sbejaoui-bump-patch, awaiting test results.

OCA-git-bot commented 1 year ago

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