Couldn't find User with 'id'= が解決できません

Question

タイムカードを申請するアプリを作成しています。
下記のカラムに値が入っている日付のみを更新したいのですが、
成功処理、失敗処理のどちらともCouldn't find User with 'id'が出てしまいます。
原因が分からないので、解決策を教えて頂けないでしょうか？
宜しくお願いします。

＊@userはbefore_actionにて@user = User.find(params[:id])を指定＊

# 勤怠変更申請



  def edit_one_month 
    @attendance = Attendance.find(params[:id])
    @superior = User.where(superior: true).where.not( id: current_user.id )
  end 

  def update_one_month
    ActiveRecord::Base.transaction do 
    attendances_params.each do |id,item|
      @attendance = Attendance.find(id)
      if item[:indicater_check_change].present? &&
        item[:note].present? &&
        item[:started_at] && item[:finished_at].present? && item[:started_at] < item[:finished_at]
        @attendance.update_attributes!(item)
        flash[:success] = "勤怠変更を受け付けました"
        redirect_to user_url(@user)
      else
        flash[:danger] = "無効な入力データがあった為、更新をキャンセルしました。"
        redirect_to edit_one_month_user_attendance_url(@user, @attendance)
        return
      end
    end 
    rescue ActiveRecord::RecordInvalid 
    flash[:danger] = "無効な入力データがあった為、更新をキャンセルしました。"
    redirect_to edit_one_month_user_attendance_url(@user, @attendance)
    return
    end
  end

  


private
    # 勤怠編集
    def attendances_params
      # userに紐ずくattendanceテーブルの（出勤,退勤,翌日,備考,指示者確認（どの上長か,指示者確認（申請かどうか））
      params.require(:user).permit(attendances: [:started_at, :finished_at, :tomorrow, :note, :indicater_check_change, :indicater_reply_change])[:attendances]
    end

<%= form_with(model: @user, url: update_one_month_user_attendance_path(@user, @attendance), local: true, method: :patch) do |f| %>
<div>
  <h1>勤怠編集画面</h1>
  <table class="table table-bordered table-condensed table-hover" id="table-attendances">
    <thead>
      <tr>
        <th>日付</th>
        <th>曜日</th>
        <th>出社</th>
        <th>退社</th>
        <th>翌日</th>
        <th>在社時間</th>
        <th>備考</th>
        <th>指示者確認<div class="maru size_small black">       
          <div class="letter3">印</div>
        </th>
      </tr>
    </thead>
    <tbody>
      <% @attendances.each do |day| %>
      <%= f.fields_for "attendances[]", day do |attendance| %>
        <% css_class = 
              case $days_of_the_week[day.worked_on.wday]
                when '土'
                  'text-primary'
                when '日'
                  'text-danger'
                end
        %>
        <tr>
          <td><%= l(day.worked_on, format: :short) %></td>
          <td class="<%= css_class %>"><%= $days_of_the_week[day.worked_on.wday] %></td>
          <td><%= attendance.time_select :started_at, {include_blank: true}, {class: "form-control bootstrap-date-only-width"} %></td>
          <td><%= attendance.time_select :finished_at, {include_blank: true}, {class: "form-control bootstrap-date-only-width"} %></td>
          <td><%= attendance.check_box :tomorrow,id: "tomorrow" %></td>
          <td>
            <% if day.started_at.present? && day.finished_at.present? %>
              <%= str_times = working_times(day.started_at, day.finished_at) %>
              <% @total_working_times = @total_working_times.to_f + str_times.to_f %>
            <% end %>
          </td>
          <td><%= attendance.text_field :note, class: "form-control" %></td>
          <td><%= attendance.collection_select(:indicater_check_change, @superior, :name, :name, {prompt: "上長を選択して下さい"}, {class: "form-control input-sm" })%></td>
        <% end %>
      <% end %>
    </tbody>
  </table>
</div>

<div class="center">
  <%= link_to "キャンセル", user_path(date: @first_day), class: "btn btn-lg btn-primary" %>
  <%= f.submit "編集を保存する", class: "btn btn-lg btn-primary" %>
</div>
<%end%>

Answer 1

エラー画面の下の方に書いてにあるparamsをみるとuser_idが11、idが218のようです。

@user = User.find(params[:id])

では、params[:id]としているのでIDが218番のUserを探してしまっています。
おそらくここでほしいユーザーデータは以下のようにして取得できるかと思います。

@user = User.find_by(id: params[:user_id])

併せて、 @userはbefore_actionにて@user = User.find(params[:id])をedit_one_monthのときには行わないようにする必要もあるかと思います。

Comments

[zoff77]

解決しました！ありがとうございます！！！