PATCH relationships

Author: jaredcnance

JsonApiDotNetCore/Abstractions/ModelAccessor.cs

@@ -24,15 +24,28 @@ public static Type GetTypeFromModelRelationshipName(Type modelType, string relat
 
     public static object SetValuesOnModelInstance(object model, Dictionary<string, object> jsonApiAttributes, Dictionary<string, object> jsonApiRelationships)
     {
-      var modelProperties = model.GetType().GetProperties().ToList();
+      var patches = GetEntityPatch(model.GetType(), jsonApiAttributes, jsonApiRelationships);
+      foreach(var patch in patches)
+      {
+         patch.Key.SetValue(model, patch.Value);
+      }
+
+      return model;
+    }
+
+    public static Dictionary<PropertyInfo, object> GetEntityPatch(Type modelType, Dictionary<string, object> jsonApiAttributes, Dictionary<string, object> jsonApiRelationships)
+    {
+      var patchDefinitions = new Dictionary<PropertyInfo, object>();
+
+      var modelProperties = modelType.GetProperties().ToList();
       foreach (var attribute in jsonApiAttributes)
       {
         modelProperties.ForEach(pI =>
         {
           if (pI.Name.ToProperCase() == attribute.Key.ToProperCase())
           {
             var convertedValue = Convert.ChangeType(attribute.Value, pI.PropertyType);
-            pI.SetValue(model, convertedValue);
+            patchDefinitions.Add(pI, convertedValue);
           }
         });
       }
@@ -49,13 +62,13 @@ public static object SetValuesOnModelInstance(object model, Dictionary<string, o
             if (pI.Name.ToProperCase() == relationshipPropertyName.ToProperCase())
             {
               var convertedValue = Convert.ChangeType(relationshipId, pI.PropertyType);
-              pI.SetValue(model, convertedValue);
+              patchDefinitions.Add(pI, convertedValue);
             }
           });
         }
       }
 
-      return model;
+      return patchDefinitions;
     }
   }
 }

JsonApiDotNetCore/Services/JsonApiDeserializer.cs

@@ -53,25 +53,9 @@ public Dictionary<PropertyInfo, object> GetEntityPatch()
     {
       var datum = GetSignularJsonApiDatum();
       var attributes = datum.Attributes;
-      // var relationships = datum.Relationships;
-
-      var patchDefinitions = new Dictionary<PropertyInfo, object>();
-
-      var modelProperties = _context.GetEntityType().GetProperties().ToList();
-
-      foreach (var attribute in attributes)
-      {
-        modelProperties.ForEach(pI =>
-        {
-          if (pI.Name.ToProperCase() == attribute.Key.ToProperCase())
-          {
-            var convertedValue = Convert.ChangeType(attribute.Value, pI.PropertyType);
-            patchDefinitions.Add(pI, convertedValue);
-          }
-        });
-      }
+      var relationships = datum.Relationships;
 
-      return patchDefinitions;
+      return ModelAccessor.GetEntityPatch(_context.GetEntityType(), attributes, relationships);
     }
 
   }