Доступ к current_user в модели

У меня есть 3 таблицы

items (columns are:  name , type)
history(columns are: date, username, item_id)
user(username, password)

Когда пользователь говорит, что "ABC" входит в систему и создает новый элемент, запись истории создается со следующим фильтром after_create. Как назначить это имя пользователя ABC в поле имени пользователя в таблице истории через этот фильтр.

class Item < ActiveRecord::Base
  has_many :histories
  after_create :update_history
  def update_history
    histories.create(:date=>Time.now, username=> ?) 
  end
end

Мой метод входа в session_controller

def login
  if request.post?
    user=User.authenticate(params[:username])
    if user
      session[:user_id] =user.id
      redirect_to( :action=>'home')
      flash[:message] = "Successfully logged in "
    else
      flash[:notice] = "Incorrect user/password combination"
      redirect_to(:action=>"login")
    end
  end
end

Я не использую плагин аутентификации. Я был бы признателен, если бы кто-нибудь мог сказать мне, как этого достичь, не используя плагин (например, userstamp и т.д.), Если это возможно.

Ответы

Ответ 1

Рельсы 5

Объявить модуль

module Current
  thread_mattr_accessor :user
end

Назначить текущего пользователя

class ApplicationController < ActionController::Base
  around_action :set_current_user
  def set_current_user
    Current.user = current_user
    yield
  ensure
    # to address the thread variable leak issues in Puma/Thin webserver
    Current.user = nil
  end             
end

Теперь вы можете обратиться к текущему пользователю как Current.user

Документация о thread_mattr_accessor

Рельсы 3,4

Это не обычная практика доступа к current_user в рамках модели. При этом, вот решение:

class User < ActiveRecord::Base
  def self.current
    Thread.current[:current_user]
  end

  def self.current=(usr)
    Thread.current[:current_user] = usr
  end
end

Установите атрибут current_user в around_filter ApplicationController.

class ApplicationController < ActionController::Base
  around_filter :set_current_user

  def set_current_user
    User.current = User.find_by_id(session[:user_id])
    yield
  ensure
    # to address the thread variable leak issues in Puma/Thin webserver
    User.current = nil
  end             
end

Задайте current_user после успешной проверки подлинности:

def login
  if User.current=User.authenticate(params[:username], params[:password])
    session[:user_id] = User.current.id
    flash[:message] = "Successfully logged in "
    redirect_to( :action=>'home')
  else
    flash[:notice] = "Incorrect user/password combination"
    redirect_to(:action=>"login")
  end
end

Наконец, обратитесь к current_user в update_history из Item.

class Item < ActiveRecord::Base
  has_many :histories
  after_create :update_history
  def update_history
    histories.create(:date=>Time.now, :username=> User.current.username) 
  end
end

Ответ 2

Контроллер должен указать экземпляр модели

Работа с базой данных - это модельное задание. Обработка веб-запросов, в том числе знание пользователя для текущего запроса, является заданием контроллера.

Следовательно, если экземпляр модели должен знать текущего пользователя, контроллер должен сказать ему.

def create
  @item = Item.new
  @item.current_user = current_user # or whatever your controller method is
  ...
end

Это предполагает, что Item имеет attr_accessor для current_user.

Ответ 3

Если пользователь создает элемент, не должен ли элемент иметь предложение belongs_to :user? Это позволит вам в after_update сделать

History.create :username => self.user.username

Ответ 4

Вы можете написать round_filter в ApplicationController

around_filter :apply_scope 

def apply_scope 

  Document.where(:user_id => current_user.id).scoping do 
  yield 

end

Ответ 5

Тройка Thread не является потокобезопасной, по иронии судьбы.

Мое решение состояло в том, чтобы переместить стек назад, ища фрейм, который отвечает на current_user. Если ни один не найден, он возвращает nil. Пример:

def find_current_user
  (1..Kernel.caller.length).each do |n|
    RubyVM::DebugInspector.open do |i|
      current_user = eval "current_user rescue nil", i.frame_binding(n)
      return current_user unless current_user.nil?
    end
  end
  return nil
end

Это можно сделать более надежным, подтвердив ожидаемый тип возвращаемого значения, и, возможно, подтвердив, что владелец кадра является типом контроллера...