Refactor Variant Into Two Types

[ghost]

https://github.com/metasys-server/basic-services-dotnet/blob/f1d936984908af6b5e4640f35d514281a97a279f/MetasysServices/Models/Variant.cs#L151

I think this points to the fact that Variant should be decomposed into two types.

1. Variant which is a type that can hold one of String, Number, Boolean, or Array.
2. AttributeValue which holds an id, attribute, Variant, (and culture for localization)

Then Variant is appropriate to use for property values as well as cells in an array. AttributeValue would be used only for property values.

[ghost]

If you do this decomposition, then Variant could have an internal Factory method that takes a JToken for use by MetasysClient.

But it could also have implicit conversion operators to convert numbers, bools, strings, etc to Variant. For convenience, and testing.

So you could do things like the following.

var v:Variant = 35
var x:Variant = "My string"

Or just have public constructors that take all the basic types that need to be supported.

[michaelgwelch]

@Federico-Artioli something to consider.