From d0e6bce99a6f10e176f59a98eb336efdbf2fced4 Mon Sep 17 00:00:00 2001
From: Jared Goodwin <jared.goodwin@immy.bot>
Date: Thu, 27 Jul 2023 10:13:46 -0700
Subject: [PATCH] Fix string interpolation in queries.

---
 Server/Services/DataService.cs | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/Server/Services/DataService.cs b/Server/Services/DataService.cs
index 9d3106fc..8bd82162 100644
--- a/Server/Services/DataService.cs
+++ b/Server/Services/DataService.cs
@@ -600,7 +600,7 @@ public class DataService : IDataService
         var user = dbContext.Users
             .Include(x => x.DeviceGroups)
             .FirstOrDefault(x =>
-                $"{x.UserName}".ToLower() == userName &&
+                x.UserName!.ToLower() == userName &&
                 x.OrganizationID == orgId);
 
         if (user == null)
@@ -997,7 +997,9 @@ public class DataService : IDataService
             return false;
         }
 
-        return dbContext.Users.Any(x => $"{x.UserName}".Trim().ToLower() == userName.Trim().ToLower());
+        return dbContext.Users
+            .Where(x => x.UserName != null)
+            .Any(x => x.UserName!.Trim().ToLower() == userName.Trim().ToLower());
     }
 
     public bool DoesUserHaveAccessToDevice(string deviceId, RemotelyUser remotelyUser)
@@ -1417,7 +1419,6 @@ public class DataService : IDataService
         return dbContext.DeviceGroups
             .AsNoTracking()
             .Include(x => x.Users)
-            .ThenInclude(x => x.DeviceGroups)
             .Where(x => x.OrganizationID == organizationId)
             .OrderBy(x => x.Name)
             .ToArray();
@@ -1715,8 +1716,8 @@ public class DataService : IDataService
 
         return dbContext.Users
             .AsNoTracking()
-            .Where(x => x.IsServerAdmin)
-            .Select(x => $"{x.UserName}")
+            .Where(x => x.IsServerAdmin && x.UserName != null)
+            .Select(x => x.UserName!)
             .ToList();
     }