From cc081a61d532b22675ccc352eb811a7342dcde9a Mon Sep 17 00:00:00 2001
From: "ICX\\Tatsiana.Hashtold" <tanya@form.io>
Date: Wed, 28 Feb 2024 11:49:33 +0300
Subject: [PATCH] FIO-7956: fixed an iisued wehre simple condition based on
 stringified checkbox value is not executed correctly

---
 src/Webform.unit.js                        | 18 +++++++++
 src/utils/conditionOperators/IsEqualTo.js  |  2 +-
 test/forms/formsWithNewSimpleConditions.js | 44 +++++++++++++++++++++-
 3 files changed, 62 insertions(+), 2 deletions(-)

diff --git a/src/Webform.unit.js b/src/Webform.unit.js
index 1b2d263758..22e0ecfd7f 100644
--- a/src/Webform.unit.js
+++ b/src/Webform.unit.js
@@ -3033,6 +3033,24 @@ describe('Webform tests', function() {
         })
         .catch(done);
     });
+
+    it('Should hide field if the checkbox based condition with string value is met', function(done) {
+      const formElement = document.createElement('div');
+      const form = new Webform(formElement);
+      const formCopy = fastCloneDeep(formsWithNewSimpleConditions.form7);
+
+      form.setForm(formCopy).then(() => {
+        const conditionalComponent = form.getComponent('textField');
+        assert.equal(conditionalComponent.visible, true, 'Component should be conditionally visible');
+
+        form.setValue({ data: { checkbox: true } });
+
+        setTimeout(() => {
+          assert.equal(conditionalComponent.visible, false, 'Component should be conditionally hidden');
+          done();
+        }, 300);
+      }).catch((err) => done(err));
+    });
   });
 
   describe('Calculate Value with allowed manual override', () => {
diff --git a/src/utils/conditionOperators/IsEqualTo.js b/src/utils/conditionOperators/IsEqualTo.js
index 53c94b657d..d8eece5f12 100644
--- a/src/utils/conditionOperators/IsEqualTo.js
+++ b/src/utils/conditionOperators/IsEqualTo.js
@@ -12,7 +12,7 @@ export default class IsEqualTo extends ConditionOperator {
     }
 
     execute({ value, comparedValue, instance, conditionComponentPath }) {
-        if (value && comparedValue && typeof value !== typeof comparedValue && _.isString(comparedValue)) {
+        if ((value || value === false) && comparedValue && typeof value !== typeof comparedValue && _.isString(comparedValue)) {
             try {
                 comparedValue = JSON.parse(comparedValue);
             }
diff --git a/test/forms/formsWithNewSimpleConditions.js b/test/forms/formsWithNewSimpleConditions.js
index dee43a8691..c22a282d34 100644
--- a/test/forms/formsWithNewSimpleConditions.js
+++ b/test/forms/formsWithNewSimpleConditions.js
@@ -1022,11 +1022,53 @@ const form6 = {
   machineName: 'cpxkpoxmfvhivle:selectBoxesCond',
 };
 
+const form7 = {
+  type: 'form',
+  display: 'form',
+  components: [
+    {
+      label: 'Checkbox',
+      tableView: false,
+      key: 'checkbox',
+      type: 'checkbox',
+      input: true,
+    },
+    {
+      label: 'Text Field',
+      applyMaskOn: 'change',
+      tableView: true,
+      key: 'textField',
+      conditional: {
+        show: true,
+        conjunction: 'all',
+        conditions: [
+          {
+            component: 'checkbox',
+            operator: 'isEqual',
+            value: 'false',
+          },
+        ],
+      },
+      type: 'textfield',
+      input: true,
+    },
+    {
+      type: 'button',
+      label: 'Submit',
+      key: 'submit',
+      disableOnInvalid: true,
+      input: true,
+      tableView: false,
+    },
+  ],
+};
+
 export default {
   form1,
   form2,
   form3,
   form4,
   form5,
-  form6
+  form6,
+  form7
 };