Skip to content

Commit 2ab4e87

Browse files
CopilotPhenX
andcommitted
Rename variable to hasAnyDatabaseGeneratedKeys for clarity
Co-authored-by: PhenX <42170+PhenX@users.noreply.github.com>
1 parent f21b3a8 commit 2ab4e87

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

src/PhenX.EntityFrameworkCore.BulkInsert/Graph/GraphBulkInsertOrchestrator.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -53,14 +53,14 @@ public async Task<GraphInsertResult<T>> InsertGraph<T>(
5353
var totalInserted = 0;
5454
var graphMetadata = new GraphMetadata(_context, options);
5555

56-
// Check if any entity types have generated keys - if so, provider must support returning IDs
57-
var hasAnyGeneratedKeys = collectionResult.InsertionOrder.Any(entityType =>
56+
// Check if any entity types have database-generated keys - if so, provider must support returning IDs
57+
var hasAnyDatabaseGeneratedKeys = collectionResult.InsertionOrder.Any(entityType =>
5858
{
5959
var efEntityType = graphMetadata.GetEntityType(entityType);
6060
return efEntityType?.FindPrimaryKey()?.Properties.Any(p => p.ValueGenerated != Microsoft.EntityFrameworkCore.Metadata.ValueGenerated.Never) == true;
6161
});
6262

63-
if (hasAnyGeneratedKeys && !provider.SupportsOutputInsertedIds)
63+
if (hasAnyDatabaseGeneratedKeys && !provider.SupportsOutputInsertedIds)
6464
{
6565
throw new NotSupportedException(
6666
$"The bulk insert provider '{provider.GetType().Name}' does not support returning generated IDs, " +

0 commit comments

Comments
 (0)