fix: getValueProps should not be executed when name does not exist (#666)

* fix: `getValueProps` should not be executed when `name` does not exist

* test: update

* feat: improve

* feat: improve

Author: madocto

src/Field.tsx

@@ -561,6 +561,7 @@ class Field extends React.Component<InternalFieldProps, FieldState> implements F
 
   public getControlled = (childProps: ChildProps = {}) => {
     const {
+      name,
       trigger,
       validateTrigger,
       getValueFromEvent,
@@ -582,7 +583,7 @@ class Field extends React.Component<InternalFieldProps, FieldState> implements F
     // eslint-disable-next-line @typescript-eslint/no-explicit-any
     const originTriggerFunc: any = childProps[trigger];
 
-    const valueProps = mergedGetValueProps(value);
+    const valueProps = name !== undefined ? mergedGetValueProps(value) : {};
 
     // warning when prop value is function
     if (process.env.NODE_ENV !== 'production' && valueProps) {

tests/index.test.tsx

@@ -412,6 +412,27 @@ describe('Form.Basic', () => {
     expect(container.querySelector('.anything')).toBeTruthy();
   });
 
+  it('getValueProps should not be executed when name does not exist', async () => {
+    const getValueProps1 = jest.fn();
+    const getValueProps2 = jest.fn();
+
+    render(
+      <div>
+        <Form initialValues={{ test: 'bamboo' }}>
+          <Field getValueProps={getValueProps1}>
+            <span className="anything" />
+          </Field>
+          <Field getValueProps={getValueProps2}>
+            {() => <span className="anything" />}
+          </Field>
+        </Form>
+      </div>,
+    );
+
+    expect(getValueProps1).not.toHaveBeenCalled();
+    expect(getValueProps2).not.toHaveBeenCalled();
+  });
+
   describe('shouldUpdate', () => {
     it('work', async () => {
       let isAllTouched: boolean;