所以我,一个铁杆新手,我正在尝试从当前的Devise会话中获取用户ID,而且我遇到了一些麻烦.
我现在在控制器中有这个:
def index @currentUser = current_user.id end
我想制作一个简单的if语句来显示/隐藏表单中的字段.
<% if @currentUser === @product.user_id %> <%= link_to "Back",products_path %> <%= link_to "Edit",edit_product_path(@product) %> <%= link_to "Delete",product_path(@product),method: :delete,data: { confirm: "Are you sure?"} %> <% else %> <span></span> <% end %>
我有一种感觉,我在定义currentUser变量时的语法很糟糕,但我不知道如何解决这个问题. Stack上有一些类似的问题,但没有一个真正适用于我或帮助过我.
提前致谢!
解决方法
我在这看到一些问题
def index @currentUser = current_user.id end
除了@HolyMoly已经评论过的内容之外,你应该使用下划线而不是camelcase,如果current_user在这里为nil,.id将失败,导致异常.
其次,你通过比较id的值检查“能力”,我会改变你的代码来做到这一点
<% if allow_product_delete?(@product) %>
在帮手
def allow_product_delete?(product) return false unless current_user @product.user_id == current_user.id end
如果你正在使用devise current_user存在于控制器和视图中,则不需要将其定义为实例变量,它已经被定义为所有操作的控制器方法(默认情况下).因此,通过在视图中调用current_user,您就完成了.
如果用户未登录,则current_user将为nil,您必须为此做好准备并防范它.
最后,我会研究能力宝石(cancan或cancancan),那些提供了一个非常好的DSL来处理你在这里尝试的东西.