[PR #1394] [MERGED] Check if an item is a child of an EnabledFolder #8569

New issue

Closed

opened 2025-12-22 06:58:33 +01:00 by backuprepo · 0 comments

backuprepo commented

2025-12-22 06:58:33 +01:00

Owner

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/1394
Author: @joern-h
Created: 5/20/2019
Status: ✅ Merged
Merged: 6/26/2019
Merged by: @dkanada

Base: release-10.3.z ← Head: bugfix-issue1347

📝 Commits (1)

1fd8164 fix issue #1347 introduced in pr #930

📊 Changes

2 files changed (+12 additions, -1 deletions)

View changed files

📝 CONTRIBUTORS.md (+1 -0)
📝 MediaBrowser.Api/UserLibrary/ItemsService.cs (+11 -1)

📄 Description

Changes
In GetQueryResult:

user.Policy.EnabledFolders.Any(i => new Guid(i) == item.Id)
returns only true if the item is the media folder.

So now it checks if an item is a child of a EnabledFolder.

Fixes issue #1347
Introduced in #930

Feel free to criticise everything you want, i am open for improvements :)

_{🔄 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/1394 **Author:** [@joern-h](https://github.com/joern-h) **Created:** 5/20/2019 **Status:** ✅ Merged **Merged:** 6/26/2019 **Merged by:** [@dkanada](https://github.com/dkanada) **Base:** `release-10.3.z` ← **Head:** `bugfix-issue1347` --- ### 📝 Commits (1) - [`1fd8164`](https://github.com/jellyfin/jellyfin/commit/1fd81647560c12728e2eb64430360d0570336f61) fix issue #1347 introduced in pr #930 ### 📊 Changes **2 files changed** (+12 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `CONTRIBUTORS.md` (+1 -0) 📝 `MediaBrowser.Api/UserLibrary/ItemsService.cs` (+11 -1) </details> ### 📄 Description **Changes** In GetQueryResult: `user.Policy.EnabledFolders.Any(i => new Guid(i) == item.Id)` returns only true if the item is the media folder. So now it checks if an item is a child of a EnabledFolder. Fixes issue #1347 Introduced in #930 Feel free to criticise everything you want, i am open for improvements :) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

backuprepo

2025-12-22 06:58:33 +01:00

closed this issue
added the
pull-request
label

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#8569

No description provided.

Rows
Columns