Use the correct builder when attaching indexes on nested owned entity…

… types Fixes #26659
dotnet · Nov 17, 2021 · 24e7b0d · 24e7b0d
1 parent 970e8a4
commit 24e7b0d
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/src/EFCore/Metadata/Internal/PropertiesSnapshot.cs b/src/EFCore/Metadata/Internal/PropertiesSnapshot.cs
@@ -121,7 +121,7 @@ public virtual void Attach(InternalEntityTypeBuilder entityTypeBuilder)
                 {
                     var originalEntityType = indexBuilder.Metadata.DeclaringEntityType;
                     var targetEntityTypeBuilder = originalEntityType.Name == entityTypeBuilder.Metadata.Name
-                        || (!originalEntityType.HasSharedClrType && originalEntityType.ClrType == entityTypeBuilder.Metadata.ClrType)
+                        || (!originalEntityType.IsInModel && originalEntityType.ClrType == entityTypeBuilder.Metadata.ClrType)
                             ? entityTypeBuilder
                             : originalEntityType.Builder;
                     indexBuilder.Attach(targetEntityTypeBuilder);

diff --git a/test/EFCore.Tests/ModelBuilding/OwnedTypesTestBase.cs b/test/EFCore.Tests/ModelBuilding/OwnedTypesTestBase.cs
@@ -1064,7 +1064,10 @@ public virtual void Can_configure_chained_ownerships()
                             l => l.AnotherBookLabel, ab =>
                             {
                                 ab.Ignore(l => l.Book);
-                                ab.OwnsOne(l => l.SpecialBookLabel).Ignore(l => l.Book).Ignore(s => s.BookLabel);
+                                ab.OwnsOne(l => l.SpecialBookLabel)
+                                    .Ignore(l => l.Book)
+                                    .Ignore(s => s.BookLabel)
+                                    .HasIndex(l => l.BookId);
                             });
                         bb.OwnsOne(
                             l => l.SpecialBookLabel, sb =>