Rails fields_for拉取模型名称而不是params id的ID

时间:2014-05-21 16:15:42

标签: ruby-on-rails ruby formbuilder fields-for

我使用fields_for创建一个页面,用于在我的应用中编辑多个用户对象。我一直在使用这个Railscast视频作为参考。 http://railscasts.com/episodes/198-edit-multiple-individually我遇到的问题是,不是在生成html时使用用户ID,而是使用表单生成器插入用户名。我不知道为什么。下面是我的控制器,视图和一些生成的html。

查看

 Manage Your User Licenses
User License Management for <%= @org.title %>
<%= form_tag update_many_user_licenses_path, :method => :put do %>
  <% for user in @users %>
    <%= fields_for "users[]", user do |user_fields| %>
      <div><%= user.name %></div>
      <div><%= user_fields.label :active_license %>
      <%= user_fields.check_box :active_license %></div>
    <% end %>
  <% end %>
  <%= submit_tag "Mark as Complete" %>
<% end %>

控制器

class UserLicensesController < ApplicationController
  before_filter :authenticate_user!

  def manage
    @org = current_user.organization
    @users = @org.users
  end
  def update_many
    @users = User.find(params[:users])
    @users.each do |user|
      user.update_attributes!(params[:user].reject { |k,v| v.blank? })
    end
    flash[:notice] = "Updated users licenses."
    redirect_to manage_user_licenses_path
  end
end   

HTML

<div>Tyrel Denison</div>
<div><label for="users_Tyrel-Denison_active_license">Active license</label>
<input name="users[Tyrel-Denison][active_license]" type="hidden" value="0" /><input   id="users_Tyrel-Denison_active_license" name="users[Tyrel-Denison][active_license]" type="checkbox" value="1" /></div>

1 个答案:

答案 0 :(得分:1)

虽然我不熟悉LinkedIn API Gem,但根据我们的讨论,似乎User模式中的to_param方法被覆盖了。

从评论中,这显然是您的to_param方法的样子:

def to_param  # TODO USe something better than this plz  
    self.name.gsub(/\s+/, '-')  
end

您报告而不是在生成html时使用用户ID,表单构建器正在插入用户的名称,正如您所看到的,这正是发生在那种方法。最终,fields_for会调用to_param,而您在这方面没有选择权。

我会非常惊讶LinkedIn gem需要你覆盖那个方法,因为它肯定会破坏任何使用它的模型,正如你所经历的那样(我无法解释为什么其他开发者决定这样做) - 他们可能无辜地编写方法而没有意识到它已经存在于基类中了)

我无法在不仔细检查系统的情况下为您提供更详细的信息,但我建议尝试查找/ to_param的使用方式(假设它在某处明确调用),并且改为创建一个单独的方法。