diff options
| author | fw-bot <otis@odoo.com> | 2019-10-31 20:09:23 +0000 |
|---|---|---|
| committer | fw-bot <otis@odoo.com> | 2019-11-04 08:16:40 +0000 |
| commit | 5e6e2184f55b51c1f55374570604dbeb9c601184 (patch) | |
| tree | 7da8d2fa57ca1d5bceaa35268d6b8b88695a433b /doc/reference/javascript_reference.rst | |
| parent | baba6be668d96d26ef2890617246cb4a6aa82f75 (diff) | |
[FIX] core: is_false has lots of false negatives over RPC
`is_false` relies on no small part on equality tests between the input
triplets and either `TRUE_LEAF` or `FALSE_LEAF`. While these are
defined as tuples, RPC domains will always be lists (as neither
XML-RPC nor JSON have tuples, and their arrays deserialize to Python
lists).
This is an issue, because tuple and list never compare equal. As a
result, while the in / not in predicates can succeed, the TRUE_LEAF /
FALSE_LEAF never will, and thus domains which contain either and might
shortcut (avoid a query entirely) will always go through the entire
process.
Fix by having domain normalization also ensure all triplets are
tuples: that's the first thing `is_false` does, it should never cause
issues and could fix / improve / shortcut other routines.
Note: Also implements TRUE_LEAF and FALSE_LEAF handling in the
SSF's modifiers evaluator. And fixes the ValueError to work
correctly if it breaks on a tuple / dict.
closes odoo/odoo#39706
X-original-commit: a44f008b918ee66742f7e943b00e4ee3d7fe2b78
Signed-off-by: Xavier Morel (xmo) <xmo@odoo.com>
Diffstat (limited to 'doc/reference/javascript_reference.rst')
0 files changed, 0 insertions, 0 deletions
