Merge branch 'bugfix/CXTDT-182396' into 'develop'

fix

See merge request BPHV_MIPS/mvm_core_ui!751
This commit is contained in:
Suresh, Kamlesh Jain 2021-08-06 16:31:13 +00:00
commit b2d7f5a0ab
2 changed files with 10 additions and 2 deletions

View File

@ -50,7 +50,11 @@ public class RuleAnyRequiredModel: RulesProtocol {
if fieldValidity {
return (fieldValidity, previousValidity)
}
}
}
// if the rule breaks all fields should be set to false
fields.forEach { (formKey) in
previousValidity[formKey] = false
}
return (valid: false, fieldValidity: previousValidity)

View File

@ -32,7 +32,7 @@ public class RuleAnyValueChangedModel: RulesProtocol {
for formKey in fields {
guard let formField = fieldMolecules[formKey] else { continue }
var fieldValidity = isValid(formField)
// If past rule is invalid forr a field, the current rule should not flip the validity of a field
// If past rule is invalid for a field, the current rule should not flip the validity of a field
if let validity = previousFieldValidity[formKey], !validity, fieldValidity {
fieldValidity = false
}
@ -40,6 +40,10 @@ public class RuleAnyValueChangedModel: RulesProtocol {
if fieldValidity {
return (true, previousValidity)
}
}
// if the rule breaks all fields should be set to false
fields.forEach { (formKey) in
previousValidity[formKey] = false
}
return (valid: false, fieldValidity: previousValidity)