Add support for binary blobs

[abe545]

Right now, they could be retrieved through an untyped data transformer (attribute, or IDataTransformer). However, it would be nice if the byte[] property could be specified without one.

[leevva]

What do you mean by "untyped data transformer"?
Could you explain what i need to do to map varbinary(max) to byte[] ?

[abe545]

There is support for data transformation classes in the library. You can implement them with either the IDataTransformer class, or subclassing DataTransformerAttributeBase, and applying it to your property. I haven't tested this on a byte[], but conceptually it should work. The one reason why it may not work is that the library seems to be detecting this should be a hierarchical result set, and there is no matching result set.

Via DataTransformerAttributeBase:

class RawDataTransformerAttribute : DataTransformerAttributeBase
{
    public override object Transform(object value, Type targetType, bool isNullable) => value;
}

 internal class ContentSearchResultSet
 {
    [Column("raw"), RawDataTransformer]
    public byte[] Raw { get; set; }
    ...
}

Via IDataTransformer:

class ByteDataTransformer : IDataTransformer
{
    public bool CanTransform(object value, Type targetType, bool isNullable, IEnumerable<Attribute> propertyAttributes)
    {
        return targetType == typeof(byte[]);
    }

    public object Transform(object value, Type targetType, bool isNullable, IEnumerable<Attribute> propertyAttributes)
    {
        return value;
    }
}
...
StoredProcedure.Create("Foo")
               .WithDataTransformer(new ByteDataTransformer())

[leevva]

Both variants don't work.
Probably the problem is in IsSimpleType method. I think for byte[] result of it should be true.
Then SimpleTypeRowFactory will be called:

            if (itemType.IsSimpleType())
                return new SimpleTypeRowFactory<T>();

instead of HierarchicalTypeRowFactory.

[abe545]

Yep, that would probably do it, to at least allow you to use the transformer. I'm hoping to be able to get to some of these issues over the next couple of weeks, but it is tough with the holidays...

[leevva]

I hope this will happen as soon as possible. Look forward to that day.