[PR #6850] [MERGED] Add Collection Validator, create collection based on nfo #11163

Closed
opened 2026-02-07 06:33:35 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/6850
Author: @PetrusZ
Created: 11/15/2021
Status: Merged
Merged: 11/19/2021
Merged by: @crobibero

Base: masterHead: collection-validator


📝 Commits (10+)

  • 5eb1fde Add Collection Validator, create collection based on nfo
  • 74459ec Fix issues mentioned in review
  • 1924d07 Fix style and performance issues mentioned in review
  • 0a0ddb0 Add AutoCollection option
  • 3c8f7d3 Improve performance
  • 52e9dc6 Remove extra blank line
  • c09e999 Use List instead of List
  • 989013c Add new line at end of file
  • 6d74c83 Fix issues mentioned in review, except for option name
  • b635b5a Paginate movies query

📊 Changes

2 files changed (+158 additions, -0 deletions)

View changed files

Emby.Server.Implementations/Library/Validators/CollectionPostScanTask.cs (+156 -0)
📝 MediaBrowser.Model/Configuration/LibraryOptions.cs (+2 -0)

📄 Description

Based on nfo's set element, automatically add movie to collection.

Signed-off-by: Petrus.Z silencly07@gmail.com

Changes
Implement the feature request posted in https://features.jellyfin.org/posts/73/create-collection-based-on-nfo.

I am new to C#, there may be some errors in the code. Thanks in advance for review.


🔄 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/6850 **Author:** [@PetrusZ](https://github.com/PetrusZ) **Created:** 11/15/2021 **Status:** ✅ Merged **Merged:** 11/19/2021 **Merged by:** [@crobibero](https://github.com/crobibero) **Base:** `master` ← **Head:** `collection-validator` --- ### 📝 Commits (10+) - [`5eb1fde`](https://github.com/jellyfin/jellyfin/commit/5eb1fde88cb32e05ea51f26e1577b20bb4c60bd6) Add Collection Validator, create collection based on nfo - [`74459ec`](https://github.com/jellyfin/jellyfin/commit/74459ec403eda3912f12122da18c15c96244d351) Fix issues mentioned in review - [`1924d07`](https://github.com/jellyfin/jellyfin/commit/1924d0740d2846aae401686e45d55e8c8358e23b) Fix style and performance issues mentioned in review - [`0a0ddb0`](https://github.com/jellyfin/jellyfin/commit/0a0ddb0eafa669256f0493d1bc2abdf32290302a) Add AutoCollection option - [`3c8f7d3`](https://github.com/jellyfin/jellyfin/commit/3c8f7d380f6f2b719dd4ededebcc2505ef24baad) Improve performance - [`52e9dc6`](https://github.com/jellyfin/jellyfin/commit/52e9dc66f5c2051a5e4b7f62ff7f2469cfa4da61) Remove extra blank line - [`c09e999`](https://github.com/jellyfin/jellyfin/commit/c09e9999163f6a6aa2f816a6f56197126c115c85) Use List<Gid> instead of List<Movie> - [`989013c`](https://github.com/jellyfin/jellyfin/commit/989013c974a4a2292f946e05a9ee7fd6d53f7c60) Add new line at end of file - [`6d74c83`](https://github.com/jellyfin/jellyfin/commit/6d74c83ddb983c30ae5ab2ffe0a403d0e07b5198) Fix issues mentioned in review, except for option name - [`b635b5a`](https://github.com/jellyfin/jellyfin/commit/b635b5a7e3b72c91922d36bce6b29265123f88f8) Paginate movies query ### 📊 Changes **2 files changed** (+158 additions, -0 deletions) <details> <summary>View changed files</summary> ➕ `Emby.Server.Implementations/Library/Validators/CollectionPostScanTask.cs` (+156 -0) 📝 `MediaBrowser.Model/Configuration/LibraryOptions.cs` (+2 -0) </details> ### 📄 Description Based on nfo's set element, automatically add movie to collection. Signed-off-by: Petrus.Z <silencly07@gmail.com> **Changes** Implement the feature request posted in https://features.jellyfin.org/posts/73/create-collection-based-on-nfo. I am new to C#, there may be some errors in the code. Thanks in advance for review. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-07 06:33:35 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#11163