Merge pull request #21 from dotarj/refactor-should-serialize-method

should serialize return value changed

Author: dotarj

src/PartialResponse.AspNetCore.Mvc.Formatters.Json/Internal/JsonSerializerExtensions.cs

@@ -34,19 +34,15 @@ public static void Serialize(this JsonSerializer jsonSerializer, JsonWriter json
                 var context = new SerializerContext(shouldSerialize);
                 var token = JToken.FromObject(value, jsonSerializer);
 
-                var array = token as JArray;
-
-                if (array != null)
+                if (token is JArray array)
                 {
                     RemoveArrayElements(array, null, context);
 
                     array.WriteTo(jsonWriter);
                 }
                 else
                 {
-                    var @object = token as JObject;
-
-                    if (@object != null)
+                    if (token is JObject @object)
                     {
                         RemoveObjectProperties(@object, null, context);
 
@@ -97,7 +93,7 @@ private static void RemoveObjectProperties(JObject @object, string currentPath,
                 {
                     var path = CombinePath(currentPath, property.Name);
 
-                    return context.ShouldSerialize(path);
+                    return !context.ShouldSerialize(path);
                 })
                 .ToList()
                 .ForEach(property => property.Remove());

src/PartialResponse.AspNetCore.Mvc.Formatters.Json/Internal/SerializerContext.cs

@@ -22,7 +22,7 @@ public bool ShouldSerialize(string path)
                 return this.cache[path];
             }
 
-            var result = !this.shouldSerialize(path);
+            var result = this.shouldSerialize(path);
 
             this.cache.Add(path, result);