starred/jellyfin
1
0
Fork 0
mirror of https://github.com/jellyfin/jellyfin.git synced 2026-05-04 18:09:12 +03:00
Code Issues 822 Packages Projects Releases 10 Wiki Activity

DVD Order not using filename numbering #758

New Issue
Closed
opened 2026-02-06 20:04:24 +03:00 by OVERLORD · 4 comments
OVERLORD commented 2026-02-06 20:04:24 +03:00
Owner
Copy Link

Originally created by @SenorSmartyPants on GitHub (Jun 20, 2019).

Describe the bug
When setting a series to DVD Order, files are assumed to be named using DVD order, not aired order. This was working fine but behavior has changed since #243.

Now TVDB metadata is looked up using filename episode numbers, assuming those are the aired order, not dvd order.

To Reproduce
Create a file named by DVD order.
The Venture Bros. 1x03 - Mid-Life Chrysalis.mkv
Scan library.
Episode will be displayed as 1x03 Home Insecurity.
Edit Venture Bros. series to DVD Order
Refresh All Metadata for Venture Bros.
Episode will now be displayed as 1x07 Home Insecurity.

Both options are incorrect.

Expected behavior

Edit Venture Bros. series to DVD Order
Refresh All Metadata for Venture Bros.
Episode will now be displayed as 1x03 Mid-Life Chrysalis.

System (please complete the following information):

  • OS: Docker
  • Browser: Chrome
  • Jellyfin Version: 10.3.5
  • Reverse proxy: no
Originally created by @SenorSmartyPants on GitHub (Jun 20, 2019). **Describe the bug** When setting a series to DVD Order, files are assumed to be named using DVD order, not aired order. This was working fine but behavior has changed since #243. Now TVDB metadata is looked up using filename episode numbers, assuming those are the aired order, not dvd order. **To Reproduce** Create a file named by DVD order. ```The Venture Bros. 1x03 - Mid-Life Chrysalis.mkv``` Scan library. Episode will be displayed as 1x03 Home Insecurity. Edit Venture Bros. series to DVD Order Refresh All Metadata for Venture Bros. Episode will now be displayed as 1x07 Home Insecurity. Both options are incorrect. **Expected behavior** <!-- A clear and concise description of what you expected to happen. --> Edit Venture Bros. series to DVD Order Refresh All Metadata for Venture Bros. Episode will now be displayed as 1x03 Mid-Life Chrysalis. **System (please complete the following information):** - OS: Docker - Browser: Chrome - Jellyfin Version: 10.3.5 - Reverse proxy: no
OVERLORD added the bugregression labels 2026-02-06 20:04:24 +03:00
OVERLORD commented 2026-02-06 20:04:27 +03:00
Author
Owner
Copy Link

@SenorSmartyPants commented on GitHub (Jun 23, 2019):

As example that other video jukeboxes also use the numbers in the filename as the DVD order numbers.

https://www.reddit.com/r/kodi/comments/59doqg/how_to_use_dvd_order_with_tv_shows_examples_with/

@SenorSmartyPants commented on GitHub (Jun 23, 2019): As example that other video jukeboxes also use the numbers in the filename as the DVD order numbers. https://www.reddit.com/r/kodi/comments/59doqg/how_to_use_dvd_order_with_tv_shows_examples_with/
OVERLORD commented 2026-02-06 20:04:29 +03:00
Author
Owner
Copy Link

@sparky8251 commented on GitHub (Jun 25, 2019):

Wonder if this is more of the fun with the database not being all that flexible right now....?

Regardless, this is definitely a regression but I'm going to mark it as blocked since to me, it reeks of "inflexible database."

@sparky8251 commented on GitHub (Jun 25, 2019): Wonder if this is more of the fun with the database not being all that flexible right now....? Regardless, this is definitely a regression but I'm going to mark it as blocked since to me, it reeks of "inflexible database."
OVERLORD commented 2026-02-06 20:04:30 +03:00
Author
Owner
Copy Link

@SenorSmartyPants commented on GitHub (Jun 25, 2019):

I made a pull request that addresses this issue. #1484

One of the CI tests fails but the other passes. Not sure why that is.

@SenorSmartyPants commented on GitHub (Jun 25, 2019): I made a pull request that addresses this issue. #1484 One of the CI tests fails but the other passes. Not sure why that is.
OVERLORD commented 2026-02-06 20:04:32 +03:00
Author
Owner
Copy Link

@sparky8251 commented on GitHub (Jun 25, 2019):

I apologize for that. I haven't been active lately and am just helping with triage since there's so much that hasn't been done around it. :)

Removed the blocking tag, hope to see your PR get merged soon!

@sparky8251 commented on GitHub (Jun 25, 2019): I apologize for that. I haven't been active lately and am just helping with triage since there's so much that hasn't been done around it. :) Removed the blocking tag, hope to see your PR get merged soon!
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
EFjellyfin.db
area:database
awaiting-feedback
backend
blocked
breaking change: web api
bug
build
ci
confirmed
discussion needed
dotnet future
downstream
duplicate
enhancement
feature
future
github-actions
good first issue
hdr
help wanted
invalid
investigation
librarydb
live-tv
lyrics
media playback
music
needs testing
nuget
performance
platform
pull-request
Mirrored from GitHub Pull Request
 question
regression
release critical
requires-web
roadmap
security
security
stale
support
syncplay
ui & ux
upstream
wontfix
No Label bug regression
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
OVERLORD
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#758
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?