NoMethodError в SessionsController#уничтожить неопределенный метод `забыть' к нулю:NilClass


Работа через рельсы учебник по M. Hartl Ch 8 Вход, Выход

При попытке выйти из приложения я получаю сообщение об ошибке, приведенное в заголовке моего вопроса.

Я очень новичок в этом, но я думаю, что это означает, что метод экземпляра forget, определенный в models/user.rb и вызываемый в SessionsHelper, вызывается на user, который по какой-то причине равен нулю. Вот фрагмент, о котором я говорю в SessionsHelper:

#Found in SessionHelper.  Forgets persistent session
def forget(user)
    user.forget
    cookies.delete(:user_id)
    cookies.delete(:remember_tolken)
end

А вот определение экземпляра forget метод:

#Found in class models/user.rb
def forget
  update_attribute(:remember_digest, nil)
end

Я не знаю, почему пользователь будет равен нулю при вызове SessionsHelper. Вот полные файлы, включая SessionsController, SessionsHelper, и User.

Я надеюсь, что был ясен и использовал правильную терминологию. Дайте мне знать, если мне понадобится уточнить.
class SessionsController < ApplicationController
  def new
  end

  def create
    user = User.find_by(email: params[:session][:email].downcase)
    if user && user.authenticate(params[:session][:password])
        log_in user # Log the user in and redirect to the user's showpage
        params[:session][:remember_me] == '1' ? remember(user) : forget(user)
        redirect_to user # same as user_url(user)
    else
        # Show error message
        flash.now[:danger] = 'Wrong email or password' 
        render 'new'
    end
    end

    def destroy
    log_out # if logged_in?    # not logging out
    redirect_to root_url
  end
end

XXXXXXXXXX

module SessionsHelper

# Logs in given user
def log_in(user)
    session[:user_id] = user.id
end

# Remembers a user in a persistant session
def remember(user)
    user.remember
    cookies.permanent.signed[:user_id] = user.id
    cookies.permanent[:remember_token] = user.remember_token
end

#Forgets persistent session
def forget(user)
    user.forget
    cookies.delete(:user_id)
    cookies.delete(:remember_tolken)
end


# Returns the current logged-in user (if any)
def current_user
    if (user_id = session[:user_id]) # CAREFUL! (not ==). Read if session of user_id exists
        @current_user ||= User.find_by(id: user_id)
    else (user_id = cookies.signed[:user_id])
        user = User.find_by(id: user_id)
        if user && user.authenticated?(cookies[:remember_token])
            log_in(user)
            @current_user = user
        end
    end
end

# Returns true if the user is logged-in, else false
def logged_in?
    !current_user.nil?
end

# logs out current user
def log_out
    session.delete(:user_id)
    forget(current_user)
    @current_user = nil
end
end

XXXXXXXXXX

class User < ActiveRecord::Base
attr_accessor :remember_token

  before_save {email.downcase!}
  validates :name, presence: true, length: {maximum: 50}
  VALID_EMAIL_REGEX = /A[w+-.]+@[a-zd-]+(.[a-zd-]+)*.[a-z]+z/i
  validates :email, presence: true, length: {maximum: 255},
                    format: { with: VALID_EMAIL_REGEX },
                    uniqueness: {case_sensitive: false}

  has_secure_password
  validates :password, length: {minimum: 6}

  # Returns the hash digest of the given string. Used to make hashed password of user in fixture model for login integration testing                                                        
  def User.digest(string)
    cost = ActiveModel::SecurePassword.min_cost ? BCrypt::Engine::MIN_COST :
                                                  BCrypt::Engine.cost
    BCrypt::Password.create(string, cost: cost)
  end

  # Returns random token for remembering users via cookies
  def User.new_token
    SecureRandom.urlsafe_base64
  end

  # Remembers a user in the database for use in persisten sessions
  def remember
    self.remember_token = User.new_token
    update_attribute(:remember_digest, User.digest(remember_token))
  end

  # Returns true if the given token matches the digest
  def authenticated?(remember_token)
    return false if remember_digest.nil?
    BCrypt::Password.new(remember_digest).is_password?(remember_token)
  end

  # Forgets a user
  def forget
    update_attribute(:remember_digest, nil)
  end

end

Спасибо

2 2

2 ответа:

Ваш метод log_out заключается в удалении сеансадо того, как забудет пользователя. Это приводит к ошибке, так как forget принимает current_user в качестве параметра и current_user использует сеанс для определения текущего пользователя:

    if (user_id = session[:user_id]) # This will attempt to assign user_id based on the value of session[:user_id], but will fail since the session was deleted

Вам нужно удалить сеанс после того, как забыли пользователя. Измените свой метод log out следующим образом:

def log_out
    session.delete(:user_id)
    forget(current_user)
    @current_user = nil
end

К этому:

def log_out
    forget(current_user)
    session.delete(:user_id)
    @current_user = nil
end

Метод current_user возвращает nil, и это отбрасывает ошибку дальше по строке.

В вашем методе current_user есть опечатка. Измените else на elsif, и это, скорее всего, исправит проблему.

Если это не исправит его, убедитесь, что метод точно такой же, как в листинге 8.57 учебника rails ; то же самое включено ниже:

# Returns the user corresponding to the remember token cookie.
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?(cookies[:remember_token])
      log_in user
      @current_user = user
    end
  end
end