2010-06-19 180 views
3

艺术家有许多事件。活动有许多艺术家。这两个模型之间的连接称为表演。rails验证嵌套属性

当前“活动”表单创建“绩效”,但为添加到“活动”表单的每位艺术家创建一位新艺术家。

我想事件的形式:

  1. 验证一个艺术家只能被添加到一个事件一旦
  2. 如果具有相同名称的艺术家的艺术家表已经存在,创建协会在连接表(表演),但不创造另一个艺术家
  3. 如果具有相同名称的艺术家已经不存在,创建和性能

我试过加将'validates_uniqueness_of:name'赋给artist.rb,但是这会阻止事件被保存。如果连接(表演)不存在,应该创建连接(表演),如果艺术家尚不存在,应创建艺术家,但艺术家的存在不应阻止创建连接/关联。

event.rb

validates_presence_of :name, :location 
has_many :performances, :dependent => :destroy 
has_many :artists, :through => :performances 
accepts_nested_attributes_for :artists, :reject_if => proc {|a| a['name'].blank?},  :allow_destroy => true 

artist.rb

has_many :performances 
has_many :events, :through => :performances 

perfomance.rb

belongs_to :artist 
belongs_to :event 

events_controller.rb

def create 
    @event = Event.new(params[:event]) 

    respond_to do |format| 
    if @event.save 
     flash[:notice] = 'Event was successfully created.' 
     format.html { redirect_to(admin_events_url) } 
     format.xml { render :xml => @event, :status => :created, :location => @event } 
    else 
     format.html { render :action => "new" } 
     format.xml { render :xml => @event.errors, :status => :unprocessable_entity } 
    end 
    end 
end 

_form.html.erb

<% form_for([:admin,@event]) do |f| %> 
<p> 
    <%= f.label :name %><br /> 
    <%= f.text_field :name %> 
</p> 
<p> 
    <%= f.label :location %><br/> 
    <%= f.text_field :location %> 
</p> 
<p> 
    <%= f.label :date %><br /> 
    <%= f.date_select :date %> 
</p> 
<p> 
    <%= f.label :description %><br /> 
    <%= f.text_area :description %> 
</p> 
<% f.fields_for :artists do |builder| %> 
    <%= render 'artist_fields', :f => builder %> 
<% end %> 
<p><%= link_to_add_fields "Add Artist", f, :artists %></p> 
<p> 
    <%= f.submit 'Submit' %> <%= link_to 'Cancel', admin_events_path %> 
</p> 
<% end %> 

artist_fields.html.erb

<p class="fields"> 
<%= f.label :name, "Artist"%><br/> 
<%= f.text_field :name %> 
<%= link_to_remove_fields "remove", f %> 
</p> 

回答

1

你有PROC拒绝艺术家的属性,如果名称为空:你也可以拒绝它,如果艺术家已经存在在模型上,但这并不能解决复制艺术家的问题。基本上你想在将它们添加到事件时执行find_or_create_by_name。

我认为在你的情况下,最好定义你自己的artist_attributes=方法,而不是依赖于accepted_nested。这样,你可以做你查找每个艺术家姓名,只需要添加:

def artist_attributes=(params) 
    if existing_artist = Artist.find_by_name(params[:name]) 
    self.artists << existing_artist unless self.artists.include? existing_artist 
    else 
    self.build_artist(params) 
    end 
end 
+0

我一直在研究你提出什么,但无法弄清楚如何开始。 – shalako 2010-07-05 17:48:55

+0

所以我需要在事件模型中添加一个自定义方法,并以某种方式合并find_or_create_by_name。我需要移除accept_nested_attributes_for吗?我需要使用虚拟属性,就像Ryan Bates在Railscasts#102中所建议的那样? Before_save,after_save?我还没有找到一个很好的例子,说明这些东西如何融合在一起。 – shalako 2010-07-05 17:58:24

+0

我添加了上面的artist_attributes =方法的示例。如果你想利用'f.fields_for:artist'的优势,我认为你仍然需要accepted_nested – 2010-07-13 23:41:03

1

你真的应该看看这些Railscasts:

  1. 不要创建各种各样的艺术家。(如果需要的话,或者创建)只使用现有的:

    http://railscasts.com/episodes/167-more-on-virtual-attributes

  2. 您还可以查看这些嵌套形式railscasts(第一部分此处链接):

    http://railscasts.com/episodes/196-nested-model-form-part-1

  3. 对于验证,你可以只用一种方法为你做一次在场的验证。喜欢的东西(在Event.rb):

    validate :artists_appear_just_once 
    
    private 
    def artists_appear_just_once 
        self.artists.size == self.artists.uniq.size 
    end 
    

或者,也可以使artsits使用uniq的默认显示只有一次!方法保存之前。只需拨打一个before_save钩和处理艺人阵...

before_save :make_artists_unique 

private 
def make_artists_unique 
    artists.uniq! 
end 

希望我得到了你所需要的正确的:P