From 7af0e223c01a8bf6aadfaabec13579ece88a66e7 Mon Sep 17 00:00:00 2001
From: James Newton-King <james@newtonking.com>
Date: Sun, 28 May 2017 14:34:27 +1200
Subject: [PATCH] -Added support for deserialing JSON to XmlNode, XObject,
 XNode, XContainer properties

---
 Src/Newtonsoft.Json.Tests/Issues/Issue1327.cs | 141 ++++++++++++++++++
 .../Converters/XmlNodeConverter.cs            |   9 +-
 2 files changed, 148 insertions(+), 2 deletions(-)
 create mode 100644 Src/Newtonsoft.Json.Tests/Issues/Issue1327.cs

diff --git a/Src/Newtonsoft.Json.Tests/Issues/Issue1327.cs b/Src/Newtonsoft.Json.Tests/Issues/Issue1327.cs
new file mode 100644
index 00000000..cc38a7b9
--- /dev/null
+++ b/Src/Newtonsoft.Json.Tests/Issues/Issue1327.cs
@@ -0,0 +1,141 @@
+#region License
+// Copyright (c) 2007 James Newton-King
+//
+// Permission is hereby granted, free of charge, to any person
+// obtaining a copy of this software and associated documentation
+// files (the "Software"), to deal in the Software without
+// restriction, including without limitation the rights to use,
+// copy, modify, merge, publish, distribute, sublicense, and/or sell
+// copies of the Software, and to permit persons to whom the
+// Software is furnished to do so, subject to the following
+// conditions:
+//
+// The above copyright notice and this permission notice shall be
+// included in all copies or substantial portions of the Software.
+//
+// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+// EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES
+// OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+// NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
+// HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY,
+// WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+// FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
+// OTHER DEALINGS IN THE SOFTWARE.
+#endregion
+
+#if !(DNXCORE50 || PORTABLE40)
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using Newtonsoft.Json;
+using Newtonsoft.Json.Linq;
+using System.Xml;
+#if !NET20
+using System.Xml.Linq;
+#endif
+#if DNXCORE50
+using Xunit;
+using Test = Xunit.FactAttribute;
+using Assert = Newtonsoft.Json.Tests.XUnitAssert;
+#else
+using NUnit.Framework;
+#endif
+
+namespace Newtonsoft.Json.Tests.Issues
+{
+    [TestFixture]
+    public class Issue1327 : TestFixtureBase
+    {
+#if !PORTABLE
+        public class PersonWithXmlNode
+        {
+            public XmlNode TestXml { get; set; }
+
+            public string Name { get; set; }
+
+            public int IdNumber { get; set; }
+        }
+#endif
+
+#if !NET20
+        public class PersonWithXObject
+        {
+            public XObject TestXml1 { get; set; }
+            public XNode TestXml2 { get; set; }
+            public XContainer TestXml3 { get; set; }
+
+            public string Name { get; set; }
+
+            public int IdNumber { get; set; }
+        }
+#endif
+
+#if !PORTABLE
+        [Test]
+        public void Test_XmlNode()
+        {
+            string json = @"{
+  ""TestXml"": {
+    ""orders"": {
+      ""order"": {
+        ""id"": ""550268"",
+        ""name"": ""vinoth""
+      }
+    }
+  },
+  ""Name"": ""Kumar"",
+  ""IdNumber"": 990268
+}";
+
+            var p = JsonConvert.DeserializeObject<PersonWithXmlNode>(json);
+
+            Assert.AreEqual("Kumar", p.Name);
+            Assert.AreEqual("vinoth", p.TestXml.SelectSingleNode("//name").InnerText);
+        }
+#endif
+
+#if !NET20
+        [Test]
+        public void Test_XObject()
+        {
+            string json = @"{
+  ""TestXml1"": {
+    ""orders"": {
+      ""order"": {
+        ""id"": ""550268"",
+        ""name"": ""vinoth""
+      }
+    }
+  },
+  ""TestXml2"": {
+    ""orders"": {
+      ""order"": {
+        ""id"": ""550268"",
+        ""name"": ""vinoth""
+      }
+    }
+  },
+  ""TestXml3"": {
+    ""orders"": {
+      ""order"": {
+        ""id"": ""550268"",
+        ""name"": ""vinoth""
+      }
+    }
+  },
+  ""Name"": ""Kumar"",
+  ""IdNumber"": 990268
+}";
+
+            var p = JsonConvert.DeserializeObject<PersonWithXObject>(json);
+
+            Assert.AreEqual("Kumar", p.Name);
+            Assert.AreEqual("vinoth", (string)((XDocument)p.TestXml1).Root.Element("order").Element("name"));
+            Assert.AreEqual("vinoth", (string)((XDocument)p.TestXml2).Root.Element("order").Element("name"));
+            Assert.AreEqual("vinoth", (string)((XDocument)p.TestXml3).Root.Element("order").Element("name"));
+        }
+#endif
+    }
+}
+#endif
\ No newline at end of file
diff --git a/Src/Newtonsoft.Json/Converters/XmlNodeConverter.cs b/Src/Newtonsoft.Json/Converters/XmlNodeConverter.cs
index 613f37ba..1edee00b 100644
--- a/Src/Newtonsoft.Json/Converters/XmlNodeConverter.cs
+++ b/Src/Newtonsoft.Json/Converters/XmlNodeConverter.cs
@@ -1627,7 +1627,11 @@ namespace Newtonsoft.Json.Converters
 #if HAVE_XLINQ
             if (typeof(XObject).IsAssignableFrom(objectType))
             {
-                if (objectType != typeof(XDocument) && objectType != typeof(XElement))
+                if (objectType != typeof(XContainer)
+                    && objectType != typeof(XDocument)
+                    && objectType != typeof(XElement)
+                    && objectType != typeof(XNode)
+                    && objectType != typeof(XObject))
                 {
                     throw JsonSerializationException.Create(reader, "XmlNodeConverter only supports deserializing XDocument or XElement.");
                 }
@@ -1640,7 +1644,8 @@ namespace Newtonsoft.Json.Converters
 #if HAVE_XML_DOCUMENT
             if (typeof(XmlNode).IsAssignableFrom(objectType))
             {
-                if (objectType != typeof(XmlDocument))
+                if (objectType != typeof(XmlDocument)
+                    && objectType != typeof(XmlNode))
                 {
                     throw JsonSerializationException.Create(reader, "XmlNodeConverter only supports deserializing XmlDocuments");
                 }