There is an bug in the imagemodule when used together with localization.
When the other language is active en you want to reset to defaults in de imagemodule setting
the required "thumbnail" and "preview" in the image sizes table get translated en become editable.
New images also get the translated filename in the database previous entries still have the old filename.
This results in the "loss" of the thumbs of the old images.
The problem can be fixed by removing the call to the t() function in _image_get_sizes();
Current code:
function _image_get_sizes() {
return variable_get('image_sizes', array(array('width' => 100, 'height' => 100, 'label' => t('thumbnail')),
array('width' => 640, 'height' => 640, 'label' => t('preview'))));
}
My fix:
function _image_get_sizes() {
return variable_get('image_sizes', array(array('width' => 100, 'height' => 100, 'label' => 'thumbnail'),
array('width' => 640, 'height' => 640, 'label' => 'preview')));
}
Comments
Comment #1
toemaz commentedHad the same problem.
Patch works fine!
Comment #2
yched commentedI stumbled on the same problem and had to apply the same correction...
I think this patch is required AND ready to be comitted.
I'm not quite sure i'm entitled to change the 'status' the way I just did, though...
Comment #3
Bèr Kessels commentedplease look at drupal.org/diffandpatch for how to provide a patch.
Comment #4
yched commentedSorry - i didn't even notice there was no actual file included in the original post.
Comment #5
Bèr Kessels commentedI can confirm this behaviour. This patch fixes it.
Though it is a pity we can no longer translate thubmnail and preview, I think the bug is too big to remain just for this translation thingy.
Comment #6
yched commentedBump.
anyone to commit that patch ?
Comment #7
yched commentedRe-bump.
Anyone to commit that patch ?
Comment #8
walkah commentedapplied, thanks.
Comment #9
(not verified) commentedComment #10
yngens commentedbut now we have these problems: http://drupal.org/node/70396