There's a bug in youtube.inc where the code fails when there's no ampersand in the url. here's a patch.

I could've nested both if-statements but that wouldn't be very readable. Also, this is the first time I create a patch (I used meld), so if I screwed up, I'm sorry ;).

CommentFileSizeAuthor
youtube.patch819 bytesjonne.freebase

Comments

vegeneric’s picture

vegeneric commented

Hey, this module was giving me problems too.... your patch fixed the issue for me. Thanks!

fokkezb’s picture

fokkezb commented
Status: Needs review » Fixed

Fixed in todays release, not by using this patch though, but by using a smarter regular expression.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.