Merge pull request #450 from nofrixion/bugfix/MOOV-3920-remittanceemail-whitespace

agranillonf · web-flow · commit 646b22bedab1 · 2024-11-07T08:57:15.000-06:00
MOOV-3920: Allow null or whitespace email address list for payruns
diff --git a/src/NoFrixion.MoneyMoov/Attributes/EmailAddressMultipleAttribute.cs b/src/NoFrixion.MoneyMoov/Attributes/EmailAddressMultipleAttribute.cs
@@ -38,7 +38,7 @@ public override bool IsValid(object? value)
 
         if (string.IsNullOrWhiteSpace(emailAddresses))
         {
-            return false;
+            return true;
         }
 
         var emails = emailAddresses.Split(new[] { ';', ' ', ',' }, StringSplitOptions.RemoveEmptyEntries);

Original file line number	Diff line number	Diff line change
`@@ -38,7 +38,7 @@ public override bool IsValid(object? value)`
`38`	`38`
`39`	`39`	`if (string.IsNullOrWhiteSpace(emailAddresses))`
`40`	`40`	`{`
`41`		`- return false;`
	`41`	`+ return true;`
`42`	`42`	`}`
`43`	`43`
`44`	`44`	`var emails = emailAddresses.Split(new[] { ';', ' ', ',' }, StringSplitOptions.RemoveEmptyEntries);`