@abgov/jsonforms-components 2.36.1 → 2.37.0

package/index.esm.js

@@ -928,7 +928,6 @@ var PROPER = EXISTS && (function something() { /* empty */ }).name === 'somethin
 var CONFIGURABLE = EXISTS && (!DESCRIPTORS$c || (DESCRIPTORS$c && getDescriptor(FunctionPrototype$1, 'name').configurable));
 
 var functionName = {
-  EXISTS: EXISTS,
   PROPER: PROPER,
   CONFIGURABLE: CONFIGURABLE
 };
@@ -2329,9 +2328,6 @@ var createMethod$2 = function (CONVERT_TO_STRING) {
 };
 
 var stringMultibyte = {
-  // `String.prototype.codePointAt` method
-  // https://tc39.es/ecma262/#sec-string.prototype.codepointat
-  codeAt: createMethod$2(false),
   // `String.prototype.at` method
   // https://github.com/mathiasbynens/String.prototype.at
   charAt: createMethod$2(true)
@@ -4042,9 +4038,7 @@ if (!set || !clear) {
 }
 
 var task$1 = {
-  set: set,
-  clear: clear
-};
+  set: set};
 
 var global$8 = global$q;
 var DESCRIPTORS$3 = descriptors;
@@ -5217,7 +5211,7 @@ const PageReviewValueCol = styled.td(_t5$5 || (_t5$5 = _$a`
 `));
 //Check and unchecked are different heights otherwise
 const CheckboxWrapper = styled.div(_t6$3 || (_t6$3 = _$a`
-  height: 28px;
+  min-height: 28px;
 `));
 styled.h4(_t7$2 || (_t7$2 = _$a`
   margin: 0 0 0.25rem 0;
@@ -5400,12 +5394,6 @@ var createMethod$1 = function (TYPE) {
 };
 
 var stringTrim = {
-  // `String.prototype.{ trimLeft, trimStart }` methods
-  // https://tc39.es/ecma262/#sec-string.prototype.trimstart
-  start: createMethod$1(1),
-  // `String.prototype.{ trimRight, trimEnd }` methods
-  // https://tc39.es/ecma262/#sec-string.prototype.trimend
-  end: createMethod$1(2),
   // `String.prototype.trim` method
   // https://tc39.es/ecma262/#sec-string.prototype.trim
   trim: createMethod$1(3)
@@ -5884,7 +5872,6 @@ const GoAInputText = props => {
           setIsVisited();
         }
         onChangeForInputControl({
-          name,
           value: formattedValue,
           controlProps: props
         });
@@ -5894,7 +5881,6 @@ const GoAInputText = props => {
           setIsVisited();
         }
         onBlurForTextControl({
-          name,
           controlProps: props,
           value: autoCapitalize ? value.toUpperCase() : value
         });
@@ -5951,14 +5937,12 @@ const MultiLineText = props => {
       }
       if (value.length === 0 || required && errors.length === 0 && value.length > 0) {
         onKeyPressForTextControl({
-          name,
           value: newValue,
           key,
           controlProps: props
         });
       }
       onChangeForInputControl({
-        name,
         value: newValue,
         controlProps: props
       });
@@ -6079,14 +6063,12 @@ const GoADateInput = props => {
         setIsVisited();
       }
       onChangeForDateControl({
-        name,
         value,
         controlProps: props
       });
     },
     onKeyPress: (name, value, key) => {
       onKeyPressForDateControl({
-        name,
         value,
         key,
         controlProps: props
@@ -6097,7 +6079,6 @@ const GoADateInput = props => {
         setIsVisited();
       }
       onBlurForDateControl({
-        name,
         value,
         controlProps: props
       });
@@ -6152,7 +6133,6 @@ const GoANumberInput = props => {
         setIsVisited();
       }
       onBlurForNumericControl({
-        name,
         value,
         controlProps: props
       });
@@ -6162,7 +6142,6 @@ const GoANumberInput = props => {
         setIsVisited();
       }
       onChangeForNumericControl({
-        name,
         value,
         controlProps: props
       });
@@ -6217,7 +6196,6 @@ const GoAInputInteger = props => {
         setIsVisited();
       }
       onBlurForNumericControl({
-        name,
         value,
         controlProps: props
       });
@@ -6227,7 +6205,6 @@ const GoAInputInteger = props => {
         setIsVisited();
       }
       onChangeForNumericControl({
-        name,
         value,
         controlProps: props
       });
@@ -6269,14 +6246,12 @@ const GoADateTimeInput = props => {
         setIsVisited();
       }
       onChangeForDateTimeControl({
-        name,
         value,
         controlProps: props
       });
     },
     onKeyPress: (name, value, key) => {
       onKeyPressForDateControl({
-        name,
         value,
         key,
         controlProps: props
@@ -6287,7 +6262,6 @@ const GoADateTimeInput = props => {
         setIsVisited();
       }
       onBlurForDateControl({
-        name,
         value,
         controlProps: props
       });
@@ -6341,7 +6315,6 @@ const GoATimeInput = props => {
     onChange: (name, value) => {},
     onKeyPress: (name, value, key) => {
       onKeyPressForTimeControl({
-        name,
         value,
         key,
         controlProps: props
@@ -6982,7 +6955,6 @@ const GoAEmailInput = props => {
               setIsVisited(true);
             }
             onChangeForInputControl({
-              name,
               value,
               controlProps: props
             });
@@ -6992,7 +6964,6 @@ const GoAEmailInput = props => {
               setIsVisited(true);
             }
             onBlurForTextControl({
-              name,
               value,
               controlProps: props
             });
@@ -7608,11 +7579,7 @@ var createMethod = function (IS_RIGHT) {
 var arrayReduce = {
   // `Array.prototype.reduce` method
   // https://tc39.es/ecma262/#sec-array.prototype.reduce
-  left: createMethod(false),
-  // `Array.prototype.reduceRight` method
-  // https://tc39.es/ecma262/#sec-array.prototype.reduceright
-  right: createMethod(true)
-};
+  left: createMethod(false)};
 
 var fails$2 = fails$v;
 
@@ -10431,7 +10398,6 @@ const NonEmptyCellComponent$1 = /*#__PURE__*/React.memo(function NonEmptyCellCom
                         error: error === null || error === void 0 ? void 0 : error.message,
                         isRequired: (_a = required === null || required === void 0 ? void 0 : required.includes(tableKeys[element])) !== null && _a !== void 0 ? _a : false,
                         errors: errors !== undefined ? errors : [],
-                        count: count !== undefined ? count : -1,
                         element,
                         rowPath,
                         index: i
@@ -12449,7 +12415,7 @@ const NameInputs = ({
   // eslint-disable-next-line
   const handleRequiredFieldBlur = (name, updatedData) => {
     const err = Object.assign({}, errors);
-    if ((!(data === null || data === void 0 ? void 0 : data[name]) || (data === null || data === void 0 ? void 0 : data[name]) === '') && requiredFields.includes(name) && (!updatedData)) {
+    if ((!(data === null || data === void 0 ? void 0 : data[name]) || (data === null || data === void 0 ? void 0 : data[name]) === '') && requiredFields.includes(name) && (true)) {
       const modifiedName = name === 'firstName' ? 'First name' : name === 'lastName' ? 'Last name' : '';
       err[name] = `${modifiedName} is required`;
     } else {
@@ -13056,71 +13022,191 @@ function resolveScope(scope, data) {
   const cleaned = scope.replace(/^#\/(properties\/)?/, '');
   const parts = cleaned.split('/').filter(Boolean);
   let cur = data;
-  for (const p of parts) {
-    if (cur == null || typeof cur !== 'object') return undefined;
-    cur = cur[p];
+  for (const part of parts) {
+    if (cur == null) return undefined;
+    if (Array.isArray(cur) && /^\d+$/.test(part)) {
+      cur = cur[Number(part)];
+    } else if (typeof cur === 'object') {
+      if (part === 'properties') continue;
+      cur = cur[part];
+    } else {
+      return undefined;
+    }
   }
   return cur;
 }
-function sumColumn(scope, data) {
-  if (!scope || typeof scope !== 'string') return undefined;
-  // normalize scope: remove leading "#/" and optional "properties/"
+function evaluateSum(scope, data) {
+  if (!scope || typeof scope !== 'string') {
+    return {
+      value: undefined,
+      error: 'SUM() requires a JSON pointer argument.'
+    };
+  }
   const cleaned = scope.replace(/^#\/(properties\/)?/, '');
   const parts = cleaned.split('/').filter(Boolean);
-  if (parts.length < 2) return undefined;
-  const colKey = parts[parts.length - 1];
+  if (parts.length < 2) {
+    return {
+      value: undefined,
+      error: `SUM() pointer "${scope}" must include an array and field name.`
+    };
+  }
+  const fieldKey = parts[parts.length - 1];
   const arrPath = parts.slice(0, -1);
   let cur = data;
-  for (const p of arrPath) {
-    if (cur == null) return undefined;
-    // handle numeric indices for arrays
-    if (Array.isArray(cur) && /^\d+$/.test(p)) {
-      cur = cur[Number(p)];
+  for (const segment of arrPath) {
+    if (cur == null) {
+      cur = undefined;
+      break;
+    }
+    if (Array.isArray(cur) && /^\d+$/.test(segment)) {
+      cur = cur[Number(segment)];
+    } else if (typeof cur === 'object') {
+      if (segment === 'properties') continue;
+      cur = cur[segment];
     } else {
-      cur = cur[p];
+      cur = undefined;
+      break;
     }
   }
-  if (!Array.isArray(cur)) return undefined;
+  if (cur == null) {
+    return {
+      value: undefined,
+      error: `Please provide values for: ${scope}`
+    };
+  }
+  if (!Array.isArray(cur)) {
+    return {
+      value: undefined,
+      error: `SUM() pointer "${scope}" does not resolve to an array.`
+    };
+  }
+  const missingIndices = [];
+  const invalidIndices = [];
   let sum = 0;
-  for (const row of cur) {
-    if (row == null) return undefined;
-    const v = row[colKey];
-    if (typeof v !== 'number') return undefined;
-    sum += v;
+  for (let i = 0; i < cur.length; i++) {
+    const row = cur[i];
+    if (row == null || typeof row !== 'object') {
+      missingIndices.push(i);
+      continue;
+    }
+    const v = row[fieldKey];
+    if (v === undefined || v === null || v === '') {
+      missingIndices.push(i);
+    } else if (typeof v !== 'number' || Number.isNaN(v)) {
+      invalidIndices.push(i);
+    } else {
+      sum += v;
+    }
+  }
+  if (invalidIndices.length > 0) {
+    return {
+      value: undefined,
+      error: `Expected numeric values for: ${scope} at rows [${invalidIndices.join(', ')}]`
+    };
   }
-  return sum;
+  if (missingIndices.length === cur.length) {
+    return {
+      value: undefined,
+      error: `Please provide values for: ${scope}`
+    };
+  }
+  return {
+    value: sum,
+    error: missingIndices.length ? `Some rows are missing values for: ${scope}` : undefined
+  };
 }
+/**
+ * General expression evaluation.
+ * - Supports SUM(#/properties/arr/c3)
+ * - Supports arithmetic with JSON pointers:
+ *   "#/properties/x * #/properties/y + #/properties/z"
+ *
+ * Returns { value, error } and NEVER throws.
+ */
 function evaluateExpression(expression, data) {
-  if (!expression || typeof expression !== 'string') return undefined;
+  if (!expression || typeof expression !== 'string') {
+    return {
+      value: undefined,
+      error: undefined
+    };
+  }
   const trimmed = expression.trim();
+  if (!trimmed) return {
+    value: undefined,
+    error: undefined
+  };
   const sumMatch = trimmed.match(/^SUM\(\s*['"]?(.+?)['"]?\s*\)$/i);
   if (sumMatch) {
-    return sumColumn(sumMatch[1], data);
+    const pointer = sumMatch[1];
+    return evaluateSum(pointer, data);
   }
-  // Find unique scope tokens like #/properties/x or #/arr/0/c3
   const scopeRegex = /#\/(?:properties\/)?[^\s"')]+/g;
   const matches = trimmed.match(scopeRegex) || [];
   const uniqueScopes = Array.from(new Set(matches));
-  // Prepare variable mapping for expr-eval
+  let exprForParse = trimmed;
+  const scopeToVar = new Map();
+  uniqueScopes.forEach((scope, index) => {
+    const varName = `v${index}`;
+    scopeToVar.set(scope, varName);
+    const pattern = new RegExp(`['"]?${escapeRegExp(scope)}['"]?`, 'g');
+    exprForParse = exprForParse.replace(pattern, varName);
+  });
+  let parsed;
+  try {
+    const parser = new Parser();
+    parsed = parser.parse(exprForParse);
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+  } catch (e) {
+    return {
+      value: undefined,
+      error: 'Invalid expression syntax'
+    };
+  }
   const vars = {};
-  let expr = trimmed;
-  for (let i = 0; i < uniqueScopes.length; i++) {
-    const scope = uniqueScopes[i];
+  const missingScopes = [];
+  const invalidTypeScopes = [];
+  for (const scope of uniqueScopes) {
     const val = resolveScope(scope, data);
-    if (val === undefined || typeof val !== 'number') return undefined;
-    const varName = `v${i}`;
+    if (val === undefined || val === null || val === '') {
+      missingScopes.push(scope);
+      continue;
+    }
+    if (typeof val !== 'number' || Number.isNaN(val)) {
+      invalidTypeScopes.push(scope);
+      continue;
+    }
+    const varName = scopeToVar.get(scope);
     vars[varName] = val;
-    const pattern = new RegExp(`['"]?${escapeRegExp(scope)}['"]?`, 'g');
-    expr = expr.replace(pattern, varName);
   }
-  // Evaluate with expr-eval parser
+  if (invalidTypeScopes.length > 0) {
+    return {
+      value: undefined,
+      error: `Expected numeric values for: ${invalidTypeScopes.join(', ')}`
+    };
+  }
+  if (missingScopes.length > 0) {
+    return {
+      value: undefined,
+      error: `Please provide values for: ${missingScopes.join(', ')}`
+    };
+  }
   try {
-    const parser = new Parser();
-    const parsed = parser.parse(expr);
     const result = parsed.evaluate(vars);
-    return typeof result === 'number' && Number.isFinite(result) ? result : undefined;
+    if (typeof result === 'number' && Number.isFinite(result)) {
+      return {
+        value: result,
+        error: undefined
+      };
+    }
+    return {
+      value: undefined,
+      error: 'Expression did not produce a numeric result'
+    };
   } catch (_a) {
-    return undefined;
+    return {
+      value: undefined,
+      error: 'Error while evaluating expression'
+    };
   }
 }
 
@@ -13128,29 +13214,44 @@ const GoACalculation = props => {
   var _a;
   const {
     uischema,
-    data,
     schema,
     path,
     id,
     visible,
     handleChange
   } = props;
-  const expression = schema === null || schema === void 0 ? void 0 : schema.description;
   const label = typeof (uischema === null || uischema === void 0 ? void 0 : uischema.label) === 'string' ? uischema.label : undefined;
+  const expression = schema === null || schema === void 0 ? void 0 : schema.description;
   const {
     core
   } = useJsonForms();
   const rootData = (_a = core === null || core === void 0 ? void 0 : core.data) !== null && _a !== void 0 ? _a : {};
-  const computedValue = expression ? evaluateExpression(expression, rootData) : undefined;
-  React.useEffect(() => {
+  const [hasInteracted, setHasInteracted] = useState(false);
+  const prevDataRef = useRef(rootData);
+  useEffect(() => {
+    setHasInteracted(was => was ? true : true);
+  }, [rootData]);
+  useEffect(() => {
+    if (prevDataRef.current !== rootData) {
+      setHasInteracted(true);
+      prevDataRef.current = rootData;
+    }
+  }, [rootData]);
+  const {
+    value: computedValue,
+    error
+  } = evaluateExpression(expression, rootData);
+  useEffect(() => {
     if (computedValue !== undefined && typeof handleChange === 'function' && path) {
       handleChange(path, computedValue);
     }
   }, [computedValue, handleChange, path]);
+  const showError = hasInteracted && !!error;
   return jsx(Visible, {
     visible: visible,
     children: jsx(GoAFormItem, {
       label: label,
+      error: showError ? error : '',
       children: jsx(GoAInput, {
         name: `computed-input-${id}`,
         testId: `computed-input-${id}`,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abgov/jsonforms-components",
-  "version": "2.36.1",
+  "version": "2.37.0",
   "license": "Apache-2.0",
   "description": "Government of Alberta - React renderers for JSON Forms based on the design system.",
   "repository": "https://github.com/GovAlta/adsp-monorepo",

package/src/lib/Controls/Calculation/CalculationEngine.d.ts

@@ -1,3 +1,15 @@
+export interface EvaluationResult {
+    value?: number;
+    error?: string;
+}
 export declare function resolveScope(scope: string, data: unknown): unknown;
-export declare function sumColumn(scope: string, data: unknown): number | undefined;
-export declare function evaluateExpression(expression: string, data: unknown): number | undefined;
+export declare function evaluateSum(scope: string | undefined, data: unknown): EvaluationResult;
+/**
+ * General expression evaluation.
+ * - Supports SUM(#/properties/arr/c3)
+ * - Supports arithmetic with JSON pointers:
+ *   "#/properties/x * #/properties/y + #/properties/z"
+ *
+ * Returns { value, error } and NEVER throws.
+ */
+export declare function evaluateExpression(expression: string | undefined, data: unknown): EvaluationResult;