Fix for Issue MikaelEliasson#50: Don't include TPH discriminator in UpdateAll

Author: MikaelEliasson

EntityFramework.Utilities/EntityFramework.Utilities/EFBatchOperation.cs

@@ -164,15 +164,6 @@ public void UpdateAll<TEntity>(IEnumerable<TEntity> items, Action<UpdateSpecific
                         IsPrimaryKey = p.IsPrimaryKey
                      }).ToList();
 
-                if (tableMapping.TPHConfiguration != null)
-                {
-                    properties.Add(new ColumnMapping
-                    {
-                        NameInDatabase = tableMapping.TPHConfiguration.ColumnName,
-                        StaticValue = tableMapping.TPHConfiguration.Mappings[typeof(TEntity)]
-                    });
-                }
-
                 var spec = new UpdateSpecification<TEntity>();
                 updateSpecification(spec);
                 provider.UpdateItems(items, tableMapping.Schema, tableMapping.TableName, properties, connectionToUse, batchSize, spec);

EntityFramework.Utilities/EntityFramework.Utilities/Properties/AssemblyInfo.cs

@@ -32,5 +32,5 @@
 // You can specify all the values or you can default the Build and Revision Numbers 
 // by using the '*' as shown below:
 // [assembly: AssemblyVersion("1.0.*")]
-[assembly: AssemblyVersion("1.0.0.0")]
-[assembly: AssemblyFileVersion("1.0.0.0")]
+[assembly: AssemblyVersion("1.0.1.0")]
+[assembly: AssemblyFileVersion("1.0.1.0")]

EntityFramework.Utilities/Tests/UpdateBulkTests.cs

@@ -36,6 +36,45 @@ public void UpdateBulk_UpdatesAll()
             }
         }
 
+        [TestMethod]
+        public void UpdateBulk_CanUpdateTPH()
+        {
+            using (var db = Context.Sql())
+            {
+                if (db.Database.Exists())
+                {
+                    db.Database.Delete();
+                }
+                db.Database.Create();
+
+                List<Contact> people = new List<Contact>();
+                people.Add(Contact.Build("FN1", "LN1", "Director"));
+                people.Add(Contact.Build("FN2", "LN2", "Associate"));
+                people.Add(Contact.Build("FN3", "LN3", "Vice President"));
+
+                EFBatchOperation.For(db, db.People).InsertAll(people);
+            }
+
+            using (var db = Context.Sql())
+            {
+                var contacts = db.Contacts.ToList();
+
+                foreach (var contact in contacts)
+                {
+                    contact.FirstName = contact.Title + " " + contact.FirstName;
+                }
+
+                EFBatchOperation.For(db, db.People).UpdateAll(contacts, x => x.ColumnsToUpdate(p => p.FirstName));
+            }
+
+            using (var db = Context.Sql())
+            {
+                var contacts = db.People.OfType<Contact>().OrderBy(c => c.LastName).ToList();
+                Assert.AreEqual(3, contacts.Count);
+                Assert.AreEqual("Director FN1", contacts.First().FirstName);
+            }
+        }
+
         [TestMethod]
         public void UpdateBulk_CanUpdateNumerics()
         {