attachment_on_the_fly 0.1.2 → 0.1.3

checksums.yaml

@@ -0,0 +1,15 @@
+---
+!binary "U0hBMQ==":
+  metadata.gz: !binary |-
+    MDIxZWM5YjI4NzY2OGNjYWExOGY1MDAzNDAxMzJiODJkMjM5MThkZg==
+  data.tar.gz: !binary |-
+    NTliMDE1OTYxOTA1MWM3NmRhNzZjMGMyMzg5MWEwNjM1ODc1ODIzNQ==
+SHA512:
+  metadata.gz: !binary |-
+    NzYwNDkzNjI3MWE2MTcwY2QyYWYyNGZlNGE4MWMzNzY3NGU2NGVjNWNmNWRl
+    M2JiOTlhODZkZWQxNGM5MWRmODRlMzRhNDA1MmE4OWIwMDM0N2E5ZDY2OWJm
+    NmMxNGE3MjJlZmRhYWY0NmUyNWMyZDQ3MDRjYjU3NzBmZjU0MjU=
+  data.tar.gz: !binary |-
+    YzdmY2M1OTRmODEzOGFiOTIxMTgyYWU0MDY3MDkzMjJiNjA1MjBjZGMzNDI5
+    YjA4NWJjNDIxNzM3OGFiYjA5YTIxNzIyMTVhNzRmNzdhMWQ4YzgyZjYyZTQz
+    MWY3YzVjMjIyNGNlMGU2ZGZjMTFjNWZjMzUzY2E0ZGI0N2FhZDk=

data/README.rdoc

@@ -42,11 +42,25 @@ proportional, and width proportional.
 my_image_model.attachment.s_640_480 will generate a 640x480 pixel image.
 
 my_image_model.attachment.s_640_width will generate an image that is 640 pixels
+
 wide and proportionally high based on the original image dimensions.
 
 my_image_model.attachment.s_480_height will generate an image that is 480 pixels
 high and proportionally wide based on the original image dimensions.
 
+== Parameters options 
+Attachement-on-the-fly uses 'Convert' from image-magick to resize image and
+you can pass any convert options in hash notation.
+my_image_model.attachement.s_800_600 :morph => 15
+
+The only exception is :extension parameter
+my_image_model.attachement.s_400_300 :quality => 75,:extension => 'jpg' will generate
+an image that is 400x300 with quality 75% and convert jpg if necessary.
+
+link:http://www.imagemagick.org/script/command-line-options.php
+
+
+
 When an image's instance is first accessed through the s_ method, a new image
 will be generated according to the image's path interpolation.  Then, every time
 the image is accessed with the same dimensions, the previously generated image
@@ -58,6 +72,8 @@ or any other of convert's command line options, you will need to do so in the
 final lines of lib/attachment_on_the_fly.rb.  We are working on a feature to do
 this dynamically, but it is not available yet.
 
+Note that, for now, you need to name your model 'attachment'.
+
 ==Contributing
 
 If you find this gem useful, please post your comments, bugs, patches and

data/VERSION

@@ -1 +1 @@
-0.1.2
+0.1.3

data/attachment_on_the_fly.gemspec

@@ -2,16 +2,17 @@
 # DO NOT EDIT THIS FILE DIRECTLY
 # Instead, edit Jeweler::Tasks in Rakefile, and run 'rake gemspec'
 # -*- encoding: utf-8 -*-
+# stub: attachment_on_the_fly 0.1.3 ruby lib
 
 Gem::Specification.new do |s|
-  s.name = %q{attachment_on_the_fly}
-  s.version = "0.1.1"
+  s.name = "attachment_on_the_fly"
+  s.version = "0.1.3"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Jeff Sutherland"]
-  s.date = %q{2011-02-18}
-  s.description = %q{A Paperclip mix-in to allow auto-generation of resized images}
-  s.email = %q{jefferey.sutherland@gmail.com}
+  s.date = "2013-12-04"
+  s.description = "A Paperclip mix-in to allow auto-generation of resized images"
+  s.email = "jefferey.sutherland@gmail.com"
   s.extra_rdoc_files = [
     "README",
     "README.rdoc"
@@ -23,15 +24,17 @@ Gem::Specification.new do |s|
     "VERSION",
     "attachment_on_the_fly.gemspec",
     "init.rb",
-    "lib/attachment_on_the_fly.rb"
+    "lib/attachment_on_the_fly.rb",
+    "spec/attachment_on_the_fly_spec.rb",
+    "spec/spec_helper.rb"
   ]
-  s.homepage = %q{http://github.com/drpentode/Attachment-on-the-Fly}
+  s.homepage = "http://github.com/drpentode/Attachment-on-the-Fly"
   s.require_paths = ["lib"]
-  s.rubygems_version = %q{1.5.0}
-  s.summary = %q{A Paperclip mix-in to allow auto-generation of resized images}
+  s.rubygems_version = "2.1.11"
+  s.summary = "A Paperclip mix-in to allow auto-generation of resized images"
 
   if s.respond_to? :specification_version then
-    s.specification_version = 3
+    s.specification_version = 4
 
     if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then
       s.add_runtime_dependency(%q<paperclip>, [">= 0"])

data/lib/attachment_on_the_fly.rb

@@ -9,8 +9,9 @@ Paperclip::Attachment.class_eval do
 
   # we respond to s_ and cls_
   def respond_to?(method,*args, &block)
-    if method.to_s.match(/^s_[0-9]+_[0-9]+/) ||  method.to_s.match(/^s_[0-9]+_[a-z]+/) || method.to_s.match(/^s[0-9]+/)  ||
-      method.to_s.match(/^cls_[0-9]+_[0-9]+/) ||  method.to_s.match(/^cls_[0-9]+_[a-z]+/) || method.to_s.match(/^cls[0-9]+/)
+    if method.to_s.match(/^(cls|s)_[0-9]+_[0-9]+$/) ||
+      method.to_s.match(/^(cls|s)_[0-9]+_(width|height|both)$/) ||
+      method.to_s.match(/^(cls|s)[0-9]+$/)
       return true
     end
     super
@@ -19,24 +20,27 @@ Paperclip::Attachment.class_eval do
   def method_missing(symbol , *args, &block )
     # We are looking for methods with S_[number]_[number]_(height | width | proportion)
     # Height and width
-    # Check to see if file exist if so return string
-    # if not generate image and return string to file  Fiel is in format S_Height_x_Width_FILE_NAME
+    # Check to see if file exists, if so return string
+    # if not generate image and return string to file
     image_name = nil
-    if symbol.to_s.match(/^s_[0-9]+_[0-9]+/) || symbol.to_s.match(/^cls_[0-9]+_[0-9]+/)
+    parameters = args.shift
+    parameters ||= {}
+    
+    if symbol.to_s.match(/^(cls|s)_[0-9]+_[0-9]+$/)
       values = symbol.to_s.split("_")
-      height = values[1]
-      width = values[2]
-      image_name = generate_image("both", height.to_i, width.to_i)
-    elsif symbol.to_s.match(/^s_[0-9]+_[a-z]+/)   || symbol.to_s.match(/^cls_[0-9]+_[a-z]+/)
+      height = values[1].to_i
+      width = values[2].to_i
+      image_name = generate_image("both", height, width, parameters)
+    elsif symbol.to_s.match(/^(cls|s)_[0-9]+_(width|height|both)$/)
       values = symbol.to_s.split("_")
-      size = values[1]
-      who = values[2]
-      image_name = generate_image(who, size.to_i)
-    elsif symbol.to_s.match(/^s[0-9]+/)  || symbol.to_s.match(/^cls[0-9]+/)
+      size = values[1].to_i
+      kind = values[2]
+      image_name = generate_image(kind, size, size, parameters)
+    elsif symbol.to_s.match(/^(cls|s)[0-9]+$/)
       values = symbol.to_s.split("s")
-      size = values[1]
-      who = "width"
-      image_name = generate_image(who, size.to_i)
+      size = values[1].to_i
+      kind = "width"
+      image_name = generate_image(kind, size, size, parameters)
     else
       # if our method string does not match, we kick things back up to super ... this keeps ActiveRecord chugging along happily
       super
@@ -44,9 +48,15 @@ Paperclip::Attachment.class_eval do
     return image_name
   end
 
-  def generate_image(kind, height = 0, width = 0)
+  def generate_image(kind, height = 0, width = 0, parameters = {})
     convert_command_path = (Paperclip.options[:command_path] ? Paperclip.options[:command_path] + "/" : "")
-
+    parameters.symbolize_keys!
+    [:extension, :quality].each do |opt|
+      parameters.reverse_merge!({opt => Paperclip.options[opt]}) if Paperclip.options[opt]
+    end
+    quality = parameters[:quality] ||= 100
+    parameters.delete :quality
+    
     prefix = ""
 
     if kind == "height"
@@ -55,27 +65,37 @@ Paperclip::Attachment.class_eval do
       width = height
       prefix = "S_" + height.to_s + "_WIDTH_"
     elsif kind == "both"
-      prefix = "S_" + height.to_s + "_" + height.to_s + "_"
+      prefix = "S_" + width.to_s + "_" + height.to_s + "_"
     end
+    presuffix = parameters.map{|k,v| "#{k}_#{v}" }.join('___') + "_q_#{quality}_"
+    prefix = "#{prefix}#{presuffix}_"
+    prefix = "#{prefix}#{Paperclip.options[:version_prefix]}_" if Paperclip.options[:version_prefix]
 
-    path = instance.attachment.path
-    url = instance.attachment.url
+    path = self.path
+    url = self.url
 
     path_arr = path.split("/")
     file_name = path_arr.pop
     path = path_arr.join("/")
 
+    base_arr = file_name.split('.');
+    original_extension = extension = base_arr.pop
+    base_name = base_arr.join('.')
+    extension = parameters[:extension] || extension
+    parameters.delete :extension
+
     url_arr = url.split("/")
     url_file_name = url_arr.pop
     url_path = url_arr.join("/")
 
-    original = path + "/" + instance.attachment.original_filename
-    newfilename = path + "/" + prefix + file_name
-    new_path = url_path + "/" + prefix + file_name
-
-    return new_path  if  File.exist?(newfilename)
+    original = path + "/" + self.original_filename    
+    newfilename = path + "/" + prefix + base_name +  '.' + extension
+    fallback_newfilename = path + "/" + prefix + base_name +  '.' + original_extension
+    new_path = url_path + "/" + prefix + base_name + '.' + extension
+    fallback_new_path = url_path + "/" + prefix + base_name + '.' + original_extension
+    return new_path if File.exist?(newfilename)
 
-    if  !File.exist?(original)
+    if !File.exist?(original)
       if Paperclip.options[:whiny]
         raise AttachmentOnTheFlyError.new("Original asset could not be read from disk at #{original}")
       else
@@ -91,24 +111,49 @@ Paperclip::Attachment.class_eval do
 
     command = ""
 
+    options_for_extension = ""
+    if original_extension != extension && has_alpha?(original)
+      # Converting extension with alpha channel is problematic.
+      # Fall back to original extension.
+      Paperclip.log("Ignoring extension parameter because original file is transparent")
+      newfilename = fallback_newfilename
+      new_path = fallback_new_path
+    end
+
+    base_command = "#{convert_command_path}convert -strip -geometry"
+
     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 = "#{base_command} x#{height} -quality #{quality} -sharpen 1 '#{original}' '#{newfilename}' 2>&1 > /dev/null"
     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 = "#{base_command} #{width} -quality #{quality} -sharpen 1 '#{original}' '#{newfilename}' 2>&1 > /dev/null"
     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 = "#{base_command} #{width}x#{height} -quality #{quality} -sharpen 1 '#{original}' '#{newfilename}' 2>&1 > /dev/null"
     end
 
-    `#{command}`
+    convert_command command
 
-    if $? != 0
-      raise AttachmentOnTheFlyError.new("Execution of convert failed. Please set path in Paperclip.options[:command_path] or ensure that file permissions are correct.")
+    return new_path
+  end
+
+  def convert_command command
+    `#{command}`
+    if ($? != 0)
+      raise AttachmentOnTheFlyError.new("Execution of convert failed. Please set path in Paperclip.options[:command_path] or ensure that file permissions are correct. Failed trying to do: #{command}")
     end
+  end
 
-    return new_path
+  def has_alpha? image
+    identify_command_path = (Paperclip.options[:identify_command_path] ? Paperclip.options[:identify_command_path] + "/" : "")
+    # http://stackoverflow.com/questions/2581469/detect-alpha-channel-with-imagemagick
+    command = "#{identify_command_path}identify -format '%[channels]' '#{image}'"
+    result = `#{command}`
+    if ($? != 0)
+      raise AttachmentOnTheFlyError.new("Execution of identify failed. Please set path in Paperclip.options[:identify_command_path] or ensure that file permissions are correct. Failed trying to do: #{command}")
+    end
+    result && result.chomp == "rgba"
   end
 end
 

data/spec/attachment_on_the_fly_spec.rb

@@ -0,0 +1,99 @@
+require_relative './spec_helper'
+
+describe "Attachment on the fly mixin" do
+
+  subject { Paperclip::Attachment.new }
+
+  context "#respond_to?" do
+    method_names = %w{s125 cls125 s_125_250 cls_125_250
+      s_125_width cls_125_width s_125_height cls_125_height
+      s_125_both cls_125_both
+    }
+
+    method_names.each do |method_name|
+      it { should respond_to(method_name.to_sym) }
+    end
+
+    it { should_not respond_to(:x_125_250) }
+    it { should_not respond_to(:s_125_250foo) }
+    it { should_not respond_to(:S_125_250) }
+  end
+
+  context "#method_missing" do
+
+    context "translates method into a generate image call" do
+      method_name_to_generate_image_call = {
+        :s_125_225 => ["both", 125, 225, {}],
+        :s125 => ["width", 125, 125, {}],
+        :s_125_height => ["height", 125, 125, {}],
+        :s_125_width => ["width", 125, 125, {}],
+        :s_125_both => ["both", 125, 125, {}]
+      }
+
+      method_name_to_generate_image_call.each do |method_name, generate_image_args|
+        it "#{method_name}" do
+          subject.should_receive(:generate_image).with(*generate_image_args)
+          subject.send(method_name)
+        end
+      end
+
+      it "passes parameters through as well" do
+        subject.should_receive(:generate_image).with("width", 125, 125, {:quality => 90, :extension => "jpeg"})
+        subject.s_125_width :quality => 90, :extension => "jpeg"
+      end
+    end
+  end
+
+  context "#generate_image" do
+
+    context "it should generate a new image" do
+      method_name_to_expectations = {
+        :s_125_width => {
+          :new => "/S_125_WIDTH__q_100__path.png",
+          :regex => /-geometry 125 /
+        },
+        :s_125_height => {
+          :new => "/S_125_HEIGHT__q_100__path.png",
+          :regex => /-geometry x125 /
+        },
+        :s_125_both => {
+          :new => "/S_125_125__q_100__path.png",
+          :regex => /-geometry 125x125 /
+        }
+      }
+      method_name_to_expectations.each do |method_name, expected|
+        it "for #{method_name}" do
+          File.should_receive(:exist?).with(expected[:new]).and_return(false)
+          File.should_receive(:exist?).with("//file.png").and_return(true)
+          subject.should_receive(:convert_command).with(expected[:regex])
+          subject.send(method_name)
+        end
+      end
+
+      it "passes in parameters for quality" do
+        File.should_receive(:exist?).with("/S_125_WIDTH__q_75__path.png").and_return(false)
+        File.should_receive(:exist?).with("//file.png").and_return(true)
+        subject.should_receive(:convert_command).with(/-quality 75/)
+        subject.s_125_width :quality => 75
+      end
+
+      it "passes in parameters for extension" do
+        File.should_receive(:exist?).with("/S_125_WIDTH_extension_jpeg_q_75__path.jpeg").and_return(false)
+        File.should_receive(:exist?).with("//file.png").and_return(true)
+        subject.should_receive(:convert_command).with(/-quality 75/)
+        subject.should_receive(:has_alpha?).with("//file.png").and_return(false)
+        subject.s_125_width(:quality => 75, :extension => "jpeg").should == "/S_125_WIDTH_extension_jpeg_q_75__path.jpeg"
+      end
+
+      it "preserves original extension if file has alpha channel" do
+        File.should_receive(:exist?).with("/S_125_WIDTH_extension_jpeg_q_75__path.jpeg").and_return(false)
+        File.should_receive(:exist?).with("//file.png").and_return(true)
+        subject.should_receive(:convert_command).with(/-quality 75/)
+        subject.should_receive(:has_alpha?).with("//file.png").and_return(true)
+        subject.s_125_width(:quality => 75, :extension => "jpeg").should == "/S_125_WIDTH_extension_jpeg_q_75__path.png"
+      end
+
+    end
+  end
+
+end

data/spec/spec_helper.rb

@@ -0,0 +1,34 @@
+require 'rubygems'
+require 'rspec'
+
+class Hash
+  def symbolize_keys!
+    keys.each do |key|
+      self[(key.to_sym rescue key) || key] = delete(key)
+    end
+    self
+  end
+end
+
+class Paperclip
+  def self.options
+    {}
+  end
+  def self.log msg
+  end
+  class Attachment
+    def initialize
+    end
+    def path
+      "/path.png"
+    end
+    def url
+      "/url"
+    end
+    def original_filename
+      "/file.png"
+    end
+  end
+end
+
+require_relative '../lib/attachment_on_the_fly'

metadata

@@ -1,39 +1,37 @@
---- !ruby/object:Gem::Specification 
+--- !ruby/object:Gem::Specification
 name: attachment_on_the_fly
-version: !ruby/object:Gem::Version 
-  prerelease: 
-  version: 0.1.2
+version: !ruby/object:Gem::Version
+  version: 0.1.3
 platform: ruby
-authors: 
+authors:
 - Jeff Sutherland
 autorequire: 
 bindir: bin
 cert_chain: []
-
-date: 2011-02-18 00:00:00 -07:00
-default_executable: 
-dependencies: 
-- !ruby/object:Gem::Dependency 
+date: 2013-12-04 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
   name: paperclip
-  prerelease: false
-  requirement: &id001 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        version: "0"
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :runtime
-  version_requirements: *id001
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 description: A Paperclip mix-in to allow auto-generation of resized images
 email: jefferey.sutherland@gmail.com
 executables: []
-
 extensions: []
-
-extra_rdoc_files: 
+extra_rdoc_files:
 - README
 - README.rdoc
-files: 
+files:
 - README
 - README.rdoc
 - Rakefile
@@ -41,33 +39,29 @@ files:
 - attachment_on_the_fly.gemspec
 - init.rb
 - lib/attachment_on_the_fly.rb
-has_rdoc: true
+- spec/attachment_on_the_fly_spec.rb
+- spec/spec_helper.rb
 homepage: http://github.com/drpentode/Attachment-on-the-Fly
 licenses: []
-
+metadata: {}
 post_install_message: 
 rdoc_options: []
-
-require_paths: 
+require_paths:
 - lib
-required_ruby_version: !ruby/object:Gem::Requirement 
-  none: false
-  requirements: 
-  - - ">="
-    - !ruby/object:Gem::Version 
-      version: "0"
-required_rubygems_version: !ruby/object:Gem::Requirement 
-  none: false
-  requirements: 
-  - - ">="
-    - !ruby/object:Gem::Version 
-      version: "0"
+required_ruby_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
+required_rubygems_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
 requirements: []
-
 rubyforge_project: 
-rubygems_version: 1.5.0
+rubygems_version: 2.1.11
 signing_key: 
-specification_version: 3
+specification_version: 4
 summary: A Paperclip mix-in to allow auto-generation of resized images
 test_files: []
-