Allow deserializing lists with null in them. Should resolve pull request #286.

Author: petejohanson

RestSharp.Tests/JsonTests.cs

@@ -71,6 +71,19 @@ public void Can_Deserialize_Simple_Generic_List_of_Simple_Types()
 			Assert.NotEmpty(output);
 		}
 
+		[Fact]
+		public void Can_Deserialize_Simple_Generic_List_of_Simple_Types_With_Nulls ()
+		{
+			const string content = "{\"users\":[\"johnsheehan\",\"jagregory\",null,\"drusellers\",\"structuremap\"]}";
+			var json = new JsonDeserializer { RootElement = "users" };
+
+			var output = json.Deserialize<List<string>> (new RestResponse { Content = content });
+
+			Assert.NotEmpty (output);
+			Assert.Equal (null, output[2]);
+			Assert.Equal (5, output.Count);
+		}
+
 		[Fact]
 		public void Can_Deserialize_Simple_Generic_List_Given_Item_Without_Array ()
 		{

RestSharp/Deserializers/JsonDeserializer.cs

@@ -209,10 +209,22 @@ private IList BuildList(Type type, object parent)
 					}
 					else if (itemType == typeof (string))
 					{
+						if (element == null)
+						{
+							list.Add (null);
+							continue;
+						}
+
 						list.Add (element.ToString ());
 					}
 					else
 					{
+						if (element == null)
+						{
+							list.Add (null);
+							continue;
+						}
+
 						var item = CreateAndMap (itemType, element);
 						list.Add (item);
 					}