1 true 2008-04-03T13:08:16-05:00 8760 24 patch-coreimage-resizes-main-attachment-as-well-as-thumbnails 8528 35 resolved attachment_fu core_image patch 2008-05-22T20:27:24-05:00 1 Rick Paul Mucur http://ar-code.lighthouseapp.com/projects/35/tickets/24 When using the CoreImage processor and specifying a thumbnail, attachment_fu will both create a thumbnail with the specified dimensions and resize the *original image* to the thumbnail size. Manually setting resize_to does not seem to affect it and it is down to the CoreImage processor's resizing method. The fix is a simple one: adding "self.temp_path = random_tempfile_filename" to the resize_image method so that it does not overwrite the original file. You can see the change in my forked version of the plugin: http://github.com/mudge/attachment_fu/commit/801dc755af806cf5b58b34408791ee66b143a8e9 When using the CoreImage processor and specifying a thumbnail, attachment_fu will both create a thumbnail with the specified dimensions and resize the *original image* to the thumbnail size. Manually setting resize_to does not seem to affect it and it is down to the CoreImage processor's resizing method. The fix is a simple one: adding "self.temp_path = random_tempfile_filename" to the resize_image method so that it does not overwrite the original file. You can see the change in my forked version of the plugin: http://github.com/mudge/attachment_fu/commit/801dc755af806cf5b58b34408791ee66b143a8e9 <div><p> When using the CoreImage processor and specifying a thumbnail, attachment_fu will both create a thumbnail with the specified dimensions and resize the <strong>original image</strong> to the thumbnail size. Manually setting resize_to does not seem to affect it and it is down to the CoreImage processor's resizing method. </p><p> The fix is a simple one: adding "self.temp_path = random_tempfile_filename" to the resize_image method so that it does not overwrite the original file. </p><p> You can see the change in my forked version of the plugin: <a href="http://github.com/mudge/attachment_fu/commit/801dc755af806cf5b58b34408791ee66b143a8e9">http://github.com/mudge/attachme...</a> </p></div> 0 When using the CoreImage processor and specifying a thumbnail, attachment_fu will both create a thumbnail with the specified dimensions and resize the *original image* to the thumbnail size. Manually setting resize_to does not seem to affect it and it seems to be down to the CoreImage processor's resizing method. I tried adding "self.temp_path = random_tempfile_filename" to the resize_image method and, while this seemed to fix it, a duplicate thumbnail was created at RAILS_ROOT. I am using the plugin as of commit 9610d9cc764026727 on Mac OS X 10.5.2 with Rails 2.0.2 and no other image processors installed. <div><p> When using the CoreImage processor and specifying a thumbnail, attachment_fu will both create a thumbnail with the specified dimensions and resize the <strong>original image</strong> to the thumbnail size. Manually setting resize_to does not seem to affect it and it seems to be down to the CoreImage processor's resizing method. </p><p> I tried adding "self.temp_path = random_tempfile_filename" to the resize_image method and, while this seemed to fix it, a duplicate thumbnail was created at RAILS_ROOT. </p><p> I am using the plugin as of commit 9610d9cc764026727 on Mac OS X 10.5.2 with Rails 2.0.2 and no other image processors installed. </p></div> false 2008-04-04T06:20:02-05:00 8760 --- {} 24 patch-coreimage-resizes-main-attachment-as-well-as-thumbnails 0 35 new 2008-04-04T06:20:02-05:00 8760 Paul Mucur Paul Mucur http://ar-code.lighthouseapp.com/projects/35/tickets/24 0 Had the same problem and attached a patch (including a test). The problem was that the original image file was overwritten by the thumbnails. <div><p> Had the same problem and attached a patch (including a test). </p><p> The problem was that the original image file was overwritten by the thumbnails. </p></div> false 2008-05-18T09:30:57-05:00 8760 --- {} 24 patch-coreimage-resizes-main-attachment-as-well-as-thumbnails 0 35 new 2008-05-18T09:30:57-05:00 20611 Christoph Lupprich Paul Mucur http://ar-code.lighthouseapp.com/projects/35/tickets/24 1 Hi All, Thanks for the information and patch. Christoph, I've just applied your patch to http://github.com/crafterm/attachment_fu/ and sent a pull request to Technoweenie to integrate upstream. Much appreciated guys, good find. Cheers, Marcus <div><p> Hi All, </p><p> Thanks for the information and patch. Christoph, I've just applied your patch to <a href="http://github.com/crafterm/attachment_fu/">http://github.com/crafterm/attac...</a> and sent a pull request to Technoweenie to integrate upstream. </p><p> Much appreciated guys, good find. </p><p> Cheers, </p><p> Marcus </p></div> false 2008-05-21T22:51:51-05:00 8760 --- {} 24 patch-coreimage-resizes-main-attachment-as-well-as-thumbnails 0 35 new 2008-05-21T22:51:51-05:00 8621 crafterm Paul Mucur http://ar-code.lighthouseapp.com/projects/35/tickets/24 1 Pushed to github: http://github.com/technoweenie/attachment_fu/commit/834553ad3d876541b986ed6de115ea0023070bb5 Thanks everyone! <div><p> Pushed to github: <a href="http://github.com/technoweenie/attachment_fu/commit/834553ad3d876541b986ed6de115ea0023070bb5">http://github.com/technoweenie/a...</a> </p><p> Thanks everyone! </p></div> true 2008-05-22T20:27:24-05:00 8760 --- :state: new 24 patch-coreimage-resizes-main-attachment-as-well-as-thumbnails 0 35 resolved 2008-05-22T20:27:24-05:00 1 Rick Paul Mucur http://ar-code.lighthouseapp.com/projects/35/tickets/24 374bf151e51fc5881021760bb2f6c4d5f1096b36 text/plain 2008-05-18T09:30:57-05:00 patch_core_image.diff 23423 1468 20611 http://ar-code.lighthouseapp.com/attachments/23423/patch_core_image.diff