11

我有这些模型：

class Organisation < ActiveRecord::Base

  has_many    :people
  has_one     :address, :as         => :addressable,
                        :dependent  => :destroy
  accepts_nested_attributes_for :address, :allow_destroy => true

end

class Person < ActiveRecord::Base

  attr_accessible :first_name, :last_name, :email, :organisation_id, :address_attributes

  belongs_to  :user
  belongs_to  :organisation
  has_one     :address, :as         => :addressable,
                        :dependent  => :destroy
  accepts_nested_attributes_for :address, :allow_destroy => true

  # These two methods seem to have no effect at all!
  validates_presence_of :organisation,  :unless => "address.present?"
  validates_associated  :address,       :unless => "organisation.present?"

end

class Address < ActiveRecord::Base

  belongs_to :addressable, :polymorphic => true

  validates_presence_of :line1, :line2, :city, :zip

end

...以及这些观点：

_fields.html.erb：

<%= render 'shared/error_messages', :object => f.object %>
<fieldset>
<div class="left">
    <%= f.label :first_name %><br/>
    <%= f.text_field :first_name %>
</div>
<div>
    <%= f.label :last_name %><br/>
    <%= f.text_field :last_name %>
</div>
<div>
    <%= f.label :email %><br/>
    <%= f.text_field :email %>
</div>
<div>
    <%= f.label :organisation_id %><br/>
    <%= f.select(:organisation_id, current_user.organisation_names, {:include_blank => "--- None ---"}, :id => 'organisation_select') %>
</div>
</fieldset>

<%= f.fields_for :address do |address| %>
  <%= render 'shared/address', :f => address %>
<% end %>

_address.html.erb：

<fieldset id="address_fields">
<div>
    <%= f.label :line1 %>
    <%= f.text_field :line1 %>
</div>
<div>
    <%= f.label :line2 %>
    <%= f.text_field :line2 %>
</div>
<div>
    <%= f.label :zip %>
    <%= f.text_field :zip %>
</div>  
<div>
    <%= f.label :city %>
    <%= f.text_field :city %>
</div>  
</fieldset>

people_controller.rb：

def new
  puts params.inspect
  @person = Person.new(:organisation_id => params[:organisation_id])
  @person.build_address
  @title = "New person"
end

{"action"=>"new", "controller"=>"people"}

def edit
  puts params.inspect
  @title = @person.name
end

{"action"=>"edit", "id"=>"69", "controller"=>"people"}

def create
  puts params.inspect
  if params[:organisation_id]
    @person = current_user.organisations.build_person(params[:person])
  else
    @person = current_user.people.build(params[:person])
  end
  if @person.save
    flash[:success] = "Person created."
    redirect_to people_path
  else
    render :action => "new"
  end
end

{"commit"=>"Create", "action"=>"create", "person"=>{"last_name"=>"Doe", "organisation_id"=>"9", "email"=>"john.doe@email.com", "first_name"=>"John", "address_attributes"=>{"city"=>"Chicago", "zip"=>"12345", "line2"=>"Apt 1", "line1"=>"1 Main Street"}}, "authenticity_token"=>"Jp3XVLbA3X1SOigPezYFfEol0FGjcMHRTy6jQeM1OuI=", "controller"=>"people", "utf8"=>"✓"}

在我的Person模型中，我需要确保只有当一个人organisation_id的地址为空白时，该人的地址字段才必须存在。

我试过这样的事情：

validates :address, :presence => true, :if => "organisation_id.blank?"

但它不起作用。

如何才能做到这一点？

谢谢你的帮助。

4

3 回答 3

33

首先，我想确定你的意思是blank?而不是present?。通常，我会看到：

validate :address, :presence_of => true, :if => 'organisation.present?'

这意味着，如果组织也存在，您只想验证地址。

关于，:accepts_nested_attributes_for您是通过传入嵌套表单属性或类似的东西来使用此功能吗？我只是想确保您绝对需要使用此功能。如果您实际上并未处理嵌套表单属性，则可以使用以下方法实现级联验证：

validates_associated :address

如果确实需要使用:accepts_nested_attributes，请务必查看:reject_if参数。基本上，如果某些条件适用，您可以完全拒绝添加属性（及其后代）：

accepts_nested_attributes_for :address, :allow_destroy => true, :reject_if => :no_organisation

def no_organisation(attributes)
  attributes[:organisation_id].blank?
end

现在，如果以上都不适用，让我们看看你的语法：

它应该可以工作，:if/:unless接受符号、字符串和 procs。您不需要指向foreign_key，但可以通过指向来简化：

:if => "organisation.blank?"

您在地址模型中有其他验证，对吗？当您不希望地址被验证时，它是否正在被验证？或者地址没有被验证？如果你能给我一些额外的细节，我可以帮助你在控制台中测试它。

  1. 为了让我自己更容易重新：大规模分配，我更改了 rails 配置：config.active_record.whitelist_attributes = false
  2. 我创建了一个要点供您遵循

  3. 我也有一个示例项目。如果您有兴趣，请告诉我。

    基本点：

  4. 将以下内容添加到 Person 以确保 Org 或 Address 有效：

    validates_presence_of :organisation, :unless => "address.present?" validates_associated :address, :unless => "organisation.present?"

  5. 向地址添加了验证以在 Org 不存在时触发错误： validates_presence_of :line1, :line2, :city, :zip

    我能够产生您正在寻求的要求。请查看我创建的要点，其中我有一个完整的控制台测试计划。

我在前面的 gist 中添加了一个控制器文件。

概述：

  1. 创建此人所需要做的就是： @person = current_user.people.build(params[:person])
  2. :organisation_id 将始终从 :person 参数节点中找到，如下所示： params[:person][:organisation_id] 所以你的 if 永远不会是真的。

我更新了要点，对控制器、模型和表单进行了必要的更改。

概述：

  1. 你需要清理你的控制器。您正在使用accepts_nested_attribute，所以在 :create 中，您只关心params[:person]. 此外，在 中render :new，您需要设置部分将使用的任何实例变量。这不会通过操作返回:new。和动作:new也:edit需要简化。
  2. 您的 Person 模型需要使用:reject_if参数，因为 Address 字段将返回到 :create 操作 as :address_attributes => {:line1 => '', :line2 => '', etc}。如果有任何值，您只想创建关联。然后你的validates_presence_offor:organisation会工作得很好。

  3. 您的表单需要将组织 ID 传递给控制器​​，而不是组织名称

    一切尽在掌握之中

应该是最后的要点。

概述：

  1. 在构建 @person 之后立即将以下内容添加到您的编辑操作中：

    @person.build_address 如果@person.address.nil？这确保您有地址输入，即使 @person.address 不存在。它不存在，因为 accept_nested_attributes 上的 :reject_if 条件

  2. 我干了 :reject_if 如下。这有点hacky，但有一些实用性： 

    accepts_nested_attributes_for :address, :allow_destroy => true, :reject_if => :attributes_blank?

def attributes_blank?(attrs)  
  attrs.except('id').values.all?(&:blank?)  
end

    一种。attrs-> params[:person][:address] 的结果
    b. .except('id')-> 返回除 'id' 之外的所有键值
    c. .values-> 将哈希中的所有值作为数组返回
    d. .all?-> 数组中的所有元素是否满足以下检查
    e. &:blank-> 块的 ruby​​ 简写，如下所示：all?{ |v| v.blank? }

于 2012-10-07T19:52:56.367 回答
0

你确定你不是这个意思：

validates :address, :presence => true, :if => organisation_id.nil?
于 2012-10-07T15:20:52.593 回答
0

更简单的方法可能是添加自定义验证器。这非常简单，您不必偶然发现语法或试图弄清楚为什么 Rails 的魔法不起作用。

在我的 Person 模型中，我需要确保只有当一个人的 organization_id 为空白时，该人的地址字段才必须存在。

class Person < ActiveRecord::Base
  ...
  validate :address_if_organisation_id_is_present

  private

  def address_if_organisation_id_is_present
    return true unless organisation_id
    errors.add(:address, "cannot be blank") unless address
  end
end

添加到模型的错误将阻止它保存。注意：您可能希望使用address.blank?或address.empty?如其他答案中所讨论的那样，但您可以为您想要的行为定义它。

于 2019-07-23T18:04:17.163 回答