ユーザーの編集ができない原因を解決したい

Question

ユーザーの編集ができるようにしたい

現在ユーザーのプロフィールの編集機能を作成しているのですが、編集画面に遷移しない原因と編集後にユーザーの情報がアップデートされない原因が掴めずにいます。

お忙しいところ大変恐縮ですが、解決に至るヒントや原因などをご教授願えますでしょうか、、

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

エラーなどは表示されておりませんが、ユーザーの情報を更新すると情報がデータが変わらない状況です。

また、現在ログインしているユーザーのみが編集できるようにしており、他のユーザーのプロフィールを編集できないように制限をかけたところ、編集ページに遷移できないようになりました。

該当のソースコード

app/cotrollers/users_controller

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

  def edit
    @user = User.find(params[:id])
    if current_user.id != @user
      redirect_to root_path
    end
  end

  def update
    @user = User.find(params[:id])
    if @user.valid?
      @user.update(user_params)
      redirect_to user_path(current_user.id)
    else
      render :edit
    end
  end

  private

  def user_params
    params.require(:user).permit(:name, :profile, :avatar)
  end
end

app/views/users/show.html.erb

<%= render "shared/header" %>

<div class="top-container">
  <div class="user-content">
    <div class="user-content__main">
      <div class="user-content__img"><%= image_tag @user.avatar, class: "user-img" %></div>
      <div class="user-content__name"><%= @user.name %></div>
      <div class="user-content__link"><%= link_to  "プロフィール編集", edit_user_path(current_user.id), class: "user-link"%></div>
    </div>
    <div class="user-profile">
      <h2 class="user-profile__title">プロフィール</h2>
      <div class="user-profile__name"><%= @user.profile %></div>
    </div>
  </div>
  <div class="user-wrapper">
    <div class="user-wrapper__title"><%= "#{@user.name}さんのおすすめカフェ一覧" %></div>
    <div class="table">
      <%= render partial: "shared/upload", collection: @user.uploads %>
    </div>
  </div>
</div>

app/views/users/edit.html.erb

<%= render "shared/header" %>

<div class="edit">
  <div class="edit__main">
    <div class="edit__left">
      <h2>Edit Profile</h2>
      <h5>プロフィール編集画面</h5>
    </div>
    <div class="edit__right">
      <%= form_with model: current_user, url: user_path, local: true do |f| %>
        <%#= render "devise/shared/error_messages", resource: resource %>

        <div class="edit__field">
          <%= f.label :name %><br/>
          <%= f.text_field :name, autofocus: true, autocomplete: "name" %>
        </div>

        <div class="edit__field">
          <%= f.label :profile %><br/>
          <%= f.text_area :profile, autofocus: true, autocomplete: "profile",rows: "5", maxlength: "300" %>
        </div>

        <div class="edit__footer">
          <div class="edit__text">
            アイコン画像
          </div>
          <div class="edit__upload">
            <p>
              Click to upload file
            </p>
            <%= f.file_field :avatar, id: "edit-image" %>
          </div>
        </div>

        <div class="edit__actions">
          <%= f.submit "Edit Profile"%>
        </div>
      <% end %>

      <%= link_to "Back", :back %>

    </div>
  </div>
</div>

app/config/routes.rb

Rails.application.routes.draw do
  devise_for :users
  get 'uploads/index'
  root to: "uploads#index"
  resources :uploads
  resources :users, only: [:show, :edit, :update]
end

app/models/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

  has_many :uploads
  has_one_attached :avatar
  with_options presence: true do
    validates :email
    validates :password, format: {with:/\A(?=.*?[a-z])(?=.*?\d)[a-z\d]+\z/i, message: "is invalid. Include both letters and numbers. Input half-width characters." }
    validates :name
    validates :profile, length: { maximum: 300}
    validates :avatar
  end
end

試したこと

・コントローラーのedit、updateで

    @user = User.find(params[:id])
    if current_user.id != @user
      redirect_to root_path
    end
  end

と記述して、条件分岐をするようにしたが、編集ページに遷移できないようになってしまった。
一度、条件分岐を切り離して、編集自体ができるか確認してみたところ、paramsの取得はできているが、データが更新されないことがわかった。

  13: def update
    14:   @user = User.find(params[:id])
 => 15:   binding.pry
    16:   if @user.update(user_params)
    17:     redirect_to user_path(current_user.id)
    18:   else
    19:     render :edit
    20:   end
    21: end

 params
=> #<ActionController::Parameters {"_method"=>"patch", "authenticity_token"=>"nb5nlpahgaNruYVvgf52kxArsFN-rVosdXCCOG1uApeXw9cqlRvkhyD_PmCVuBj1iOqU28dn4-Hh5zrfm-Riag", "user"=>{"name"=>"yuu00", "profile"=>"カフェ巡りが", "avatar"=>#<ActionDispatch::Http::UploadedFile:0x00007f93289a8860 @tempfile=#<Tempfile:/var/folders/gw/k3mqyqh15y37h56g1b17ymvc0000gn/T/RackMultipart20210526-14898-uuwj6j.png>, @original_filename="初心者でも使用て開発ツール3選.png", @content_type="image/png", @headers="Content-Disposition: form-data; name=\"user[avatar]\"; filename=\"\xE5\x88\x9D\xE5\xBF\x83\xE8\x80\x85\xE3\x81\xA6\xE3\x82\x99\xE3\x82\x82\xE4\xBD\xBF\xE7\x94\3\x81\xA6\xE3\x82\x99\xE3\x81\x8D\xE3\x82\x8B\xE3\x82\xA2\xE3\x83\x95\xE3\x82\x9A\xE3\x83\xAA\xE9\x96\x8B\xE7\x99\xBA\xE3\x83\x84\xE3\x83\xBC\xE3\x83\xAB3\xE9\x81\xB8.png\"\r\nContent-Type: image/png\r\n">}, "commit"=>"Edit Profile", "controller"=>"users", "action"=>"update", "id"=>"1"} permitted: false>

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

開発環境
・rubymine
・ruby(3.0.1)
・Ruby on rails (6.1.3.1)

参考文献
qiita
qiita

Answer 1

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

current_user.idと比較するならば@user.idですね

deviseを用いている場合
@user.update(user_params)の代わりに@user.update_without_password(user_params)
(パスワードも更新する場合はupdate_with_password)
を使うことになります。

Comments

[yuu0000]

ありがとうございます

下記のように修正したところ、条件分岐ができるようになりました！

def edit
@user = User.find(params[:id])
unless @user == current_user
redirect_to user_path(@user)
end
end

ユーザーの編集については挙動確認してみます！

[yuu0000]

編集機能の挙動確認をしてみたところ、更新されず編集ページに遷移してしまいました、、

また、下記のように@userをcurrent_userと記述を変更してみても動作は同じでした。
挙動のイメージとしては、emaiとパスワードの編集とname、profile、avatarはプロフィール編集として分離させる実装です。

def update
@user = User.find(params[:id])
if current_user.update_without_password(user_params)
redirect_to user_path(current_user)
else
redirect_to edit_user_path(current_user)
end
end

[asm]

passwordに自分でvalidate掛けている場合にはpasswordを検証する条件を付けないといけません。

if: :password_required?
がちょうどいいと思いますが

[yuu0000]

検証する条件の時は、users_controllerに下記のように記述する方法でしょうか？
def password_required?
#判定ロジック
end

[asm]

devise 略, :validatable
しているのであればpassword_required?メソッドは存在している筈です。

https://github.com/heartcombo/devise/blob/5d5636f03ac19e8188d99c044d4b5e90124313af/lib/devise/models/validatable.rb#L60

[yuu0000]

ありがとうございます

password_required?が記述されているファイルが見つけられないのですが、こちらの記述が書かれているファイルはどちらになるのでしょうか、、

初歩的な質問で申し訳ありません、、

[asm]

devise gemの中のlib/devise/models/validatable.rbです

[yuu0000]

bundle open でgemの中身を確認する方法でお間違いないでしょうか？

[asm]

そうですね。
もう一度言いますが
devise 略, :validatableしているので
VlidatableはUserにインクルードしており、Userクラスにpassword_required?メソッドは存在する筈です

[yuu0000]

そうですね、

userクラスの確認をしてみたのですが、password_required?メソッドが見当たりませんでした、

インクルード済みなので基本的には存在するメソッドですよね。

コードは下記のように表記されておりました。

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

has_many :uploads
has_one_attached :avatar
with_options presence: true do
validates :email
validates :password, format: {with:/\A(?=.*?[a-z])(?=.*?\d)[a-z\d]+\z/i, message: "is invalid. Include both letters and numbers. Input half-width characters." }
validates :name
validates :profile, length: { maximum: 300}
validates :avatar
end

end

[asm]

Moduleをincludeしているわけですのでuser.rbにコードが存在している訳ではないです。

[yuu0000]

そうなんですね、
失礼いたしました、

では、user.rbに　password_required?メソッドを記述する方法で処理するんですね

[asm]

コードは(user.rbに)存在してないけど、メソッドは(Userクラスに)存在している筈です。
別に記述してもいいけど、deviseの方のvalidateの条件に使ってるメソッドなので
ヘタに上書きするとdeviseの方が壊れる可能性がありますね。

[yuu0000]

既にdeviseのgemに記述されているメソッドなんですね、
私の認識の誤りでした、、

[yuu0000]

コードを下記のように修正後、編集ができるようになりました。
allow_blank: trueを追加しました。

こちらのコードにした場合、別ページでemailとpasswordを変更する場合でも編集はできるのでしょうか？

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

has_many :uploads
has_one_attached :avatar
with_options presence: true do
validates :email
validates :password, format: {with:/\A(?=.*?[a-z])(?=.*?\d)[a-z\d]+\z/i, message: "is invalid. Include both letters and numbers. Input half-width characters." }, allow_blank: true
validates :name
validates :profile, length: { maximum: 300}
validates :avatar
end
end

[asm]

悪くないと思いますよ
必要な場面では別で空でないかがが検証されていますので

[yuu0000]

かしこまりました！

お忙しいところ誠にありがとうございました。
実装でご相談したい時に改めてよろしくお願いいたします！

Answer 2

エラーメッセージを表示するようにすることを薦めます、
おそらく validates :password, format: .... としているので、パスワードがない というエラーでしょう。
if: パスワードがある をつけるか、allow_nil: true するかしましょう。

Comments

[yuu0000]

ご連絡ありがとうございます！

そうですね、可能性としてパスワードとemailなどは別のページで編集する実装にしておりますが、deviseでのユーザー登録のためプロフィール用の編集ページを作成しても、プロフィール情報もdevise管理になっているためにバリデーションがかかり更新ができない状態になっているみたいですね、、