use fast-deep-equal

igorbrasileiro · Sep 5, 2024 · 6ba3bdf · 6ba3bdf
1 parent 6ca6edd
commit 6ba3bdf
Show file tree

Hide file tree

Showing 9 changed files with 55 additions and 31 deletions.
diff --git a/package.json b/package.json
@@ -79,5 +79,6 @@
     "packages/validator-ajv6",
     "packages/validator-ajv8",
     "packages/snapshot-tests"
-  ]
+  ],
+  "packageManager": "yarn@1.22.22+sha512.a6b2f7906b721bba3d67d4aff083df04dad64c399707841b7acf00f6b133b7ac24255f2652fa22ae3534329dc6180534e98d17432037ff6fd140556e2bb3137e"
 }
diff --git a/packages/core/src/components/Form.tsx b/packages/core/src/components/Form.tsx
@@ -1,10 +1,11 @@
-import { Component, ElementType, FormEvent, ReactNode, Ref, RefObject, createRef } from 'react';
+import { Component, createRef, ElementType, FormEvent, ReactNode, Ref, RefObject } from 'react';
 import {
   createSchemaUtils,
   CustomValidator,
   deepEquals,
   ErrorSchema,
   ErrorTransformer,
+  Experimental_DefaultFormStateBehavior,
   FormContextType,
   GenericObjectType,
   getTemplate,
@@ -14,31 +15,31 @@ import {
   mergeObjects,
   NAME_KEY,
   PathSchema,
-  StrictRJSFSchema,
   Registry,
   RegistryFieldsType,
   RegistryWidgetsType,
+  RJSF_ADDITIONAL_PROPERTIES_FLAG,
   RJSFSchema,
   RJSFValidationError,
-  RJSF_ADDITIONAL_PROPERTIES_FLAG,
   SchemaUtilsType,
   shouldRender,
+  StrictRJSFSchema,
   SUBMIT_BTN_OPTIONS_KEY,
   TemplatesType,
   toErrorList,
-  UiSchema,
   UI_GLOBAL_OPTIONS_KEY,
   UI_OPTIONS_KEY,
+  UiSchema,
   ValidationData,
   validationDataMerge,
   ValidatorType,
-  Experimental_DefaultFormStateBehavior,
 } from '@rjsf/utils';
 import _forEach from 'lodash/forEach';
 import _get from 'lodash/get';
 import _isEmpty from 'lodash/isEmpty';
 import _pick from 'lodash/pick';
 import _toPath from 'lodash/toPath';
+import fastDeepEqual from 'fast-deep-equal';
 
 import getDefaultRegistry from '../getDefaultRegistry';
 
@@ -282,7 +283,7 @@ export default class Form<
     }
 
     this.state = this.getStateFromProps(props, props.formData);
-    if (this.props.onChange && !deepEquals(this.state.formData, this.props.formData)) {
+    if (this.props.onChange && !fastDeepEqual(this.state.formData, this.props.formData)) {
       this.props.onChange(this.state);
     }
     this.formElement = createRef();
@@ -309,10 +310,14 @@ export default class Form<
   getSnapshotBeforeUpdate(
     prevProps: FormProps<T, S, F>,
     prevState: FormState<T, S, F>
-  ): { nextState: FormState<T, S, F>; shouldUpdate: true } | { shouldUpdate: false } {
+  ):
+    | { nextState: FormState<T, S, F>; shouldUpdate: true }
+    | {
+        shouldUpdate: false;
+      } {
     if (!deepEquals(this.props, prevProps)) {
-      const isSchemaChanged = !deepEquals(prevProps.schema, this.props.schema);
-      const isFormDataChanged = !deepEquals(prevProps.formData, this.props.formData);
+      const isSchemaChanged = !fastDeepEqual(prevProps.schema, this.props.schema);
+      const isFormDataChanged = !fastDeepEqual(prevProps.formData, this.props.formData);
       const nextState = this.getStateFromProps(
         this.props,
         this.props.formData,
@@ -346,14 +351,18 @@ export default class Form<
   componentDidUpdate(
     _: FormProps<T, S, F>,
     prevState: FormState<T, S, F>,
-    snapshot: { nextState: FormState<T, S, F>; shouldUpdate: true } | { shouldUpdate: false }
+    snapshot:
+      | { nextState: FormState<T, S, F>; shouldUpdate: true }
+      | {
+          shouldUpdate: false;
+        }
   ) {
     if (snapshot.shouldUpdate) {
       const { nextState } = snapshot;
 
       if (
-        !deepEquals(nextState.formData, this.props.formData) &&
-        !deepEquals(nextState.formData, prevState.formData) &&
+        !fastDeepEqual(nextState.formData, this.props.formData) &&
+        !fastDeepEqual(nextState.formData, prevState.formData) &&
         this.props.onChange
       ) {
         this.props.onChange(nextState);
@@ -692,7 +701,6 @@ export default class Form<
    * Callback function to handle reset form data.
    * - Reset all fields with default values.
    * - Reset validations and errors
-   *
    */
   reset = () => {
     const { onChange } = this.props;
@@ -772,7 +780,14 @@ export default class Form<
         },
         () => {
           if (onSubmit) {
-            onSubmit({ ...this.state, formData: newFormData, status: 'submitted' }, event);
+            onSubmit(
+              {
+                ...this.state,
+                formData: newFormData,
+                status: 'submitted',
+              },
+              event
+            );
           }
         }
       );
@@ -956,9 +971,14 @@ export default class Form<
 
     let { [SUBMIT_BTN_OPTIONS_KEY]: submitOptions = {} } = getUiOptions<T, S, F>(uiSchema);
     if (disabled) {
-      submitOptions = { ...submitOptions, props: { ...submitOptions.props, disabled: true } };
+      submitOptions = {
+        ...submitOptions,
+        props: { ...submitOptions.props, disabled: true },
+      };
     }
-    const submitUiSchema = { [UI_OPTIONS_KEY]: { [SUBMIT_BTN_OPTIONS_KEY]: submitOptions } };
+    const submitUiSchema = {
+      [UI_OPTIONS_KEY]: { [SUBMIT_BTN_OPTIONS_KEY]: submitOptions },
+    };
 
     return (
       <FormTag

diff --git a/packages/core/src/components/fields/MultiSchemaField.tsx b/packages/core/src/components/fields/MultiSchemaField.tsx
@@ -4,7 +4,6 @@ import isEmpty from 'lodash/isEmpty';
 import omit from 'lodash/omit';
 import {
   ANY_OF_KEY,
-  deepEquals,
   ERRORS_KEY,
   FieldProps,
   FormContextType,
@@ -18,6 +17,7 @@ import {
   TranslatableString,
   UiSchema,
 } from '@rjsf/utils';
+import fastDeepEqual from 'fast-deep-equal';
 
 /** Type used for the state of the `AnyOfField` component */
 type AnyOfFieldState<S extends StrictRJSFSchema = RJSFSchema> = {
@@ -67,15 +67,15 @@ class AnyOfField<T = any, S extends StrictRJSFSchema = RJSFSchema, F extends For
     const { formData, options, idSchema } = this.props;
     const { selectedOption } = this.state;
     let newState = this.state;
-    if (!deepEquals(prevProps.options, options)) {
+    if (!fastDeepEqual(prevProps.options, options)) {
       const {
         registry: { schemaUtils },
       } = this.props;
       // re-cache the retrieved options in state in case they have $refs to save doing it later
       const retrievedOptions = options.map((opt: S) => schemaUtils.retrieveSchema(opt, formData));
       newState = { selectedOption, retrievedOptions };
     }
-    if (!deepEquals(formData, prevProps.formData) && idSchema.$id === prevProps.idSchema.$id) {
+    if (!fastDeepEqual(formData, prevProps.formData) && idSchema.$id === prevProps.idSchema.$id) {
       const { retrievedOptions } = newState;
       const matchingOption = this.getMatchingOption(selectedOption, formData, retrievedOptions);
 

diff --git a/packages/utils/src/enumOptionsDeselectValue.ts b/packages/utils/src/enumOptionsDeselectValue.ts
@@ -1,4 +1,4 @@
-import isEqual from 'lodash/isEqual';
+import isEqual from 'fast-deep-equal';
 
 import { EnumOptionsType, RJSFSchema, StrictRJSFSchema } from './types';
 import enumOptionsValueForIndex from './enumOptionsValueForIndex';

diff --git a/packages/utils/src/enumOptionsIsSelected.ts b/packages/utils/src/enumOptionsIsSelected.ts
@@ -1,4 +1,4 @@
-import isEqual from 'lodash/isEqual';
+import isEqual from 'fast-deep-equal';
 
 import { EnumOptionsType, RJSFSchema, StrictRJSFSchema } from './types';
 

diff --git a/packages/utils/src/parser/ParserValidator.ts b/packages/utils/src/parser/ParserValidator.ts
@@ -1,5 +1,5 @@
 import get from 'lodash/get';
-import isEqual from 'lodash/isEqual';
+import isEqual from 'fast-deep-equal';
 
 import { ID_KEY } from '../constants';
 import hashForSchema from '../hashForSchema';

diff --git a/packages/utils/src/parser/schemaParser.ts b/packages/utils/src/parser/schemaParser.ts
@@ -1,10 +1,10 @@
 import forEach from 'lodash/forEach';
-import isEqual from 'lodash/isEqual';
+import isEqual from 'fast-deep-equal';
 
 import { FormContextType, RJSFSchema, StrictRJSFSchema } from '../types';
-import { PROPERTIES_KEY, ITEMS_KEY } from '../constants';
+import { ITEMS_KEY, PROPERTIES_KEY } from '../constants';
 import ParserValidator, { SchemaMap } from './ParserValidator';
-import { retrieveSchemaInternal, resolveAnyOrOneOfSchemas } from '../schema/retrieveSchema';
+import { resolveAnyOrOneOfSchemas, retrieveSchemaInternal } from '../schema/retrieveSchema';
 
 /** Recursive function used to parse the given `schema` belonging to the `rootSchema`. The `validator` is used to
  * capture the sub-schemas that the `isValid()` function is called with. For each schema returned by the

diff --git a/packages/utils/src/schema/retrieveSchema.ts b/packages/utils/src/schema/retrieveSchema.ts
@@ -1,5 +1,5 @@
 import get from 'lodash/get';
-import isEqual from 'lodash/isEqual';
+import isEqual from 'fast-deep-equal';
 import set from 'lodash/set';
 import times from 'lodash/times';
 import transform from 'lodash/transform';
@@ -15,10 +15,10 @@ import {
   ANY_OF_KEY,
   DEPENDENCIES_KEY,
   IF_KEY,
+  ITEMS_KEY,
   ONE_OF_KEY,
-  REF_KEY,
   PROPERTIES_KEY,
-  ITEMS_KEY,
+  REF_KEY,
 } from '../constants';
 import findSchemaDefinition, { splitKeyElementFromObject } from '../findSchemaDefinition';
 import getDiscriminatorFieldFromSchema from '../getDiscriminatorFieldFromSchema';
@@ -196,7 +196,10 @@ export function resolveSchema<T = any, S extends StrictRJSFSchema = RJSFSchema,
       )
     );
     const allPermutations = getAllPermutationsOfXxxOf<S>(allOfSchemaElements);
-    return allPermutations.map((permutation) => ({ ...schema, allOf: permutation }));
+    return allPermutations.map((permutation) => ({
+      ...schema,
+      allOf: permutation,
+    }));
   }
   // No $ref or dependencies or allOf attribute was found, returning the original schema.
   return [schema];

diff --git a/packages/utils/src/schema/toIdSchema.ts b/packages/utils/src/schema/toIdSchema.ts
@@ -1,5 +1,5 @@
 import get from 'lodash/get';
-import isEqual from 'lodash/isEqual';
+import isEqual from 'fast-deep-equal';
 
 import { ALL_OF_KEY, DEPENDENCIES_KEY, ID_KEY, ITEMS_KEY, PROPERTIES_KEY, REF_KEY } from '../constants';
 import isObject from '../isObject';