Browse Source

Clean up user renaming

Patrick Barron 4 years ago
parent
commit
a7b29e2fe0
1 changed files with 7 additions and 1 deletions
  1. 7 1
      Jellyfin.Server.Implementations/Users/UserManager.cs

+ 7 - 1
Jellyfin.Server.Implementations/Users/UserManager.cs

@@ -144,7 +144,13 @@ namespace Jellyfin.Server.Implementations.Users
                 throw new ArgumentException("The new and old names must be different.");
             }
 
-            if (Users.Any(u => u.Id != user.Id && u.Username.Equals(newName, StringComparison.OrdinalIgnoreCase)))
+            await using var dbContext = _dbProvider.CreateContext();
+
+            if (await dbContext.Users
+                .AsQueryable()
+                .Where(u => u.Username == newName && u.Id != user.Id)
+                .AnyAsync()
+                .ConfigureAwait(false))
             {
                 throw new ArgumentException(string.Format(
                     CultureInfo.InvariantCulture,