Favoritesのhref属性は"likes_user_path"ではない？

erb ファイルは，<%=... %>のような記載をしないと，
そのまま HTML のように扱われます．

ですので，現状ではリンク先は文字列 likes_user_path であるのがブラウザから確認できると思います．

例えば，

<%= link_to 'Users', users_path ...

のように記載されている部分をブラウザから確認すると，href の値は users_path ではなく，/users のように別の値になっているかと思います．

この辺りを確認するとよいかと思いました．

①自分の投稿のお気に入りボタンを押しても表示が変わらない。

favorites/favorite_buttonパーシャルを使えば、解決できそうな気がします。
パーシャル使用時にどんなエラーが出てますか？

あと、これはパーシャルの参考になればと、https://qiita.com/ITmanbow/items/1e826f2a4c0244ac3543

@atm-snag様

ご回答ありがとうございます。
hrefを「<%= likes_user_path(@user) %>」に修正後、②のエラー画面と同様の表示が出るようになりました。(ご指摘の点はおそらく正しかったと思います)
②のエラー画面が表示される場合はどのようなことが考えられますか？

@ohakutsu様

ご回答ありがとうございます。

favorites/favorite_buttonパーシャル使用時には_micropost.html.erb上で、上記のエラー画面が表示されてしまいます。_favorites_button.html.erb上で何かコーディングミスがありますでしょうか？

@Tarzan3154
おそらくなのですが、パーシャルを使う際に、変数micropostをパーシャルに渡していないためだと思われます。
パーシャルに変数を渡したいときは、

<%= render partial: "form", locals: { thing: @thing } %>

のようにlocalsに記述する必要があります。
なお、partial, localsは省略ができるようなので、

<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="" />
    </li>
    <div>
      <%= link_to micropost.user.name, user_path(micropost.user) %>
      <span class="text-muted">posted at <%= micropost.created_at %></span>
    </div>
    <div>
      <p><%= 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 %>
-      <%= link_to "Favorite", favorites_path(micropost_id: micropost_id), method: :post, class: "btn btn-primary btn-sm" %>
+      <%= render "favorites/favorite_button", micropost: micropost %>
    </div>
  <% end %>
</ul>

のようにするとうまくいくのかなと思います。

ちなみに、パーシャルについては、先程も取り上げましたが、部分テンプレートでスッキリ！！ ❏Rails❏ - Qiitaの記事が参考になります。

ActionController::UnknownFormat in <コントローラ名>#<アクション名>

<コントローラ名>#<アクション名> is missing template ...

のエラーの場合は，メッセージとしては コントローラのアクションに対する template がありません
と言っているようです．

default では app/views/コントローラ名/アクション名.html.erb にtemplate ファイルがあることを期待されています．
UsersController#likesの場合は，apps/views/users/likes.html.erbを確認すると良いかもしれません．

リンク元ではなくて，リンク先(likes_user_path) 側の動作を確認すると良いかと思います．

ご参考

• https://rails-ambassador.herokuapp.com/debugs/UnknownFormatError

@ohakutsu様

ご回答ありがとうございます。
ご指摘の点、修正したところ、トップページおよび詳細画面で「NameError」が表示されます。(詳細画面では「NameError in Users#show」)

他に修正事項はありますでしょうか？

NameError in Toppages#index
Showing xxxx/app/views/favorites/_favorite_button.html.erb where line #8 raised.
undefined local variable or method user for xxx

とあるので，favorites/favorite_button 側で user というローカル変数かメソッドが無いようです．

favorites/favorite_button の呼び出し元で，
変数 micropost は渡しているけど，変数 user は渡していないのが原因なんじゃないかと予想します．

@atm-snag様

返信遅くなりすみません。
favorites/favorite_button の呼び出し元で変数userを渡すよう修正してみましたが、改善しませんでした。


user.rbの記述等は間違いないでしょうか？

エラーを見てないのでなんとも言いにくいのですが，
改善しないということが，エラーが変更にならないということでしたら．
user 変数が届いていれば，上記指摘したエラーが変更されているように思います．

又，Userモデル(＃favoites?)についてですが，
ソースを簡単に眺めてみた感じだとおかしな記述ではないように見えました．

favorite_button パーシャルを見てみたのですが，

micropost_id: user.id

になっているので，この辺りはおかしいのではないか．と思っています．
micropost_id は micropost の id なのでそちらが正しいのではないかと思っています．

@atm-snag @atm-snag

ご回答ありがとうございます。
少し話は逸れるのですが、詳細画面(もしくはナビバー)のFavoritesボタンを押すと、登録したお気に入りの一覧ではなく、下記のエラー画面が表示されます。

Users#likesが定義されていないエラーかと思いましたが、

上記のcontrollerの記述は間違っておりますでしょうか？

エラーとしては

/app/view/users/_users.html.erb where line #1 raised:

undefined method `any?` for nil:NilClass

簡単に訳すと，

/app/view/users/_users.html.erb の1行目で起きています．

値 `nil` : クラス(NilClass) には，`any?` というメソッドが未定義です．

となります．

場所としては if users.any? の行なので，users が nil の状態であることが予想されます．
そのために，nil.any?と解釈されて，nilには any?メソッドが無いというエラーになっていると推測します．
users/users パーシャルの中に，usersが渡されていない状態ではないかと推測します．

UsersController#likes の定義ですが

class UsersController
  def likes
    @user = User.find(params[:id])
    @favolites = @user.favorites.page(params[:page])
    counts(@user)
  end
end

特に問題はないように感じました．
counts メソッドの定義は見てないのでわからないです．

@atm-snag
ご回答ありがとうございます。
ご指摘いただいた点を修正したところ、ページが表示されるようになりました。ありがとうございます。

続けて質問ですが、Micropostを作成し、「Favorite」ボタンを押してお気に入りに登録しようとすると、

の画面が表示され、登録ができません。(既にお気に入り登録済みのMicropostで「Unfavorite」を押した場合も同様で、お気に入り解除もできず)
お気に入り(もしくは削除)対象のidが見つからないため発生しているエラーかと思いましたが、routes.rbおよびfavorites_controller.rbの記述は間違いないでしょうか？

リクエストパラメータの micropost_id が空文字になっています。

routes.rb や favorites_controller.rb ではなく、 view側で micropost_id のパラメータを設定している箇所が怪しそうです

Micropost#find のエラー

Couldn't find Micropost with `id`=

となっているので，id = のid と比較している部分が空になっているため，Micropost が find (発見) できていないようです．(id=100のように，探すための値が入っていれば，エラーにそう表示される筈)
ソース的には Micropost.find(params[:micropost_id]) なので，params[:micropost_id]が怪しいです．
parameters を見ると，"micropost_id" => "" となっているので，空文字列("")がわたってきているようです．

routes と controller

上記のように，FavoritesController#create では micropost_id が渡されてきていないようです．
コントローラには，おそらく view からリクエスト(input や submit などを使って)しているので，
view の micropost_id を input している部分が怪しいと思っています．

過去の favoite_button パーシャルを見ると，<%= hidden_field_tag :micropost_id, user.id %> になっています．
この辺りで値が渡されているか確認すると良いかと思います．

<%= form_with(model: current_user.favorites.build, local: true) do |f| %>
  <%= hidden_field_tag :micropost_id, user.id %>
  <%= f.submit 'Favoite', class: 'btn btn-primary btn-block' %>
<% end %>

@mishiwata1015 @atm-snag
返信遅くなりすみません。
<%= hidden_field_tag :micropost_id, micropost.id %>
に修正したら、正常に動作するようになりました！
ありがとうございました。

@Tarzan3154
問題が無くなったなら良かったと思います．

@atm-snag
追加でお聞きしたいのですが、ログイン後、詳細画面の「Favorites」タブ(もしくはナビバーのFavoritesタブ)をクリックしても、お気に入り一覧のページに行けず、下記のエラー画面が表示されてしまいます。(お気に入りに登録している投稿が0の場合のみ正常に表示される)

fovoritesテーブルに"email"というカラムがないからかと思い、emailカラムを作成してみましたが、改善せず。(usersテーブルのmigrationファイル内でemailカラムの行に:null => falseをつけてみても変わりませんでした)
こちらのエラーの原因をご教示お願いできないでしょうか？
よろしくお願い致します。

@atm-snag

追加で情報ですが、　_navbar.html.erb　で　
Favoritesのhref属性を　<%= likes_user_path(@user) %>
にして、ログアウトしてToppages/indexの画面を表示させようとすると、下記のエラー画面が表示されてしまいます。(href属性を　/　にしたら、正常に表示される)

現時点では？？？です。
aplication.html.erbの記述等は間違いないでしょうか？

NoMethodError in Users#likes について

• https://qiita.com/Tarzan3154/questions/bc958bc26fb6a804b772#answer-7334e550b35806898c8c

この件ですが

undefined method `email` for #<Favorite:xxxx>

直訳ですと，「Favorite クラスのオブジェクトには，emailメソッドが未定義です」となります．
Favorite クラスに存在しないという推論は正しいと思います．

しかし，下記エラー部分を見ると，

gravatar_id = Digest::MD5::hexdigest(user.email.downcase)

user.email と書いてあり，変数名 user の email メソッドを呼んでいます．
この user には Favorite クラスのオブジェクトではなく，User クラスのオブジェクトが入るのが正しいように思います．

ですので，gravatar_urlメソッドに渡している値が，Userクラスのオブジェクトかどうかを確認すると良さそうです．

ActionController::UrlGenerationError in Toppages#index について

• https://qiita.com/Tarzan3154/questions/bc958bc26fb6a804b772#answer-6e2d89bb6b272f6ac8d7

エラーの行は，

<a class="navbar-brand" href="<%= likes_user_path(@user) %>">Favorites</a>

となっていて，おそらく likes_user_path(@user) で問題があります．
(それ以外は Ruby の処理ではないので，エラーの要因にならなそう)

likes_user_path() は，routes.rb を参照すると．

• https://qiita.com/Tarzan3154/questions/bc958bc26fb6a804b772#answer-90e1c771ca1a691620ec

resources :users, only: [:index, :show, :new, :create] do
  member do
    get :likes
  end
end

とあるので，
likes_user_path(user) は /users/#{user.id}/likes(controller = users / action = likes / id = user.id と同じ) を返すように設定されています．
(この辺り，routes を調べてもらうといいと思います．)

ここで，エラーメッセージを見ると

No route matches {:action=>"likes", :controller=>"users", :id=>nil}, missing required keys: [:id]

となっていて，上記の id = user.id の部分が，nil になっていることがわかります．

とすると，最初の likes_user_path(@user) で考えると，
@user が nil になっているか，@user.id が nil になっていると推測されます．

ここで，@user がどうやって設定されているかを確認します．
一般的には，変数はコントローラ方面で設定されているので，コントローラを確認します．
(=> Toppage#index メソッド内)
という流れで確認すると良いと思います．

参考

• https://railsguides.jp/routing.html#crud%E3%80%81%E5%8B%95%E8%A9%9E%E3%80%81%E3%82%A2%E3%82%AF%E3%82%B7%E3%83%A7%E3%83%B3
  • routes 回り

@atm-snag
ご回答ありがとうございます。

gravatar_urlメソッドに渡している値が，Userクラスのオブジェクトかどうかを確認すると良さそうです．
⇒それは、どのファイルのどの部分を確認すればよろしいでしょうか？

おそらく，どこかの URL にアクセスしたときに，view が呼ばれて，
その中で，gravater_urlメソッドが呼ばれていると思います．
その関数の呼び出しているところの変数がどうなっているか，を確認すると良いと思っています．

関数を呼び出している，view で Ruby の変数の確認をするには，

<%= user %>

のようにしてみるのはどうでしょうか？(user は変数名)
そうすると，同じURL にアクセスしたときにブラウザで表示される HTML のソースコードの中に変数の内容が出力されていると思います．

更に詳しく知りたい場合は，

<%= user.email %>

などのように，特定のプロパティを参照することもできます．

<%= user %> したときに，#<User:0xxxxxxxxx〜 みたいな文字列が出れば，
user は User クラスのinstance だと推測できます．

又，参考にも載せておきましたが，以下の方法を使ってもいいかもしれません．

<%= debug user %>

<%= simple_format user.to_yaml %>

<%= user.inspect %>

参考

• https://railsguides.jp/debugging_rails_applications.html

@atm-snag

回答遅くなりすみません。
NoMethodError in Users#likesについて、
デバッグしてみたところ、@userに値が入っていないのかと思うのですが、
toppages_controller.rbおよびtoppages/indexにて、@userの値を加えればよろしいでしょうか？

過去の情報をまとめると，以下でした

• likes_user_path(@user) は，_navbar.html.erb というファイルにある
• TopPages#index をリクエストして，上記の場所でエラーになった
  • (予想) toppages/index.html.erb の内部で，navbar をrender している
• TopPages#index の内容は，コレ

で，

toppages_controller.rbおよびtoppages/indexにて、@userの値を加えればよろしいでしょうか？

Toppages#indexの内容から見ると，@user は足りなそうです．
それはそうなんですが，Favorite させたいユーザって誰なんでしょうか？

UI を見てないのでわからないのですが，例えばそれが micropost をした user のことだったら，
対象の micropost から user を得るのが正しいと思います．

なので，Toppages#index 内で @user を与えることが直接正解ではないかもしれません．
Toppages#index で likes_user_path に与えるユーザって決められそうでしょうか？