Include bools in UpdateNonDefaults

Author: mythz

src/ServiceStack.OrmLite/Expressions/SqlExpression.cs

@@ -744,7 +744,10 @@ public virtual string ToUpdateStatement(T item, bool excludeDefaults = false)
                 if (updateFields.Count > 0 && !updateFields.Contains(fieldDef.Name)) continue; // added
 
                 var value = fieldDef.GetValue(item);
-                if (excludeDefaults && (value == null || value.Equals(value.GetType().GetDefaultValue()))) continue; //GetDefaultValue?
+                if (excludeDefaults 
+                    && (value == null || value.Equals(value.GetType().GetDefaultValue()))
+                    && !(value is bool)) 
+                    continue;
 
                 fieldDef.GetQuotedValue(item, DialectProvider);
 

tests/ServiceStack.OrmLite.Tests/OrmLiteUpdateTests.cs

@@ -166,7 +166,7 @@ public void Can_Update_Only_Blobs()
         {
             db.DropAndCreateTable<SomeBlobs>();
 
-            db.Insert(new SomeBlobs {FirstName = "Bro", LastName = "Last"});
+            db.Insert(new SomeBlobs { FirstName = "Bro", LastName = "Last" });
             db.Insert(new SomeBlobs { FirstName = "Sis", LastName = "Last" });
 
             var existing = db.Select<SomeBlobs>(p => p.FirstName == "Bro").First();
@@ -183,7 +183,7 @@ public void Can_Update_Only_Blobs()
             existing.Blob2 = new byte[blob2Bytes];
             Buffer.BlockCopy(blob2Array, 0, existing.Blob2, 0, blob2Bytes);
 
-            db.UpdateOnly(existing, p => new {p.Blob1, p.Blob2, p.FirstName}, r => r.LastName == "Last" && r.FirstName == "Bro");
+            db.UpdateOnly(existing, p => new { p.Blob1, p.Blob2, p.FirstName }, r => r.LastName == "Last" && r.FirstName == "Bro");
 
             var verify = db.Select<SomeBlobs>(p => p.FirstName == "Bro").First();
 
@@ -193,6 +193,46 @@ public void Can_Update_Only_Blobs()
             Assert.That(existing.Blob1, Is.EquivalentTo(verify.Blob1));
             Assert.That(existing.Blob2, Is.EquivalentTo(verify.Blob2));
         }
+
+        public class PocoWithBool
+        {
+            public int Id { get; set; }
+            public bool Bool { get; set; }
+        }
+
+        public class PocoWithNullableBool
+        {
+            public int Id { get; set; }
+            public bool? Bool { get; set; }
+        }
+
+        [Test]
+        public void Can_UpdateOnly_bool_columns()
+        {
+            using (var db = OpenDbConnection())
+            {
+                db.DropAndCreateTable<PocoWithBool>();
+
+                db.Insert(new PocoWithBool { Id = 1, Bool = true });
+                db.UpdateNonDefaults(new PocoWithBool { Bool = false }, x => x.Id == 1);
+                var row = db.SingleById<PocoWithBool>(1);
+                Assert.That(row.Bool, Is.False);
+            }
+        }
+
+        [Test]
+        public void Can_UpdateOnly_nullable_bool_columns()
+        {
+            using (var db = OpenDbConnection())
+            {
+                db.DropAndCreateTable<PocoWithNullableBool>();
+
+                db.Insert(new PocoWithNullableBool { Id = 1, Bool = true });
+                db.UpdateNonDefaults(new PocoWithNullableBool { Bool = false }, x => x.Id == 1);
+                var row = db.SingleById<PocoWithNullableBool>(1);
+                Assert.That(row.Bool, Is.False);
+            }
+        }
     }
 
     [CompositeIndex("FirstName", "LastName")]
@@ -205,4 +245,5 @@ public class SomeBlobs
         public byte[] Blob1 { get; set; }
         public byte[] Blob2 { get; set; }
     }
+
 }