Still do ToArray as the first thing, to avoid test fails.

This commit is contained in:
Bjarke Berg
2021-08-03 12:29:15 +02:00
parent 06951d2e2e
commit ac2f567f0e

View File

@@ -18,14 +18,15 @@ namespace Umbraco.Cms.Infrastructure.Persistence
public int BulkInsertRecords<T>(IUmbracoDatabase database, IEnumerable<T> records)
{
if (!records.Any()) return 0;
var recordsA = records.ToArray();
if (recordsA.Length == 0) return 0;
var pocoData = database.PocoDataFactory.ForType(typeof(T));
if (pocoData == null) throw new InvalidOperationException("Could not find PocoData for " + typeof(T));
return database.DatabaseType.IsSqlServer2008OrLater()
? BulkInsertRecordsSqlServer(database, pocoData, records)
: BasicBulkSqlInsertProvider.BulkInsertRecordsWithCommands(database, records.ToArray());
? BulkInsertRecordsSqlServer(database, pocoData, recordsA)
: BasicBulkSqlInsertProvider.BulkInsertRecordsWithCommands(database, recordsA);
}
/// <summary>