ユーザー情報編集時のURLへのアクセス制限のかけ方について

Question

ProgateでTwitterのようなアプリを作っているのですが、ユーザー情報の更新時のURLに対してアクセス制限をかけようと思ったのですが、自分のやり方だと制限がかからなかったので質問させていただきました。更新時のURLというのは、ユーザー情報を編集するページ自体のURLではなく、更新をした瞬間のURL、いわゆるviewのないURLのことです。(うまく説明できずすみません)

上手くいかない点に関して下に具体的に書いたので、なぜ自分のやり方だとうまくいかないのかアドバイスいただきたいです。因みに正しいやり方については既に見つかっておりますので、下のやり方が何故ダメなのかについてのみご教授いただければ幸いです。

※アクセス制限というのはどんな制限かというと、ある特定のユーザーが自分自身以外のユーザの編集を行うことができないようにするという制限のことです。

うまくいかない点

下のrailsコード欄の中の　@user.id != @current_user.id　を条件としたif文が制限を試みているコードの部分です。しかし、更新時のURLにアクセスしてみたところ、redirect_toが働いてないからかずっとページ読み込み中のような状態になってしまいうまくいきません。

※　@userには編集ページのidをもとにデータベースから取得したユーザーのデータが入っており、@current_userにはbefore_actionにおいてsession[:user_id]を代入しておくことで、ログイン中のユーザーのidが入っています。

def update
    @user = User.find_by(id: params[:id])
    @user.name = params[:name]
    @user.email = params[:email]
    
    if @user.id != @current_user.id
      flash[:notice] = "権限がありません"
      redirect_to("/posts/index")
    end
    
    if params[:image]
      @user.image_name = "#{@user.id}.jpg"
      image = params[:image]
      File.binwrite("public/user_images/#{@user.image_name}", image.read)
    end

    if @user.save
      flash[:notice] = "ユーザー情報を編集しました"
      redirect_to("/users/#{@user.id}")
    else
      render("users/edit")
    end
  end

Answer 1

@current_userにはbefore_actionにおいてsession[:user_id]を代入しておくことで、ログイン中のユーザーのidが入っています。

@current_userにはuser_idが入っているということなので、

if @user.id != @current_user

ではないでしょうか？

Comments

[FreddyMarcury]

回答ありがとうございます。すみません自分の文章が間違っていて、@current_userにはデータベースのユーザー情報が入っており、idだけではありませんでした。そのため、@current_user.idに関しては間違っていないことになります。質問文の方でそのように書いてしまい大変申し訳ございません。