smplayer-dev / smplayer

Free Media Player for Windows, Linux and Mac OS with YouTube support.
https://smplayer.info
GNU General Public License v2.0
681 stars 95 forks source link

Title of YouTube video isn't display correctly when using MPV media engine #803

Open remcodj opened 1 year ago

remcodj commented 1 year ago

Describe the bug When loading a Youtube URL in the playlist, the title of the video isn't displayed. Instead it shows some code. The video is played without any problems

To Reproduce Steps to reproduce the behavior:

  1. Go to Add URL and paste i.e. https://www.youtube.com/watch?v=OPf0YbXqDm0
  2. DoubleClick on the new entry to start playing the video
  3. the "name" of the video changes from the URL to: webm&gir=yes&clen=34181159&dur=270.394&lmt=1681176425456820&mt=1695714628&fvip=1&keepalive=yes&fexp=24007246&beids=24350017&c=IOS&txp=453D434&sparams=expire,ei,ip,id,itag,source,requiressl,gcr,vprv,svpuc,mime,gir,clen,dur,lmt&sig=AOq0QJ8wRAIgTLiZhFA9MIGVVP2SJU1JSoTkNO24J85XU2MqANRE_UoCIGSwLiBF55Tm1s4Dj7yh7MElWBrpq8moEGSoQmzQg30x&lsparams=mh,mm,mn,ms,mv,mvi,pl&lsig=AG3C_xAwRQIgPsahrogN16Q7NTGu8HoisikcmRGeKl6sYOFUsnu76AMCIQCmy4j1sOcY4z-xIq8eA4zX0ZBJWAbhxIksp0nef61t5w==

Expected behavior I should expect that it will show the title of the Youtube video

Your Enviroment

Additional context when i change the multimedia engine to mplayer, is it working a expected.

remcodj commented 1 year ago

I think i found a deviation in de SMplayer log... The variable of INFO_MEDIA_TITLE was stated like this:

INFO_MEDIA_TITLE=${=media-title:}

but all the other variables have or a "=" between the brackets, or there is a ":" at the end. but the INFO_MEDIA_TITLE has both... Those it have somtehing to do with that?

INFO_TITLES=${=disc-titles}
INFO_CHAPTERS=${=chapters}
INFO_TRACKS_COUNT=${=track-list/count}
METADATA_TITLE=${metadata/by-key/title:}
METADATA_ARTIST=${metadata/by-key/artist:}
METADATA_ALBUM=${metadata/by-key/album:}
METADATA_GENRE=${metadata/by-key/genre:}
METADATA_DATE=${metadata/by-key/date:}
METADATA_TRACK=${metadata/by-key/track:}
METADATA_COPYRIGHT=${metadata/by-key/copyright:}
INFO_MEDIA_TITLE=${=media-title:}
INFO_STREAM_PATH=${stream-path}
 --term-status-msg=STATUS: ${=time-pos} / ${=duration:${=length:0}} P: ${=pause} B: ${=paused-for-cache} I: ${=core-idle} VB: ${=video-bitrate:0} AB: ${=audio-bitrate:0} https://rr3---sn-j5pfqpxonuxo-xone.googlevideo.com/videoplayback?expire=1696266583&ei=96QaZajIAdGt6dsPn8eOwAs&ip=204.168.170.155&id=o-AJR1BrjUwMYwNBeC6FHxFTrBfi3xJqhFuMsZ74HHqPGq&itag=244&source=youtube&requiressl=yes&mh=xr&mm=31%2C29&mn=sn-j5pfqpxonuxo
remcodj commented 1 year ago

i think it is in this file /src/mpvoptions.cpp on line 105.

I have no idea how to change it And maybe I'm completely wrong :-)