Skip to content
Snippets Groups Projects

MediaWrapper: add cond to updateMeta for title

Closed Duncan McNamara requested to merge Skantes/vlc-android:webradio into master
1 unresolved thread

If libvlc doesn't have any meta for title it will provide the filename. This can be problematic with m3u8 #EXTINF that provides a title. This adds a check when updating the title meta. If the libvlc title provided is equal to the filename then it will proritize the medialibrary title. Network share will have null filenames though, meaning that libvlc will never be chosen, hence the check for directory type in the condition

Fixes #2722 (closed)

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • added 1 commit

    • ab50698f - MediaWrapper: add cond to udpateMeta for title

    Compare with previous version

  • Robert Stone resolved all threads

    resolved all threads

  • added 1 commit

    • 46fb67ff - MediaWrapper: add cond to updateMeta for title

    Compare with previous version

  • Duncan McNamara changed title from Draft: MediaWrapper: add cond to udpateMeta for title to Draft: MediaWrapper: add cond to updateMeta for title

    changed title from Draft: MediaWrapper: add cond to udpateMeta for title to Draft: MediaWrapper: add cond to updateMeta for title

  • Duncan McNamara marked this merge request as ready

    marked this merge request as ready

  • Superseed by !1702 (merged)

  • Contributor

    Sorry guys for the late reply. Tested the fix of the fix and it's also working as it should on my side.

    thx

  • mentioned in issue #2722 (closed)

  • Please register or sign in to reply
    Loading