odoo
odoo copied to clipboard
[IMP] core: introduce condition operator 'any*' to simplify joins in queries
Description of the issue/feature this PR addresses:
Current behavior before PR:
Desired behavior after PR is merged:
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr
I just fixed up some commit messages.
@odoo/rd-security could you check if this is ok in terms of security?
A mere
*suffix seems a bit too glib and easy to miss for disabling ACL checks, and very much not in keeping with the usual lingo for that. Even more so for something which seems mostly targeted at ORM-internal optimisations but will inevitably leak out in short order.
Indeed, we thought of other names like "any_sudo" (the operator is not really a sudo, it bypasses only security at the level of the domain, but not sub-domains). Any suggestion for a better name?
Indeed, we thought of other names like "any_sudo" (the operator is not really a sudo, it bypasses only security at the level of the domain, but not sub-domains). Any suggestion for a better name?
Not really, any_sudo was also what I was thinking about. And while it's not as bad as an an actual sudo due to not being recursive / infective, that is its effect on security checks.
Merge method set to rebase and fast-forward.
