Hello.

I verified this with a fresh Drupal Install.

Drupal 6.26
emfield 6.x-2.6+0-dev
media_youtube 6.x-1.x-dev
cck 6.x-2.8+33-dev
colorbox 6.x-1.2+3-dev

I created a new content type with a Embedded Video field (Youtube) and Display type to Colorbox: Image Thumbnail -> full size video.

The link that shows up is:
http://test/?width=429&height=357&iframe=true (test is the name of the server).

And I guess it should be similar to:
http://test/emvideo/modal/55/640/385/field_youtubevideo/youtube/Uv6QrlHh...

I also tried with recommended releases of all modules involved with same results.

Thanks for any help that anyone could give me with this in advance!

CommentFileSizeAuthor
#9 emvideo-colorbox-display-1849076-9.patch939 bytessheise
#5 emvideo-colorbox-display-1849076-5.patch590 byteszerolab

Comments

scarvajal’s picture

scarvajal commented

I tested with media_vimeo and got exactly same results.
So looks like the problem its in emfield module.

Hubris-1’s picture

Hubris-1 commented

Had the same issue. Both YouTube videos and Slideshare presentations when using Colorbox format it Views. Manually coded colorbox links work ok.
Drupal 6.26
Colorbox 6.x-1.3
Embedded Media Field 6.x-2.6
Media: YouTube 6.x-1.3
Slideshare: 6.x-1.0-Beta 04

Looked at the file: emfield\contrib\emvideo\emvideo.theme.inc and made the following change

Line 248: $destination = $item['data']['embedded_url'];
change to
Line 248: $destination .= $item['data']['embedded_url'];

That made colorbox & emvideo work for me.

-Chris

goldlilys’s picture

goldlilys commented

Thanks, changing that line worked for me too. Recently updated colorbox and suddenly the video inside colorbox won't work. But normally the video file works. Wasn't colorbox fault, but this did the trick. Should be added to next release of emfield.

drupalfan81’s picture

drupalfan81 commented
Version: 6.x-2.x-dev » 6.x-2.6
Priority: Normal » Critical

@Hubris...THANK YOU SO MUCH! I was bashing my head into the wall all night trying to figure out what was going on. Finally found this in the morning when searching google. This fixed the issue. Such a simple fix with a critical issue, why has nobody bothered to comment on this open issue and 2.) why has the fixed not been put into the module?

I am changing this version to the recommend release production version so hopefully it will get attention as this bug made it's way from the dev version into production. Since colorbox has now gain traction and taking over in number of installs to lightbox2, I think this should be addressed asap. Hopefully the maintainers will get a chance to throw this into an update. Thanks again!

zerolab’s picture

zerolab commented
StatusFileSize
new emvideo-colorbox-display-1849076-5.patch590 bytes

Attaching #2 as a patch against 6.x-2.6

Tested the fix with the YouTube and Vimeo providers.

+1

ytsejam’s picture

ytsejam commented

Works for me too. Many thanks! Hope this gets committed.

zerolab’s picture

zerolab commented
Status: Active » Needs review
imclean’s picture

imclean commented
Version: 6.x-2.6 » 6.x-2.x-dev

Thanks, this is pretty major.

All patches should be against dev.

imclean’s picture

imclean commented
Issue summary: View changes

formatting

sheise’s picture

sheise commented
Issue summary: View changes
StatusFileSize
new emvideo-colorbox-display-1849076-9.patch939 bytes

I'm having the same issue with an added twist.

As far as I can tell, $item['data']['embedded_url'] never gets set. So the entire line of code from comment #5 can be deleted.

Additionally, colorbox is then loading twice, once within and on top of itself.

Patch attached for both problems.

zerolab’s picture

zerolab commented

@sheise, if you read the issue description carefully
$item['data']['embedded_url'] does get set and in the mentioned case is ?width=429&height=357&iframe=true. That last gives Colorbox indication on what needs to be done.

So I would re-test.