New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Refactor and extend external subtitle and audio handling #7255
Refactor and extend external subtitle and audio handling #7255
Conversation
Changes in OpenAPI specification found. Expand to see details.What's Changed
|
9e0224b
to
f4fd390
Compare
c1553dd
to
aed0afe
Compare
76f3e45
to
f2790b2
Compare
fc2ca88
to
1a61f7b
Compare
1a61f7b
to
5223a0f
Compare
4372660
to
701b9e6
Compare
0befa53
to
4cbacee
Compare
Tests are failing because #7233 changed |
881d4f3
to
f08ee32
Compare
…file recognition, consolidate and extend NamingOptions
f08ee32
to
a36e34f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's not needed, just extra overhead
Co-authored-by: Bond-009 <bond.009@outlook.com>
After this pr was merged external subtitles are no longer being scanned and added to the database |
Please open an issue and include your file structure. |
done |
First time doing C# things, so any input is welcome.
This PR aims to add more support to the external media resolvers for subtitles and audio.
Changes
forced
anddefault
flags based on what the (external) container (e.g. mka, mks) providesTitle
of streams for external audio and subtitles if present, otherwise try to parse it from the filenamenamingOptions
and extract language from filename if presentAddExternalSubtitleStreams
(LibraryManager
andILibraryManager
)InternalMetadataPath
Questions
DlnaProfileType
but there wasn't one for subtitles, so I added one. I'm not sure if this is the correct approachLibraryManager
exposed a method to explicitly add a subtitle stream. It wasn't used anywhere and I had problems properly keeping its functionality, so I removed it. If it should be kept I may need further assistance on how this can be achieved.Supercedes: #7060
Fixes: #7057
PR to update docs: jellyfin-archive/jellyfin-docs#635