ImageScience GIF thumbnail references incorrect
Reported by Matt Baron | September 17th, 2007 @ 04:05 PM
When using ImageScience, attachment_fu translates GIF thumbnails to PNG to avoid transparency issues with the ImageScience resizer. However, attachment_fu still stores the reference as a gif so when the image is displayed, it will not be found.
Comments and changes to this ticket
-
-
David Eisinger March 20th, 2008 @ 02:50 PM
On line 235 of attachment_fu.rb:
ext.sub!(/gif$/, 'png') if attachment_options[:processor] == "ImageScienceProcessor" Getting rid of 'Processor' will fix the problem:ext.sub!(/gif$/, 'png') if attachment_options[:processor] == "ImageScience"
-
Benjamin Behr May 19th, 2008 @ 07:18 AM
On line 235 of attachment_fu.rb:
>@@@ ruby
ext.sub!(/gif$/, 'png') if attachment_options[:processor] == "ImageScienceProcessor"
@@@
Getting rid of 'Processor' will fix the problem:
@@@ ruby
ext.sub!(/gif$/, 'png') if attachment_options[:processor] == "ImageScience"
@@@
This doesn't fix the problem if "ImageSience" is explicitly set in the has_attachment definition. One approach to solve this problem is to remove the "Processor" at the end of line 85 of attachment_fu.rb and put it after attachment_options[:processor] on line 86. After the change it should look like:
attachment_options[:processor] = "#{processors.first}" processor_mod = Technoweenie::AttachmentFu::Processors.const_get("#{attachment_options[:processor]}Processor") -
Virginio Carfagno October 9th, 2008 @ 04:24 AM
- Tag set to “bug”
I tried the solution of Benjamin Behr and it works.
-
edraut October 15th, 2008 @ 08:39 AM
This is a repeat of the previous post from Oct. 6. The correct solution is in there.
Please Sign in or create a free account to add a new ticket.
With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.
Create new ticket
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป
Benjamin Behr
edraut
Virginio Carfagno