refactor writing resource identifier objects

Chris Santero · Chris Santero · commit dc5cc3c682f5 · 2015-06-19T14:57:59.000-04:00
diff --git a/JSONAPI/Json/JsonApiFormatter.cs b/JSONAPI/Json/JsonApiFormatter.cs
@@ -336,10 +336,8 @@ protected void Serialize(object value, Stream writeStream, JsonWriter writer, Js
 
                     Lazy<string> objId = new Lazy<String>(() => GetIdFor(propertyValue));
 
-                    // Write the data object
-                    writer.WriteStartObject();
-                    WriteTypeAndId(writer, prop.PropertyType, propertyValue);
-                    writer.WriteEndObject();
+                    // Write the resource identifier object
+                    WriteResourceIdentifierObject(writer, prop.PropertyType, propertyValue);
 
                     if (iip)
                         if (aggregator != null)
@@ -962,6 +960,13 @@ protected string GetIdFor(object obj)
             return GetValueForIdProperty(idprop, obj);
         }
 
+        private void WriteResourceIdentifierObject(JsonWriter writer, Type propertyType, object propertyValue)
+        {
+            writer.WriteStartObject();
+            WriteTypeAndId(writer, propertyType, propertyValue);
+            writer.WriteEndObject();
+        }
+
         private void WriteTypeAndId(JsonWriter writer, Type propertyType, object propertyValue)
         {
             writer.WritePropertyName("type");
@@ -977,9 +982,7 @@ private void WriteIdsArrayJson(Newtonsoft.Json.JsonWriter writer, IEnumerable<ob
             writer.WriteStartArray();
             while (collectionEnumerator.MoveNext())
             {
-                writer.WriteStartObject();
-                WriteTypeAndId(writer, collectionEnumerator.Current.GetType(), collectionEnumerator.Current);
-                writer.WriteEndObject();
+                WriteResourceIdentifierObject(writer, collectionEnumerator.Current.GetType(), collectionEnumerator.Current);
             }
             writer.WriteEndArray();
         }
