Handle SYBVARIANT types from SQL function. Fixes #317. Fixed #321.

Since we can not rely on determining the type from SYBVARIANT we rely on checking for a simple 4-byte (assumed integer) since this is the first time we have encountered it. Else, return binary/string representation.

Author: metaskills

CHANGELOG

@@ -1,5 +1,6 @@
 * 1.0.6 *
 
+* Handle SYBVARIANT types from SQL function. Fixes #317. Fixed #321.
 * Fix `use_utf16` optoin for booleans. Fixes #314
 * Add `-q` check for bin puts. Fixes #318
 * Use FreeTDS 1.00.21.

ext/tiny_tds/result.c

@@ -325,6 +325,15 @@ static VALUE rb_tinytds_result_fetch_row(VALUE self, ID timezone, int symbolize_
         case SYBTEXT:
           val = ENCODED_STR_NEW(data, data_len);
           break;
+        case 98: { // SYBVARIANT
+          if (data_len == 4) {
+            val = INT2NUM(*(DBINT *)data);
+            break;
+          } else {
+            val = ENCODED_STR_NEW(data, data_len);
+            break;
+          }
+        }
         default:
           val = ENCODED_STR_NEW(data, data_len);
           break;

test/result_test.rb

@@ -335,6 +335,11 @@ class ResultTest < TinyTds::TestCase
       assert_nil result.return_code
     end
 
+    it 'with LOGINPROPERTY function' do
+      v = @client.execute("SELECT LOGINPROPERTY('sa', 'IsLocked') as v").first['v']
+      v.must_equal 0
+    end
+
     describe 'with multiple result sets' do
 
       before do