Fix DeleteById with RowVersion for SQL Server

Author: mythz

src/ServiceStack.OrmLite.SqlServer/Converters/SqlServerSpecialConverters.cs

@@ -4,9 +4,6 @@ namespace ServiceStack.OrmLite.SqlServer.Converters
 {
     public class SqlServerRowVersionConverter : RowVersionConverter
     {
-        public override string ColumnDefinition
-        {
-            get { return "rowversion"; }
-        }
+        public override string ColumnDefinition => "rowversion";
     }
 }

src/ServiceStack.OrmLite/OrmLiteWriteCommandExtensions.cs

@@ -573,6 +573,9 @@ internal static string DeleteByIdSql<T>(this IDbCommand dbCmd, object id, ulong
 
             var rowVersionParam = dbCmd.CreateParameter();
             rowVersionParam.ParameterName = dialectProvider.GetParam("rowVersion");
+            var converter = dialectProvider.GetConverterBestMatch(typeof(ulong));
+            converter.InitDbParam(rowVersionParam, typeof(ulong));
+
             rowVersionParam.Value = rowVersion;
             dbCmd.Parameters.Add(rowVersionParam);
 

tests/ServiceStack.OrmLite.Tests/RowVersionTests.cs

@@ -360,6 +360,18 @@ public void Can_delete_with_current_rowversion()
             var count = db.Count<ModelWithRowVersion>(m => m.Id == rowId);
             Assert.That(count, Is.EqualTo(0));
         }
+        
+        [Test]
+        public void Can_DeleteById_with_current_rowversion()
+        {
+            var rowId = db.Insert(new ModelWithRowVersion { Text = "Four" }, selectIdentity: true);
+            var row = db.SingleById<ModelWithRowVersion>(rowId);
+
+            db.DeleteById<ModelWithRowVersion>(row.Id, rowVersion:row.RowVersion);
+
+            var count = db.Count<ModelWithRowVersion>(m => m.Id == rowId);
+            Assert.That(count, Is.EqualTo(0));
+        }
 
         [Test]
         public void Update_with_outdated_rowversion_throws()