search

OCA / edi-framework

GNU Affero General Public License v3.0
7 stars 29 forks source link

[16.0][FIX] edi_oca: consider parent record when checking access rule #109

Closed JordiMForgeFlow closed 6 days ago

JordiMForgeFlow commented 2 months ago

This fix prevents an error raised when you have child exchanges that do not have a specific model and res_id assigned.

When getting the related record of a child we are already looking at the parent if the exchange record does not have a specific model set (https://github.com/OCA/edi-framework/blob/d484f0dcbb7b1caca1aaae0f9a3c2b3510a714f5/edi_oca/models/edi_exchange_record.py#L210). The same should be done when checking the access.

OCA-git-bot commented 2 months ago

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

JordiMForgeFlow commented 1 week ago

@etobella @simahawk could you review this one? :)

JordiMForgeFlow commented 1 week ago

@simahawk done :)

simahawk commented 6 days ago

/ocabot merge patch

OCA-git-bot commented 6 days ago

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

OCA-git-bot commented 6 days ago

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