Skip to content

Commit 1c18e14

Browse files
committed
Simplify validation initialization
1 parent 44eb962 commit 1c18e14

File tree

2 files changed

+6
-5
lines changed

2 files changed

+6
-5
lines changed

Sources/MSLSwiftUI/FormValidation/FormExample.swift

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,13 +17,13 @@ import SwiftUI
1717

1818
struct UserFormInfo: FormValidatable, DefaultValueProvider {
1919
@FormFieldValidated(
20-
requirement: .required(nil),
20+
requirement: .required("Name is required"),
2121
validation: { $0.count < 3 ? "Name must be at least 3 characters" : nil }
2222
)
2323
var name = ""
2424

2525
@FormFieldValidated(
26-
requirement: .required("This field is required"),
26+
requirement: .required("Phone number is required"),
2727
validation: { value in
2828
!(value?.isPhoneNumberValid() ?? false) ? "Phone number is not valid." : nil
2929
}
@@ -79,6 +79,7 @@ struct ExampleForm: View {
7979
)
8080

8181
TextField("Phone Number", text: self.$userInfo.phoneNumber ?? "")
82+
.focused(self.$fieldToFocus, equals: "_phoneNumber")
8283
.keyboardType(.phonePad)
8384
.overlay(
8485
ValidationErrorView(errorMessage: self.userInfo.$phoneNumber.errorMessage),

Sources/MSLSwiftUI/FormValidation/ValidatedPropertyWrapper.swift

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ public enum FormFieldRequirement: Equatable {
55
case required(_ message: String?)
66

77
/// A value is not required for this field
8-
case notRequired
8+
case optional
99
}
1010

1111
public enum FormFieldStatus: Equatable {
@@ -94,8 +94,8 @@ public struct FormFieldValidated<Value: Equatable>: ValidatedProtocol {
9494

9595
public init(
9696
wrappedValue: Value,
97-
requirement: FormFieldRequirement = .notRequired,
98-
validation: @escaping (Value) -> String?
97+
requirement: FormFieldRequirement = .optional,
98+
validation: @escaping (Value) -> String? = { _ in return nil }
9999
) {
100100
self.originalValue = wrappedValue
101101

0 commit comments

Comments
 (0)