[PR #2039] [MERGED] Fix always false condition #8798

Closed
opened 2025-12-22 07:07:50 +01:00 by backuprepo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/2039
Author: @Bond-009
Created: 11/22/2019
Status: Merged
Merged: 11/24/2019
Merged by: @dkanada

Base: masterHead: fixcondition


📝 Commits (1)

  • d884ed3 Fix always false condition

📊 Changes

1 file changed (+1 additions, -1 deletions)

View changed files

📝 MediaBrowser.Api/UserLibrary/PersonsService.cs (+1 -1)

📄 Description

Before it would be evaluated as query.IsFavorite ?? (false && query.User != null)


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/jellyfin/jellyfin/pull/2039 **Author:** [@Bond-009](https://github.com/Bond-009) **Created:** 11/22/2019 **Status:** ✅ Merged **Merged:** 11/24/2019 **Merged by:** [@dkanada](https://github.com/dkanada) **Base:** `master` ← **Head:** `fixcondition` --- ### 📝 Commits (1) - [`d884ed3`](https://github.com/jellyfin/jellyfin/commit/d884ed353f0434d8832a25a8cd065aa82777a70c) Fix always false condition ### 📊 Changes **1 file changed** (+1 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `MediaBrowser.Api/UserLibrary/PersonsService.cs` (+1 -1) </details> ### 📄 Description Before it would be evaluated as `query.IsFavorite ?? (false && query.User != null)` --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
backuprepo 2025-12-22 07:07:50 +01:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: starred/jellyfin#8798
No description provided.