dragonfly 0.5.0 → 0.5.1

data/VERSION

@@ -1 +1 @@
-0.5.0
+0.5.1

data/dragonfly.gemspec

@@ -5,7 +5,7 @@
 
 Gem::Specification.new do |s|
   s.name = %q{dragonfly}
-  s.version = "0.5.0"
+  s.version = "0.5.1"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Mark Evans"]

data/extra_docs/Processing.md

@@ -44,12 +44,7 @@ Each method takes the temp_object, and the (optional) processing options hash as
     end
 
     app = Dragonfly::App[:images]
-    
     app.register_processor(MyProcessor)
-    
-    temp_object = app.create_object(File.new('path/to/image.png'))
-    
-    temp_object.process(:black_and_white, :some => 'option')
 
 You can register multiple processors.
 
@@ -59,6 +54,11 @@ As with analysers and encoders, if the processor is {Dragonfly::Configurable con
       p.some_attribute = 'hello'
     end
 
+Your new processing method is now available to use:
+    
+    temp_object = app.create_object(File.new('path/to/image.png'))
+    temp_object.process(:black_and_white, :some => 'option')         # processed temp_object
+
 To get the url for content processed by your custom processor, the long way is using something like:
 
     app.url_for('some_uid',
@@ -79,7 +79,7 @@ However, this could soon get tedious if using more than once, so the best thing
 So in your configuration of the Dragonfly app (or in an initializer if using 'dragonfly/rails/images') you
 could do something like:
 
-    Dragonfly::App[:images].parameters.add_shortcut(/^bw-(\d*x\d*)$/) do |string, match_data|
+    app.parameters.add_shortcut(/^bw-(\d*x\d*)$/) do |string, match_data|
       {
         :processing_method => :black_and_white,
         :processing_options => {:size => match_data[1]},

data/lib/dragonfly/analysis/file_command_analyser.rb

@@ -7,13 +7,18 @@ module Dragonfly
       
       configurable_attr :file_command, "file"
       configurable_attr :use_filesystem, false
+      configurable_attr :num_bytes_to_check, 255
       
       def mime_type(temp_object)
         if use_filesystem
           `#{file_command} -b --mime '#{temp_object.path}'`
         else
           IO.popen("#{file_command} -b --mime -", 'r+') do |io|
-            io.write temp_object.data
+            if num_bytes_to_check
+              io.write temp_object.data[0, num_bytes_to_check]
+            else
+              io.write temp_object.data 
+            end
             io.close_write
             io.read
           end

data/spec/dragonfly/analysis/file_command_analyser_spec.rb

@@ -36,6 +36,20 @@ describe Dragonfly::Analysis::FileCommandAnalyser do
         @analyser.mime_type(@temp_object)
         @temp_object.instance_eval{@tempfile}.should be_nil
       end
+      
+      {
+        :jpg => 'image/jpeg',
+        :png => 'image/png',
+        :gif => 'image/gif',
+        :tif => 'image/tiff'
+      }.each do |format, mime_type|
+        it "should work properly (without a broken pipe error) for big files of format #{format}" do
+          processor = Dragonfly::Processing::RMagickProcessor.new
+          temp_object = Dragonfly::TempObject.new(processor.generate(1000, 1000, format))
+          @analyser.mime_type(temp_object).should == mime_type
+        end
+      end
+      
     end
   
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: dragonfly
 version: !ruby/object:Gem::Version 
-  version: 0.5.0
+  version: 0.5.1
 platform: ruby
 authors: 
 - Mark Evans