fix(Form): internal logic fixes * 8

Author: tenphi

src/components/form/Form/use-field/use-field-props.tsx

@@ -82,13 +82,6 @@ export function useFieldProps<
 
   // eslint-disable-next-line react-hooks/rules-of-hooks
   const onChangeEvent = useEvent((value, dontTouch: boolean) => {
-    console.log(
-      '! uikit onChange',
-      value,
-      dontTouch,
-      field?.validateTrigger || defaultValidationTrigger,
-    );
-
     return field?.onChange?.(
       value,
       dontTouch,

src/components/form/Form/use-field/use-field.ts

@@ -1,4 +1,4 @@
-import { useEffect, useMemo, useState } from 'react';
+import { useEffect, useState } from 'react';
 
 import { ValidateTrigger } from '../../../../shared/index';
 import { useEvent, useWarn } from '../../../../_internal/index';
@@ -127,16 +127,6 @@ export function useField<T extends FieldTypes, Props extends CubeFieldProps<T>>(
 
     const field = form.getFieldInstance(fieldName);
 
-    console.log(
-      '! uikit set field before',
-      fieldName,
-      val,
-      !dontTouch,
-      false,
-      dontTouch,
-      shouldUpdate,
-    );
-
     if (shouldUpdate) {
       const fieldsValue = form.getFieldsValue();
 
@@ -156,15 +146,6 @@ export function useField<T extends FieldTypes, Props extends CubeFieldProps<T>>(
 
     form.setFieldValue(fieldName, val, !dontTouch, false, dontTouch);
 
-    console.log(
-      '! uikit set field after',
-      fieldName,
-      val,
-      !dontTouch,
-      false,
-      dontTouch,
-    );
-
     if (
       !dontTouch &&
       (validateTrigger === 'onChange' ||
@@ -185,43 +166,25 @@ export function useField<T extends FieldTypes, Props extends CubeFieldProps<T>>(
 
   let inputValue = field?.inputValue;
 
-  return useMemo(
-    () => ({
-      id: fieldId,
-      name: fieldName,
-      value: inputValue,
-      validateTrigger,
-      form,
-      field,
-      nonInput,
-
-      validationState:
-        validationState ??
-        (field?.errors?.length
-          ? 'invalid'
-          : showValid && field?.status === 'valid'
-            ? 'valid'
-            : undefined),
-      ...(isRequired && { isRequired }),
-      message: message ?? (field?.status === 'invalid' && field?.errors?.[0]),
-      onBlur: onBlurHandler,
-      onChange: onChangeHandler,
-    }),
-    [
-      form,
-      field,
-      field?.errors?.length,
-      field?.status,
-      inputValue,
-      fieldId,
-      fieldName,
-      isRequired,
-      onBlurHandler,
-      onChangeHandler,
-      validateTrigger,
-      validationState,
-      showValid,
-      nonInput,
-    ],
-  );
+  return {
+    id: fieldId,
+    name: fieldName,
+    value: inputValue,
+    validateTrigger,
+    form,
+    field,
+    nonInput,
+
+    validationState:
+      validationState ??
+      (field?.errors?.length
+        ? 'invalid'
+        : showValid && field?.status === 'valid'
+          ? 'valid'
+          : undefined),
+    ...(isRequired && { isRequired }),
+    message: message ?? (field?.status === 'invalid' && field?.errors?.[0]),
+    onBlur: onBlurHandler,
+    onChange: onChangeHandler,
+  };
 }

src/components/form/Form/use-form.tsx

@@ -162,10 +162,7 @@ export class CubeFormInstance<
     const field = this.fields[name];
 
     if (!field || isEqual(value, inputOnly ? field.inputValue : field.value)) {
-      console.log('! uikit equal', field, value, inputOnly);
       return;
-    } else {
-      console.log('! uikit setting', field, value, inputOnly);
     }
 
     if (!inputOnly) {