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

Downloading wrong playlist [bug] #661

Closed
kevin-hanselman opened this issue Jan 31, 2013 · 6 comments
Closed

Downloading wrong playlist [bug] #661

kevin-hanselman opened this issue Jan 31, 2013 · 6 comments
Assignees

Comments

@kevin-hanselman
Copy link

@kevin-hanselman kevin-hanselman commented Jan 31, 2013

I am trying to downloading this playlist:

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

I have tried:

youtube-dl http://www.youtube.com/playlist?list=PLMCmkNmxw6Z9eduM7BZjSEh7HiU543Ig0

youtube-dl PLMCmkNmxw6Z9eduM7BZjSEh7HiU543Ig0

youtube-dl http://www.youtube.com/view_play_list?p=PLMCmkNmxw6Z9eduM7BZjSEh7HiU543Ig0

All of them start downloading a 100 video playlist that starts with Gangnam Style...

I have tried downloading a few other playlists and they work just fine, which leads me to believe there is a potential bug in the program. If not, am I just doing something wrong?

@ghost ghost assigned FiloSottile Feb 3, 2013
@FiloSottile
Copy link
Collaborator

@FiloSottile FiloSottile commented Feb 3, 2013

Hm, it is a bug related to the PL prefix that we strip.

Here we have to include it; sometimes we must not include it; sometimes including it changes the page layout. Great.

@phihag
Copy link
Contributor

@phihag phihag commented Mar 3, 2013

The playlist you referenced has been modified significantly. Do you have a new testcase for this problem? Should be solved anyways though.

phihag added a commit that referenced this issue Mar 3, 2013
@kevin-hanselman
Copy link
Author

@kevin-hanselman kevin-hanselman commented Mar 3, 2013

@FiloSottile commited a fix to my issue. I cleared my playlist because I successfully downloaded those videos.

@FiloSottile
Copy link
Collaborator

@FiloSottile FiloSottile commented Mar 4, 2013

The issue was not related to playlist content, but to the fact that the web
interface required the PL prefix, while the API eats it also without.

The official docs say that it is mandatory now, so (I think) that I edited
the regex to include it.

@kevin-hanselman
Copy link
Author

@kevin-hanselman kevin-hanselman commented Mar 4, 2013

That's a long random character string to get matched to YouTube Daily Top Tracks playlist. What are the chances? 😃

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.