(Ruby) 会員登録でユーザー情報がしっかり登録されない。ログインされない問題

Question

お世話になっております。

現在、簡単な投稿サイトweb applicationを作成中で、新規会員登録やログインをしようと必要事項を入力後会員ページに行けないエラーが発生しております。

どうすれば、通常通り登録し、投稿画面（会員限定に制限しています。）へいけるでしょうか。

アドバイス宜しくお願い致します。

以下、コードになります。解決へむけて他に必要な項目があれば仰って頂ければと思います。

routes.rb

Rails.application.routes.draw do

  post "likes/:post_id/create" => "likes#create"
  post "likes/:post_id/destroy" => "likes#destroy"

  post "users/:id/update" => "users#update"
  get "users/:id/edit"=> "users#edit"
  post "users/create"=> "users#create"
  get "signup"=> "users#new"
  get "users/index"=> "users#index"
  get "users/:id"=> "users#show"
  post "login"=> "users#login"
  post "logout"=> "users#logout"
  get "login" => "users#login_form"

 
  get "users/:id/likes" => "users#likes"
  
  get "posts/index"=> "posts#index"
  get "posts/new"=> "posts#new"
  get "posts/:id" => "posts#show"
  post "posts/create"=> "posts#create"
  get "posts/:id/edit"=> "posts#edit"
  post "posts/:id/update" => "posts#update"
  post  "posts/:id/destroy" => "posts#destroy"

  get "/" => "home#top"
  get "about" => "home#about"
end 

application_controller.rb

class ApplicationController < ActionController::Base
  

    def set_current_user
        @current_user = User.find_by(id: session[:user_id])
    end 

    def authenticate_user
        if @current_user == nil 
            flash[:notice]= "ログインが必要です"
            redirect_to("/login")
        end 
    end 

    def forbid_login_user
        if @current_user
            flash[:notice]= "すでにログインしています"
            redirect_to("/posts/index")
        end
    end 

end 

users_controller.rb

class UsersController < ApplicationController
    before_action :authenticate_user, {only: [:index, :show, :edit, :update]}
    before_action :forbid_login_user, {only: [:new, :create, :login_form, :login]}
     before_action :ensure_correct_user, {only: [:edit, :update]}

    def index 
        @users= User.all
    end 

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

    def new 
        @user = User.new
    end

    def create 
        @user = User.new(
            name: params[:name],
            email: params[:email],
            image_name: "default_user.jpg",
            password: params[:password]
        )
        if @user.save
            session[:user_id] = @user.id
            flash[:notice]= "パワーの源入りました。どすこい。"
            redirect_to("/users/#{@user.id}")
        else
            render("users/new")
        end 
    end 

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


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

        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 

    def login_form
    end 

    def login
        @user= User.find_by(email: params[:email])
        if @user && @user.authenticate(params[:password])
            session[:user_id]= @user.id
            flash[:notice]= "ログインしました"
            redirect_to("/posts/index")
        else
            @error_message= "メールアドレスまたはパスワードが間違っています"
            @email= params[:email]
            @password= params[:password]
            render("users/login_form")
        end 
    end

      def logout 
        session[:user_id]= nil 
        flash[:notice]= "ログアウトしました"
        redirect_to("/login")
      end 

      def likes 
        @user= User.find_by(id: params[:id])
        @likes= Like.where(user_id: @user.id)
      end 

      def ensure_correct_user
        if @current_user.id != params[:id].to_i
            flash[:notice]= "今のあなたには不可能です。精進なさい。"
            redirect_to("/posts/index")
        end
    end 


end 

users/new.html.erb

<div class="main users-new">
 <div class="container">
   <div class="form-heading">新規ユーザー登録</div>
   <div class="form users-form">
    <div class="form-body">
     <% @user.errors.full_messages.each do |message| %>
        <div class="form-error">
         <%= message %>
        </div>
        <% end %>

        <%= form_tag("/users/create") do %>
            <p>ユーザー名</p>
            <input name="name" value="<%= @user.name %>">
            <p>メールアドレス</p>
            <input name= "email" value="<%= @user.email %>">
            <p>パスワード</p>
            <input type="password" name="password" value= "<%= @user.password %>" >
            <input type="submit" value="新規登録">
            <% end %>
        </div>
        </div>
        </div>
        </div>

users.login_form.html.erb

<div class= "main users-new">
 <div class="container">
  <div class="form-heading">ログイン</div>
   <div class="form users-form">
    <div class="form-body">
     <% if @error_message %>
      <div class= "form-error">
        <%= @error_message %>
    </div>
    <% end %>

     <%= form_tag("/login") do %>
     <p> メールアドレス</p>
     <input name= "email" value= "<%= @email %>">
     <p>パスワード</p>
     <input type="password", name="password", value="<%= @password %>">
     <input type="submit" value="ログイン">
     <% end %> 
    </div>
    </div>
    </div>
    </div>

users/edit.html.erb

<div class= "main users-edit">
 <div class= "container">
  <div class="form-heading">応援を強化する </div>
   <div class= "form users-from">
    <div class="form-body">
     <% @user.errors.full_messages.each do |message| %>
        <div class= "form-error"> 
        <%= message %>
        </div>
     <% end %>

    <%= form_tag("/users/#{@user.id}/update", {multipart: true}) do %>
    <p>ユーザー名</p>
    <input name="name" value="<%= @user.name %>">
    <p>画像</p>
    <input name= "image" type= "file">
    <p>メールアドレス</p>
    <input name= "email" value="<%= @user.email %>">
    <input type= "submit" value="保存">
    <% end %>

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

Models/User.rb

class User < ApplicationRecord
    
    has_secure_password
    validates :name, {presence: true}
    validates :email, {presence: true, uniqueness: true}
  
     def posts
        return Post.where(user_id: self.id)
     end 
     
    end 

追加情報　"Userが登録されたかどうかどう確認したか
「会員ページに行けない」とは、どういう操作をしたらどうなったのか
どういう操作：どのview のどこをどうしたか
は？" に関して

A. 以下、確認の流れを再現

まず、この1枚目の写真が自分が作っているデモのアプリケーションのホーム画面です。
右上に新規登録とログインがあり、投稿をする、投稿を閲覧するには会員登録が必要な仕様です。

2枚目の写真のように、新規登録をしてみます。
パスワードは "666"と設定しました。
新規登録が完成すれば、投稿一覧ページにいけるはずですが。。

3枚目の写真のように、”ログインが必要です”とログイン画面に飛ばされてしまいました。
新規登録が終わったら、そのまま投稿一覧ページに行って欲しいです（問題箇所1 ）

ログイン画面に飛ばされてしまったので、4枚目の写真のように、先ほど新規登録したメールアドレスと、パスワードを入力してログインし投稿一覧ページに行きたいです。

5枚目のように、正しい情報でログインを試みるも、”ログインが必要です”というflashとともに、またログイン画面が出てきてしまいました（問題箇所2 )

6枚目のように、もしかしたらUserの新規登録がしっかりと出来ていないのか？と思い、新規登録画面に戻り、先ほどと同じ情報で登録ボタンを押してみました。
ここで既にUserとして情報が入っていれば、登録内容の重複でエラーが起きるはずです。

7枚目のように、”既に登録されているよ”と出てきました。
従って、User情報はしっかりと登録されているけれど、会員のログインをして、投稿画面に行けない状態になっています。

以上が、確認の流れになります。

宜しくお願い致します。

Comments

[winterboum]

不足情報
model User のcode
Userが登録されたかどうかどう確認したか
「会員ページに行けない」とは、どういう操作をしたらどうなったのか
どういう操作：どのview のどこをどうしたか

[yoshidesu]

winterboumさん　ありがとうございます。
質問文の中にmodels/user.rbのコードと、添付画像を追加させて頂きました！

[winterboum]

Userが登録されたかどうかどう確認したか
「会員ページに行けない」とは、どういう操作をしたらどうなったのか
どういう操作：どのview のどこをどうしたか
は？

[yoshidesu]

ありがとうございます。
以上の内容を本文の追加情報に記載しました

Answer 1

class UsersController < ApplicationController before_action :authenticate_user,
なので、loginを要求されます。
crateしてredirectのまえに 「login成功したときに行うこと」を実行すれば良いでしょう

def set_current_user もしくはそれ相当の処理がないので、@current_userは常に nilです

Comments

[yoshidesu]

無事解決できました！！