TypeError in KarutesController#update hash key "pain" is not a Symbolのエラー

Question

初学者です。
devise,cancancan,rails_adminを利用して、管理者のみがユーザーに紐づくカルテを作成、編集などができるように実装を進めています。

カルテを編集し、保存したしようとした際に、保存されていたuser_idがNULLになってしまい、バリテーション（User must exist User can't be blank）のエラーメッセージが出てしまいました。そのため、updateアクションを以下のように書き換えました。

karutes_contoroller.rb

 def edit
    @karute = Karute.find(params[:id])
  end
def update
    @karute = Karute.find(params[:id])
    if @karute.update(karute_params)
      redirect_to root_path
    else
      render :edit
    end
  end

これを変更

karutes_contoroller.rb

 def edit
    @karute = Karute.find(params[:id])
  end
def update
    @karute = Karute.find(params[:id])
    if @karute.update(
      **karute_params,
      user_id: @karute.user.id
     )
      redirect_to root_path
    else
      render :edit
    end
  end

そうすると
TypeError in KarutesController#update
hash key "pain" is not a Symbol
のエラーが出ました。
https://gyazo.com/b289957cadf4ff5a009fc875246adb3d

このエラーについて調べてみたのですが、探し方が悪いのか、参考になりそうな記事等が見つからず、困っています。
お力を貸していただければありがたいです。

###該当のソースコード

karutes_contoroller.rb

class KarutesController < ApplicationController
  def index
    @users = User.all
    # where.not(id: current_user.id) 
    # .all
  end

  def new
    @karute = Karute.new

  end

  def create
    @karute = Karute.new(karute_params)
    if @karute.save
      redirect_to user_path(@karute.user_id)
    else
      render :new
    end
  end

  def edit
    @karute = Karute.find(params[:id])
  end

  def update
    @karute = Karute.find(params[:id])
    if @karute.update(
      **karute_params,
      user_id: @karute.user.id
     )

      redirect_to root_path
    else
      render :edit
    end
  end

  private

  def karute_params
    params.require(:karute).permit(:pain, :illness, :history, :medicine, :habit, :memo, :date).merge(user_id: params[:user_id])
  end
end

users_controller.rb

class UsersController < ApplicationController
  def show
    @user = User.find(params[:id])
  end
end

karute.rb

class Karute < ApplicationRecord
  belongs_to :user
  validates :user_id,{presence: true}
end

user.rb

class User < ApplicationRecord
  # Include default devise modules. Others available are:
  # :confirmable, :lockable, :timeoutable, :trackable and :omniauthable
  devise :database_authenticatable, :registerable,
         :recoverable, :rememberable, :validatable

  with_options presence: true do
    validates :password, format: { with: /\A(?=.*?[a-z])(?=.*?\d)[a-z\d]{6,100}+\z/i }
    validates :family_name, format: { with: /\A[ぁ-んァ-ン一-龥]/ }
    validates :first_name, format: { with: /\A[ぁ-んァ-ン一-龥]/ }
    validates :family_name_kana, format: { with: /\A[ァ-ヶー－]+\z/ }
    validates :first_name_kana, format: { with: /\A[ァ-ヶー－]+\z/ }
    validates :birth_date_id
    validates :phone_number, format: { with: /\A\d{11}\z/ }
  end
  has_one :address
  has_one :karute
end

edit.htmi.erb

<h2 class="karute-title">カルテ編集</h2>

<%= form_with model: @karute, local: true do |f| %> 
<%= render 'shared/error_messages', model: f.object %> 

  

  <div class="field">
    <%= f.label "作成日" %><br />
    <%= f.text_field :date %>
  </div>

  <div class="field">
    <%= f.label "痛みのある部位" %><br />
    <%= f.text_field :pain %>
  </div>

  <div class="field">
    <%= f.label "既往症" %><br />
    <%= f.text_field :illness %>
  </div>

  <div class="field">
    <%= f.label "通院歴" %><br />
    <%= f.text_field :history %>
  </div>

  <div class="field">
    <%= f.label "服用薬" %><br />
    <%= f.text_field :medicine %>
  </div>

  <div class="field">
    <%= f.label "日常的に行っている行動" %><br />
    <%= f.text_field :habit %>
  </div>

  <div class="field">
    <%= f.label "メモ" %><br />
    <%= f.text_field :memo %>
  </div>

  <div class="actions">
    <%= f.submit "カルテ編集", class: :form__btn  %>
    <%=link_to 'もどる', root_path, class:"back-btn" %>
  </div>

<% end %>

show.html.erb
<h1>カルテ</h1>
<%= link_to '一覧に戻る', root_path %>
<% unless @user.karute.present?%> 
  <%= link_to 'カルテを追加', new_user_karute_path(@user.id), method: :get %>
<% else %> 
  <%= link_to 'カルテを編集', edit_karute_path(@user.karute.id), method: :get %>
  <h2 class="page-heading"><%= "#{@user.family_name}#{@user.first_name}さんのカルテ"%></h2>
省略

routes.rb
Rails.application.routes.draw do
  mount RailsAdmin::Engine => '/admin', as: 'rails_admin'
  devise_for :users, controllers: {
    registrations: 'users/registrations'
  }
  devise_scope :user do
    get 'addresses', to: 'users/registrations#new_address'
    post 'addresses', to: 'users/registrations#create_address'
  end
  root to: "karutes#index"
  resources :users, only: [:show] do
    resources :karutes, shallow: true
  end
end

rails routes
                                 root GET    /                                                                                        karutes#index
                         user_karutes GET    /users/:user_id/karutes(.:format)                                                        karutes#index
                                      POST   /users/:user_id/karutes(.:format)                                                        karutes#create
                      new_user_karute GET    /users/:user_id/karutes/new(.:format)                                                    karutes#new
                          edit_karute GET    /karutes/:id/edit(.:format)                                                              karutes#edit
                               karute GET    /karutes/:id(.:format)                                                                   karutes#show
                                      PATCH  /karutes/:id(.:format)                                                                   karutes#update
                                      PUT    /karutes/:id(.:format)                                                                   karutes#update
                                      DELETE /karutes/:id(.:format)                                                                   karutes#destroy
                                 user GET    /users/:id(.:format)                                                                     users#show

ご指摘をいただけると嬉しいです。

Comments

[winterboum]

user_idがNULLになってしまうのがおかしいです。
edit.htmlに問題がありそう。省略しないで載せてください。

[noe.black]

winterboum様
コメントありがとうございます。editの内容を追加いたしましたので、ご確認いただけると幸いです。

Answer 1

edit.htmlには user_id のfieldがありません。
なのに　karute_params　で params[:user_id] をmergeしているのでnilになっています。
このmergeは　create で必要なのだと思いますので、createにて
@karute = Karute.new(karute_params.merge(user_id: params[:user_id]) とし、def karute_params　の方では mergeを外してください

Comments

[noe.black]

winterboum様解決いたしました。
mergeが原因とのこと納得いたしました。
ありがとうございました。