fix: Clarify conditional by explicitly checking for HasValue.

paulhazen · paulhazen · commit 2c8202be2ccc · 2025-02-07T12:10:37.000-08:00
diff --git a/com.playeveryware.eos/Runtime/Core/Config/Attributes/Validators/LengthValidationAttribute.cs b/com.playeveryware.eos/Runtime/Core/Config/Attributes/Validators/LengthValidationAttribute.cs
@@ -68,7 +68,7 @@ public override bool ValidateStringField(string value, out string errorMessage)
                 return false;
             }
 
-            if (value.Length > _maxLength)
+            if (_maxLength.HasValue && value.Length > _maxLength)
             {
                 errorMessage = $"String must be no more than {_maxLength.Value} characters long.";
                 return false;

Original file line number	Diff line number	Diff line change
`@@ -68,7 +68,7 @@ public override bool ValidateStringField(string value, out string errorMessage)`
`68`	`68`	`return false;`
`69`	`69`	`}`
`70`	`70`
`71`		`- if (value.Length > _maxLength)`
	`71`	`+ if (_maxLength.HasValue && value.Length > _maxLength)`
`72`	`72`	`{`
`73`	`73`	`errorMessage = $"String must be no more than {_maxLength.Value} characters long.";`
`74`	`74`	`return false;`