Closed (fixed)
Project:
Embedded Media Field
Version:
6.x-1.17
Component:
Embedded Image Field
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
4 Nov 2009 at 20:03 UTC
Updated:
4 Mar 2010 at 23:30 UTC
Jump to comment: Most recent file
Comments
Comment #1
Plazmus commentedYes, there is definitely something wrong.
For me it looks like 'Image Preview Settings' is overwriting 'Full size display settings' and 'Thumbnail'.
It is not possible to set thumbnail to be linked to content and full size image to provider at the moment, you can only have both linked to content or both linked to provider.
Comment #2
Plazmus commentedIt's definitely a bug.
Comment #3
gilgabar commentedThe problem is in emimage.theme.inc. It looks like a sloppy copy/paste job. The same line that is appropriate for theming preview links is used for full and thumbnail links as well.
It's pretty straightforward to fix. Just change the word 'preview' to 'full' or 'thumbnail' in each place. Patch attached.
Comment #4
heyqule commentedHey gilgabar
Your patch is giving "patch unexpectedly ends in middle of line" error because it's missing a return character at end of file.
After adding that return character, the patch worked and it solved the issue for me.
Comment #5
aaron commentedfixed. thanks @gilgabar.