JsonIgnore Needs to be Added wherever XmlIgnore is

[mtrutledge]

Description of problem

I am currently working on updating the Redis Cache Provider to support JSON serialization with a binary fall back. A common error I am currently seeing is:

Newtonsoft.Json.JsonSerializationException: Self referencing loop detected for property 'Vocabulary' with type 'DotNetNuke.Entities.Content.Taxonomy.Vocabulary'. Path '$values[0].DesktopModule.Terms.$values[0].Vocabulary.Terms.$values[0]'.

Looking at the file DNN Platform/Library/Entities/Content/Taxonomy/Term.cs I see that in order to avoid this issue on the XML side, XmlIgnore was added to multiple properties. In order to support Json Serialization we will need to add JsonIgnore to the attributes that have XmlIgnore

Description of solution

In order to support Json Serialization we will need to add JsonIgnore to the attributes that have XmlIgnore

[mtrutledge]

I can work on this one

[mtrutledge]

Are there any preferences on how I do this? I can do just the Terms.cs class now. Or I can search for [XmlIgnore] and add [JsonIgnore] as well. I just don't want to create a PR that has 100's of file changes. Let me know.

[mitchelsellers]

@mtrutledge My preference for this would be lets go for the full solution in a single PR, targeting the 9.11.0 release branch. Improving the serialization ability of core objects is a positive change for sure.