Use MetadataToken when searching for IProperty for a specific MemberInfo

* Using equality comparison yields false when the property comes from a base class because the reflected type is taken in consideration as well

Author: PawelGerr

Directory.Build.props

@@ -2,7 +2,7 @@
 
   <PropertyGroup>
     <Copyright>(c) $([System.DateTime]::Now.Year), Pawel Gerr. All rights reserved.</Copyright>
-    <VersionPrefix>4.4.1</VersionPrefix>
+    <VersionPrefix>4.4.2</VersionPrefix>
     <Authors>Pawel Gerr</Authors>    
     <GenerateDocumentationFile>true</GenerateDocumentationFile>
     <PackageProjectUrl>https://dev.azure.com/pawelgerr/Thinktecture.EntityFrameworkCore</PackageProjectUrl>

src/Thinktecture.EntityFrameworkCore.BulkOperations/Extensions/BulkOperationsCollectionExtensions.cs

@@ -46,7 +46,8 @@ internal static IReadOnlyList<PropertyWithNavigations> ConvertToEntityProperties
 
    private static IProperty? FindProperty(IEntityType entityType, MemberInfo memberInfo)
    {
-      return entityType.GetProperties().FirstOrDefault(property => property.PropertyInfo == memberInfo || property.FieldInfo == memberInfo);
+      return entityType.GetProperties().FirstOrDefault(property => property.PropertyInfo?.MetadataToken == memberInfo.MetadataToken
+                                                                   || property.FieldInfo?.MetadataToken == memberInfo.MetadataToken);
    }
 
    private static INavigation? FindOwnedProperty(

tests/Thinktecture.EntityFrameworkCore.SqlServer.Tests/EntityFrameworkCore/BulkOperations/SqlServerBulkOperationExecutorTests/BulkUpdateAsync.cs

@@ -325,4 +325,25 @@ public async Task Should_update_entity_with_auto_increment_having_custom_propert
                                               Name = "Name"
                                            });
    }
+
+   [Fact]
+   public async Task Should_update_property_of_base_class()
+   {
+      var entity = new TestEntityWithBaseClass { Id = new Guid("3D3AECE7-3B5A-48C6-9C8C-CAB7FFE2120D"), Name = "Initial" };
+      ArrangeDbContext.Add(entity);
+      await ArrangeDbContext.SaveChangesAsync();
+
+      entity.Name = "changed";
+
+      var affectedRows = await SUT.BulkUpdateAsync(new[] { entity }, new SqlServerBulkUpdateOptions
+                                                                     {
+                                                                        PropertiesToUpdate = IEntityPropertiesProvider.Include<TestEntityWithBaseClass>(e => e.Name)
+                                                                     });
+
+      affectedRows.Should().Be(1);
+
+      var loadedEntity = await AssertDbContext.TestEntitiesWithBaseClass.FirstOrDefaultAsync();
+      loadedEntity.Should().NotBeNull();
+      loadedEntity!.Name.Should().Be("changed");
+   }
 }