ZeroQI / YouTube-Agent.bundle

Plex Metadata Agent for Movies and TV Series libraries
452 stars 43 forks source link

Season Grouping Bug when Channel ID in youtube.id file #16

Closed selkiess closed 5 years ago

selkiess commented 5 years ago

Cody'sLab\Cheap Camera, Light bulb, Survive [cjFSyBsRkCw].mkv channel ID in youtube.id All episodes put in specials

Internet Comment Etiquette with Erik [UCyWDmyZRjrGHeKF-ofFsT5Q]\Internet Comment Etiquette - 'Things You're Doing Wrong' [0m_cuVJu1Ig].mkv Episodes put into season by year

I discovered this because of another (feature request? bug?) in that channels that have nicknames instead of the string of characters id are not identified with the agent. Youtube-dl's uploader_id filename variable always grabs the nickname if it exists so autograbbed channels don't work properly yet.

Thanks a bunch for working on this agent, proper youtube support for plex is almost there. With the above bug fixed it would now be possible to mass download entire channels with a simple Channelname [ID]/ VIDEONAME [ID] fileformat.

Edit: A good channel for testing this that does get its nickname grabbed by youtube-dl is CodysLab https://www.youtube.com/channel/UCu6mSoMNzHQiBIOCkHUa2Aw

ZeroQI commented 5 years ago

Season number is decided by the scanner Are you using Absolute Series Scanner with the agent?

selkiess commented 5 years ago

Yes I am using the latest version of Absolute Series Scanner

ZeroQI commented 5 years ago

Some if file simplification was published recently... Could you please attach the scanner logs located in: PLEX_ROOT/Plug-in Support/Data/com.plexapp.agents.hama/DataItems/_Logs (even if Hama isn't installed) I will change the location to the Plex logs folder at some point but have no time at the moment Will reproduce with the same filenames to test and solve the issue

selkiess commented 5 years ago

sure: Logs.zip

Edit: after looking at them myself maybe the problem is [youtube-[UCu6mSoMNzHQiBIOCkHUa2Aw]] when youtube.id file is there when it is normally [UCu6mSoMNzHQiBIOCkHUa2Aw]

ZeroQI commented 5 years ago

@Selkie707 That helps, thanks. Notes

selkiess commented 5 years ago

Thanks man, removing the brackets totally fixed it. With how much development youtube-dl has I'm suprised there isn't an option to force the channel id to the UC format by now.

ZeroQI commented 5 years ago

Playlist URL works with youtube-dl... Not sure about channel URL, but worth a try