【Rails5】ユーザ情報が正常に更新されない

Question

実現したいこと

ユーザ編集画面からユーザ情報を更新したい。

発生している事象

ユーザID=2 のユーザでログインした状態でユーザ情報更新フォームにて編集を行い更新すると
ユーザID=1 のユーザが更新されてしまう。

ソースコード

#users/edit.html.erb

<h1>ユーザ情報編集</h1>
<div class="regist_form">
<%= form_with model: @user, local: true, :url => {:action => :update} do |f| %>
  <div class="form-group">
  	<h2><%= f.label "氏名" %> : <%= @user.user_name %></h2 class="form-control">
  </div>
  <div class="form-group">
      <%= f.label "ユーザ名" %><br>
      <%= f.text_field :display_name, class: 'form_control' %>
  </div>
  <div class="form-group">
      <%= f.label "メールアドレス" %><br>
      <%= f.email_field :email, class: 'form_control' %>
  </div>
  <div class="form-group">
      <%= f.label "パスワード" %><br>
      <%= f.password_field :password, class: 'form_control' %>
  </div>
  <div class="form-group">
      <%= f.label "パスワード(確認用)" %><br>
      <%= f.password_field :password_confirmation, class: 'form_control' %>
  </div>
      <%= f.submit "Save changes", class: "btn btn-primary" %>
<% end %>
</div>

#users_controller.rb

class UsersController < ApplicationController
  before_action :set_user, only: [:show, :edit, :update, :delete]

  def show
    # before_action :set_user でユーザ情報取得
  end

  def edit
    # before_action :set_user でユーザ情報取得
  end

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

    # if current_user = @user
      if @user.update_attributes(user_params)
        flash[:success] = 'successfully update user information.'
        # params[:session][:display_name] = @user[:display_name]
        redirect_to users_show_path(id: @user[:id])
      else
        flash.now[:danger] = 'FAILED! Please check parameters.'
        render 'edit'
      end
    # else
    #   redirect_to home_path
    # end
  end

  private
    def set_user
      @user = User.find_by(display_name: params[:display_name])
    end

    def user_params
      params.require(:user).permit(:user_name, :display_name, :email, :password, :password_confirmation)
    end
end

class CreateUsers < ActiveRecord::Migration[5.2]
  def change
    create_table :users do |t|
      t.string :user_name
      t.string :display_name
      t.string :email
      t.string :password_digest
      

      t.timestamps
    end
  end
end

---

追記

routes.rb は以下のようになっております。

Rails.application.routes.draw do
<中略>
  get    '/edit_user',  to: 'users#edit'
  patch  '/edit_user',  to: 'users#update'
  put    '/edit_user',  to: 'users#update'
<中略>
end

また、edit.html.erb へ遷移するための、show.html.erb は以下の通りです。

<h1>マイページ</h1>
<h3>氏名      : <%= @user.user_name %></h3>
<h4>メールアドレス: <%= @user.email %></h4>
<br>
<% if logged_in? %>
  <%= link_to "ユーザ情報の変更", edit_user_path(display_name: @user.display_name), class: "btn btn-primary" %>
  <%= link_to 'ログアウト', logout_path, method: :delete %>
<% end %>

---

編集画面からupdateメソッドに飛んだ際にどの項目も正常に更新されません。
ご助力いただけると幸いです。

足りない情報などあればご指摘ください。

Comments

[bamboo-nova]

editへ遷移するときのリンクなどはどのように記載されているのでしょうか？

[bamboo-nova]

routesではなくて、editのビューへ移動するリンクが貼られているビューです。もしかしたら、一つ前のリンクで飛ばしているんじゃないかと思ったので...

[zvub1123]

失礼しました。。。
更新ページの直前に該当する「マイページ」ビューを追記しました。

[bamboo-nova]

get '/edit_user', to: 'users#edit'
patch '/edit_user', to: 'users#update'
put '/edit_user', to: 'users#update'
:idがパスに割り振られていないです。多分、これによる影響な気がします。

モデルと紐づけた方が楽なので、例えばですがresourcesを使うのをお勧めします。

Answer 1

気になる点はいくつかあります。。
１）show.html.erb　で表示しているUserがログインユーザで有るか否か保証がありません。
２）update では id でUser を取り出してますが、edit.html.erb　には id のfieldがありません
3) update で if current_user = @user　をコメントアウトしているのでログインユーザ以外のものも修正できます。
4) display_nameでUserを呼んでますが、 id で有る方が安全です。
５）routeが標準でないのが気になります。
resorces :users
の１行で index,show,create,update,destroy 等のrouteを定義し、かつ標準の形 例えば　/users/1/edit を作ってくれます。特に理由がなければそれを使うことをお薦めします

＃＃＃＃
書いていたつもりで落ちてました。
routesの修正の他にも修正必要です
update では　User.find(params[:id]) と id を使うのに、
viewでは edit_user_path(display_name: @user.display_name)　とIDを渡していません。
edit_user_path(@user) として下さい

Comments

[zvub1123]

色々とご指摘ありがとうございます。
デフォルトのパスが見えるのが個人的に好きでは無かったのでパスを細かくカスタマイズしていたつもりだったのですが、それに付随してコントローラのアクションやViewで、統一感が損なわれてしまっていました。。。
ご指摘通りresources: <model> という表記に変え、対応する処理なども修正したところ可読性も上がりました。ご助力ありがとうございます。