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

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

新規登録して質問してみよう
ただいま回答率
85.48%
Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

Q&A

解決済

2回答

231閲覧

【delete後にホーム画面に戻っても削除されていない】

takabeee

総合スコア19

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

0グッド

0クリップ

投稿2020/01/09 01:04

編集2020/01/09 03:24

お世話になっております。
題目にある通りなのですが、それに対して質問をさせていただきます。

まず、削除ボタンを以下のように書きました。
・削除ボタン

<% if current_user == micropost.user %> <%= link_to "Delete", micropost, method: :delete, data: { confirm: "You sure?" }, class: 'btn btn-danger btn-sm' %> <% end %>

念のために削除ボタンを含めたページの全容を載せておきます。
・_micropost.html.erbファイル

<ul class="list-unstyled"> <% microposts.each do |micropost| %> <li class="media mb-3"> <img class="mr-2 rounded" src="<%= gravatar_url(micropost.user, { size: 50 }) %>" alt=""> <div class="media-body"> <div> <%= link_to micropost.user.name, user_path(micropost.user) %> <span class="text-muted">posted at <%= micropost.created_at %></span> </div> <div> <p class="mb-0"><%= micropost.content %></p> </div> <div> <% if current_user == micropost.user %> <%= link_to "Delete", micropost, method: :delete, data: { confirm: "You sure?" }, class: 'btn btn-danger btn-sm' %> <% end %> </div> </div> </li> <% end %> <%= paginate microposts %> </ul>

また、コントローラのdestroyアクションは以下の通りです。
・micropost#destroy

def destroy @micropost.destroy flash[:success] = 'メッセージを削除しました。' redirect_back(fallback_location: root_path) end

before_actionは以下の通りです。

before_action :correct_user, only: [:destroy]

・correct_userメソッド

def correct_user @micropost = current_user.microposts.find_by(id: params[:id]) unless redirect_to root_url end end

以上のコードで投稿を削除したら、ページがリロードされている感じはありますが削除されずにホーム画面に戻ってしまいます。

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

ベストアンサー

def correct_user @micropost = current_user.microposts.find_by(id: params[:id]) unless redirect_to root_url end end

unless の条件節が redirect_to root_url になってしまい、必ず成功してしまいます。
というか、条件判定のためにredirectしてしまう。

unless @micropost
では?

投稿2020/01/09 04:16

winterboum

総合スコア23329

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

takabeee

2020/01/09 04:26

うまくいきました。泣 ありがとうございます。
guest

0

@micropost.destroy
とありますが、@micropost への代入が def destroy には見当たりません。
before_action とか定義してありますか?
あればその定義を見せてください
なければ @micropost = Micropost.find(params[:id]) を

投稿2020/01/09 03:03

winterboum

総合スコア23329

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

takabeee

2020/01/09 03:22

回答ありがとうございます。そしてまた説明が不足していたこと、申し訳ございません。
takabeee

2020/01/09 03:24

質問内容を追記しました。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問