merged in pr restsharp#737

Author: Michael Hallett

RestSharp.Tests/JsonTests.cs

@@ -791,6 +791,18 @@ public void Can_Deserialize_Plain_Values()
             Assert.AreEqual(result, new Guid("c02bdd1e-cce3-4b9c-8473-165e6e93b92a"));
         }
 
+        [Test]
+        public void Can_Deserialize_Dictionary_with_Null()
+        {
+            string doc = File.ReadAllText(Path.Combine("SampleData", "jsondictionary_null.txt"));
+            JsonDeserializer json = new JsonDeserializer { RootElement = "response" };
+            IDictionary<string, object> output = json.Deserialize<Dictionary<string, object>>(new RestResponse { Content = doc });
+
+            IDictionary<string, object> dictionary = (IDictionary<string, object>)output["SomeDictionary"];
+            Assert.AreEqual("abra", dictionary["NonNull"]);
+            Assert.IsNull(dictionary["Null"]);
+        }
+
         private static string CreateJsonWithUnderscores()
         {
             JsonObject doc = new JsonObject();

RestSharp.Tests/RestSharp.Tests.Signed.csproj

@@ -142,6 +142,9 @@
     <Content Include="SampleData\jsondictionary.txt">
       <CopyToOutputDirectory>Always</CopyToOutputDirectory>
     </Content>
+    <Content Include="SampleData\jsondictionary_null.txt">
+      <CopyToOutputDirectory>Always</CopyToOutputDirectory>
+    </Content>
     <Content Include="SampleData\jsonenumtypes.txt">
       <CopyToOutputDirectory>Always</CopyToOutputDirectory>
     </Content>

RestSharp.Tests/RestSharp.Tests.csproj

@@ -129,6 +129,9 @@
     <Content Include="SampleData\iso8601datetimes.txt">
       <CopyToOutputDirectory>Always</CopyToOutputDirectory>
     </Content>
+    <Content Include="SampleData\jsondictionary_null.txt">
+      <CopyToOutputDirectory>Always</CopyToOutputDirectory>
+    </Content>
     <Content Include="SampleData\jsondictionary_KeysType.txt">
       <CopyToOutputDirectory>Always</CopyToOutputDirectory>
     </Content>

RestSharp.Tests/SampleData/jsondictionary_null.txt

@@ -0,0 +1,7 @@
+{
+    "SomeDictionary" : 
+    {
+        "NonNull": "abra",
+        "Null": null
+    }
+}

RestSharp/Deserializers/JsonDeserializer.cs

@@ -204,8 +204,12 @@ private object ConvertValue(Type type, object value)
                 type = type.GetGenericArguments()[0];
             }
 
-            if (type == typeof(object) && value != null)
+            if (type == typeof(object))
             {
+                if (value == null)
+                {
+                    return null;
+                }
                 type = value.GetType();
             }