如何通过关联更新Has_Many的记录

Question

我有三个模型 - User，Client，Topic。

用户

has_many :clients
has_many :topics, :through => :clients, :uniq => true

客户

has_and_belongs_to_many :topics

主题

has_and_belongs_to_many :clients

我正在尝试做的是在客户的edit视图上，更改该客户的主题。

这是我的客户端控制器的更新操作：

  def update
        if params[:topic_ids]
             @client = current_user.clients.find(params[:id])
             @client.topic_ids = params[:client][:topic_ids]
             @client.save
        else
             @client = current_user.clients.find(params[:id])
        end

    respond_to do |format|
      if @client.update_attributes(params[:client])
        format.html { redirect_to @client, notice: 'Client was successfully updated.' }
        format.json { head :no_content }
      else
        format.html { render action: "edit" }
        format.json { render json: @client.errors, status: :unprocessable_entity }
      end
    end
  end

日志如下所示：

Started PUT "/clients/6" for 127.0.0.1 at 2012-10-07 18:56:14 -0500
Processing by ClientsController#update as HTML
  Parameters: {"utf8"=>"✓", "authenticity_token"=>"J172mxxCX0OdxcGm4GSPv8=", "client"=>{"name"=>"Testeeee Johnson", "email"=>"[e-mail]", "phone"=>"4320981234", "firm_id"=>"1", "personal_priority"=>"1", "last_contact"=>"2012-06-08", "vote"=>"1", "vote_for_user"=>"0", "next_vote"=>"2012-10-10", "vote_ii"=>"0", "vote_ii_for_us"=>"0"}, "topic_ids"=>["2"], "commit"=>"Update Client", "id"=>"6"}
  User Load (0.2ms)  SELECT "users".* FROM "users" WHERE "users"."id" = 1 LIMIT 1
  Client Load (0.1ms)  SELECT "clients".* FROM "clients" WHERE "clients"."user_id" = 1 AND "clients"."id" = ? LIMIT 1  [["id", "6"]]
  Topic Load (0.2ms)  SELECT "topics".* FROM "topics" INNER JOIN "clients_topics" ON "topics"."id" = "clients_topics"."topic_id" WHERE "clients_topics"."client_id" = 6
   (0.1ms)  begin transaction
   (0.0ms)  commit transaction
   (0.0ms)  begin transaction
   (0.0ms)  commit transaction
   (0.0ms)  begin transaction
   (0.0ms)  commit transaction
Redirected to http://localhost:3000/clients/6
Completed 302 Found in 8ms (ActiveRecord: 0.8ms)

不用说，它不会更新client.topics的记录。

如何更新客户记录的topics属性？

编辑 1

这是_form部分的外观：

<%= form_for(@client) do |f| %>
  <% if @client.errors.any? %>
    <div id="error_explanation">
      <h2><%= pluralize(@client.errors.count, "error") %> prohibited this client from being saved:</h2>
      <ul>
      <% @client.errors.full_messages.each do |msg| %>
        <li><%= msg %></li>
      <% end %>
      </ul>
    </div>
  <% end %>
  <div class="field">
    <%= f.label :name %><br />
    <%= f.text_field :name %>
  </div>
  <div class="field">
    <%= f.label :email %><br />
    <%= f.text_field :email %>
  </div>
  <div class="field">
    <%= f.label :firm %><br />
    <%= f.select :firm_id, Firm.all.collect { |firm| [firm.name, firm.id] }, {:include_blank => 'None'} %>
  </div>
  <div class="field">
  <h4>Topics</h4>  
    <% Topic.all.each do |topic| %>
      <% checked = @client.topics.include?(topic) %>
        <%= f.label(:name, topic.name) %> <%= check_box_tag "topic_ids[]", topic.id, checked %>
    <% end %>
  </div>
.
. - reduced for brevity
.

  <br /><br />
  <div class="actions">
    <%= f.submit %>
  </div>
<% end %>

Answer 1

无论

出于何种原因，form助手都不适用于check_box。

因此，这是有效的代码：

<%= check_box_tag "client[topic_ids][]", topic.id, checked %>

根据类似问题的其他答案，帮助程序f.check_box是模型绑定的，提供给复选框的值是从窗体上的模型隐式的。问题是，我无法弄清楚如何获取form_helper的隐式值以生成正确的标签 - 即 client[topic_ids][]，所以我不得不求助于check_box_tag。

Answer 2

您在

下面的评论中提到，您还想为每个主题添加权重。这在has_and_belongs_to_many关联中不再受支持，因此您应该使用 has_many ：through 关联：

class User < ActiveRecord::Base
  has_many :clients
end
class Clients < ActiveRecord::Base
  belongs_to :user
  has_many :client_topics
  has_many :topics, :through => :clients_topics
end
# Create this table like any other, with a "weight" field
class ClientsTopics < ActiveRecord::Base
  belongs_to :client
  belongs_to :topic
end
class Topics < ActiveRecord::Base
  has_many :clients_topics
  has_many :clients, :through => :clients_topics
end

现在，您的更新将需要删除所有现有clients_topics，然后遍历传递的topic_ids和权重，并将它们添加到客户端，如下所示：

def update
  if params[:topic_ids]
     @client = current_user.clients.find(params[:id])
     @client.clients_topics.delete_all
     params[:client][:topic_ids].each_with_index do |topic_id, index|
       weight = params[:client][:weights][index]
       @client.clients_topics.create!(:topic_id => topic_id, :weight => weight)
     end
  else
     @client = current_user.clients.find(params[:id])
  end
  (etc...)
end