Patch (to be ported)
Project:
Media: Video Flotsam
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
14 Sep 2010 at 12:56 UTC
Updated:
9 Dec 2010 at 17:13 UTC
Hello
I had a problem with Daily motion thumbnails as soon as i changed some code for the thumbnail url
I have replaced
function emvideo_dailymotion_thumbnail($field, $item, $formatter, $node, $width, $height) {
return 'http://www.dailymotion.com/thumbnail/160x120/video/'. $item['value'];
}
by
function emvideo_dailymotion_thumbnail($field, $item, $formatter, $node, $width, $height) {
return 'http://www.dailymotion.com/thumbnail/160x120/'. $item['value'];
}
in emfield/contrib/emvideo/providers/dailymotion.inc
on line 125
Comments
Comment #1
alex ua commentedThe thumbnails work as expected for me. Marking as postponed for now- can you give me a url of a video that you're having trouble getting thumbnails for?
Thanks!
Comment #2
alex ua commentedMoving to Media: Video Flotsam module, where this will be fixed (if it still needs to be fixed).
Comment #3
alex ua commentedComment #4
dale commentedHi,
The above fix works for me.
Apparently the url that gets stored for the video ends with /video
so emthumb tries to fetch :
http://www.dailymotion.com/thumbnail/160x120/video/video/xfuarw_caputo-s...
I also noticed Dailymotion supports larger thumnailsize these days.
for example this video :
http://www.dailymotion.com/video/xfuarw_caputo-slaat-ploeggenoot-knockou...
has a 160x120 thumbnail at :
http://www.dailymotion.com/thumbnail/160x120/video/xfuarw_caputo-slaat-p...
and a 320x240 thumbnail at :
http://www.dailymotion.com/thumbnail/320x240/video/xfuarw_caputo-slaat-p...
Thanks for the very useful module emfield module and all the providers that come with it!
Comment #5
dale commentedComment #6
dale commented