画像がnilのまま投稿するとエラーになる

Question

題名の通り、画像を空のまま投稿するとエラーが返ってきます。
Micropostモデルのimgカラムの中身がnilのまま投稿したら、
root_pathに飛んでflashを表示させたいのですがどうしたら良いのでしょうか。
ご教授よろしく御願い致します。

micropostモデルの中で下記実装中です。
↓
validates :img, presence:true

[試したこと]
MicropostsControllerのcreateアクションの中で下記コードを実装済。
→redirect_to root_path if micropost_params == nil（３行目）

def create
    @micropost = current_user.microposts.build(micropost_params)
    redirect_to root_path if micropost_params == nil
    if @micropost.save
      flash[:success] = "投稿しました！"
      redirect_back(fallback_location: root_path)
    else
      flash[:danger] = "投稿に失敗しました!"
      @feed_items = current_user.feed.paginate(page: params[:page])
      render 'pages/home'
    end
 end

  private
    def micropost_params
      params.require(:micropost).permit(:img)
    end

[viewファイル]

<h4 class="modal-title">写真投稿</h4>      
<div class="modal-body">
   <% @micropost = Micropost.new unless @micropost %>
   <%= form_for(@micropost) do |f| %>
     <%= f.file_field :img, accept: 'image/jpeg,image/gif,image/png' %>
     <%= f.submit "投稿する", class: 'btn btn-primary' %>
   <% end %>
</div>

Comments

[siruku6]

投稿formのviewファイルを質問内に追記していただいてもいいでしょうか？
errorを見る限りでは、paramsの中にmicropostというkeyが存在しないせいでエラーになっているように見えます。

[jqk]

返信ありがとうございます。viewfile追加しました。
因みに、画像を添付すれば、画像投稿はできます。

Answer 1

原因の調査結果

viewファイルを真似たところ、動作を再現することができました。
f.file_fieldについては、中身が入力されていない場合、その枠（今回でいうところの micropost）もない状態で処理がcontrollerに移るようです。

その結果、requireに失敗してしまいます。

ところが、text_fieldなどであれば、入力値が空であったとしても枠（今回はmicropost）が存在する状態で処理が行われるので、今回のようなエラーにはなりません。

つまり、file_field特有のバグのようです。

対策

ここで、対策としてfile_field以外の入力フォームを追加することでこの問題は回避できるのですが、micropostにはimg以外のカラムは存在しないのでしょうか？？

他のカラムがあれば、その入力フォームをviewに追加すれば、requireに失敗するerrorは回避できそうです。

もしカラムがもう何もないようであれば、特に存在意義はないのですが、次のようなコードをviewに差し込んでおくことで、micropostの枠が生成されてrequireは成功するようになると思いますので試してみてください。

# これはだめだった(20200214判明)
= f.hidden_field :id

# こっちはどうでしょう？
= f.hidden_field :user_id, value: current_user.id

補足(以下、間違いでした)

今回は次のようなvalidationを設定していると思いますので、

validates :img, presence: true

次の1行は削除してもらって大丈夫だと思います。

redirect_to root_path if micropost_params == nil

これがなくても、次の条件が自動でfalseになるのでelse文の処理に入ります。

if @micropost.save

Comments

[siruku6]

すみません、rootに遷移させたいのであれば、redirect_toは要りますね。
通常そんなことはしないので見落としてしまっていました。

[jqk]

丁寧なご説明ありがとうございます！言われた通りに<%= f.hidden_field :id %>をviewに挿入するとimgがnilの時のエラー表示が無くなりました！
ですが逆に、imgに画像を挿入して、投稿をしたところ、別のエラーが発生する様になりました。
→　No route matches [GET] "/microposts"
micropostモデルのカラムはimgとuser_idだけです。
どうしてこの様なエラーが表示されてしまうのでしょうか？宜しければご教授御願い致します。

[siruku6]

困りました、、、idカラムを仮で追加しても、どうせpermit(:img)をすり抜けられないから問題ないと思っていたのですが、そう簡単には行かないみたいですね。

エラーの理由としては、書かれているとおりで、micropostテーブルが想定していない情報(:id)が与えられたためだと考えられます。

user_idならあるということなので、実際に存在するデータを渡すことにして、

= f.hidden_field :user_id, value: current_user.id

に変更すると、新しいエラーも回避できるかもしれません。

[jqk]

<%= f.hidden_field :user_id, value: current_user.id%>に変えても同じエラーが表示されてしまします。
　→　No route matches [GET] "/microposts"
ご丁寧な対応ありがとうございます！もう一度試行錯誤してみます！
解決次第追加で情報流します！

[siruku6]

user_idは実際に存在するカラムなので、permit対象に追加してみるとどうでしょうか？(今は：imgだけのようですが)

[siruku6]

あ、勘違いしてました。

これ全部エラー文だと思ってましたが、後半はShunPさんの説明文でしたか、、、

>No route matches [GET] "/microposts"
micropostモデルのカラムはimgとuser_idだけです。

そういうことであれば、routingエラーが発生しているようなので、routingエラーの全文と、rails routes コマンドの結果を見比べてみてください。
分からなければ両方を掲載してもらえれば私か他の誰かが回答出来ると思います。

頑張ってみてください~

[jqk]

siruku6さん解決しました！！本当にケアしていただき有難う御座います！！

[解決方法]
①言われた通りにviewに<%= f.hidden_field :user_id, value: current_user.id%>を追加。
次にidが定義されていないというエラー文が表示されたので、
viewに<% if user_signed_in? %>を追加（deviseを使用している為）。
②次に、No route matches [GET] "/microposts"というエラーが表示されたので、
MicropostsControllerのcreateアクションのリダイレクト先を変更。
redirect_back(fallback_location: root_path)→→redirect_to root_path
ストロングパラメータで許可するのは、imgカラムだけでも大丈夫です。

因みにredirect_to root_pathに変えた理由は、ただの勘です。
どうしてこれでエラーが表示されなくなったのか、分かる範囲で良いので、
ご教授頂きたいです。よろしく御願い致します。

[siruku6]

実は、私も`redirect_back(fallback_location: パス)`という記述を見るのは初めてで、ここは理解できていません。

調べてみると、どうやらつい最近追加されたばかりのメソッドのようです（日本語の記事が少ない

そして、このメソッドの機能としては、直前に開いていたページにリダイレクトさせることのようです。

これはあくまでも予想なのですが、もしかしたら redirect_back() は直前に開いていたページに強制的に 「get リクエスト」で飛ばそうとするのかもしれません。
そしてその遷移先が、「実は get 以外の postなどのrequestで表示されていたページ」だった場合、routingが存在しないためにエラーになってしまいます。

この説明でなんとなくイメージできればいいなと思っています。。。