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

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

ただいまの
回答率

90.35%

  • Ruby on Rails 4

    2566questions

    Ruby on Rails4はRubyによって書かれたオープンソースのウェブフレームワークです。 Ruby on Railsは「設定より規約」の原則に従っており、効率的に作業を行うために再開発を行う必要をなくしてくれます。

ファイルがないと投稿できない問題

解決済

回答 1

投稿

  • 評価
  • クリップ 1
  • VIEW 529

hrc

score 50

クチコミ投稿フォームを作成しています。
タイトル、評価、コメント、画像ファイル全て無いと投稿できないのですが
ファイルだけは任意にしたいと考えております。
以下ソースなのですが、どこを修正すればいいでしょう?
StrongParameterを修正すればいいだけかと思っていたらそうでなかったので
どなたかお分かりになる方よろしくお願いします。

def create

    @review = Review.new(review_params)

    file = review_params[:file]
    file_name = file.original_filename
    ext = File.extname(file_name).downcase
    new_filename = 'review/' + SecureRandom.hex + ext
    @review.image1 = new_filename

    result = uploadimage(file, file_name, new_filename)

    respond_to do |format|
      if result=="success" && @review.save
        format.html { redirect_to "/#{I18n.locale}/restaurant/#{@review.restaurant_id}", notice: 'Review was successfully created.' }
        format.json { render :show, status: :created, location: @review }
      else
        format.html { redirect_to "/restaurant/#{@review.restaurant_id}", notice: result }
        format.json { render json: @review.errors, status: :unprocessable_entity }
      end
    end
  end



private
  def review_params
    params.require(:review).permit(:restaurant_id, :user_id, :food_rating, :decor_rating, :service_rating, :file, :title, :comment)
  end

  def uploadimage(file_object, file_name, new_filename)
    perms = ['.jpeg', '.jpg', '.png', '.gif']
    if !perms.include?(File.extname(file_name).downcase)
      result = I18n.t :error_file_restriction
    elsif file_object.size > 2.megabyte
      result = I18n.t :error_file_size_limit
    else

      Aws.config[:credentials] = Aws::Credentials.new(ENV['AWS_CREDENTIAL_ACCESS_KEY_ID'], ENV['AWS_CREDENTIAL_SECRET_ACCESS_KEY'])
      Aws.config[:region] = ENV['AWS_REGION']

      s3 = Aws::S3::Client.new

      s3.put_object(
        bucket: ENV['AWS_BUCKET_NAME'],
        body: file_object,
        key: "images/#{new_filename}"
      )

      result = "success"
    end

こちらはViewです。

.div
  = form_for @review, data: { remote: true } do |f|
    div.rating
      dl
        dt = t :review_food
        dd = f.select :food_rating, options_for_select((1..5).map{|n| @review_rating[n]}, selected:@review_rating[5]), {prompt: t('placeholder.food_rating')}, {class: 'form-control'}
      dl
        dt  = t :review_decor
        dd = f.select :decor_rating, options_for_select((1..5).map{|n| @review_rating[n]}, selected:@review_rating[5]), {prompt: t('placeholder.decor_rating')}, {class: 'form-control'}
      dl
        dt = t :review_service
        dd = f.select :service_rating, options_for_select((1..5).map{|n| @review_rating[n]}, selected:@review_rating[5]), {prompt: t('placeholder.service_rating')}, {class: 'form-control'}
    div.comment
      dl  
        dt = f.label t :review_title
        dd = f.text_field :title, :size => 64
      dl
        dt = f.label t :review_comment
        dd = f.text_area :comment, :size => "64x20"
      dl
        dt = f.label t :review_image
        dd = f.file_field(:file)
        = f.hidden_field :restaurant_id, :value =>  params[:id]
        = f.hidden_field :user_id, :value => current_user.id
    div.review_btn
      = f.submit
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

0

おそらく

file = review_params[:file]
    file_name = file.original_filename
    ext = File.extname(file_name).downcase
    new_filename = 'review/' + SecureRandom.hex + ext
    @review.image1 = new_filename

    result = uploadimage(file, file_name, new_filename)

のぶぶんを

#
   result = if file = review_params[:file]
      file_name = file.original_filename
      ext = File.extname(file_name).downcase
      new_filename = 'review/' + SecureRandom.hex + ext
      @review.image1 = new_filename

      uploadimage(file, file_name, new_filename)
    else
      'success'
    end


かなぁ。。。。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/11/04 10:24

    ありがとうございます!こちらの修正で投稿できるようにはなったのですが、def createへのリクエストが2回走るようになってしまい、同じポストが2回更新されるようになってしまいました。それと投稿したら元のページにリダイレクトしてフラッシュメッセージを出すようにしていたのですが、これも表示されない状態になりました。ブラウザから更新かけるとフラッシュメッセージが出て投稿が2回分反映された状態です。これはなぜなのでしょうか??お分かりになりますでしょうか?

    キャンセル

  • 2016/11/05 11:33

    自己解決しました。form_forでremote: true→remote: falseでajaxをやめたらイベントが2度発生しなくなりました。諸々ありがとうございました。

    キャンセル

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

  • ただいまの回答率 90.35%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

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

  • Ruby on Rails 4

    2566questions

    Ruby on Rails4はRubyによって書かれたオープンソースのウェブフレームワークです。 Ruby on Railsは「設定より規約」の原則に従っており、効率的に作業を行うために再開発を行う必要をなくしてくれます。