Merge branch 'fix/attrs-undefined-bug' into v2-develop

gisce · Sep 6, 2024 · 0f83250 · 0f83250
2 parents c299b6c + 7f39e0d
commit 0f83250
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 1 deletion.
diff --git a/src/helpers/attributeParser.ts b/src/helpers/attributeParser.ts
@@ -94,7 +94,8 @@ const evaluateFieldComparison = ({
     valueInObject === undefined &&
     fields[fieldName].type !== "boolean" &&
     fields[fieldName].type !== "many2one" &&
-    fields[fieldName].type !== "selection"
+    fields[fieldName].type !== "selection" &&
+    typeof expectedValue !== "boolean"
   ) {
     return {
       modifiedValueInObject: null,

diff --git a/src/spec/Form.spec.ts b/src/spec/Form.spec.ts
@@ -5773,4 +5773,27 @@ describe("A Form", () => {
     expect(form.contextForFields.street).toBeDefined();
     expect(form.contextForFields.street).toEqual({ active_test: false });
   });
+  it("Should be able to parse attributes with an undefined char value and exist condition in attrs", () => {
+    const arch = `<form><group><alert colspan="4" name="alert1" alert_type="warning" title="Atenci&#243;n" text="Debes introducir un CUPS para iniciar tu oferta correctamente" attrs="{'invisible': [('cups', '!=', False)]}" json_attrs="{&quot;invisible&quot;: {&quot;rules&quot;: [{&quot;operator&quot;: &quot;!=&quot;, &quot;field&quot;: &quot;cups&quot;, &quot;value&quot;: false}], &quot;condition&quot;: &quot;AND&quot;}}"/>
+                <alert colspan="2" name="alert2" alert_type="info" title="Atenci&#243;n" text="alert2" attrs="{'invisible': [('cups', '=', False)]}" json_attrs="{&quot;invisible&quot;: {&quot;rules&quot;: [{&quot;operator&quot;: &quot;=&quot;, &quot;field&quot;: &quot;cups&quot;, &quot;value&quot;: false}], &quot;condition&quot;: &quot;AND&quot;}}"/>
+             </group><input type="char" name="cups" /></form>`;
+    const fields = {
+      cups: {
+        is_function: false,
+        size: 22,
+        string: "CUPS (*)",
+        type: "char",
+        views: {},
+      },
+    };
+    const form = new Form(fields);
+    form.parse(arch, { values: { cups: undefined } });
+    expect(form.type).toBe("form");
+    const alert1 = form.findById("alert1")!;
+    const alert2 = form.findById("alert2")!;
+    expect(alert1).toBeDefined();
+    expect(alert2).toBeDefined();
+    expect(alert1.invisible).toBeFalsy();
+    expect(alert2.invisible).toBeTruthy();
+  });
 });