renderされているが、flashが表示されない・・・・・・

Question

modelでバリデーションをかけ、失敗した場合に、errorsに入るメッセージをflash.nowを使い、viewで取り出したいのですが、上手くできません。

#コントローラー

def create
 ・・・中略
 if @hoge.save
    redirect_to hoge_path
 end
 flash.now[:error] = @hoge.errors.full_messages
 render :new
end

/<new.html.haml>

-if flash[:error].present?
  -binding.pry
  [e-mail]_messages.each do |message|
    %h2
      =message
 ・・・中略

new.html.hamlの２行目のbinding.pryでとまります。
ということは、renderはされていて、if文の中まで処理が進んでいるということだと思います。
flash[:error]の中身を見ても、ちゃんと入っていることが確認できます。
しかし、なぜなのか、viewに表示されません。
ちなみに、render :newの部分を、redirect_to + flashに書き換えたら表示できました。

renderとredirect_toの違いは自分でも調べてみて、
render=>画面描画のみ、
redirect_to => コントローラーを介してのviewの表示ということはわかりました。

ifの中まで処理が進んでいるので、画面は適切にrenderされているのだと思います。
しかし、なぜflashメッセージが表示されないのでしょうか？？
周りにrubyに詳しい人がいないため、昨日の夕方から悩み続けています。
どなたか教えて下さい。お願いします。

Answer 1

if @hoge.save
    redirect_to hoge_path
end
 flash.now[:error] = @hoge.errors.full_messages
 render :new

これだと全てnewに戻ってきてしまうので、以下のようにしたらどうでしょう。

if @hoge.save
    redirect_to hoge_path
else
 flash.now[:error] = @hoge.errors.full_messages
 render :new
end

flashはviewで拾った方がいいとは思いますが。取り急ぎ。

Comments

[shinichiro81]

zooiioozさん、解答ありがとうございます。
何かの手違いだと思うのですが、
zooiioozさんに掲げていただいたコードが私のものと同じになっているので、
よろしければ、改善したコードを教えていただけないでしょうか？？

[zooiiooz]

最初のコードが
if @hoge.save
redirect_to hoge_path
end
flash.now[:error] = @hoge.errors.full_messages
render :new
end
となってたので、最初のendをelseと間違えたのかと思ったのですが...。

[shinichiro81]

zooiioozさん,大変失礼しました。
私のコードを修正して頂けていたのですね。
そうと気づかずに、無礼なコメントをしてしまいました。
申し訳ございません。
しかし、直して頂いたコードで試して見ても、解決しませんでした・・・・・・
new.html.hamlがrenderされるところまではいきますし（binding.pryで止まる）、
flashに中身が入っていることも確認済みなのですが、
なぜか表示されません。
renderでなくredirect_toを使えば、表示できるので、そんなに困ってはいないのですが、
なぜなのかとても気になってしまって。

Answer 2

2017年の質問ですが、同じような問題にハマったので記述しておきます。
同じように、redirectすると flash[:alert] などを表示できるが、renderではできませんでした。

質問者のshinichiro81さんのviewでの実装が不明なので解決するかわかりませんが、
もし、viewで form_with を使用している場合、local: true オプションをつけるとrenderで表示されるようになります。

= form_with url: xxx_path, method: :post, local: true do |f| 

https://qiita.com/Tatsu88/items/8ea9b944681a48cce589

Answer 3

flash[:error] にエラーメッセージを格納してるのに、
なぜView側では@address.errorsを見てるのでしょう？

flash[:error].each do |message|で普通に取れると思うのですが、
いかがでしょうか？

Comments

[shinichiro81]

tashua314さん、ありがとうございます。
なるほど！ご指摘のコードの方が短くてスッキリしますね。
訂正させていただきました。
だけど、やっぱりrenderだと表示されませんね。redirect_toだと可能なのですが。