Merge pull request #780 from stsrki/dev091-datagrid-null-field

fix: DataGrid must be specified
Megabit · Apr 21, 2020 · ab07588 · ab07588
2 parents 09983a5 + 60dfef1
commit ab07588
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/Source/Extensions/Blazorise.DataGrid/Utils/FunctionCompiler.cs b/Source/Extensions/Blazorise.DataGrid/Utils/FunctionCompiler.cs
@@ -20,6 +20,9 @@ public static Func<TItem> CreateNewItem<TItem>()
         /// <returns>Returns the requested field if it exists.</returns>
         private static Expression GetSafeField( Expression item, string fieldName )
         {
+            if ( string.IsNullOrEmpty( fieldName ) )
+                throw new ArgumentException( $"{nameof( fieldName )} is not specified." );
+
             var parts = fieldName.Split( new char[] { '.' }, 2 );
 
             Expression field = Expression.PropertyOrField( item, parts[0] );
@@ -58,6 +61,9 @@ private static bool IsNullable( Type type )
         /// <returns>Returns the requested field if it exists.</returns>
         private static Expression GetField( Expression item, string fieldName )
         {
+            if ( string.IsNullOrEmpty( fieldName ) )
+                throw new ArgumentException( $"{nameof( fieldName )} is not specified." );
+
             var parts = fieldName.Split( new char[] { '.' }, 2 );
 
             Expression subProperty = Expression.PropertyOrField( item, parts[0] );