attachment_on_the_fly 0.1.0 → 0.1.1

data/VERSION

@@ -1 +1 @@
-0.1.0
+0.1.1

data/lib/attachment_on_the_fly.rb

@@ -79,20 +79,27 @@ Paperclip::Attachment.class_eval do
       return new_path
     end
 
+    command = ""
+
     if kind == "height"
       # resize_image infilename, outfilename , 0, height
       command = "#{convert_command_path}convert -colorspace RGB -geometry x#{height} -quality 100 -sharpen 1 #{original} #{newfilename} 2>&1 > /dev/null"
-      `#{command}`
     elsif kind == "width"
       # resize_image infilename, outfilename, width
       command = "#{convert_command_path}convert -colorspace RGB -geometry #{width} -quality 100 -sharpen 1 #{original} #{newfilename} 2>&1 > /dev/null"
-      `#{command}`
     elsif kind == "both"
       # resize_image infilename, outfilename, height, width
       command = "#{convert_command_path}convert -colorspace RGB -geometry #{width}x#{height} -quality 100 -sharpen 1 #{original} #{newfilename} 2>&1 > /dev/null"
-      `#{command}`
+    end
+
+    `#{command}`
+
+    if $? != 0
+      raise AttachmentOnTheFlyError.new("Execution of convert failed")
     end
 
     return new_path
   end
 end
+
+class AttachmentOnTheFlyError < StandardError; end

metadata

@@ -2,7 +2,7 @@
 name: attachment_on_the_fly
 version: !ruby/object:Gem::Version 
   prerelease: 
-  version: 0.1.0
+  version: 0.1.1
 platform: ruby
 authors: 
 - Jeff Sutherland
@@ -10,7 +10,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-02-17 00:00:00 -07:00
+date: 2011-02-18 00:00:00 -07:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency