CarrierWave + RMagickを使って画像とPDFをアップロードしたい

Question

概要

Ruby On Rails 5 で投稿機能を作っています。
アップローダーはCarrierWave、投稿画像についてはサムネイル化したかったのでRMagickを導入しています。
かかる中、以下のような設定をしたことで、pdfファイルのアップロードができなくなり詰まってしまいました。

やりたいこと

carrierWaveを使って画像、PDFをアップロードできるようにしたい。ただし、画像についてはRMagickを使ってサムネイル化させたい。

現状

RMagickを使わない場合（サムネイル化しない）、はうまく機能しています。

現状のアップローダーの設定は以下の通りです。

class ArticleUploader < CarrierWave::Uploader::Base

  include CarrierWave::RMagick
 
  # サムネイルを生成する設定
  version :thumb do
    process :resize_to_limit => [300, 300]
  end
 
  # jpg,jpeg,gif,png pdfしか受け付けない
  def extension_white_list
    %w(jpg jpeg gif png pdf)
  end
  
  protected
  def secure_token
    var = :"@#{mounted_as}_secure_token"
    model.instance_variable_get(var) or model.instance_variable_set(var, SecureRandom.uuid)
  end
 
  if Rails.env.production?
    storage :fog
  else
    storage :file
  end

  def store_dir
    "uploads/#{model.class.to_s.underscore}/#{mounted_as}/#{model.id}"
  end

  def cache_dir
    "#{Rails.root}/tmp/uploads"
  end

end

PDFがアップロードできない原因と解決方法の自己考察

上記コード内の「＃サムネイルを生成する設定」が効いてしまって画像以外のアップロードができないようです。

このため、
該当箇所を拡張子が（jpg jpeg gif png）なら「＃サムネイルを生成する設定」を有効、
拡張子が（pdf）なら「＃サムネイルを生成する設定」を無視する、
といったような、if文を作れば良いのでは？と考えています。

しかしながら、この条件式の作り方がわからず困っております。
良い解決策がございましたらご教授いただけますと幸いです。

Answer 1

まだまだ初学者なので、良い回答が出せるかどうかわかりませんが…。
私はgemのメソッドが自分の思った通りに動作してくれない場合、そのメソッドをオーバーライドすることにしています。

まずは以下にサンプルコードを記載させていただきますね。

class ArticleUploader < CarrierWave::Uploader::Base

  include CarrierWave::RMagick

  # サムネイルを生成する設定
  version :thumb do
    process :resize_to_limit => [300, 300]
  end

  #以下の記述を追記し、resize_to_limitをオーバーライド
  def resize_to_limit(width, height)
    if File.extname(original_filename) == '.pdf'
      width = dimension_from width
      height = dimension_from height
      manipulate! do |img|
        geometry = Magick::Geometry.new(width, height, 0, 0,　Magick::GreaterGeometry)
        new_img = img.change_geometry(geometry) do |new_width, new_height|
          img.resize(new_width, new_height)
        end
        destroy_image(img)
        new_img = yield(new_img) if block_given?
        new_img
      end
    end
  end
  #ここまで

  # jpg,jpeg,gif,png pdfしか受け付けない
  def extension_white_list
    %w(jpg jpeg gif png pdf)
  end

  protected
  def secure_token
    var = :"@#{mounted_as}_secure_token"
    model.instance_variable_get(var) or model.instance_variable_set(var, SecureRandom.uuid)
  end

  if Rails.env.production?
    storage :fog
  else
    storage :file
  end

  def store_dir
    "uploads/#{model.class.to_s.underscore}/#{mounted_as}/#{model.id}"
  end

  def cache_dir
    "#{Rails.root}/tmp/uploads"
  end

end

さて、こちらの記述で何が行われているのか、簡単に説明させていただきますね。
画像のリサイズに使われているresize_to_limitはCarriawaveの/lib/carrierwave/processing/rmagick.rbで定義されているメソッドです。
このファイルに記載されているメソッドを使えるようにするためにinclude CarrierWave::RMagickという記載があるわけです。

では、ここからメソッドのオーバーライドの話に映ります。
carrierwaveuploader（github）
上記のURLはCarrierwaveの本体のコードがしまってあるページです。
こちらページの/lib/carrierwave/processing/rmagick.rbの一部に以下のような記載があります。

def resize_to_limit(width, height)
  width = dimension_from width
  height = dimension_from height
  manipulate! do |img|
    geometry = Magick::Geometry.new(width, height, 0, 0, Magick::GreaterGeometry)
    new_img = img.change_geometry(geometry) do |new_width, new_height|
      img.resize(new_width, new_height)
    end
    destroy_image(img)
    new_img = yield(new_img) if block_given?
    new_img
  end
end

これが、今回オーバーライドすべきメソッドになります。
この文頭にif File.extname(original_filename) == '.pdf'を追加することによって、拡張子がpdfの場合にのみresize_to_limitの処理を実行することができるようになります。
…なる、はず、、です……。
ちなみにextnameは、rubyにおいて、ファイルの拡張子を取得ためのメソッドですね。
詳しい内容に関しては以下のURLに乗っているので、ご参考まで。
extnameに関するドキュメント

方向性としては間違っていないと思うのですが、正直私も自信がありません。
こちらの回答がzin2さんのお役に立てることをお祈りしております。
よろしくお願いいたします。

Comments

[zin2]

また違うエラーが出ており、まだ解決はできていないですが、理論的で丁寧なご説明、ありがとうございました。もう少し検討してみます。