タグ付き投稿のパラメーターにuser_idを取り込む方法について

Question

前提・実現したいこと

formオブジェクトを用いてタグ付きの投稿を実装しましたが、user_idがなかったので、付け直しています。
マイグレートのし直し、アソシエーションは記述しました。

コントローラーに問題があると思うのですが、一日費やしても解決できませんでした。

検索しても試した記事しか出てこなく、解決方法が見つかりません。

エラー画面のコンソールで@user, current_userなどとしてもundefinedになってしまいます。

User.allは取得できました。

発生している問題・エラーメッセージ

NameError in PostsController#create

undefined local variable or method `current_user' for #<PostTag:0x00007fc867956140>
Extracted source (around line #12):

deviseを使った時に、ログイン中のユーザー情報を取得する方法
使用するコントローラ内にて、
before_action :authenticate_user!
とすると、current_userでユーザー情報を取得できます。

ログイン中のユーザーIDであれば、以下のようにすれば取得可能です。
@user = current_user.id

これが書いてある記事などを参考にしました。

該当のソースコード

class PostsController < ApplicationController

  before_action :authenticate_user!, only: [:new, :create]

  def index
    @posts = Post.all.order(created_at: :desc)
  end

  def new
    @post = PostTag.new
  end

  def create
    @user = current_user
    @post = PostTag.new(post_params)
    if @post.valid?
      @post.save
      return redirect_to root_path
    else
      render "new"
    end
  end

  def search
    return nil if params[:input] == ""
    tag = Tag.where(['name LIKE ?', "%#{params[:input]}%"])
    render json:{ keyword: tag }
  end


  private

  def post_params
    @user = current_user

    params.require(:post_tag).permit(:message, :name).merge(user_id: current_user)
  end
end

補足情報（FW/ツールのバージョンなど）

rails 6.0.0
ruby 2.6.5

Comments

[winterboum]

エラーメッセージとcodeが整合しません。エラーが起きた時のcodeですか１２行目は空行に見えます。

Answer 1

提示していただいているコードで気になったのは下記ですが、これで期待通りの動作をするでしょうか？

- params.require(:post_tag).permit(:message, :name).merge(user_id: current_user)

+ params.require(:post_tag).permit(:message, :name).merge(user_id: current_user.id)

Answer 2

変数名やシンボルにいろいろ入れてたので、どこが問題だったのが正確には把握できなかったのですが、
おそらく、formオブジェクトにおいてattr_accessor :user_idとしてたので、Post.new(user_id: user_id)としたのが良かったのかなと思います。

また、コンソールでbinding.pryを使いながらformオブジェクトの流れを把握しつつ解決することの重要性を知りました。

コメントいただいた方、ありがとうございました。

class PostTag
  include ActiveModel::Model



  attr_accessor :message, :name, :user_id

  with_options presence: true do
    validates :message
    validates :name
    validates :user_id
  end

  def save
      post = Post.new(message: message, user_id: user_id)
      post.save
      tag = Tag.where(name: name).first_or_initialize
      tag.save

      PostTagRelation.create(post_id: post.id, tag_id: tag.id)
  end

end

class PostsController < ApplicationController

  before_action :authenticate_user!, only: [:new, :create]
  before_action :getuser

  def index
    @posts = Post.all.order(created_at: :desc)
  end

  def new
    @post = PostTag.new
  end

  def create
    @post = PostTag.new(post_params)
    if @post.valid?

      @post.save
      return redirect_to root_path
    else
      render "new"
    end
  end

  def search
    return nil if params[:input] == ""
    tag = Tag.where(['name LIKE ?', "%#{params[:input]}%"])
    render json:{ keyword: tag }
  end
  

  private

  def post_params
    params.require(:post_tag).permit(:message, :name).merge(user_id: @user)
  end

  def getuser
    @user = current_user.id
  end
end

いろいろいじってしまったので、リファクタリングできたらしようと思います。