hotfix: Custom EF Core include statements were not correctly added. N…

…ow they are. Wow.
StudioStoy · Jul 29, 2023 · fd6f6b4 · fd6f6b4
1 parent e8fdcb3
commit fd6f6b4
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/GainsTracker.CoreAPI/Components/UserProfiles/Data/BigBrainUserProfile.cs b/GainsTracker.CoreAPI/Components/UserProfiles/Data/BigBrainUserProfile.cs
@@ -19,13 +19,14 @@ public BigBrainUserProfile(AppDbContext context) : base(context)
     public UserProfile GetUserProfileByUserHandle(string userHandle, params PropertyToInclude<UserProfile>[] properties)
     {
         string gainsId = GetGainsIdByUsername(userHandle);
+        List<PropertyToInclude<UserProfile>> includes = properties.ToList();
 
         // If there's no include expression provided, get the standard user profile with icon.
-        if (properties.Length <= 0)
-            properties.ToList().Add(() => up => up.Icon);
+        if (includes.Count <= 0)
+            includes.Add(() => up => up.Icon);
 
         IQueryable<UserProfile> query = Context.UserProfiles.AsQueryable();
-        foreach (PropertyToInclude<UserProfile> property in properties)
+        foreach (PropertyToInclude<UserProfile> property in includes)
             query = query.Include(property.Invoke());
 
         return query.FirstOrDefault(e => e.GainsAccountId == gainsId)