- is missing at the and of ?<video_id>[0-9A-Za-z_] part. So urls like this is splitted up.

CommentFileSizeAuthor
#4 wrong_youtube_playlist-2732143-4.patch1.71 KBSam152
#2 wrong_youtube_playlist-2732143-2.patch854 bytesszantog
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

szantog created an issue. See original summary.

szantog’s picture

szantog CreditAttribution: szantog as a volunteer commented
Status: Active » Needs review
FileSize
wrong_youtube_playlist-2732143-2.patch854 bytes

BTW the same is in Youtube class.

Status: Needs review » Needs work

The last submitted patch, 2: wrong_youtube_playlist-2732143-2.patch, failed testing.

Sam152’s picture

Sam152 CreditAttribution: Sam152 as a volunteer and at PreviousNext commented
Status: Needs work » Needs review
FileSize
wrong_youtube_playlist-2732143-4.patch1.71 KB

I've added a test case to your patch.

  • Sam152 committed 16ad6f6 on 8.x-1.x authored by szantog 
    Issue #2732143 by szantog, Sam152: Wrong youtube playlist regexp
Sam152’s picture

Sam152 CreditAttribution: Sam152 as a volunteer and at PreviousNext commented
Status: Needs review » Fixed

Nice work, thanks for your contribution!

Status: Fixed » Closed (fixed)

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