Skip to content

[19.0][FIX] base_user_role: make role_user_ids searchable#469

Open
AaronHForgeFlow wants to merge 1 commit into
OCA:19.0from
ForgeFlow:19.0-base_user_role-fix-role-user-search
Open

[19.0][FIX] base_user_role: make role_user_ids searchable#469
AaronHForgeFlow wants to merge 1 commit into
OCA:19.0from
ForgeFlow:19.0-base_user_role-fix-role-user-search

Conversation

@AaronHForgeFlow

Copy link
Copy Markdown

Searching by user in the res.users.role list/search view raised 'Cannot convert res.users.role.role_user_ids to SQL because it is not stored' because role_user_ids is a non-stored computed One2many.

Add a search method redirecting the query to the stored line_ids.user_id relation.

Searching by user in the res.users.role list/search view raised
'Cannot convert res.users.role.role_user_ids to SQL because it is not
stored' because role_user_ids is a non-stored computed One2many.

Add a search method redirecting the query to the stored line_ids.user_id
relation.
@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hi @sebalix, @jcdrubay, @novawish,
some modules you are maintaining are being modified, check this out!

@LoisRForgeFlow LoisRForgeFlow left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

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.

5 participants