fix: add type validation for cell min/max

siemens · Jan 24, 2024 · f88793e · f88793e
1 parent 4a461da
commit f88793e
Show file tree

Hide file tree

Showing 2 changed files with 31 additions and 4 deletions.
diff --git a/Src/DevAgeSourcePack4/ComponentModel/Validator/ValidatorBase.cs b/Src/DevAgeSourcePack4/ComponentModel/Validator/ValidatorBase.cs
@@ -498,6 +498,11 @@ public object MinimumValue
                 if (m_MinimumValue != value)
                 {
                     m_MinimumValue = value;
+
+                    if (m_MinimumValue.GetType() != ValueType)
+                    {
+                        throw new ValidationErrorException(ValueTypeName, ObjectToStringForError(m_MinimumValue));
+                    }
                     OnChanged(EventArgs.Empty);
                 }
             }
@@ -515,6 +520,11 @@ public object MaximumValue
                 if (m_MaximumValue != value)
                 {
                     m_MaximumValue = value;
+
+                    if (m_MaximumValue.GetType() != ValueType)
+                    {
+                        throw new ValidationErrorException(ValueTypeName, ObjectToStringForError(m_MaximumValue));
+                    }
                     OnChanged(EventArgs.Empty);
                 }
             }

diff --git a/Src/DevAgeSourcePack4/Data/Exceptions/Exceptions.cs b/Src/DevAgeSourcePack4/Data/Exceptions/Exceptions.cs
@@ -127,10 +127,27 @@ public ConversionErrorException(string destinationType, string value, Exception
 		}
 	}
 
-	/// <summary>
-	/// Common EventArgs class used to store and raise events with an Exception associated
-	/// </summary>
-	public class ExceptionEventArgs : EventArgs
+    /// <summary>
+    /// Validation exception
+    /// </summary>
+    [Serializable]
+    public class ValidationErrorException : DevAgeApplicationException
+    {
+        /// <summary>
+        /// Constructor
+        /// </summary>
+        /// <param name="type"></param>
+        /// <param name="value"></param>
+		public ValidationErrorException(string type, string value) :
+            base(value + " is not of type " + type + ".")
+        {
+        }
+    }
+
+    /// <summary>
+    /// Common EventArgs class used to store and raise events with an Exception associated
+    /// </summary>
+    public class ExceptionEventArgs : EventArgs
 	{
         /// <summary>
         /// Constructor