shouldUpdate accepts true

Author: zombieJ

README.md

@@ -73,16 +73,16 @@ We use typescript to create the Type definition. You can view directly in IDE. B
 
 ## Field
 
-| Prop              | Description                             | Type                                  | Default  |
-| ----------------- | --------------------------------------- | ------------------------------------- | -------- |
-| dependencies      | Will re-render if dependencies changed  | [NamePath](#namepath)[]               | -        |
-| getValueFromEvent | Specify how to get value from event     | (..args: any[]) => any                | -        |
-| name              | Field name path                         | [NamePath](#namepath)                 | -        |
-| normalize         | Normalize value before update           | (value, prevValue, prevValues) => any | -        |
-| rules             | Validate rules                          | [Rule](#rule)[]                       | -        |
-| shouldUpdate      | Check if Field should update            | (prevValues, nextValues): boolean     | -        |
-| trigger           | Collect value update by event trigger   | string                                | onChange |
-| validateTrigger   | Config trigger point with rule validate | string \| string[]                    | onChange |
+| Prop              | Description                             | Type                                         | Default  |
+| ----------------- | --------------------------------------- | -------------------------------------------- | -------- |
+| dependencies      | Will re-render if dependencies changed  | [NamePath](#namepath)[]                      | -        |
+| getValueFromEvent | Specify how to get value from event     | (..args: any[]) => any                       | -        |
+| name              | Field name path                         | [NamePath](#namepath)                        | -        |
+| normalize         | Normalize value before update           | (value, prevValue, prevValues) => any        | -        |
+| rules             | Validate rules                          | [Rule](#rule)[]                              | -        |
+| shouldUpdate      | Check if Field should update            | true \| (prevValues, nextValues): boolean | -        |
+| trigger           | Collect value update by event trigger   | string                                       | onChange |
+| validateTrigger   | Config trigger point with rule validate | string \| string[]                           | onChange |
 
 ## List
 

src/Field.tsx

@@ -47,7 +47,9 @@ export interface FieldProps {
   name?: NamePath;
   normalize?: (value: StoreValue, prevValue: StoreValue, allValues: Store) => StoreValue;
   rules?: Rule[];
-  shouldUpdate?: (prevValues: Store, nextValues: Store, info: { source?: string }) => boolean;
+  shouldUpdate?:
+    | true
+    | ((prevValues: Store, nextValues: Store, info: { source?: string }) => boolean);
   trigger?: string;
   validateTrigger?: string | string[] | false;
 }
@@ -224,7 +226,10 @@ class Field extends React.Component<FieldProps, FieldState> implements FieldEnti
           dependencies.some(dependency =>
             containsNamePath(namePathList, getNamePath(dependency)),
           ) ||
-          (shouldUpdate ? shouldUpdate(prevStore, values, info) : prevValue !== curValue)
+          shouldUpdate === true ||
+          (typeof shouldUpdate === 'function'
+            ? shouldUpdate(prevStore, values, info)
+            : prevValue !== curValue)
         ) {
           this.reRender();
         }

tests/index.test.js

@@ -319,4 +319,29 @@ describe('Basic', () => {
 
     errorSpy.mockRestore();
   });
+
+  it('shouldUpdate return true will update whatever notification updated', async () => {
+    let renderCount = 0;
+
+    const wrapper = mount(
+      <Form>
+        <Field rules={[{ required: true }]}>
+          <Input />
+        </Field>
+        <Field shouldUpdate>
+          {() => {
+            renderCount += 1;
+            return null;
+          }}
+        </Field>
+      </Form>,
+    );
+
+    const baseRenderCount = renderCount;
+    changeValue(getField(wrapper), '');
+    expect(renderCount).toEqual(baseRenderCount + 1);
+
+    await timeout();
+    expect(renderCount).toEqual(baseRenderCount + 2);
+  });
 });