Repository navigation
Fix series merging - #17417
Merged
Merged
Fix series merging#17417
Conversation
Contributor
Jellyfin API (v 12.0.0)ResultNo differences. Specifications are equivalent |
|
Affected by #17408 here — a 7-folder multi-part series where series-level extras / special features stop aggregating after the folders merge (only the "survivor" folder's extras show up). The approach in this PR looks like the right root-cause fix to me, and it should be a no-op for single-folder series. +1 for getting this in — happy to test a build against my multi-folder setup and report back if that helps. |
|
theguymadmax
approved these changes
Jul 25, 2026
Bond-009
approved these changes
Jul 25, 2026
IDisposable
approved these changes
Jul 25, 2026
crobibero
approved these changes
Jul 25, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changes
Series spread across multiple folders stopped merging into one tile because
Series.PresentationUniqueKeyembedded the set of owning libraries, so sibling folder-items diverged whenever libraries overlapped or folder order changed. This drops the library set from the key (library scoping already happens at query time) and adds a name-based fallback so series without a provider ID still group.Additionally,
Series.GetExtraOwnerIdsnow aggregates extras/special features from every folder-item of a merged series instead of only the grouping survivor. A migration recomputes existing series keys and re-points their seasons/episodes so the fix applies without a manual metadata rescan.Code assistance
Claude Opus 4.8 was used to trace the presentation-key grouping path.
Issues
Fixes #16593
Fixes #17408
Fixes #904