Рельсы logged_in? метод, не меняющий панель навигации

Question

У меня есть панель навигации, которая отображает другой набор ссылок, основываясь на том, вошел ли пользователь в систему или нет. Я установил logged_in? метод в условном выражении для достижения этой цели. Проблема заключается в том, что после того, как я нажал кнопку входа в систему и успешно вошел в систему. Система не изменяет панель навигации должным образом. Я должен обновить страницу или щелкнуть другую ссылку, чтобы зарегистрировать ее.

application.html.erb

<% if logged_in? %>
    <li><%= link_to "Oppurtunity", companies_path %></li>
    <li><%= link_to "Pipeline", deals_path %></li>
    <li><%= link_to "Tasks", tasks_path %></li>
    <li><%= link_to "My Account", edit_user_path(current_user) %></li>          
    <li><%= link_to "Log Out", logout_path, method: :delete, class: 'btn btn-default navbar-btn' %></li>
<% else %>
    <li><%= link_to "Log In", login_path, class: 'btn btn-default navbar-btn' %></li>
    <li><%= link_to "Contact", new_contact_path %></li>
<% end %>

session_helper.rb

 # Returns the current logged-in user (if any).
def current_user
  if (user_id = session[:user_id])
    @current_user ||= User.find_by(id: user_id)
  elsif (user_id = cookies.signed[:user_id])
    user = User.find_by(id: user_id)
    if user && user.authenticated?(:remember, cookies[:remember_token])
      log_in user
      @current_user = user
    end
  end
end

def logged_in?
    !current_user.nil?
end

session_controller.rb

class SessionsController < ApplicationController

def new
  end

  def create
    user = User.find_by(email: params[:session][:email].downcase)
    if user && user.authenticate(params[:session][:password])
     if user.activated?
      # Log the user in and redirect to the user's show page.
      log_in user
      params[:session][:remember_me] == '1' ? remember(user) : forget(user)
      redirect_to root_url(subdomain: user.subdomain), notice: "Logged in!"
     else
      message  = "Account not activated."
      message += "Check your email for the activation link."
      flash[:warning] = message
      redirect_to root_url
    end
  else
    flash.now[:danger] = 'Invalid email/password combination'
    render 'new'
  end
end

  def destroy
    log_out if logged_in?
    redirect_to root_url(subdomain: false)
  end
end

Answer 1

Вы создали current_user логика неверна. При сравнении любых двух объектов вам нужно использовать == но вы используете =, Попробуйте изменить current_user метод ниже

def current_user
  if (user_id == session[:user_id]) #here
    @current_user ||= User.find_by(id: user_id)
  elsif (user_id == cookies.signed[:user_id]) #and here
    user = User.find_by(id: user_id)
    if user && user.authenticated?(:remember, cookies[:remember_token])
      log_in user
      @current_user = user
    end
  end
end