尝试使用CURRENT_USER它是未定义

Question

我有这个呼叫我的投票模式：

fires :vote_updated, :on => :update, 
        :actor => :user, 
        :secondary_subject => :video, 
        :if => lambda { |vote| ((vote.value == 1) || (vote.value == -1)) && (vote.video.user != current_user)} 

如果你不熟悉，它的工作原理与timeline_fu plugin。

如果拥有投票视频的用户是当前用户，我不希望该电话被解雇。这就是这条线进来：

:if => lambda { |vote| ((vote.value == 1) || (vote.value == -1)) && (vote.video.user != current_user)} 

不过，我没有current_user这里访问。我如何解决这个问题？

下面是我的票控制器创建方法（实际上有没有更新的方法）：

def create  
    @video = Video.find(params[:video_id]) 
    @vote = current_user.video_votes.find_or_create_by_video_id(@video.id) 

    if @vote.value.nil? 
    if params[:type] == "up" 
     @vote.value = 1 
    else 
     @vote.value = -1 
    end 
    elsif (params[:type] == "up" && @vote.value == 1) || (params[:type] == "down" && @vote.value == -1) 
    @vote.value = 0 
    elsif ((params[:type] == "up" && @vote.value == -1) || (params[:type] == "down" && @vote.value == 1)) || (@vote.value == 0) 
    if params[:type] == "up" 
     @vote.value = 1 
    else 
     @vote.value = -1 
    end 
    end 

    if @vote.save 
    respond_to do |format| 
     format.html { redirect_to @video } 
     format.js 
    end 
    else 
    respond_to do |format| 
     format.html 
     format.js 
    end 
    end 
end 

Answer 1

我相信这样做的控制器将确认这是正确的。我将过滤器之前创建一个针对这种情况

UPDATE：

就像一个简单的例子：

before_filter :valid_vote, :only => :update 

def update 
    @vote.update_attributes(params[:vote]) # or whatever 
end 
.. 

private 

def valid_vote 
    @vote = Vote.find params[:id] 
    unless (@vote.video.user.id != current_user.id) 
     render :text => 'You can't vote for your own video', :status => 403 
    end 
end 

所以@vote在声明并处理连线您的“更新”行动之前验证。 如果它不是有效的，那么你的“更新”行动保持不变

更新2：

不知道你会喜欢它，但你也可以做如下：

在投票模型

：

attr_accessor :skip_timeline 

然后用与之前过滤器的概念，但做@vote.skip_timeline = true，而不是渲染文本

则声明可能如下：

:if => lambda { |vote| ((vote.value == 1) || (vote.value == -1)) && !vote.skip_timeline } 

你也可以移动((vote.value == 1) || (vote.value == -1))您之前过滤：

def valid_vote 
    @vote = Vote.find params[:id] 
    unless ([1,-1].include? @vote.value && @vote.video.user.id != current_user.id) 
     @vote.skip_timeline = true 
    end 
end 

和

：如果=>拉姆达{|投票| ！vote.skip_timeline}

Answer 2

您会收到此错误消息，因为通常不建议您访问模型中的current_user（或会话信息）。我不是那么熟悉timeline_fu宝石，所以这个答案不会是你可能得到的最好答案。我只是要告诉你如何从任何模型访问current_user。

首先转到您的应用程序控制器。你会想制作一个设置当前用户的方法。您需要在之前的过滤器中调用该方法。

before_filter :loadCurrentUser 

def loadCurrentUser 
  User.currentUser = current_user 
end 

然后在您的用户模型中，您需要定义'currentUser'。

def self.currentUser 
    Thread.currentUser[:user] 
end 

你不一定要声明在应用程序控制器的CURRENT_USER，但因为它是一块宝石，我不知道，如果它有一个方便的控制器。

编辑：这种方式可能容易出现问题，但我不完全确定您是否在问如何使current_user在模型中可用或完全不同的解决方法，以便您没有该问题......并阅读另一个答案的回答，我想这不是你问的。