GetCaml method of Value class doesn't add <Value> tag unless IncludeTimeValue hasValue

[gsalatino]

Hi! The API is beautiful. It's really light and simple to use.

I would just like to point out that there is a Xml render issue when Value doesn't need to include time value. In this case, the Value tag isn't rendered:

        internal string GetCaml()
        {
            var sb = new StringBuilder();

            if (IncludeTimeValue.HasValue)
            {
                sb.AppendLine($"<Value Type='{Type}' IncludeTimeValue='{(IncludeTimeValue.Value ? "TRUE" : "FALSE")}'>");
            }

            sb.AppendLine(GetCamlValue());
            sb.AppendLine("</Value>");

            return sb.ToString();
        }

I've a workaround the issue. It's an ad-hoc solution for queries that not involve a datetime field as part of where clause. Here is it:

      var value = Value.ObjectValue(ValueType.Text, true, filterValue); 
      var viewXml = Query
        .Build(Operator.Equal("fieldName", value))
        .GetCaml()
        .Replace(" IncludeTimeValue='TRUE'", string.Empty);

It would be great if you can fix it, perhaps just by adding:

            else
            {
                sb.AppendLine($"<Value Type='{Type}'>");
            }

Thanks in advance!

[gsalatino]

Thank you very much for your quick response! The fix works great!