如何在Rails中正确创建自定义操作？

Question

我试图创建自定义表单，允许管理员切换用户的用户权限，并将其保存到数据库中。我必须做一些完全错误的事情，因为页面生成良好，并且提交按钮提交给操作正常，但没有任何操作保存到数据库，它只是再次呈现我的初始视图。任何人都可以看到我做错了什么？

守则

路线：

resources :users do 
    member do 
    get 'assign' 
    put 'assign_update' 
    end 
end 
... 

控制器（这样做是为了规避的事实，管理员和其他属性都无法访问企图的这种奇怪的方式这可能是一个烂摊子。 ）：

... 
def assign 
    @user = User.find(params[:id]) 
end 

def assign_update 
    admin_protected = params[:user].delete(:admin) 

    @user = User.find(params[:id]) 
    @user.admin = admin_protected 

    if @user.save 
    flash[:success] = "User updated" 
    redirect_to users_path 
    else 
    render 'assign' 
    end 
end 

查看：

... 
<%= form_for(@user, url: { controller: 'users', 
    action: 'assign_update'}, method: 'put') do |f| %> 
    <%= f.label :admin, 'Is admin?', class: 'checkbox inline' %> 
    <%= f.check_box :admin %> 
    <%= f.submit "Save changes", class: "btn btn-large btn-primary" %> 
<% end %> 

Answer 1

因此，要总结我们在注释部分下的对话问题...

问题被发现是对其他用户模型属性的验证。用户实例无法保存，因为在执行save方法时验证未通过。结果，Rails只是渲染分配视图。

update_attribute方法更新一个属性，没有模型验证也没有质量分配保护。在这种情况下，admin属性适合这两个标准。

Answer 2

你应该在你的模型使用attr_accessible从PARAMS手工采摘这些领域拥有并赋予它们分别确保管理领域不包括在分配给attr_accessible声明

所以

字段列表

def assign_update 
    admin_protected = params[:user].delete(:admin) 

    @user = User.find(params[:id]) 
    @user.admin = admin_protected 

    if @user.save 
    flash[:success] = "User updated" 
    redirect_to users_path 
    else 
    render 'assign' 
    end 
end 

成为

def assign_update 
#  admin_protected = params[:user].delete(:admin) 

     @user = User.find(params[:id]) 
     @user.admin = params[:user][:admin] 

     if @user.save 
     flash[:success] = "User updated" 
     redirect_to users_path 
     else 
     render 'assign' 
     end 
    end 

的问题是，在你的方法，你的仍然是质量分配。

调试的是实际发生的log文件输出

UPDATE

检查错误列表中，您应该仔细看看。 以下内容添加到您的窗体

<% if @user.errors.any? %> 
    <div id="error_explanation"> 
     <h2><%= pluralize(@user.errors.count, "error") %> prohibited this account from being saved:</h2> 

     <ul> 
     <% @user.errors.full_messages.each do |msg| %> 
     <li><%= msg %></li> 
     <% end %> 
     </ul> 
    </div> 
    <% end %> 

它应该给你，广告用户的什么是错的一个清晰的概念，以及如何解决它