Thanks for the great module, just what I need!. It works OK for me displaying the full video in the node (or in views) but I haven't got it to work in any other way. I'm trying to use it in views with 'Lightbox2 ImageThumbnail -> full size video. I can't get it to workeven just on thumbnail linked to original video. Thumbnails themselves are all OK, just the link to video that looks to be malformed. Having a look at the code to see how it works. I know this is very new and still in development.

CommentFileSizeAuthor
#2 media_ted.tar_.gz4.9 KBzerolab

Comments

zerolab’s picture

zerolab commented

Thank you for the bug report.

Did not get a notification, thus the late reply. Will try looking into it in the following days. I tried to follow the emvideo providers as close as possible.
Which version of Lightbox2 are you using?

zerolab’s picture

zerolab commented
Assigned: Unassigned » zerolab
Status: Active » Needs work
Issue tags: +quickfix
StatusFileSize
new media_ted.tar_.gz4.9 KB

Hey Tony,

here's a quick version that fixes the Lightbox integration. The problem was in the way the video ID was being stored. I was saving a pretty long and unnecessary string.
The emvideo module i checking the supplied to /emvideo/modal with the value stored in the node, which in this case was different because of URL encoding.

Please note that this version changes the way the ID is stored and old nodes may not be displayed correctly (I am travelling and no time to do proper testing, nor CVS commits)

Let me know if you find any other bugs/inconsistencies. Hope to push this to an alpha next week.

Tony Sharpe’s picture

Tony Sharpe commented

Hi Zerolab,
This worked a treat thank you, although I had to re-create new nodes like you said. Test embed code and Wordpress form.
Thanks

zerolab’s picture

zerolab commented
Status: Needs work » Closed (fixed)

Commited the latest changes to CVS. Added some backwards compatibility between the old dev and current one.