Browse Source

Backport pull request #8166 from jellyfin/release-10.8.z

Bind @userid only when it's in the statement

Authored-by: Joseph <1315585+joseph39@users.noreply.github.com>

Merged-by: Claus Vium <cvium@users.noreply.github.com>

Original-merge: 057e8ef24061a969d4ca31b5b115e14d5ba76fe4
Joshua Boniface 2 years ago
parent
commit
bffda19bbc
1 changed files with 1 additions and 5 deletions
  1. 1 5
      Emby.Server.Implementations/Data/SqliteItemRepository.cs

+ 1 - 5
Emby.Server.Implementations/Data/SqliteItemRepository.cs

@@ -4934,6 +4934,7 @@ SELECT key FROM UserDatas WHERE isFavorite=@IsFavorite AND userId=@UserId)
 AND Type = @InternalPersonType)");
                 statement?.TryBind("@IsFavorite", query.IsFavorite.Value);
                 statement?.TryBind("@InternalPersonType", typeof(Person).FullName);
+                statement?.TryBind("@UserId", query.User.InternalId);
             }
 
             if (!query.ItemId.Equals(default))
@@ -4988,11 +4989,6 @@ AND Type = @InternalPersonType)");
                 statement?.TryBind("@NameContains", "%" + query.NameContains + "%");
             }
 
-            if (query.User != null)
-            {
-                statement?.TryBind("@UserId", query.User.InternalId);
-            }
-
             return whereClauses;
         }