Minor change for avoiding array creation in favor of enumerator.

Ed-Fi-Alliance-OSS · Dec 12, 2024 · 7e8fda1 · 7e8fda1
1 parent 4dbebe0
commit 7e8fda1
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/...Fi.Ods.Features/ChangeQueries/Repositories/TrackedChangesIdentifierProjectionsProvider.cs b/...Fi.Ods.Features/ChangeQueries/Repositories/TrackedChangesIdentifierProjectionsProvider.cs
@@ -66,7 +66,8 @@ IEnumerable<SelectColumn> GetSelectColumns(ResourceProperty resourceProperty)
                 if (entityProperty.IsSurrogateIdentifierUsage())
                 {
                     // For Descriptors, this returns Namespace/CodeValue, for Student/Staff/Parent it returns the UniqueId
-                    var naturalIdentifyingPropertyNames = entityProperty.DefiningProperty.Entity.NaturalIdentifyingProperties().Select(p => p.PropertyName).ToArray();
+                    var naturalIdentifyingPropertyNames = entityProperty.DefiningProperty.Entity.NaturalIdentifyingProperties()
+                        .Select(p => p.PropertyName);
 
                     foreach (SelectColumn selectColumn in TrackedChangesSelectColumnHelpers
                                  .GetSelectColumnsForSurrogateIdentifierUsage(

diff --git a/...ication/EdFi.Ods.Features/ChangeQueries/Repositories/TrackedChangesSelectColumnHelpers.cs b/...ication/EdFi.Ods.Features/ChangeQueries/Repositories/TrackedChangesSelectColumnHelpers.cs
@@ -18,7 +18,7 @@ public static class TrackedChangesSelectColumnHelpers
 
     public static IEnumerable<SelectColumn> GetSelectColumnsForSurrogateIdentifierUsage(
         string entityPropertyName,
-        string[] naturalIdentifyingPropertyNames,
+        IEnumerable<string> naturalIdentifyingPropertyNames,
         IDatabaseNamingConvention namingConvention)
     {
         var allTerms = SplitTerms(entityPropertyName);