fix: add type validation for cell min/max (#27)

- add type validation for cell min/max
siemens · Jan 25, 2024 · cab52e7 · cab52e7
1 parent 4a461da
commit cab52e7
Show file tree

Hide file tree

Showing 3 changed files with 36 additions and 5 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,12 +1,16 @@
 # SourceGrid
 
-## [3.0.0] - 2024-01-18
+## [3.0.0] - 2024-01-29
 
 ### Major Changes
 
 - Migrate to .net 8.0 related to #3
 - Deprecated DevAge Serialization 
 
+### Fixed
+
+- Fixed bug in validator base related to #25
+
 ## [2.0.0] - 2023-11-24
 
 ### Major Changes

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