better type check for mssql+ado binary null strip
authorRafael Kitover <rkitover@cpan.org>
Sun, 24 Jun 2012 06:59:11 +0000 (02:59 -0400)
committerRafael Kitover <rkitover@cpan.org>
Tue, 10 Jul 2012 22:06:09 +0000 (18:06 -0400)
commit5efba7fcc89e113c60d78fa246a0217c405ea1fc
tree35a129c5caa29e684fb3bd7fb7b6beefc4851ecb
parenta036182226af9edc62ef393cebca797030f44a9a
better type check for mssql+ado binary null strip

The data_type in the t/lib result class for the blob for the tests
changed to 'blob', while the binary type null stripper only expected
'binary' or 'image' data types, which is wrong for possibly
cross-deployed schemas.

Change _strip_trailing_binary_nulls in ::ADO::CursorUtils to use the
newly added and more generic ::Storage::DBI::_is_binary_type instead of
just checking for 'binary' or 'image'.
lib/DBIx/Class/Storage/DBI.pm
lib/DBIx/Class/Storage/DBI/ADO/CursorUtils.pm
lib/DBIx/Class/Storage/DBI/ADO/Microsoft_SQL_Server.pm
lib/DBIx/Class/Storage/DBI/ADO/Microsoft_SQL_Server/Cursor.pm
lib/DBIx/Class/Storage/DBI/Replicated.pm