From 06aac9899607f724919510ac1eee46f3f877ebf7 Mon Sep 17 00:00:00 2001
From: dkanada <dkanada@users.noreply.github.com>
Date: Sun, 24 Nov 2019 21:41:19 +0900
Subject: [PATCH] Merge pull request #2039 from Bond-009/fixcondition

Fix always false condition

(cherry picked from commit 47ad21b6e396de836480211515de32f213cef590)
Signed-off-by: Joshua Boniface <joshua@boniface.me>
---
 MediaBrowser.Api/UserLibrary/PersonsService.cs | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/MediaBrowser.Api/UserLibrary/PersonsService.cs b/MediaBrowser.Api/UserLibrary/PersonsService.cs
index c264577788..2024e9e637 100644
--- a/MediaBrowser.Api/UserLibrary/PersonsService.cs
+++ b/MediaBrowser.Api/UserLibrary/PersonsService.cs
@@ -109,6 +109,11 @@ namespace MediaBrowser.Api.UserLibrary
                 NameContains = query.NameContains ?? query.SearchTerm
             });
 
+            if ((query.IsFavorite ?? false) && query.User != null)
+            {
+                items = items.Where(i => UserDataRepository.GetUserData(query.User, i).IsFavorite).ToList();
+            }
+
             return new QueryResult<(BaseItem, ItemCounts)>
             {
                 TotalRecordCount = items.Count,