fix part 2

Author: StefH

src/System.Linq.Dynamic.Core.NewtonsoftJson/Config/NewtonsoftJsonParsingConfig.cs

@@ -10,7 +10,10 @@ public class NewtonsoftJsonParsingConfig : ParsingConfig
     /// <summary>
     /// The default ParsingConfig for <see cref="NewtonsoftJsonParsingConfig"/>.
     /// </summary>
-    public new static NewtonsoftJsonParsingConfig Default { get; } = new();
+    public new static NewtonsoftJsonParsingConfig Default { get; } = new NewtonsoftJsonParsingConfig
+    {
+        ConvertObjectToSupportComparison = true
+    };
 
     /// <summary>
     /// The default <see cref="DynamicJsonClassOptions"/> to use.
@@ -28,7 +31,7 @@ public class NewtonsoftJsonParsingConfig : ParsingConfig
     /// <remarks>
     /// Use this property to control how the normalization process handles properties that are missing or undefined.
     /// The selected behavior may affect the output or error handling of normalization operations.
-    /// The default value is <see cref="NormalizationNonExistingPropertyBehavior.UseDefaultValue"/>.
+    /// The default value is <see cref="NormalizationNonExistingPropertyBehavior.UseNull"/>.
     /// </remarks>
     public NormalizationNonExistingPropertyBehavior NormalizationNonExistingPropertyValueBehavior { get; set; }
 }

src/System.Linq.Dynamic.Core.NewtonsoftJson/Config/NormalizationNonExistingPropertyBehavior.cs

@@ -6,12 +6,12 @@
 public enum NormalizationNonExistingPropertyBehavior
 {
     /// <summary>
-    /// Specifies that the default value should be used.
+    /// Specifies that a null value should be used.
     /// </summary>
-    UseDefaultValue = 0,
+    UseNull = 0,
 
     /// <summary>
-    /// Specifies that null values should be used.
+    /// Specifies that the default value should be used.
     /// </summary>
-    UseNull = 1
+    UseDefaultValue = 1    
 }

src/System.Linq.Dynamic.Core.NewtonsoftJson/Extensions/JObjectExtensions.cs

@@ -44,10 +44,7 @@ private class JTokenResolvers : Dictionary<JTokenType, Func<JToken, DynamicJsonC
         foreach (var prop in src.Properties())
         {
             var value = Resolvers[prop.Type](prop.Value, options);
-            if (value != null)
-            {
-                dynamicPropertiesWithValue.Add(new DynamicPropertyWithValue(prop.Name, value));
-            }
+            dynamicPropertiesWithValue.Add(new DynamicPropertyWithValue(prop.Name, value));
         }
 
         return DynamicClassFactory.CreateInstance(dynamicPropertiesWithValue);

src/System.Linq.Dynamic.Core.NewtonsoftJson/NewtonsoftJsonExtensions.cs

@@ -871,8 +871,11 @@ private static JArray ToJArray(Func<IQueryable> func)
 
     private static IQueryable ToQueryable(JArray source, NewtonsoftJsonParsingConfig? config = null)
     {
-        var normalized = config?.Normalize == true ?
-            NormalizeUtils.NormalizeArray(source, config.NormalizationNonExistingPropertyValueBehavior):
+        config = config ?? NewtonsoftJsonParsingConfig.Default;
+        config.ConvertObjectToSupportComparison = true;
+
+        var normalized = config.Normalize == true ?
+            NormalizeUtils.NormalizeArray(source, config.NormalizationNonExistingPropertyValueBehavior) :
             source;
 
         return normalized

src/System.Linq.Dynamic.Core.NewtonsoftJson/Utils/NormalizeUtils.cs

@@ -86,7 +86,7 @@ private static JObject NormalizeObject(JObject source, Dictionary<string, JsonVa
                 }
                 else
                 {
-                    result[key] = normalizationBehavior == NormalizationNonExistingPropertyBehavior.UseDefaultValue ? GetDefaultValue(schema[key]) : JValue.CreateNull();
+                    result[key] = GetDefaultOrNullValue(normalizationBehavior, schema[key]);
                 }
             }
         }
@@ -105,7 +105,7 @@ private static JObject CreateEmptyObject(Dictionary<string, JsonValueInfo> schem
             }
             else
             {
-                obj[key] = normalizationBehavior == NormalizationNonExistingPropertyBehavior.UseDefaultValue ? GetDefaultValue(schema[key]) : JValue.CreateNull();
+                obj[key] = GetDefaultOrNullValue(normalizationBehavior, schema[key]);
             }
         }
 
@@ -125,7 +125,28 @@ private static JToken GetDefaultValue(JsonValueInfo jType)
             JTokenType.Integer => default(int),
             JTokenType.String => string.Empty,
             JTokenType.TimeSpan => TimeSpan.MinValue,
+            _ => GetNullValue(jType),
+        };
+    }
+
+    private static JValue GetNullValue(JsonValueInfo jType)
+    {
+        return jType.Type switch
+        {
+            JTokenType.Boolean => new JValue((bool?)null),
+            JTokenType.Bytes => new JValue((byte[]?)null),
+            JTokenType.Date => new JValue((DateTime?)null),
+            JTokenType.Float => new JValue((float?)null),
+            JTokenType.Guid => new JValue((Guid?)null),
+            JTokenType.Integer => new JValue((int?)null),
+            JTokenType.String => new JValue((string?)null),
+            JTokenType.TimeSpan => new JValue((TimeSpan?)null),
             _ => JValue.CreateNull(),
         };
     }
+
+    private static JToken GetDefaultOrNullValue(NormalizationNonExistingPropertyBehavior behavior, JsonValueInfo jType)
+    {
+        return behavior == NormalizationNonExistingPropertyBehavior.UseDefaultValue ? GetDefaultValue(jType) : GetNullValue(jType);
+    }
 }

src/System.Linq.Dynamic.Core.SystemTextJson/Config/NormalizationNonExistingPropertyBehavior.cs

@@ -6,12 +6,12 @@
 public enum NormalizationNonExistingPropertyBehavior
 {
     /// <summary>
-    /// Specifies that the default value should be used.
+    /// Specifies that a null value should be used.
     /// </summary>
-    UseDefaultValue = 0,
+    UseNull = 0,
 
     /// <summary>
-    /// Specifies that null values should be used.
+    /// Specifies that the default value should be used.
     /// </summary>
-    UseNull = 1
+    UseDefaultValue = 1    
 }

src/System.Linq.Dynamic.Core.SystemTextJson/Config/SystemTextJsonParsingConfig.cs

@@ -24,7 +24,7 @@ public class SystemTextJsonParsingConfig : ParsingConfig
     /// <remarks>
     /// Use this property to control how the normalization process handles properties that are missing or undefined.
     /// The selected behavior may affect the output or error handling of normalization operations.
-    /// The default value is <see cref="NormalizationNonExistingPropertyBehavior.UseDefaultValue"/>.
+    /// The default value is <see cref="NormalizationNonExistingPropertyBehavior.UseNull"/>.
     /// </remarks>
     public NormalizationNonExistingPropertyBehavior NormalizationNonExistingPropertyValueBehavior { get; set; }
 }

src/System.Linq.Dynamic.Core.SystemTextJson/Extensions/JsonDocumentExtensions.cs

@@ -34,10 +34,7 @@ private class JTokenResolvers : Dictionary<JsonValueKind, Func<JsonElement, Dyna
         foreach (var prop in src.Value.EnumerateObject())
         {
             var value = Resolvers[prop.Value.ValueKind](prop.Value, options);
-            if (value != null)
-            {
-                dynamicPropertiesWithValue.Add(new DynamicPropertyWithValue(prop.Name, value));
-            }
+            dynamicPropertiesWithValue.Add(new DynamicPropertyWithValue(prop.Name, value));
         }
 
         return DynamicClassFactory.CreateInstance(dynamicPropertiesWithValue);
@@ -129,7 +126,7 @@ private static IEnumerable ConvertJsonElementToEnumerable(JsonElement arg, Dynam
     private static IEnumerable ConvertToTypedArray(IEnumerable<object?> src, Type newType)
     {
         var method = ConvertToTypedArrayGenericMethod.MakeGenericMethod(newType);
-        return (IEnumerable)method.Invoke(null, new object[] { src })!;
+        return (IEnumerable)method.Invoke(null, [src])!;
     }
 
     private static readonly MethodInfo ConvertToTypedArrayGenericMethod = typeof(JsonDocumentExtensions).GetMethod(nameof(ConvertToTypedArrayGeneric), BindingFlags.NonPublic | BindingFlags.Static)!;

src/System.Linq.Dynamic.Core.SystemTextJson/Utils/NormalizeUtils.cs

@@ -113,7 +113,7 @@ private static JsonObject NormalizeObject(JsonObject source, Dictionary<string,
                 }
                 else
                 {
-                    result[key] = normalizationBehavior == NormalizationNonExistingPropertyBehavior.UseDefaultValue ? GetDefaultValue(jType) : null;
+                    result[key] = GetDefaultOrNullValue(normalizationBehavior, jType);
                 }
             }
         }
@@ -135,7 +135,7 @@ private static JsonObject CreateEmptyObject(Dictionary<string, JsonValueInfo> sc
             }
             else
             {
-                obj[key] = normalizationBehavior == NormalizationNonExistingPropertyBehavior.UseDefaultValue ? GetDefaultValue(jType) : null;
+                obj[key] = GetDefaultOrNullValue(normalizationBehavior, jType);
             }
         }
 
@@ -151,7 +151,25 @@ private static JsonObject CreateEmptyObject(Dictionary<string, JsonValueInfo> sc
             JsonValueKind.Number => default(int),
             JsonValueKind.String => string.Empty,
             JsonValueKind.True => false,
+            _ => GetNullValue(jType),
+        };
+    }
+
+    private static JsonNode? GetNullValue(JsonValueInfo jType)
+    {
+        return jType.Type switch
+        {
+            JsonValueKind.Array => null,
+            JsonValueKind.False => JsonValue.Create<bool?>(false),
+            JsonValueKind.Number => JsonValue.Create<int?>(null),
+            JsonValueKind.String => JsonValue.Create<string?>(null),
+            JsonValueKind.True => JsonValue.Create<bool?>(true),
             _ => null,
         };
     }
+
+    private static JsonNode? GetDefaultOrNullValue(NormalizationNonExistingPropertyBehavior behavior, JsonValueInfo jType)
+    {
+        return behavior == NormalizationNonExistingPropertyBehavior.UseDefaultValue ? GetDefaultValue(jType) : GetNullValue(jType);
+    }
 }

test/System.Linq.Dynamic.Core.NewtonsoftJson.Tests/NewtonsoftJsonTests.cs

@@ -568,6 +568,6 @@ public void NormalizeArray_When_NormalizeIsFalse_ShouldThrow()
         Action act = () => JArray.Parse(array).Where(config, "Age >= 30");
 
         // Assert
-        act.Should().Throw<InvalidOperationException>().WithMessage("The binary operator GreaterThanOrEqual is not defined for the types 'System.Object' and 'System.Int32'.");
+        act.Should().Throw<InvalidOperationException>().WithMessage("Unable to find property 'Age' on type '<>*");
     }
 }