Lazily update formState

Author: dungntm58

Sources/FormHook/Hook/UseController.swift

@@ -16,9 +16,15 @@ public func useController<FieldName, Value>(
     unregisterOption: UnregisterOption = []
 ) -> ControllerRenderOption<FieldName, Value> where FieldName: Hashable {
     let form = useContext(Context<FormControl<FieldName>>.self)
-    let registration = form.register(name: name, options: RegisterOption(rules: rules, defaultValue: defaultValue, shouldUnregister: shouldUnregister))
+    let registrationRef = useRef(form.register(name: name, options: RegisterOption(rules: rules, defaultValue: defaultValue, shouldUnregister: shouldUnregister)))
 
-    useEffect {{
+    let preservedChangedArray = [
+        AnyEquatable(name),
+        AnyEquatable(shouldUnregister),
+        AnyEquatable(unregisterOption),
+        AnyEquatable(form)
+    ]
+    useEffect(.preserved(by: preservedChangedArray)) {{
         guard shouldUnregister else { return }
         Task {
             await form.unregister(name: name, options: unregisterOption)
@@ -27,7 +33,7 @@ public func useController<FieldName, Value>(
 
     let field = FieldOption(
         name: name,
-        value: registration
+        value: registrationRef.current
     )
     let fieldState = form.getFieldState(name: name)
     return (field: field, fieldState: fieldState, formState: form.instantFormState)

Sources/FormHook/Util/AnyEquatable.swift

@@ -53,3 +53,19 @@ func areEqual(first: Any?, second: Any?) -> Bool {
     }
     return true
 }
+
+struct AnyEquatable: Equatable {
+    static func == (lhs: AnyEquatable, rhs: AnyEquatable) -> Bool {
+        areEqual(first: lhs.base, second: rhs.base)
+    }
+
+    let base: Any?
+
+    init(_ base: Any?) {
+        if let obj = base as? AnyEquatable {
+            self = obj
+        } else {
+            self.base = base
+        }
+    }
+}

Tests/FormHookTests/FormHookTests.swift

@@ -253,7 +253,7 @@ final class FormHookTests: QuickSpec {
                         it("validity of \"a\" remains") {
                             let fieldState = await formControl.getFieldState(name: .a)
                             expect(fieldState.isInvalid) == true
-                            expect(fieldState.error.isEmpty) == true
+                            expect(fieldState.error).to(beEmpty())
                         }
                     }
                 }
@@ -268,7 +268,7 @@ final class FormHookTests: QuickSpec {
                         it("key \"a\" remains errors") {
                             let fieldState = await formControl.getFieldState(name: .a)
                             expect(fieldState.isInvalid) == false
-                            expect(fieldState.error.isEmpty) == true
+                            expect(fieldState.error).to(beEmpty())
                         }
                     }
 
@@ -447,7 +447,7 @@ final class FormHookTests: QuickSpec {
                             it("key \"a\" remains errors") {
                                 let fieldState = await formControl.getFieldState(name: .a)
                                 expect(fieldState.isInvalid) == false
-                                expect(fieldState.error.isEmpty) == true
+                                expect(fieldState.error).to(beEmpty())
                             }
                         }
 
@@ -697,7 +697,7 @@ final class FormHookTests: QuickSpec {
                     it("key \"b\" remains errors") {
                         let fieldState = await formControl.getFieldState(name: .b)
                         expect(fieldState.isInvalid) == false
-                        expect(fieldState.error.isEmpty) == true
+                        expect(fieldState.error).to(beEmpty())
                     }
                 }
 
@@ -846,7 +846,7 @@ final class FormHookTests: QuickSpec {
                     it("errors of key \"a\" has gone") {
                         let fieldState = await formControl.getFieldState(name: .a)
                         expect(fieldState.isInvalid) == false
-                        expect(fieldState.error.isEmpty) == true
+                        expect(fieldState.error).to(beEmpty())
                     }
 
                     it("key \"b\" remains errors") {