refactored hasError to showError
Signed-off-by: Matt Bruce <matt.bruce@verizon.com>
This commit is contained in:
parent
1b75bfb369
commit
fa3d139742
@ -82,8 +82,8 @@ open class CheckboxBase<ModelType: CheckboxModel>: Control<ModelType> {
|
||||
@Proxy(\.model.childText)
|
||||
open var childText: String?
|
||||
|
||||
@Proxy(\.model.hasError)
|
||||
open var hasError: Bool
|
||||
@Proxy(\.model.showError)
|
||||
open var showError: Bool
|
||||
|
||||
@Proxy(\.model.errorText)
|
||||
open var errorText: String?
|
||||
@ -219,8 +219,8 @@ open class CheckboxBase<ModelType: CheckboxModel>: Control<ModelType> {
|
||||
/// This will checkbox the state of the Selector and execute the actionBlock if provided.
|
||||
open func toggle() {
|
||||
//removed error
|
||||
if hasError && isSelected == false {
|
||||
hasError.toggle()
|
||||
if showError && isSelected == false {
|
||||
showError.toggle()
|
||||
}
|
||||
isSelected.toggle()
|
||||
sendActions(for: .valueChanged)
|
||||
@ -354,7 +354,7 @@ open class CheckboxBase<ModelType: CheckboxModel>: Control<ModelType> {
|
||||
public let error = BinarySurfaceColorConfiguration<ModelType>()
|
||||
override func getColor(_ viewModel: ModelType) -> UIColor {
|
||||
//only show error is enabled and showError == true
|
||||
let showErrorColor = !viewModel.disabled && viewModel.hasError
|
||||
let showErrorColor = !viewModel.disabled && viewModel.showError
|
||||
|
||||
if showErrorColor {
|
||||
return error.getColor(viewModel)
|
||||
|
||||
@ -11,8 +11,8 @@ import UIKit
|
||||
public class CheckboxGroup: CheckboxGroupBase<DefaultCheckboxGroupModel, Checkbox> {
|
||||
public override func didSelect(_ selectedControl: Checkbox) {
|
||||
selectedControl.toggle()
|
||||
if selectedControl.isSelected, hasError{
|
||||
hasError.toggle()
|
||||
if selectedControl.isSelected, showError{
|
||||
showError.toggle()
|
||||
}
|
||||
valueChanged()
|
||||
}
|
||||
@ -23,18 +23,18 @@ public class CheckboxGroupBase<GroupModelType: CheckboxGroupModel, ModelHandlerT
|
||||
//--------------------------------------------------
|
||||
// MARK: - Public Properties
|
||||
//--------------------------------------------------
|
||||
public var hasError: Bool {
|
||||
get { model.hasError }
|
||||
public var showError: Bool {
|
||||
get { model.showError }
|
||||
set {
|
||||
var newHasError = newValue
|
||||
var newShowError = newValue
|
||||
let anySelected = selectorViews.filter { $0.isSelected == true }.count > 0
|
||||
if anySelected && newHasError {
|
||||
newHasError = false
|
||||
if anySelected && newShowError {
|
||||
newShowError = false
|
||||
}
|
||||
selectorViews.forEach { handler in
|
||||
handler.hasError = newHasError
|
||||
handler.showError = newShowError
|
||||
}
|
||||
model.hasError = newHasError
|
||||
model.showError = newShowError
|
||||
|
||||
}
|
||||
}
|
||||
|
||||
@ -19,7 +19,7 @@ public struct DefaultCheckboxGroupModel: CheckboxGroupModel {
|
||||
public var surface: Surface = .light
|
||||
public var disabled: Bool = false
|
||||
public var selectors: [SelectorModelType]
|
||||
public var hasError: Bool = false
|
||||
public var showError: Bool = false
|
||||
public var errorText: String?
|
||||
public init() { selectors = [] }
|
||||
public init(selectors: [SelectorModelType]){
|
||||
|
||||
@ -18,7 +18,7 @@ public protocol CheckboxModel: Modelable, FormFieldable, Errorable, DataTrackabl
|
||||
extension CheckboxModel {
|
||||
|
||||
public var shouldShowError: Bool {
|
||||
guard hasError && !disabled && errorText?.isEmpty == false else { return false }
|
||||
guard showError && !disabled && errorText?.isEmpty == false else { return false }
|
||||
return true
|
||||
}
|
||||
|
||||
@ -52,7 +52,7 @@ extension CheckboxModel {
|
||||
}
|
||||
|
||||
public var errorModel: DefaultLabelModel? {
|
||||
guard let errorText = errorText, hasError else { return nil }
|
||||
guard let errorText = errorText, showError else { return nil }
|
||||
var model = DefaultLabelModel()
|
||||
model.textPosition = .left
|
||||
model.typograpicalStyle = .BodyMedium
|
||||
@ -71,7 +71,7 @@ public struct DefaultCheckboxModel: CheckboxModel {
|
||||
public var childText: String?
|
||||
public var childTextAttributes: [any LabelAttributeModel]?
|
||||
|
||||
public var hasError: Bool = false
|
||||
public var showError: Bool = false
|
||||
public var errorText: String?
|
||||
|
||||
public var inputId: String?
|
||||
@ -103,7 +103,7 @@ public struct DefaultCheckboxModel: CheckboxModel {
|
||||
&& labelTextAttributes == equatable.labelTextAttributes
|
||||
&& childText == equatable.childText
|
||||
&& childTextAttributes == equatable.childTextAttributes
|
||||
&& hasError == equatable.hasError
|
||||
&& showError == equatable.showError
|
||||
&& errorText == equatable.errorText
|
||||
&& inputId == equatable.inputId
|
||||
&& value == equatable.value
|
||||
|
||||
@ -17,7 +17,7 @@ public struct DefaultRadioBoxGroupModel: RadioBoxGroupModel {
|
||||
public var surface: Surface = .light
|
||||
public var disabled: Bool = false
|
||||
public var selectors: [SelectorModelType]
|
||||
public var hasError: Bool = false
|
||||
public var showError: Bool = false
|
||||
public var errorText: String?
|
||||
public init() { selectors = [] }
|
||||
public init(selectors: [SelectorModelType]){
|
||||
|
||||
@ -14,8 +14,8 @@ public class RadioButton: RadioButtonBase<DefaultRadioButtonModel>{
|
||||
//for groups allows "toggle"
|
||||
open override func toggle() {
|
||||
//removed error
|
||||
if hasError && isSelected == false {
|
||||
hasError.toggle()
|
||||
if showError && isSelected == false {
|
||||
showError.toggle()
|
||||
}
|
||||
isSelected.toggle()
|
||||
}
|
||||
@ -90,8 +90,8 @@ open class RadioButtonBase<ModelType: RadioButtonModel>: Control<ModelType> {
|
||||
@Proxy(\.model.childText)
|
||||
open var childText: String?
|
||||
|
||||
@Proxy(\.model.hasError)
|
||||
open var hasError: Bool
|
||||
@Proxy(\.model.showError)
|
||||
open var showError: Bool
|
||||
|
||||
@Proxy(\.model.errorText)
|
||||
open var errorText: String?
|
||||
@ -229,8 +229,8 @@ open class RadioButtonBase<ModelType: RadioButtonModel>: Control<ModelType> {
|
||||
guard !isSelected else { return }
|
||||
|
||||
//removed error
|
||||
if hasError && isSelected == false {
|
||||
hasError.toggle()
|
||||
if showError && isSelected == false {
|
||||
showError.toggle()
|
||||
}
|
||||
isSelected.toggle()
|
||||
sendActions(for: .valueChanged)
|
||||
@ -346,7 +346,7 @@ open class RadioButtonBase<ModelType: RadioButtonModel>: Control<ModelType> {
|
||||
public let error = BinarySurfaceColorConfiguration<ModelType>()
|
||||
override func getColor(_ viewModel: ModelType) -> UIColor {
|
||||
//only show error is enabled and showError == true
|
||||
let showErrorColor = !viewModel.disabled && viewModel.hasError
|
||||
let showErrorColor = !viewModel.disabled && viewModel.showError
|
||||
|
||||
if showErrorColor {
|
||||
return error.getColor(viewModel)
|
||||
|
||||
@ -14,8 +14,8 @@ public class RadioButtonGroup: RadioButtonGroupBase<DefaultRadioButtonGroupModel
|
||||
let oldSelectedControl = selectorViews.filter { $0.isSelected == true }.first
|
||||
oldSelectedControl?.toggle()
|
||||
selectedControl.toggle()
|
||||
if hasError {
|
||||
hasError = false
|
||||
if showError {
|
||||
showError = false
|
||||
}
|
||||
valueChanged()
|
||||
}
|
||||
@ -26,17 +26,17 @@ public class RadioButtonGroupBase<GroupModelType: RadioButtonGroupModel, ModelHa
|
||||
//--------------------------------------------------
|
||||
// MARK: - Public Properties
|
||||
//--------------------------------------------------
|
||||
public var hasError: Bool {
|
||||
get { model.hasError }
|
||||
public var showError: Bool {
|
||||
get { model.showError }
|
||||
set {
|
||||
var newHasError = newValue
|
||||
if selectedModel != nil, newHasError {
|
||||
newHasError = false
|
||||
var newShowError = newValue
|
||||
if selectedModel != nil, newShowError {
|
||||
newShowError = false
|
||||
}
|
||||
selectorViews.forEach { handler in
|
||||
handler.hasError = newHasError
|
||||
handler.showError = newShowError
|
||||
}
|
||||
model.hasError = newHasError
|
||||
model.showError = newShowError
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
@ -21,7 +21,7 @@ public struct DefaultRadioButtonGroupModel: RadioButtonGroupModel {
|
||||
public var surface: Surface = .light
|
||||
public var disabled: Bool = false
|
||||
public var selectors: [SelectorModelType]
|
||||
public var hasError: Bool = false
|
||||
public var showError: Bool = false
|
||||
public var errorText: String?
|
||||
public init() { selectors = [] }
|
||||
public init(selectors: [SelectorModelType]){
|
||||
|
||||
@ -18,7 +18,7 @@ public protocol RadioButtonModel: Modelable, FormFieldable, Errorable, DataTrack
|
||||
extension RadioButtonModel {
|
||||
|
||||
public var shouldShowError: Bool {
|
||||
guard hasError && !disabled && errorText?.isEmpty == false else { return false }
|
||||
guard showError && !disabled && errorText?.isEmpty == false else { return false }
|
||||
return true
|
||||
}
|
||||
|
||||
@ -52,7 +52,7 @@ extension RadioButtonModel {
|
||||
}
|
||||
|
||||
public var errorModel: DefaultLabelModel? {
|
||||
guard let errorText = errorText, hasError else { return nil }
|
||||
guard let errorText = errorText, showError else { return nil }
|
||||
var model = DefaultLabelModel()
|
||||
model.textPosition = .left
|
||||
model.typograpicalStyle = .BodyMedium
|
||||
@ -72,7 +72,7 @@ public struct DefaultRadioButtonModel: RadioButtonModel {
|
||||
public var childText: String?
|
||||
public var childTextAttributes: [any LabelAttributeModel]?
|
||||
|
||||
public var hasError: Bool = false
|
||||
public var showError: Bool = false
|
||||
public var errorText: String?
|
||||
|
||||
public var inputId: String?
|
||||
@ -106,7 +106,7 @@ public struct DefaultRadioButtonModel: RadioButtonModel {
|
||||
&& childText == equatable.childText
|
||||
&& childTextAttributes == equatable.childTextAttributes
|
||||
|
||||
&& hasError == equatable.hasError
|
||||
&& showError == equatable.showError
|
||||
&& errorText == equatable.errorText
|
||||
|
||||
&& inputId == equatable.inputId
|
||||
|
||||
@ -8,6 +8,6 @@
|
||||
import Foundation
|
||||
|
||||
public protocol Errorable {
|
||||
var hasError: Bool { get set }
|
||||
var showError: Bool { get set }
|
||||
var errorText: String? { get set }
|
||||
}
|
||||
|
||||
Loading…
Reference in New Issue
Block a user