@ukhomeoffice/cop-react-form-renderer 6.15.8-alpha → 6.15.8

package/dist/components/FormRenderer/helpers/clearOutUncompletedRoutesUtils.js

@@ -3,28 +3,43 @@
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
-exports.removeObjectWithSingleIdFieldInPlace = exports.removeEmptyArraysAndUnusedCollectionIDs = exports.isShowEntity = exports.getNestedQuestionPath = exports.getImmediateParent = exports.getDependencyObjectFromPath = exports.getDependencies = exports.findComponentDefinitionInForm = exports.deleteNodeByPath = exports.deleteCorrespondingMetaInfo = void 0;
+exports.toArray = exports.removeObjectWithOnlySingleIdField = exports.removeEmptyArraysAndUnusedCollectionIDs = exports.pruneCollectionEntry = exports.isShowEntity = exports.getNestedQuestionPath = exports.getImmediateParent = exports.getDependencyObjectFromPath = exports.getDependencies = exports.findComponentDefinitionInForm = exports.deleteNodeByPath = exports.deleteNodeAndOptions = exports.deleteCorrespondingMetaInfo = exports.deleteComponentData = exports.addValue = void 0;
 var _Condition = _interopRequireDefault(require("../../../utils/Condition"));
 function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e }; }
+/* eslint-disable no-param-reassign */
+
 /**
- * Given a path, remove a node from this path within an object.
+ * Add a value to a map of arrays. If the key exists, append the value to the array.
+ * If not, create the map entry with that key.
  * 
+ * @param {*} key 
+ * @param {*} value 
+ * @param {*} multiMap 
+ */
+const addValue = (key, value, multiMap) => {
+  if (!multiMap.has(key)) {
+    multiMap.set(key, []);
+  }
+  multiMap.get(key).push(value);
+};
+
+/**
+ * Given a path, remove a node from this path within an object. 
  * 
- * @param {Object} obj Javascript object from which the node will be deleted. Updated by the method.
+ * @param {Object} payload Javascript object from which the node will be deleted. Updated by the method.
  * @param {String} path A string containing a decimal point delimited path specifying the node to delete.
- * @return Nothing, obj above updated.
+ * @return {void}, obj above updated.
  */
-
-/* eslint-disable consistent-return, no-param-reassign */
-const deleteNodeByPath = (obj, path) => {
-  if (Array.isArray(obj)) {
-    // If obj is an array, recursively call deleteNodeByPath on each element
-    for (let i = 0; i < obj.length; i += 1) {
-      deleteNodeByPath(obj[i], path);
+exports.addValue = addValue;
+const deleteNodeByPath = (payload, path) => {
+  if (Array.isArray(payload)) {
+    // If payload is an array, recursively call deleteNodeByPath on each element
+    for (let i = 0; i < payload.length; i += 1) {
+      deleteNodeByPath(payload[i], path);
     }
   }
   const keys = path.split('.');
-  let current = obj;
+  let current = payload;
   for (let i = 0; i < keys.length - 1; i += 1) {
     current = current[keys[i]];
     if (current === undefined) {
@@ -32,27 +47,8 @@ const deleteNodeByPath = (obj, path) => {
     }
   }
   if (current[keys[keys.length - 1]]) {
-    console.log("Deleting element : ".concat(keys[keys.length - 1]));
     delete current[keys[keys.length - 1]];
   }
-  return obj;
-};
-
-/**
- * Pruning a collection payload may have resulted in objects that are only left with their 'id' field, which isn't data 
- * but added by the renderer to find the activeId. If so, remove these objects entirely as they have been pruned.
- * 
- * @param {Array} array Array of objects. Each object which has only 1 remaining field called 'id' should be removed.
- * @return Nothing, array above updated.
- */
-exports.deleteNodeByPath = deleteNodeByPath;
-const removeObjectWithSingleIdFieldInPlace = array => {
-  for (let i = array.length - 1; i >= 0; i -= 1) {
-    const obj = array[i];
-    if (Object.keys(obj).length === 1 && Object.keys(obj)[0] === 'id') {
-      array.splice(i, 1);
-    }
-  }
 };
 
 /**
@@ -63,7 +59,7 @@ const removeObjectWithSingleIdFieldInPlace = array => {
  * @param {String} path Decimal point delimited path.
  * @returns {String} Immediate parent of the path passed in.
  */
-exports.removeObjectWithSingleIdFieldInPlace = removeObjectWithSingleIdFieldInPlace;
+exports.deleteNodeByPath = deleteNodeByPath;
 const getImmediateParent = path => {
   if (typeof path !== 'string' || !path.includes('.')) {
     return null;
@@ -89,6 +85,53 @@ const getNestedQuestionPath = (optionPath, nestedFieldId) => {
   return parentPath ? "".concat(parentPath, ".").concat(nestedFieldId) : nestedFieldId;
 };
 
+/** 
+ * If the component has options, go through each option removing the data for any nested fields. 
+ * Required as nested options are in the payload at the same heirarchical level.
+ * 
+ * There are occasions when a field shouldn't be removed from the payload, eg if a form is being used for 2 purposes
+ * eg the cop-reassign-task-to-rcc.json, and the field is only being hidden for display purposes. Allow these fields
+ * to be preserved with a property on the component.
+
+ * @param {Object} payload Javascript object from which the node will be deleted. Updated by the method.
+ * @param {String} path A string containing a decimal point delimited path specifying the node to delete.
+ * @param {Object} component The form component representing the path being deleted
+ * @return {void}, obj above updated.
+ */
+exports.getNestedQuestionPath = getNestedQuestionPath;
+const deleteNodeAndOptions = (payload, path, component) => {
+  var _component$data;
+  if (component.preserveInPayload) return;
+  deleteNodeByPath(payload, path);
+  // If the component has options, go through each option removing the data for any nested fields. Required as nested options are in the payload at the same heirarchical level.
+  if (component !== null && component !== void 0 && (_component$data = component.data) !== null && _component$data !== void 0 && _component$data.options) {
+    var _component$data2;
+    component === null || component === void 0 || (_component$data2 = component.data) === null || _component$data2 === void 0 || (_component$data2 = _component$data2.options) === null || _component$data2 === void 0 || _component$data2.forEach(option => {
+      var _option$nested;
+      (_option$nested = option.nested) === null || _option$nested === void 0 || _option$nested.forEach(nested => {
+        deleteNodeByPath(payload, getNestedQuestionPath(path, nested.fieldId));
+      });
+    });
+  }
+};
+
+/**
+ * Pruning a collection payload may have resulted in objects that are only left with their 'id' field, which isn't data 
+ * but added by the renderer to find the activeId. If so, remove these objects entirely as they have been pruned.
+ * 
+ * @param {Array} array Array of objects. Each object which has only 1 remaining field called 'id' should be removed.
+ * @return {void}, array above updated.
+ */
+exports.deleteNodeAndOptions = deleteNodeAndOptions;
+const removeObjectWithOnlySingleIdField = array => {
+  for (let i = array.length - 1; i >= 0; i -= 1) {
+    const obj = array[i];
+    if (Object.keys(obj).length === 1 && Object.keys(obj)[0] === 'id') {
+      array.splice(i, 1);
+    }
+  }
+};
+
 /**
  * Helper method to establish all the payload paths (dependencies) that an entity (page or component) is 
  * dependent on through its show_when rule.
@@ -102,7 +145,7 @@ const getNestedQuestionPath = (optionPath, nestedFieldId) => {
  * @param {Object} entity Entity whose show_when rule is to be searched for 'field' data items within.
  * @returns {Set} Set of payload paths that the entity (page or component) is dependent on.
  */
-exports.getNestedQuestionPath = getNestedQuestionPath;
+exports.removeObjectWithOnlySingleIdField = removeObjectWithOnlySingleIdField;
 const getDependencies = entity => {
   const findShowWhenFields = (showWhenObject, showWhenFields) => {
     if (typeof showWhenObject === 'object' && showWhenObject !== null) {
@@ -200,11 +243,13 @@ exports.isShowEntity = isShowEntity;
 const findComponentDefinitionInForm = (pageComponentDef, componentByIdMap, componentByFieldIdMap) => {
   var _ref, _componentByIdMap$get;
   const componentInForm = (_ref = (_componentByIdMap$get = componentByIdMap.get(pageComponentDef.use)) !== null && _componentByIdMap$get !== void 0 ? _componentByIdMap$get : componentByFieldIdMap.get(pageComponentDef.use)) !== null && _ref !== void 0 ? _ref : pageComponentDef;
+
+  // Create clone of component, so processing can make changes to it without changing the form
+  const componentInFormClone = JSON.parse(JSON.stringify(componentInForm));
   if (pageComponentDef.use && pageComponentDef.show_when) {
-    componentInForm.show_when = pageComponentDef.show_when;
+    componentInFormClone.show_when = pageComponentDef.show_when;
   }
-  // Retun clone of component, so subsequent processing can make changes to it without changing the form
-  return JSON.parse(JSON.stringify(componentInForm));
+  return componentInFormClone;
 };
 
 /**
@@ -216,7 +261,7 @@ const findComponentDefinitionInForm = (pageComponentDef, componentByIdMap, compo
  * @param {*} component The component definition being deleted that needs corresponding meta data also deleted
  * @param {*} collectionDataObject The payload containing the file data being deleted (for which the corresponding meta needs deleting)
  * @param {*} formData The entire payload, which will include the meta section
- * @returns Nothing, as the formData will be updated in situ
+ * @returns {void}, as the formData will be updated in situ
  */
 exports.findComponentDefinitionInForm = findComponentDefinitionInForm;
 const deleteCorrespondingMetaInfo = (component, collectionDataObject, formData) => {
@@ -236,6 +281,7 @@ const deleteCorrespondingMetaInfo = (component, collectionDataObject, formData)
     }
   }
 };
+
 /**
  * After the payload has been cleansed of individual data items, empty arrays and objects may remain.
  * Removing an array (when the payload for a collection) may leave a redundant activeId field. The active Id 
@@ -252,6 +298,7 @@ const deleteCorrespondingMetaInfo = (component, collectionDataObject, formData)
  * 3. The function operates recursively to handle deeply nested structures.
  *
  * @param {any} payload - The input data structure, which can be an array or an object.
+ * 
  */
 exports.deleteCorrespondingMetaInfo = deleteCorrespondingMetaInfo;
 const removeEmptyArraysAndUnusedCollectionIDs = payload => {
@@ -274,12 +321,10 @@ const removeEmptyArraysAndUnusedCollectionIDs = payload => {
         // If the array being removed has an activeId associated with it, remove it
         if (payload["".concat(key, "ActiveId")]) {
           delete payload["".concat(key, "ActiveId")];
-          console.log("Deleted element : ".concat(key, "ActiveId}"));
         }
         delete payload["".concat(key, "ActiveId")];
         if (payload[key]) {
           delete payload[key];
-          console.log("Deleted element : ".concat(key));
         }
       } else {
         removeEmptyArraysAndUnusedCollectionIDs(payload[key]); // Recurse for nested structures
@@ -287,4 +332,58 @@ const removeEmptyArraysAndUnusedCollectionIDs = payload => {
     });
   }
 };
-exports.removeEmptyArraysAndUnusedCollectionIDs = removeEmptyArraysAndUnusedCollectionIDs;
+
+/**
+ * Delete a component's payload item from the overall payload.
+ * A component can be defined in >1 places in the form spec. To cater for this, 
+ * if the componentsToKeep counter tells us that there is > 1 uses of this component
+ * still unaccounted for in the form then don't delete, but reduce the count by 1.
+ * 
+ * When the counter reaches 1 we know all other occurences of the component have been resolved
+ * so it is safe to delete.
+ * 
+ * 
+ * @param {*} payload The form payload from which to delete the component data
+ * @param {*} path The payload path of the component
+ * @param {*} component The component whose data we should attempt to delete
+ * @param {*} componentsToKeep A list of all components with a count of their number of uses in the form
+ */
+exports.removeEmptyArraysAndUnusedCollectionIDs = removeEmptyArraysAndUnusedCollectionIDs;
+const deleteComponentData = (payload, path, component, componentsToKeep) => {
+  if (componentsToKeep[path] > 1) {
+    componentsToKeep[path] -= 1;
+  } else {
+    deleteNodeAndOptions(payload, path, component);
+  }
+};
+
+/**
+ * 
+ * Takes a single page collection payload object and removes the payload items specified in the componentsToPrune list
+ * as long as they don't appear in the componentsToKeep list.
+ * Additionally, if the component type is multifile, remove the corresponding data entries that will have been created 
+ * in the meta section of the payload by the form renderer.
+ * 
+ * @param {Set} pathsToKeep paths that we cannot delete from the collectionDataObject
+ * @param {Map} componentsToPrune paths that we should delete, as long as they are not in the pathsToKeep
+ * @param {Object} collectionDataObject the payload from which to delete the paths
+ * @param {Object} formData The form data, whose meta section may include corresponding documents entries for multifile entries
+ */
+exports.deleteComponentData = deleteComponentData;
+const pruneCollectionEntry = (pathsToKeep, componentsToPrune, collectionDataObject, formData) => {
+  componentsToPrune.forEach(component => {
+    if (!pathsToKeep.has(component.fieldId)) {
+      if (component.type === "multifile") {
+        deleteCorrespondingMetaInfo(component, collectionDataObject, formData);
+      }
+      deleteNodeAndOptions(collectionDataObject, component.fieldId, component);
+    }
+  });
+};
+
+/*
+ * Converts an object to an array if it isn't already, for use when combining show when rules.
+ */
+exports.pruneCollectionEntry = pruneCollectionEntry;
+const toArray = value => Array.isArray(value) ? value : [value];
+exports.toArray = toArray;

package/dist/components/FormRenderer/helpers/clearOutUncompletedRoutesUtils.test.js

@@ -19,8 +19,8 @@ describe('deleteNodeByPath', () => {
       epsilon: "epsilon"
     };
     const path = 'type';
-    const result = Utils.deleteNodeByPath(nested, path);
-    expect(result).toEqual({
+    Utils.deleteNodeByPath(nested, path);
+    expect(nested).toEqual({
       alpha: "alpha",
       bravo: "bravo",
       person: {
@@ -44,8 +44,8 @@ describe('deleteNodeByPath', () => {
       epsilon: "epsilon"
     };
     const path = 'person.charlie';
-    const result = Utils.deleteNodeByPath(nested, path);
-    expect(result).toEqual({
+    Utils.deleteNodeByPath(nested, path);
+    expect(nested).toEqual({
       type: "Example form",
       alpha: "alpha",
       bravo: "bravo",
@@ -57,7 +57,7 @@ describe('deleteNodeByPath', () => {
     });
   });
 });
-describe('removeObjectWithSingleIdFieldInPlace', () => {
+describe('removeObjectWithOnlySingleIdField', () => {
   it('removes objects with only the "id" field', () => {
     const inputArray = [{
       id: 1
@@ -73,7 +73,7 @@ describe('removeObjectWithSingleIdFieldInPlace', () => {
       id: 4,
       name: 'John'
     }];
-    Utils.removeObjectWithSingleIdFieldInPlace(inputArray);
+    Utils.removeObjectWithOnlySingleIdField(inputArray);
     expect(inputArray).toEqual([{
       name: 'John'
     }, {
@@ -90,7 +90,7 @@ describe('removeObjectWithSingleIdFieldInPlace', () => {
       age: 30
     }];
     const originalArray = [].concat(inputArray);
-    Utils.removeObjectWithSingleIdFieldInPlace(inputArray);
+    Utils.removeObjectWithOnlySingleIdField(inputArray);
     expect(inputArray).toEqual(originalArray);
   });
 });
@@ -499,4 +499,61 @@ describe("removeEmptyArraysAndUnusedCollectionIDs", () => {
       nested: {}
     }]);
   });
+});
+describe("pruneCollectionEntry", () => {
+  let pathsToKeep;
+  let componentsToPrune;
+  let collectionDataObject;
+  let formData;
+  let expectedFormData;
+  beforeEach(() => {
+    pathsToKeep = new Set(['keepThis']);
+    componentsToPrune = [{
+      fieldId: 'removeThis',
+      type: 'text'
+    }, {
+      fieldId: 'keepThis',
+      type: 'text'
+    }, {
+      fieldId: 'removeMultifile',
+      type: 'multifile'
+    }];
+    collectionDataObject = {
+      'removeThis': 'value',
+      'keepThis': 'value',
+      'removeMultifile': {
+        'url': 'https://example.com/file1.pdf'
+      }
+    };
+    formData = {
+      meta: {
+        documents: [{
+          url: 'https://example.com/file1.pdf'
+        }, {
+          url: 'https://example.com/file2.pdf'
+        }, {
+          url: 'https://example.com/file3.pdf'
+        }]
+      }
+    };
+    expectedFormData = {
+      meta: {
+        documents: [{
+          url: 'https://example.com/file2.pdf'
+        }, {
+          url: 'https://example.com/file3.pdf'
+        }]
+      }
+    };
+  });
+  it('should remove components not in pathsToKeep', () => {
+    Utils.pruneCollectionEntry(pathsToKeep, componentsToPrune, collectionDataObject, formData);
+    expect(collectionDataObject).toEqual({
+      'keepThis': 'value'
+    });
+  });
+  it('should delete meta.documents for multifile components', () => {
+    Utils.pruneCollectionEntry(pathsToKeep, componentsToPrune, collectionDataObject, formData);
+    expect(formData).toEqual(expectedFormData);
+  });
 });

package/dist/components/FormRenderer/onCYAAction.js

@@ -36,7 +36,6 @@ const onCYAAction = (setPagePoint, action, pages, validate, components, data, se
         pages,
         components
       };
-      console.log("Invoking clearOutUncompletedRoutes");
       _helpers.default.clearOutUncompletedRoutes(formPagesAndComponents, submissionData);
       submissionData.formStatus = _helpers.default.getSubmissionStatus(type, pages, pageId, action, submissionData, currentTask, true);
       setData(submissionData);
@@ -81,7 +80,6 @@ const onCYAAction = (setPagePoint, action, pages, validate, components, data, se
           pages,
           components
         };
-        console.log("Invoking clearOutUncompletedRoutes");
         _helpers.default.clearOutUncompletedRoutes(formPagesAndComponents, submissionData);
         submissionData.formStatus = _helpers.default.getSubmissionStatus(type, pages, pageId, action, submissionData, currentTask, true);
         setData(submissionData);

package/dist/components/FormRenderer/onCYAAction.test.js

@@ -55,6 +55,11 @@ jest.mock('./helpers', () => ({
     this.getNextPageIdCalls = 0;
     this.getFormStateCalls = 0;
     this.getSubmissionStatusCalls = 0;
+  },
+  clearOutUncompletedRoutesCalls: 0,
+  clearOutUncompletedRoutes(formPagesAndComponents, formData) {
+    this.clearOutUncompletedRoutesCalls += 1;
+    return formData;
   }
 }));
 jest.mock('../../utils', () => ({

package/dist/components/FormRenderer/onPageAction.js

@@ -112,7 +112,6 @@ const onPageAction = (action, patch, patchLabel, hooks, data, formState, validat
         components
       };
       if (action.type === _models.PageAction.TYPES.SUBMIT) {
-        console.log("Invoking clearOutUncompletedRoutes");
         _helpers.default.clearOutUncompletedRoutes(formPagesAndComponents, submissionData);
       }
 

package/dist/hooks/useGetRequest.js

@@ -8,10 +8,6 @@ var _axios = _interopRequireDefault(require("axios"));
 var _react = require("react");
 var _useAxios = _interopRequireDefault(require("./useAxios"));
 function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e }; }
-// Global imports
-
-// Local imports
-
 // Caches for responses and errors.
 const cache = {};
 const errorCache = {};
@@ -31,15 +27,18 @@ const STATUS_IDLE = exports.STATUS_IDLE = 'idle';
 const STATUS_FETCHING = exports.STATUS_FETCHING = 'fetching';
 const STATUS_FETCHED = exports.STATUS_FETCHED = 'fetched';
 const STATUS_ERROR = exports.STATUS_ERROR = 'error';
-const useGetRequest = url => {
+const useGetRequest = function (url) {
+  let caching = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : true;
   const axiosInstance = (0, _useAxios.default)();
-  const cancelToken = _axios.default.CancelToken.source();
-  const cancelRequests = () => {
-    if (cancelToken) cancelToken.cancel();
-  };
+  const cancelTokenRef = (0, _react.useRef)(_axios.default.CancelToken.source());
   const [status, setStatus] = (0, _react.useState)(STATUS_IDLE);
   const [error, setError] = (0, _react.useState)(null);
   const [data, setData] = (0, _react.useState)(null);
+  const cancelRequests = () => {
+    if (cancelTokenRef.current) {
+      cancelTokenRef.current.cancel();
+    }
+  };
   (0, _react.useEffect)(() => {
     if (!url || !axiosInstance) return;
     const fetchData = async () => {
@@ -47,9 +46,9 @@ const useGetRequest = url => {
         setError(null);
         setStatus(STATUS_FETCHING);
         let fetchedData;
-        if (cache[url]) {
+        if (caching && cache[url]) {
           fetchedData = cache[url];
-        } else if (errorCache[url]) {
+        } else if (caching && errorCache[url]) {
           /**
            * This logic is intended to stop multiple requests being made in succession
            * that all fail. Presently, this will only allow the first request to be
@@ -60,24 +59,25 @@ const useGetRequest = url => {
           throw errorCache[url];
         } else {
           const response = await axiosInstance.get(url, {
-            cancelToken: cancelToken.token
+            cancelToken: cancelTokenRef.current.token
           }).catch(e => {
+            setError(e);
             throw e;
           });
           fetchedData = response.data;
-          cache[url] = fetchedData;
+          if (caching) cache[url] = fetchedData;
         }
         setData(fetchedData);
         setStatus(STATUS_FETCHED);
       } catch (e) {
-        errorCache[url] = e;
+        if (caching) errorCache[url] = e;
         setError(e);
         setData(null);
         setStatus(STATUS_ERROR);
       }
     };
     fetchData();
-  }, [axiosInstance, url, cancelToken.token]);
+  }, [axiosInstance, url, caching]);
   return {
     status,
     error,

package/dist/hooks/useRefData.js

@@ -28,13 +28,14 @@ const getRefDataUrl = component => {
   return undefined;
 };
 const useRefData = (component, formData) => {
+  var _component$data;
   const url = getRefDataUrl(_objectSpread(_objectSpread({}, component), {}, {
     formData
   }));
   const {
     status: _status,
     data: _data
-  } = (0, _useGetRequest.default)(url);
+  } = (0, _useGetRequest.default)(url, component === null || component === void 0 || (_component$data = component.data) === null || _component$data === void 0 ? void 0 : _component$data.useCache);
   const [data, setData] = (0, _react.useState)([]);
   const [status, setStatus] = (0, _react.useState)(STATUS_LOADING);
   (0, _react.useEffect)(() => {
@@ -52,7 +53,7 @@ const useRefData = (component, formData) => {
       setData([]);
       setStatus(STATUS_COMPLETE);
     }
-  }, [component, _status, _data, url, setData, setStatus]);
+  }, [component.id, _status, _data, url]);
   return {
     data,
     status

package/dist/utils/Component/getComponentTests/getComponent.multifile.test.js

@@ -23,7 +23,8 @@ describe('utils.Component.get', () => {
       fieldId: FIELD_ID,
       label: LABEL,
       onChange: ON_CHANGE,
-      'data-testid': ID
+      'data-testid': ID,
+      allowedTypes: []
     };
     const {
       container

package/dist/utils/Component/getDefaultValueFromConfig.js

@@ -75,7 +75,8 @@ const setupDefaultObjectValue = (defaultObject, data) => {
   // defaultObject.sourced is an object with values that should
   // be used as field names to get values from data.
   if (defaultObj.sourced) {
-    result = result !== null && result !== void 0 ? result : {};
+    var _result;
+    result = (_result = result) !== null && _result !== void 0 ? _result : {};
     Object.keys(defaultObj.sourced).every(key => {
       const sourcedValue = typeof defaultObj.sourced[key] === 'string' && (0, _getSourceData.default)(data, defaultObj.sourced[key]);
       if (sourcedValue === undefined && defaultObj.skipIfSourceInvalid) {

package/dist/utils/Condition/meetsCondition.js

@@ -20,6 +20,26 @@ const getComparisonValue = (condition, data) => {
   return condition.value;
 };
 
+/**
+ * Processes a value based on its type.
+ * - If it's an array, returns its length.
+ * - If it's a string, parses it as a float.
+ * - If it's a number, returns it as is.
+ * - Returns `undefined` for other types.
+ * 
+ * @param {any} value - The value to process.
+ * @returns {number | undefined} - The processed value or `undefined` if not applicable.
+ */
+const getProcessedValue = obj => {
+  if (Array.isArray(obj)) return obj.length;
+  if (typeof obj === "string") {
+    const cleaned = obj.replace(/,/g, ''); // Remove all commas
+    return Number.isNaN(cleaned) ? null : parseFloat(cleaned);
+  }
+  if (typeof obj === "number") return obj;
+  return null;
+};
+
 /**
  * Looks at a condition object to see if the supplied value meets it.
  * The condition object contains an `op` property, along with a comparator
@@ -66,21 +86,15 @@ const meetsCondition = (condition, value, data) => {
         }
       case '<':
         {
-          if (!value || !compare) {
-            return false;
-          }
-          const valFloat = parseFloat(value.replace(',', ''));
-          const compareFloat = parseFloat(compare.replace(',', ''));
-          return valFloat < compareFloat;
+          const valFloat = getProcessedValue(value);
+          const compareFloat = getProcessedValue(compare);
+          return valFloat != null && compareFloat != null && valFloat < compareFloat;
         }
       case '>':
         {
-          if (!value || !compare) {
-            return false;
-          }
-          const valFloat = parseFloat(value.replace(/,/g, ''));
-          const compareFloat = parseFloat(compare.replace(/,/g, ''));
-          return valFloat > compareFloat;
+          const valFloat = getProcessedValue(value);
+          const compareFloat = getProcessedValue(compare);
+          return valFloat != null && compareFloat != null && valFloat > compareFloat;
         }
       case 'contains':
         {

package/dist/utils/Condition/meetsCondition.test.js

@@ -551,6 +551,27 @@ describe('utils.Condition.meetsCondition', () => {
       };
       expect((0, _meetsCondition.default)(CONDITION, '10,000,000')).toBeTruthy();
     });
+    it('greater than should handle value is 0', () => {
+      const CONDITION = {
+        op: '>',
+        value: 0
+      };
+      expect((0, _meetsCondition.default)(CONDITION, 1)).toBeTruthy();
+    });
+    it('greater than should handle compear is 0', () => {
+      const CONDITION = {
+        op: '<',
+        value: 1
+      };
+      expect((0, _meetsCondition.default)(CONDITION, 0)).toBeTruthy();
+    });
+    it('greater than should handle array is 0', () => {
+      const CONDITION = {
+        op: '>',
+        value: 1
+      };
+      expect((0, _meetsCondition.default)(CONDITION, ["A", "B", "C"])).toBeTruthy();
+    });
   });
   describe('operator includes', () => {
     it('should accept a string that exists within the value', () => {