Fix for 15759. Update an error message for FK properties mismatch

src/EFCore/Diagnostics/CoreLoggerExtensions.cs

@@ -1081,10 +1081,14 @@ private static string RedundantForeignKeyWarning(EventDefinitionBase definition,
         ///     Logs for the <see cref="CoreEventId.IncompatibleMatchingForeignKeyProperties" /> event.
         /// </summary>
         /// <param name="diagnostics"> The diagnostics logger to use. </param>
+        /// <param name="principalEntityType"> The entity type on the principal end of the relationship. </param>
+        /// <param name="dependentEntityType"> The entity type on the dependent end of the relationship. </param>
         /// <param name="foreignKeyProperties"> The properties that make up the foreign key. </param>
         /// <param name="principalKeyProperties"> The corresponding keys on the principal side. </param>
         public static void IncompatibleMatchingForeignKeyProperties(
             [NotNull] this IDiagnosticsLogger<DbLoggerCategory.Model> diagnostics,
+            [NotNull] IConventionEntityType principalEntityType,
+            [NotNull] IConventionEntityType dependentEntityType,
             [NotNull] IReadOnlyList<IPropertyBase> foreignKeyProperties,
             [NotNull] IReadOnlyList<IPropertyBase> principalKeyProperties)
         {
@@ -1094,15 +1098,19 @@ public static void IncompatibleMatchingForeignKeyProperties(
             {
                 definition.Log(
                     diagnostics,
+                    principalEntityType.DisplayName(),
+                    dependentEntityType.DisplayName(),
                     foreignKeyProperties.Format(includeTypes: true),
                     principalKeyProperties.Format(includeTypes: true));
             }
 
             if (diagnostics.NeedsEventData(definition, out var diagnosticSourceEnabled, out var simpleLogEnabled))
             {
-                var eventData = new TwoPropertyBaseCollectionsEventData(
+                var eventData = new IncompatibleMatchingForeignKeyEventData(
                     definition,
                     IncompatibleMatchingForeignKeyProperties,
+                    principalEntityType,
+                    dependentEntityType,
                     foreignKeyProperties,
                     principalKeyProperties);
 
@@ -1112,9 +1120,11 @@ public static void IncompatibleMatchingForeignKeyProperties(
 
         private static string IncompatibleMatchingForeignKeyProperties(EventDefinitionBase definition, EventData payload)
         {
-            var d = (EventDefinition<string, string>)definition;
-            var p = (TwoPropertyBaseCollectionsEventData)payload;
+            var d = (EventDefinition<string, string, string, string>)definition;
+            var p = (IncompatibleMatchingForeignKeyEventData)payload;
             return d.GenerateMessage(
+                p.PrincipalEntityType.DisplayName(),
+                p.DependentEntityType.DisplayName(),
                 p.FirstPropertyCollection.Format(includeTypes: true),
                 p.SecondPropertyCollection.Format(includeTypes: true));
         }

src/EFCore/Diagnostics/IncompatibleMatchingForeignKeyEventData.cs

@@ -0,0 +1,50 @@
+// Copyright (c) .NET Foundation. All rights reserved.
+// Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
+
+using System;
+using System.Collections.Generic;
+using System.Diagnostics;
+using JetBrains.Annotations;
+using Microsoft.EntityFrameworkCore.Metadata;
+
+namespace Microsoft.EntityFrameworkCore.Diagnostics
+{
+    /// <summary>
+    ///     A <see cref="DiagnosticSource" /> event payload class for the 
+    ///     IncompatibleMatchingForeignKeyProperties event.
+    /// </summary>
+    public class IncompatibleMatchingForeignKeyEventData : TwoPropertyBaseCollectionsEventData
+    {
+        /// <summary>
+        ///     Constructs the event payload.
+        /// </summary>
+        /// <param name="eventDefinition"> The event definition. </param>
+        /// <param name="messageGenerator"> A delegate that generates a log message for this event. </param>
+        /// <param name="principalEntityType"> The entity type on the principal end of the relationship. </param>
+        /// <param name="dependentEntityType"> The entity type on the dependent end of the relationship. </param>
+        /// <param name="firstPropertyCollection"> The first property collection. </param>
+        /// <param name="secondPropertyCollection"> The second property collection. </param>
+        public IncompatibleMatchingForeignKeyEventData(
+            [NotNull] EventDefinitionBase eventDefinition,
+            [NotNull] Func<EventDefinitionBase, EventData, string> messageGenerator,
+            [NotNull] IConventionEntityType principalEntityType,
+            [NotNull] IConventionEntityType dependentEntityType,
+            [NotNull] IReadOnlyList<IPropertyBase> firstPropertyCollection,
+            [NotNull] IReadOnlyList<IPropertyBase> secondPropertyCollection)
+            : base(eventDefinition, messageGenerator, firstPropertyCollection, secondPropertyCollection)
+        {
+            PrincipalEntityType = principalEntityType;
+            DependentEntityType = dependentEntityType;
+        }
+
+        /// <summary>
+        ///     The principal entity type.
+        /// </summary>
+        public virtual IConventionEntityType PrincipalEntityType { get; }
+
+        /// <summary>
+        ///     The dependent entity type.
+        /// </summary>
+        public virtual IConventionEntityType DependentEntityType { get; }
+    }
+}

src/EFCore/Metadata/Conventions/ForeignKeyPropertyDiscoveryConvention.cs

@@ -418,7 +418,8 @@ private bool TryFindMatchingProperties(
                     p => !p.IsShadowProperty()
                         || p.GetConfigurationSource().Overrides(ConfigurationSource.DataAnnotation)))
                 {
-                    Dependencies.Logger.IncompatibleMatchingForeignKeyProperties(foreignKeyProperties, propertiesToReference);
+                    Dependencies.Logger.IncompatibleMatchingForeignKeyProperties(
+                        principalEntityType, dependentEntityType, foreignKeyProperties, propertiesToReference);
                 }
 
                 // Stop searching if match found, but is incompatible

src/EFCore/Properties/CoreStrings.resx

@@ -992,8 +992,8 @@
     <comment>Debug CoreEventId.RedundantIndexRemoved string string string</comment>
   </data>
   <data name="LogIncompatibleMatchingForeignKeyProperties" xml:space="preserve">
-    <value>The foreign key properties haven't been configured by convention because the best match {foreignKey} are incompatible with the current principal key {principalKey}. This message can be disregarded if explicit configuration has been specified.</value>
-    <comment>Debug CoreEventId.IncompatibleMatchingForeignKeyProperties string string</comment>
+    <value>For the relationship between principal entity type '{principalEntityType}' and dependent entity type '{dependentEntityType}', the foreign key properties haven't been configured by convention because the best match {foreignKey} are incompatible with the current principal key {principalKey}. This message can be disregarded if explicit configuration has been specified.</value>
+    <comment>Debug CoreEventId.IncompatibleMatchingForeignKeyProperties string string string string</comment>
   </data>
   <data name="LogRequiredAttributeInverted" xml:space="preserve">
     <value>The navigation property '{navigation}' has a RequiredAttribute causing the entity type '{entityType}' to be configured as the dependent side in the corresponding relationship.</value>

test/EFCore.Tests/Infrastructure/CoreEventIdTest.cs

@@ -45,6 +45,7 @@ public void Every_eventId_has_a_logger_method_and_logs_when_level_enabled()
                 { typeof(ExpressionPrinter), () => new ExpressionPrinter() },
                 { typeof(Expression), () => Expression.Constant("A") },
                 { typeof(IEntityType), () => entityType },
+                { typeof(IConventionEntityType), () => entityType },
                 { typeof(IKey), () => new Key(new[] { property }, ConfigurationSource.Convention) },
                 { typeof(IPropertyBase), () => property },
                 { typeof(IServiceProvider), () => new FakeServiceProvider() },

test/EFCore.Tests/Metadata/Conventions/ForeignKeyPropertyDiscoveryConventionTest.cs

@@ -507,6 +507,7 @@ public void Does_not_match_principal_type_plus_PK_name_property_of_different_typ
             Assert.Equal(LogLevel.Debug, logEntry.Level);
             Assert.Equal(
                 CoreResources.LogIncompatibleMatchingForeignKeyProperties(new TestLogger<TestLoggingDefinitions>()).GenerateMessage(
+                    nameof(PrincipalEntity), nameof(DependentEntity),
                     "{'PrincipalEntityPeeKay' : string}", "{'PeeKay' : int}"), logEntry.Message);
 
             ValidateModel();