@dartech/arsenal-ui 1.3.76 → 1.3.78

package/index.js

@@ -1473,10 +1473,24 @@ const formatTableRowValue = ({
 
 const digitsOnly = new RegExp('^[-+]?[0-9]+$');
 const floatsOnly = new RegExp(/^[-+]?[0-9]*\.?[0-9]+([eE][-+]?[0-9]+)?$/);
-const isExpression = value => {
+const isExpression = (value, useRecursion) => {
   if (typeof value === 'string') return /\$(?:S?JS)?\(/g.test(value);
+  if (useRecursion) {
+    if (Array.isArray(value)) {
+      for (const item of value) {
+        if (isExpression(item, useRecursion)) return true;
+      }
+    }
+    if (typeof value === 'object') {
+      for (const key in value) {
+        if (isExpression(value[key], useRecursion)) return true;
+      }
+    }
+  }
   return false;
+  // return typeof value === 'string' ? /\$(?:S?JS)?\(/g.test(value) : false;
 };
+
 const isDateType = propertyType => {
   return propertyType === PropertyType.DATE || propertyType === PropertyType.DATE_TIME || propertyType === PropertyType.TIME;
 };
@@ -3921,18 +3935,19 @@ const usePropertyFiller = ({
         return {};
     }
   }, [propertyType]);
+  const needRecursionCheck = useMemo(() => propertyType === PropertyType.JSON, [propertyType]);
   const checkFillOption = useCallback(() => {
-    if (isExpression(value) && fillOption !== 'expression') {
+    if (isExpression(value, needRecursionCheck)) {
       setFillOption('expression');
-    } else if (value !== null && value !== undefined && !isExpression(value) && fillOption !== 'widget' && fillOption !== 'string') {
+    } else if (value !== null && value !== undefined && !isExpression(value, needRecursionCheck) && fillOption !== 'widget' && fillOption !== 'string') {
       setFillOption('widget');
     }
-  }, [value, fillOption]);
+  }, [value, fillOption, needRecursionCheck]);
   useEffect(() => {
     if (!fillOption) {
       if (isPropertyValueEmpty(value) && (fillOptions === null || fillOptions === void 0 ? void 0 : fillOptions.length)) {
         setFillOption(fillOptions[0].value);
-      } else if (_useExpression && isExpression(value)) {
+      } else if (_useExpression && isExpression(value, needRecursionCheck)) {
         setFillOption('expression');
       } else {
         setFillOption('widget');
@@ -3940,7 +3955,7 @@ const usePropertyFiller = ({
     } else if (!isDirty) {
       checkFillOption();
     }
-  }, [value, fillOption, fillOptions, _useExpression, isDirty, checkFillOption]);
+  }, [value, fillOption, fillOptions, _useExpression, isDirty, checkFillOption, needRecursionCheck]);
   return {
     propertyType,
     valueLabel,
@@ -4155,53 +4170,6 @@ const PropertyFiller = ({
   }));
 };
 
-const JsonEditor = /*#__PURE__*/forwardRef(({
-  validate: _validate = false,
-  name,
-  useParsedValue
-}, ref) => {
-  const {
-    control,
-    setValue,
-    setError,
-    clearErrors
-  } = useFormContext();
-  const value = useWatch({
-    control,
-    name
-  });
-  const handleChange = value => {
-    try {
-      if (value) setValue(name, useParsedValue ? JSON.parse(value) : value);
-      clearErrors(name);
-    } catch (e) {
-      if (_validate) setError(name, {
-        message: 'Invalid JSON'
-      });
-    }
-  };
-  return jsx(CodeMirror, {
-    ref: ref,
-    basicSetup: {
-      autocompletion: true,
-      history: true
-    },
-    theme: abcdef,
-    width: "100%",
-    height: "200px",
-    style: {
-      fontSize: 14,
-      lineHeight: 1.4,
-      zIndex: 0,
-      isolation: 'isolate'
-    },
-    value: typeof value !== 'string' ? value === null ? '' : JSON.stringify(value) : value,
-    onChange: handleChange,
-    extensions: [langs.json()]
-  });
-});
-var JsonEditor$1 = JsonEditor;
-
 const MultiplePropertyWidget = ({
   name,
   property,
@@ -4367,7 +4335,7 @@ const MultiplePropertyFiller = ({
   };
   useEffect(() => {
     if (!value) {
-      onChange(property.isRequired ? [] : null);
+      onChange(property.isRequired && property.isMultiple ? [] : null);
     }
   }, [value, property, onChange]);
   useEffect(() => {
@@ -4410,21 +4378,29 @@ const MultiplePropertyFiller = ({
           }), option.value))
         }))]
       }))
-    })) : null, fillOption === 'expression' && (propertyType === PropertyType.JSON || propertyType === PropertyType.ENTITY || propertyType === PropertyType.ANY ? jsx(JsonEditor$1, {
+    })) : null, fillOption === 'expression' && (propertyType === PropertyType.JSON || propertyType === PropertyType.ENTITY || propertyType === PropertyType.ANY ? jsx(ControlAceEditor, {
       name: name,
-      ref: ref
+      control: control,
+      label: valueLabel,
+      parseValue: true,
+      validateJson: true,
+      hideErrorMessage: true
     }) : jsx(ControlInput, {
       required: true,
       hideErrorMessage: true,
       control: control,
       name: name,
       label: valueLabel
-    })), fillOption === 'json_valid' && jsx(JsonEditor$1, {
-      validate: true,
+    })), fillOption === 'json_valid' && jsx(ControlAceEditor, {
       name: name,
-      ref: ref,
-      useParsedValue: true
-    }), fillOption === 'widget' && jsx(MultiplePropertyWidget$1, {
+      control: control,
+      label: valueLabel,
+      parseValue: true,
+      validateJson: true,
+      hideErrorMessage: true
+    })
+    // <JsonEditor validate name={name} ref={ref} useParsedValue />
+    , fillOption === 'widget' && jsx(MultiplePropertyWidget$1, {
       property: property,
       name: name,
       useExpression: useExpression,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dartech/arsenal-ui",
-  "version": "1.3.76",
+  "version": "1.3.78",
   "author": "DAR",
   "publishConfig": {
     "registry": "https://registry.npmjs.org/"

package/src/utils/validators.d.ts

@@ -1,5 +1,5 @@
 import { PropertyType } from '../interfaces';
 export declare const digitsOnly: RegExp;
 export declare const floatsOnly: RegExp;
-export declare const isExpression: (value: unknown) => boolean;
+export declare const isExpression: (value: unknown, useRecursion?: boolean) => boolean;
 export declare const isDateType: (propertyType: PropertyType) => boolean;