embed code from google video doesn't work

anonymous5190 - September 21, 2007 - 17:07
Project:Embedded Media Field
Version:5.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:anonymous5190
Status:closed
Description

URLs for google videos seem to be parsed fine, but if you use the "embed code" that Google provides, the video ID isn't extracted and the video won't be added/displayed for the node.

Here's a patch that adds a regexp to google.inc to allow parsing of the embed code.

AttachmentSize
google.inc_.patch409 bytes

#1

Alex UA - November 9, 2007 - 00:01

I'll give it a look, test it out, and if it works for all browsers, it will be rolled into the next release.

#2

Alex UA - November 9, 2007 - 17:16
Status:patch (code needs review)» closed

Works great for me. Thanks for the patch!

#3

aaron - November 9, 2007 - 19:57
Status:closed» patch (reviewed & tested by the community)

#4

aaron - January 4, 2008 - 20:58
Status:patch (reviewed & tested by the community)» fixed

committed; thanks anonymous5190

#5

Anonymous - January 18, 2008 - 21:01
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.