[PlaylistPlayer] Resolve plugin paths to fill the item infotag #16192
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.
Description
Backport for #16158:
When a
plugin://
path is played for the first time from the jsonrpc API, there is no information stored in the database for the item and it does not contain any music or video infotags (item->IsAudio() || item->IsVideo()
isfalse
). That makes Kodi interpret the item as a playlist (g_application.PlayMedia
) and information such as the item subtitles is not available. Users using remote control applications are not able to select the item provided subtitles. Subs will only be available the next time the item is played since at that moment the item info is already in the database and is filled byFillFileItemList
. This PR makes sure the plugin path is resolved before checking if it is audio or video, solving the issue.Motivation and Context
Fixes #16151
How Has This Been Tested?
With a proof of concept video plugin, changing the
plugin://
play path to make sure it is always an unknown path not in the db. Played the item via Kore remote app and checked the option to select subtitles was available.Types of change
Checklist: