Thumbnails not being generated
sphism - October 30, 2009 - 14:07
| Project: | CDN2 Video |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | critical |
| Assigned: | Unassigned |
| Status: | closed |
Description
I know they are working on this but here's a specific problem with the thumbs...
Thumbnails aren't being generated and when i edit a node and click the link to see the thumbnail the url it takes me to has some illegal characters in it: (the thumb_blah bit, not the xxxxxx's)
The requested URL /xxxx-xxxxxxxx.............xxxxxxxx_320x240_thumb_d.jpg was not found on this server.I think it should be this, but this doesn't work either
The requested URL /xxxx-xxxxxxxx.............xxxxxxxx_320x240_thumb.jpg was not found on this server.I think they are trying to generate multiple thumbnails and i'm guessing that the thumb_xx is going to be the number of the thumbnail. Which is a cool idea, but what's with the weird characters thumb_d.jpg?
#1
This is still a CRITICAL issue.
I thought it had been fixed last week with the updates you did, but alas not.
This really really needs to be fixed :(
#2
Im having the same problem - transcoding seems to work fine now after recent hiccups but no thumbnails showing at all. This is the last stumbling block for us on an otherwise excellent module. Any news guys on a fix please?
#3
Hello,
This is still a CRITICAL BUG, no thumbnails are being generated for any content and no one has replied to this issue...
erm... please give me some indication of when this will be fixed, at least just roll back the changes you've made and do your development work on a development system, NOT the live one.
Our site is waiting to go live and we are considering moving to a different system entirely.
Perhaps the module should be tagged as BETA since it currently doesn't work.
Other changes made recently have been great but you've got to provide thumbnails...
Good Luck,
matt
#4
EDIT, dont use this patch, has bad formatting.
#5
Apparently
Use this one.
--- cdn2_flowplayer/cdn2_flowplayer.module 6 Oct 2009 23:20:56 -0000 1.1.4.11
+++ cdn2_flowplayer/cdn2_flowplayer.module 10 Nov 2009 19:58:31 -0000 1.1.4.12
@@ -47,13 +47,8 @@ function theme_cdn2_flowplayer($node, $a
$field_info = _cdn2_get_field_by_content_type($node->type);
$name = $field_info['field_name'];
$assets = $node->{$name}[0]['assets'];
- foreach($assets as $new_asset) {
- if($new_asset['preset_name'] == '320x240_thumb') {
- $splash_image_file = '<img src="'. $new_asset['asset_fetch_url'] .'">';
- }
- }
+ $splash_image_file = preg_replace("%\.[^\.]*$%", "_thumb_009.jpg", $video_url);
-// $splash_image_file = preg_replace('%_([^/]*)$%', '_'. $width .'x'. $height .'.jpg', $video_url);
$flash_js = theme('cdn2_flowplayer_get_js',$node->nid, $preset->shortName, $player_path, $width, $height, $video_url, $splash_image_file);
$top = ($height/2)-(46/2);
$left = ($width/2)-(46/2);
#6
Dont use the attachments, they have bad characters in them for some reason..
Use the stuff in < code >
#7
Thank you so much Kyle. I'll try this out in the morning and get back to you.
#8
Hmm, i'm not getting any thumbnails from the patch you supplied but it's quite possible that my custom code is getting in the way somewhere.
I updated to the latest dev release yesterday and applied your patch to that.
Can you please send me a link to a new style thumbnail that works and / or a description of how the url is formed from the data in the cdn2_videos table...
i think it's:
http://us.network3.cdn2.net/[our unique client code]/[video_token]_flash_flv_low_res_thumb_009.jpg
but currently this doesn't work???
#9
Once the dev version has been re-generated try again.
I do believe I have fixed this now.
#10
#11
--deleted--
i think i can fix this myself...
#12
Ok i got this working...
I really like that you are generating multiple thumbnails.
You have hard coded it to choose thumbnail 009 but we have short clips and they don't always get 9 thumbs, i changed it to the first thumb and it works great.
The reason i figured this out is that you haven't protected the parent directory from listing all it's files, please can you do this, i don't particularly want people being able to go and grab all the images and movies without using the site.
Sorry, i've changed this to needs work just so you check this post, feel free to set it back to fixed :)
Thanks for sorting this all out for us...
#13
FYI, this will be committed to Dev later today.
http://skitch.com/kylebrowning/n8enm/test-demo.cdn2.net
#14
Ah. That look great :)
#15
Should be in dev snapshot as soon as its regenerated
#16
All works very nicely, being able to select thumbnail on the node edit page is great.
Thanks for doing that.
matt
#17
aarahhaharrrr...
:)
So it turns out only $user->uid == 1 gets a thumbnail made for them.
If you add this permission to a module...
/*** Implementation of hook_perm()
*/
function mymodule_perm() {
return array(
'transcode cdn2 video to 320x240_thumb'
);
}
then anyone you give that permission will get thumbs, which you want to be everyone.
ps i'm working on site with lots of overrides so it's possible that this is fixed but i haven't copied the right changes over to my over rides, but i'm pretty sure this isn't the case.
#18
hmm...
ok, so the above fix does indeed show a tickbox for all users to generate a thumbnail but it doesn't actually get made on the server....???
Any ideas kyle?
#19
Ok, well this is still an issue. The user with id = 1 gets thumbnails but no one else does.
Any ideas? Is it your end or my end? I can't bugfix your end and everything apart from the uid looks the same to me at my end...
I'm stumped.
matt
#20
I did some work this morning on the module to support permissions for generating thumbnails.
I was able to test and confirm that thumbnails are being generated for a uid that is != 1
If you want, cvs checkout to get the latest changes, or you can wait till a new dev snapshot is generated
#21
#22
Note also that as of the latest dev release, the cdn2 thumb preset is deprecated, as 10 thumbnails are now generated per transcode format. The latest theming functions reflect this as well, and include the ability to select which thumbnail you wish to use.
-=Aaron
#23
Just updated to latest dev release (nov25) and can confirm that the thumbnails are working for everyone.
Thanks for that, and thanks for the other new features as well.
matt
#24
Automatically closed -- issue fixed for 2 weeks with no activity.