Ruby on Rails:在表单上为另一个模型添加表单

时间:2017-01-09 20:14:23

标签: ruby-on-rails ruby forms ruby-on-rails-3 ruby-on-rails-4

在我的Ruby on Rails应用程序中,我正在尝试实现Group和Contact之间的关系,其中一个组可以包含许多联系人,一个联系人可以是许多组的一部分。我正在使用一个名为Contactgroup的模型来处理这种关系,因此表格是:

Group (id, name)
Contact (id, firstname, surname)
Contactgroup (group_id, contact_id)

示例数据为:

Groups:
ID      Name
1       Singers
2       Drummers

Contacts:
ID      Firstname    Surname
1       Freddy       Mercury
2       Roger        Taylor
3       Kurt         Cobain
4       Dave         Grohl

Contact Groups:
Group_ID        Contact_ID
1               1
1               3
1               4
2               2
2               4

我要做的是获取它,以便当用户创建组时,他们可以选择要添加到该组的联系人。这意味着有组表单,用户可以在其中键入组名称,在此表单上,我想显示每个用户联系人的复选框,以便用户可以选择要添加到组中的联系人,以及何时他们点击提交新组将保存在组表中,新的联系人组记录将保存在Contactgroup表中。

这是app/views/groups/_form.html.erb代码:

<%= form_for @group do |f| %>

  <% if @group.errors.any? %>
    <div id="error_explanation">
      <h2>
        <%= pluralize(@group.errors.count, "error") %> prohibited this group from being saved:
      </h2>
      <ul>
        <% @group.errors.full_messages.each do |msg| %>
          <li><%= msg %></li>
        <% end %>
      </ul>
    </div>
  <% end %>

  <p>
    <%= f.label :name %><br>
    <%= f.text_field :name %>
  </p>

  <h2>Add members:</h2>
    <%= form_for([@group, @group.contactgroups.build]) do |f| %>
      <p>
        <%= f.collection_check_boxes(:contact_id, @contacts, :id, :firstname) %>
      </p>
      <p>
        <%= f.submit %>
      </p>
    <% end %>
  <p>
    <%= f.submit %>
  </p>

<% end %>

在这里,您可以看到我尝试使用的代码:

<h2>Add members:</h2>
    <%= form_for([@group, @group.contactgroups.build]) do |f| %>
      <p>
        <%= f.collection_check_boxes(:contact_id, @contacts, :id, :firstname) %>
      </p>
      <p>
        <%= f.submit %>
      </p>
    <% end %>
  <p>
    <%= f.submit %>
  </p>

<% end %>

我从rails指南(http://guides.rubyonrails.org/getting_started.html)得到了这个,但是我收到错误undefined method contactgroups'给#,并且不认为这会给我我想要的东西。

我的路线档案是:

Rails.application.routes.draw do
  get 'sessions/new'
  get 'sessions/create'
  get 'sessions/destroy'
  resources :users
  get 'welcome/index'
  root 'welcome#index'
  resources :contacts

  resources :groups do
    resources :contactgroups
  end
  resources :contactgroups


  get 'sessions/new'
  get 'sessions/create'
  get 'sessions/destroy'
  controller :sessions do
    get  'login' => :new
    post 'login' => :create
    get 'logout' => :destroy
  end 
end

我的groups_controller:

class GroupsController < ApplicationController
    def index
        @groups = Group.where(user_id: session[:user_id])
    end
    def show
        @group = Group.find(params[:id])
        @members = Contactgroup.where(group_id: @group.id)
    end
    def new
        @group = Group.new
        @contacts = Contact.where(user_id: session[:user_id])
    end
    def edit
        @group = Group.find(params[:id])
    end
    def create
        @group = Group.new(group_params)
        @group.user_id = session[:user_id]
        if @group.save
            redirect_to @group
        else
            render 'new'
        end
    end
    def update
        @group = Group.find(params[:id])
        if @group.update(group_params)
            redirect_to @group
        else
            render 'edit'
        end
    end
    def destroy
        @group = Group.find(params[:id])
        @group.destroy
        redirect_to groups_path
    end
    private
    def group_params
        params.require(:group).permit(:name, :user_id)
    end
end

和contactgroups_controller:

class ContactgroupsController < ApplicationController
    def destroy
        @contactgroup = Contactgroup.find(params[:id])
        @contactgroup.destroy
        redirect_to(:back)
    end
end

我的模型如下:

Contact.rb

class Contact < ActiveRecord::Base
end

Group.rb

class Group < ActiveRecord::Base
end

Contactgroup.rb

class Contactgroup < ActiveRecord::Base
  belongs_to :contact
  belongs_to :group
end

必须有一个简单的解决方案来解决这个问题,因为我认为它通常在其他系统上完成,但我不知道如何做到这一点。

有人可以帮忙。

3 个答案:

答案 0 :(得分:1)

您无法在表单内使用表单。使用collection_check_boxes的正确方法如下。

替换

<%= form_for([@group, @group.contactgroups.build]) do |f| %>
  <p>
    <%= f.collection_check_boxes(:contact_id, @contacts, :id, :firstname) %>
  </p>
  <p>
    <%= f.submit %>
  </p>
<% end %>

只需

<p>
  <%= f.collection_check_boxes(:contact_ids, @contacts, :id, :firstname) %>
</p>

答案 1 :(得分:0)

好的,问题很简单。您正在呼叫@group.contactgroups,但您尚未在group模型上实际设置该关联。仅从contactgroup方设置关联。因此,您可以contactgroup.group但不能group.contactgroups

你最好的选择是将其建模为habtm - 就像我之前提到的那样。这就是你如何做到的:

Contact.rb:

class Contact < ActiveRecord::Base
  has_and_belongs_to_many :groups 
end

Group.rb:

class Group < ActiveRecord::Base
  has_and_belongs_to_many :contacts
end

注意:您仍然拥有HABTM联系人组的概念,但使用Rails标准命名它将作为contacts_groups表存在于您的数据库中。然后你可以用这种方式构建你的表单。

使用快速谷歌,这里有一个关于使用HABTM复选框的S / O问题(尚未对其进行审查以证明您的情况有用):Rails 4 - checkboxes for has_and_belongs_to_many association

使用HABTM是Rails的标准做法,有很多很好的理由。它确实非常适合你的情况(诚实!)并且它实际上并没有打破你想要在SQL中看到它的要求(严重!)。

先尝试一下:) 我可以告诉你如何打破Rails约定......但是一般都很清楚你不应该违反约定,直到你知道它们的约定为止。

答案 2 :(得分:0)

这比最初的想法/建议简单得多。

我需要做的是将模型更改为:

Contactgroup
  belongs_to :contact
  belongs_to :group

Contact
  has_many :contactgroups
  has_many :groups, through: :contactgroups, :dependent => :destroy

Group
  has_many :contactgroups
  has_many :contacts, through: :contactgroups, :dependent => :destroy

groups_controller我需要将新方法和参数更改为:

def new
    @group = Group.new
    @group.contactgroups.build
end

private
def group_params
    params.require(:group).permit(:name, :user_id, { contact_ids: [] })
end

然后将以下代码行添加到app/views/groups/_form.html.erb

<%= f.collection_check_boxes :contact_ids, Contact.where(user_id: session[:user_id]), :id, :firstname ,{ prompt: "firstname" } %>

这为每个联系人提供了一个复选框,并允许从组表单创建联系人组记录。