Merge pull request #1805 from mehlah/rmagick_width_and_height

Add `#width` and `#height` methods to the RMagick processor

lib/carrierwave/processing/mini_magick.rb

@@ -318,8 +318,6 @@ def append_combine_options(cmd, combine_options)
         end
       end
 
-    private
-
       def mini_magick_image
         if url
           ::MiniMagick::Image.open(url)

lib/carrierwave/processing/rmagick.rb

@@ -251,6 +251,28 @@ def resize_to_geometry_string(geometry_string)
       end
     end
 
+    ##
+    # Returns the width of the image.
+    #
+    # === Returns
+    #
+    # [Integer] the image's width in pixels
+    #
+    def width
+      rmagick_image.columns
+    end
+
+    ##
+    # Returns the height of the image.
+    #
+    # === Returns
+    #
+    # [Integer] the image's height in pixels
+    #
+    def height
+      rmagick_image.rows
+    end
+
     ##
     # Manipulate the image with RMagick. This method will load up an image
     # and then pass each of its frames to the supplied block. It will then
@@ -351,5 +373,9 @@ def destroy_image(image)
       image.destroy! if image.respond_to?(:destroy!)
     end
 
+    def rmagick_image
+      ::Magick::Image.read(current_path).first
+    end
+
   end # RMagick
 end # CarrierWave

spec/processing/rmagick_spec.rb

@@ -196,6 +196,14 @@
     end
   end
 
+  describe "#width and #height" do
+    it "should return the width and height of the image" do
+      @instance.resize_to_fill(200, 300)
+      expect(@instance.width).to eq(200)
+      expect(@instance.height).to eq(300)
+    end
+  end
+
   describe "test errors" do
     context "invalid image file" do
       before do