#793 Do not use builder.HasKey on derived types where the PK columns are hidden. Thanks to Måns Tånneryd.

Author: sjh37

EntityFramework.Reverse.POCO.Generator/EF.Reverse.POCO.v3.ttinclude

@@ -6274,6 +6274,9 @@
             if (isEfCore3Plus && t.IsView && !t.HasPrimaryKey)
                 return "builder.HasNoKey();";
 
+            if (t.PrimaryKeys.All(k => k.Hidden))
+                return string.Empty;
+
             var defaultKey = $"builder.HasKey({t.PrimaryKeyNameHumanCase()})";
             if (t.Indexes == null || !t.Indexes.Any())
                 return defaultKey + ";";

Generator/Generators/GeneratorEfCore.cs

@@ -172,6 +172,9 @@ public override string PrimaryKeyModelBuilder(Table t)
             if (isEfCore3Plus && t.IsView && !t.HasPrimaryKey)
                 return "builder.HasNoKey();";
 
+            if (t.PrimaryKeys.All(k => k.Hidden))
+                return string.Empty;
+
             var defaultKey = $"builder.HasKey({t.PrimaryKeyNameHumanCase()})";
             if (t.Indexes == null || !t.Indexes.Any())
                 return defaultKey + ";";