[REF] web: condition_tree: split file

The file condition_tree.js has become quite big. We split it in several
files so that the role of various parts are clearer. Along the way, we
have reworked a bit some functions, notably toAST.

Part-of: odoo#216038
Related: odoo/enterprise#88759
Signed-off-by: Michaël Mattiello (mcm) <[email protected]>

Author: Polymorphe57

addons/web/static/src/core/domain_selector/domain_selector.js

@@ -1,21 +1,18 @@
 import { Component, onWillStart, onWillUpdateProps } from "@odoo/owl";
-import { Domain } from "@web/core/domain";
-import { TreeEditor } from "@web/core/tree_editor/tree_editor";
-import {
-    domainFromTree,
-    treeFromDomain,
-    formatValue,
-    condition,
-    constructTree,
-} from "@web/core/tree_editor/condition_tree";
 import { CheckBox } from "@web/core/checkbox/checkbox";
-import { deepEqual } from "@web/core/utils/objects";
+import { Domain } from "@web/core/domain";
 import { getDomainDisplayedOperators } from "@web/core/domain_selector/domain_selector_operator_editor";
-import { getOperatorEditorInfo } from "@web/core/tree_editor/tree_editor_operator_editor";
 import { _t } from "@web/core/l10n/translation";
 import { ModelFieldSelector } from "@web/core/model_field_selector/model_field_selector";
-import { useService } from "@web/core/utils/hooks";
+import { condition, formatValue } from "@web/core/tree_editor/condition_tree";
+import { constructTreeFromDomain } from "@web/core/tree_editor/construct_tree_from_domain";
+import { domainFromTree } from "@web/core/tree_editor/domain_from_tree";
+import { TreeEditor } from "@web/core/tree_editor/tree_editor";
+import { getOperatorEditorInfo } from "@web/core/tree_editor/tree_editor_operator_editor";
+import { treeFromDomain } from "@web/core/tree_editor/tree_from_domain";
 import { useMakeGetFieldDef } from "@web/core/tree_editor/utils";
+import { useService } from "@web/core/utils/hooks";
+import { deepEqual } from "@web/core/utils/objects";
 import { getDefaultCondition } from "./utils";
 
 const ARCHIVED_CONDITION = condition("active", "in", [true, false]);
@@ -69,9 +66,11 @@ export class DomainSelector extends Component {
             return;
         }
 
-        const getFieldDef = await this.makeGetFieldDef(p.resModel, constructTree(domain), [
-            "active",
-        ]);
+        const getFieldDef = await this.makeGetFieldDef(
+            p.resModel,
+            constructTreeFromDomain(domain),
+            ["active"]
+        );
 
         this.tree = treeFromDomain(domain, {
             getFieldDef,

addons/web/static/src/core/domain_selector/utils.js

@@ -1,8 +1,9 @@
-import { getDefaultValue } from "@web/core/tree_editor/tree_editor_value_editors";
 import { getDomainDisplayedOperators } from "@web/core/domain_selector/domain_selector_operator_editor";
-import { useService } from "@web/core/utils/hooks";
-import { domainFromTree, condition } from "@web/core/tree_editor/condition_tree";
+import { condition } from "@web/core/tree_editor/condition_tree";
+import { domainFromTree } from "@web/core/tree_editor/domain_from_tree";
+import { getDefaultValue } from "@web/core/tree_editor/tree_editor_value_editors";
 import { getDefaultPath } from "@web/core/tree_editor/utils";
+import { useService } from "@web/core/utils/hooks";
 
 export function getDefaultCondition(fieldDefs) {
     const defaultPath = getDefaultPath(fieldDefs);

addons/web/static/src/core/expression_editor/expression_editor.js

@@ -1,16 +1,14 @@
 import { Component, onWillStart, onWillUpdateProps } from "@odoo/owl";
 import { getExpressionDisplayedOperators } from "@web/core/expression_editor/expression_editor_operator_editor";
-import {
-    condition,
-    expressionFromTree,
-    treeFromExpression,
-} from "@web/core/tree_editor/condition_tree";
+import { _t } from "@web/core/l10n/translation";
+import { ModelFieldSelector } from "@web/core/model_field_selector/model_field_selector";
+import { condition } from "@web/core/tree_editor/condition_tree";
+import { expressionFromTree } from "@web/core/tree_editor/expression_from_tree";
 import { TreeEditor } from "@web/core/tree_editor/tree_editor";
 import { getOperatorEditorInfo } from "@web/core/tree_editor/tree_editor_operator_editor";
 import { getDefaultValue } from "@web/core/tree_editor/tree_editor_value_editors";
+import { treeFromExpression } from "@web/core/tree_editor/tree_from_expression";
 import { getDefaultPath } from "@web/core/tree_editor/utils";
-import { ModelFieldSelector } from "@web/core/model_field_selector/model_field_selector";
-import { _t } from "@web/core/l10n/translation";
 
 export class ExpressionEditor extends Component {
     static template = "web.ExpressionEditor";

addons/web/static/src/core/tree_editor/ast_utils.js

@@ -0,0 +1,30 @@
+import { COMPARATORS, TERM_OPERATORS_NEGATION_EXTENDED } from "./operators";
+
+export function isBool(ast) {
+    return ast.type === 8 && ast.fn.type === 5 && ast.fn.value === "bool" && ast.args.length === 1;
+}
+
+export function isNot(ast) {
+    return ast.type === 6 && ast.op === "not";
+}
+
+export function not(ast) {
+    if (isNot(ast)) {
+        return ast.right;
+    }
+    if (ast.type === 2) {
+        return { ...ast, value: !ast.value };
+    }
+    if (ast.type === 7 && COMPARATORS.includes(ast.op)) {
+        return { ...ast, op: TERM_OPERATORS_NEGATION_EXTENDED[ast.op] }; // do not use this if ast is within a domain context!
+    }
+    return { type: 6, op: "not", right: isBool(ast) ? ast.args[0] : ast };
+}
+
+export function isValidPath(ast, options) {
+    const getFieldDef = options.getFieldDef || (() => null);
+    if (ast.type === 5) {
+        return getFieldDef(ast.value) !== null;
+    }
+    return false;
+}