Skip to content
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

Support "newstyle" Youtube playlist IDs. #188

Merged
merged 1 commit into from Oct 4, 2011

Conversation

rbrito
Copy link
Contributor

@rbrito rbrito commented Oct 4, 2011

Many playlists reported by Youtube now have the form like in:

http://www.youtube.com/playlist?list=PL520044A3524F5E5D

where PL is prefixed to what youtube-dl used to use as playlist IDs. So,
while matching it, we adapt the regular expression so as to discard the PL
and only get the 520044A3524F5E5D in the case of the playlist of the
example.

Many playlists reported by Youtube now have the form like in:

    http://www.youtube.com/playlist?list=PL520044A3524F5E5D

where `PL` is prefixed to what youtube-dl used to use as playlist IDs. So,
while matching it, we adapt the regular expression so as to discard the `PL`
and only get the `520044A3524F5E5D` in the case of the playlist of the
example.
@phihag phihag merged commit 7125a7c into ytdl-org:master Oct 4, 2011
@phihag
Copy link
Contributor

phihag commented Oct 4, 2011

Merged (and added course).

Yes, it would be a good idea to remove that in favor of a generic solution, but that requires some testing. If you have the time, feel free to simplify the regexp ;)

l29ah referenced this pull request in l29ah/youtube-dl Oct 14, 2020
[SouthparkDE/MTV] another mgid extraction (mtv_base) feed url updated
joedborg referenced this pull request in joedborg/youtube-dl Nov 17, 2020
[pull] master from ytdl-org:master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants