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

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

ただいまの
回答率

89.21%

railsでAjaxを実装したい ActionView::Template::Error

受付中

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 128

daaaichi

score 16

やりたいこと

RailsTutorialを参考にしてフォロー機能のAjaxで実装しようとしています。
現在の状態は非同期ではフォローされないものの、ページ遷移をするわけでもなく、ページをリロードすればフォローされるというような状況です。

class User < ApplicationRecord
  mount_uploader :avatar_image, AvatarImagesUploader
  validates :nick_name, presence: true
  # Include default devise modules. Others available are:
  # :confirmable, :lockable, :timeoutable, :trackable and :omniauthable
  devise :database_authenticatable, :registerable,
  :recoverable, :rememberable, :validatable, :omniauthable, omniauth_providers: [:twitter]
  has_many :active_relationships, class_name: "Relationship",
                                  foreign_key: "follower_id",
                                        dependent: :destroy
  has_many :passive_relationships, class_name: "Relationship",
                                  foreign_key: "followed_id",
                                  dependent: :destroy
  has_many :following, through: :active_relationships, source: :followed
  has_many :followers, through: :passive_relationships, source: :follower


  def self.from_omniauth(auth)
    find_or_create_by(provider: auth['provider'], uid: auth['uid']) do |user|
      user.provider = auth['provider']
      user.uid = auth['uid']
      user.username = auth['info']['nickname']
    end
  end

  def self.new_with_session(params, session)
    if session['devise.user_attributes']
      new(session['devise.user_attributes']) do |user|
        user.attributes = params
      end
    else
      super
    end
  end

  def follow(other_user)
    active_relationships.create(followed_id: other_user.id)
  end

  def unfollow(other_user)
    active_relationships.find_by(followed_id: other_user.id).destroy
  end

  def following?(other_user)
    following.include?(other_user)
  end
end
class Relationship < ApplicationRecord
  belongs_to :follower, class_name: "User"
  belongs_to :followed, class_name: "User"
  validates :follower_id, presence: true
  validates :followed_id, presence: true
end
class UsersController < ApplicationController
  def index
    @search = User.ransack(params[:q])
    @result = @search.result(distinct: true)
    @check = params[:q]
  end

  def show
  end

  def following
    @users = current_user.following
    render "show_follow"
  end

  def followers
    @users = current_user.followers
    render "show_follow"
  end
end
class RelationshipsController < ApplicationController
  def create
    @user = User.find(params[:followed_id])
    current_user.follow(@user)
    respond_to do |format|
      format.html {redirect_to @user}
      format.js
    end
  end

  def destroy
    @user = Relationship.find(params[:id]).followed
    current_user.unfollow(@user)
    respond_to do |format|
      format.html {redirect_to @user}
      format.js
    end
  end
end
<% @users.each do |users| %>
  <div class="d-inline-flex p-2 bd-highlight col-12 border">
    <p><%= image_tag users.avatar_image.thumb.url,:size =>'50x50' %></p>
    <p class="pl-3 col-4 d-block"><%= users.nick_name.truncate(8) %></p>
    <div class="d-flex flex-row justify-content-end">
      <% if current_user != users %> <!-- フォローフォーム開始 -->
        <div id="show_follow_form">
          <% if current_user.following?(users) %>
            <%= form_for(current_user.active_relationships.find_by(followed_id: @users.id),
            html: { method: :delete }, remote: true) do |f| %>
              <%= f.submit "フォロー中", class: "btn mt-3" %>
            <% end %>
          <% else %>
            <%= form_for(current_user.active_relationships.build, remote: true) do |f| %>
              <div><%= hidden_field_tag :followed_id, @users.id %></div>
              <%= f.submit "フォロー", class: "btn btn-primary ml-3 mt-3" %>
            <% end %>                  <!-- フォローフォーム終了 -->
          <% end %>
        </div>
        <p><%= link_to "送る", "#", class: "btn btn-info m-3" %></p>
      <% end %>
    </div>
  </div>
<% end %>
_show_follow.html.erb

<%= form_for(current_user.active_relationships.build, remote: true) do |f| %>
  <div><%= hidden_field_tag :followed_id, @users.id %></div>
  <%= f.submit "フォロー", class: "btn btn-primary ml-3 mt-3" %>
<% end %>

_show_unfollow.html.erb

<%= form_for(current_user.active_relationships.find_by(followed_id: @users.id),
        html: { method: :delete }, remote: true) do |f| %>
  <%= f.submit "フォロー中", class: "btn mt-3" %>
<% end %>
create.js.erb

$("#show_follow_form").html("<%= escape_javascript(render('users/show_follow')) %>");
$("#index_follow_form").html("<%= escape_javascript(render('users/index_follow')) %>");

destroy.js.erb

$("#show_follow_form").html("<%= escape_javascript(render('users/show_unfollow')) %>");
$("#index_follow_form").html("<%= escape_javascript(render('users/index_unfollow')) %>");

出ているエラー

 Rendering relationships/create.js.erb
  Rendered users/_show_follow.html.erb (2.3ms)
  Rendered relationships/create.js.erb (3.7ms)
Completed 500 Internal Server Error in 64ms (ActiveRecord: 30.5ms)



ActionView::Template::Error (undefined method `id' for nil:NilClass):
    1: <%= form_for(current_user.active_relationships.build, remote: true) do |f| %>
    2:   <div><%= hidden_field_tag :followed_id, @users.id %></div>
    3:   <%= f.submit "フォロー", class: "btn btn-primary ml-3 mt-3" %>
    4: <% end %>

app/views/users/_show_follow.html.erb:2:in `block in _app_views_users__show_follow_html_erb__2645418965663559337_70272209315780'

わからないこと

「undefined method `id' for nil:NilClass」と出ていてもUsersテーブルにもRelationshipsテーブルにもidカラムがあり値はちゃんと入っているのになぜNilになるのかがわかりません。

どなたかご教授願います。。。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

0

typoですね。
def create では  @user = User.find(params[:followed_id])

_show_follow.html.erb では
<div><%= hidden_field_tag :followed_id, @users.id %></div>

@users と @user になってます。 @user.idにしましょう

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2020/02/11 21:33 編集

    winterboumさん。このエラーに関しては解決できました。ありがとうございます。
    ですが、まだ非同期でのフォローができなくてフォローするときもフォローを外すときもページ更新をしないと表示が切り替わりません。フォローを外したときに下記のようなエラーログが出ています
    ActionView::Template::Error (First argument in form cannot contain nil or be empty):
    1: <%= form_for(current_user.active_relationships.find_by(followed_id: @user.id),
    2: html: { method: :delete }, remote: true) do |f| %>
    3: <%= f.submit "フォロー中", class: "btn mt-3" %>
    4: <% end %>
    また下の方にいるユーザーをフォローしたときに、一番上にいるユーザーのボタンの表示が非同期で切り替わるという現象が起きています。私ではなにが原因かわかりません。。教えていただきたいです。。よろしくお願いいたします。

    キャンセル

  • 2020/02/11 21:42

    @users がないか全体見なおして。
    @usersが正しいのか@userが正しいのか。

    キャンセル

  • 2020/02/11 21:47

    @users.each do |users| の loopの中に @users の数だけ` <div id="show_follow_form">` があります。
    最初のid="show_follow_form"を見つけてそこを書きなおしてますから、一番上が書き換わってます・
    id はユニークに。

    キャンセル

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

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