質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.53%

  • Ruby

    9189questions

    Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

  • Ruby on Rails 5

    2967questions

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

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 341

zin2

score 6

 概要

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文を作れば良いのでは?と考えています。

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

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

+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_limitCarriawave/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さんのお役に立てることをお祈りしております。
よろしくお願いいたします。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/11/13 13:30

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

    キャンセル

同じタグがついた質問を見る

  • Ruby

    9189questions

    Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

  • Ruby on Rails 5

    2967questions