-Fixed deserializing DataTable with error handling

Src/Newtonsoft.Json.Tests/Converters/DataTableConverterTests.cs

@@ -26,6 +26,7 @@
 using System.IO;
 using System.Text;
 using Newtonsoft.Json.Bson;
+using Newtonsoft.Json.Converters;
 using Newtonsoft.Json.Linq;
 #if !(NETFX_CORE || PORTABLE || DNXCORE50 || PORTABLE40)
 using System;
@@ -546,6 +547,53 @@ namespace Newtonsoft.Json.Tests.Converters
             }
         }
 #endif
+
+        [Test]
+        public void HandleColumnOnError()
+        {
+            string json = "[{\"timeCol\":\"\"}]";
+            DataTable table = JsonConvert.DeserializeObject<CustomDataTable>(json);
+
+            Assert.AreEqual(DBNull.Value, table.Rows[0]["timeCol"]);
+        }
+
+        [JsonConverter(typeof(DataTableConverterTest))]
+        public class CustomDataTable : DataTable
+        {
+        }
+
+        public class DataTableConverterTest : DataTableConverter
+        {
+            protected DataTable CreateTable()
+            {
+                CustomDataTable table = new CustomDataTable();
+                table.Columns.Add("timeCol", typeof(DateTime));
+                return table;
+            }
+
+            public override object ReadJson(JsonReader reader, Type objectType, object existingValue, JsonSerializer serializer)
+            {
+                if (existingValue == null)
+                {
+                    existingValue = CreateTable();
+                }
+
+                serializer.Error += OnError;
+                try
+                {
+                    return base.ReadJson(reader, objectType, existingValue, serializer);
+                }
+                finally
+                {
+                    serializer.Error -= OnError;
+                }
+            }
+
+            private void OnError(object sender, Json.Serialization.ErrorEventArgs e)
+            {
+                e.ErrorContext.Handled = true;
+            }
+        }
     }
 }
 

Src/Newtonsoft.Json.Tests/Serialization/TraceWriterTests.cs

@@ -979,7 +979,6 @@ Newtonsoft.Json Error: 0 : Error!
     1,
     1,
     1,
-    1,
     true,
     9999999990000000000000000000000000000000000,
     true,

Src/Newtonsoft.Json/Converters/DataTableConverter.cs

@@ -187,7 +187,11 @@ namespace Newtonsoft.Json.Converters
                 }
                 else
                 {
-                    dr[columnName] = (reader.Value != null) ? serializer.Deserialize(reader, column.DataType) : DBNull.Value;
+                    object columnValue = (reader.Value != null)
+                        ? serializer.Deserialize(reader, column.DataType) ?? DBNull.Value
+                        : DBNull.Value;
+
+                    dr[columnName] = columnValue;
                 }
 
                 reader.ReadAndAssert();