基于Rails中的链接表字段分配一个实例变量

我试图创建一个系统来检查当前用户是否有权编辑记录。我想这样做的方法是将@user的变量赋值给显示记录的链接ID和重定向的私有函数，除非@user = current_user，但是当我这样做时，我每次都会得到重定向行为（即使在用户拥有该记录）。基于Rails中的链接表字段分配一个实例变量

这里是控制器设置

def edit 
    @deal = Deal.find(params[:id]) 
    @user = User.where(params[:id] === @deal.user_id) 
    correct_user 
    end 
... 
private 

    def correct_user 
    unless @user === current_user 
     redirect_to root_path 
     flash[:error] = 'You can only edit deals that you own' 
    end 
    end

用户和交易模型是通过有许多相关的，拥有

class User < ApplicationRecord 
    has_many :deals 
... 
class Deal < ApplicationRecord 
    belongs_to :user

而对于交易的模式（用户ID是在创建分配给user_id）

create_table "deals", force: :cascade do |t| 
    t.text  "headline" 
    t.string "matter" 
    t.text  "summary" 
    t.integer "user_id" 
    t.datetime "created_at", null: false 
    t.datetime "updated_at", null: false 
    t.index ["user_id"], name: "index_deals_on_user_id" 
    end

该项目设置与设计，所以这些助手是也提供。

来源

2017-03-22 oneWorkingHeadphone

使用==代替===你只是想current_user

@user == current_user

比较@user看到这里的区别=== vs. == in Ruby

你也需要改变

@user = User.where(params[:id] === @deal.user_id)

到

@user = @deal.user

来源

2017-03-22 12:56:04

这得到了它，谢谢！ – oneWorkingHeadphone

好了，从哪里开始？

首先，不要使用===比较记录，使用==代替。

其次：

def edit 
    @deal = Deal.find(params[:id]) 
    @user = User.find_by(params[:id]: @deal.user_id) 
    correct_user 
end

当试图找到用户，您需要提供AR选项哈希，而不是布尔参数。如果要在记录未找到时引发异常，或者使用find_by简单地返回nil，则使用find。

并在correct_user再次使用==而不是===。

但是这个代码仍然不是很漂亮，可以使用一些重构;）

来源

2017-03-22 12:59:04 Esse

基于Rails中的链接表字段分配一个实例变量

回答

相关问题