search

OCA / operating-unit

Manage Operating Units in Odoo
GNU Affero General Public License v3.0
72 stars 237 forks source link

[14.0][FIX] OperatingUnit.name_search() omits args domain in "other search" #524

Closed MSokol00 closed 1 year ago

MSokol00 commented 1 year ago

Fix: omit domain passed in args to OperatingUnit.name_search() in "other search"

Currently when we create model with relation to operating.unit with domain (for example Many2one field with ('company_id','=',company_id) domain) and make field available in form view, users will be able to select only operating units that fulfill domain condition, unless they start to type name to field to search for it. Then all operating units that user have access to will be available, not taking into account domain condition.

This PR fixes that issue.

AaronHForgeFlow commented 1 year ago

/ocabot merge patch

@MSokol00 can you do the forward port?

OCA-git-bot commented 1 year ago

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 14.0-ocabot-merge-pr-524-by-AaronHForgeFlow-bump-patch, awaiting test results.

OCA-git-bot commented 1 year ago

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