Ruby on rails 验证无效的关联模型和嵌套表单

Update2:我已经清理了代码，似乎解决了一些问题。我已将新代码作为新问题发布

更新：组织和用户有1:多的关系。我的问题涉及一个联合注册表单，其中需要组织和用户。在maxcal对原始帖子的帮助之后，我为我的嵌套表单（“组织有很多用户”）编写了一个新的

create

方法，如下所示。我还添加了

create

方法。目前的情况/问题：

• 与所有有效信息一起提交，它工作正常
• 提交时包含无效的组织信息（不管用户是否也无效），它会按照我们的要求呈现带有错误消息的页面，但它只显示组织详细信息的错误。此外，对于用户详细信息，它已经清空了所有字段，而这是不应该的
• 仅为用户提交了无效信息，它再次呈现表单，但没有任何错误消息，并且用户的所有字段都已清空

有人知道代码出了什么问题吗？问题似乎更多的是嵌套用户，而不是组织（父级）。此外，

users\u attributes.empty？

不起作用，因为根据日志，提交的空表单仍然包含此类属性：

Parameters: {"utf8"=>"✓", "authenticity_token"=>"***", "organization"=>{"name"=>"", "bag"=>"", "users_attributes"=>{"0"=>{"email"=>"", "username"=>"", "password"=>"[FILTERED]", "password_confirmation"=>"[FILTERED]", "usertype"=>"2", "admin"=>"true"}}}, "commit"=>"Register"}

---

---

原始问题： 我有两个关联的模型和一个带有验证的嵌套表单。不幸的是，它不起作用。1） 种子设定时，它生成错误

验证失败：用户组织不能为空
。我对此提出了一个问题，过早地得出结论：它已经解决了这个问题。没有。2） 提交正确填写了所有字段的嵌套注册表单时，会生成flash错误消息
表单包含1个错误。用户组织不能为空

我应该如何调整代码以解决这些问题

模型文件：

#User model
belongs_to :organization, inverse_of: :users
validates_presence_of :organization_id, :unless => 'usertype == 1'

# Organization model
has_many :users, dependent: :destroy
accepts_nested_attributes_for :users, :reject_if => :all_blank, :allow_destroy => true

validate  :check_user
private
  def check_user
    if users.empty?
      errors.add(:base, 'User not present')
    end
  end

组织控制器方法

  def new
    @organization = Organization.new
    @user = @organization.users.build
  end

  def create
    @organization = Organization.new(new_params)
    if @organization.save
      @organization.users.each do |single_user|
        single_user.send_activation_email                 # Method in user model file.
      end
      flash[:success] = "Confirmation email sent."
      redirect_to root_url
    else
      @organization.users.build if @organization.users.blank?
      render 'new'
    end
  end

def new_params
  params.require(:organization).permit(:name, :bag,
             users_attributes: [:email, :username, :usertype, :password, :password_confirmation])
end

表格：

  <%= form_for @organization, url: organizations_path do |f| %>
    <%= render 'shared/error_messages', object: f.object %>
    <%= f.text_field :name %>
    <%= f.text_field :bag %>
    <%= f.fields_for :users do |p| %>
      <%= p.email_field :email %>
      <%= p.text_field :username %>
      <%= p.text_field :fullname %>
      <%= p.password_field :password %>
      <%= p.password_field :password_confirmation %>
      <%= p.hidden_field :usertype, value: 2 %>
    <% end %>

提交注册表单时出现错误的日志：

Started POST "/organizations" 
Processing by OrganizationsController#create as HTML
  Parameters: {"utf8"=>"✓", "authenticity_token"=>"0cR***Nnx4iReMiePg==", "organization"=>{"name"=>"test21", "bag"=>"tes21", "users_attributes"=>{"0"=>{"email"=>"test21@example.com", "username"=>"test21", "password"=>"[FILTERED]", "password_confirmation"=>"[FILTERED]", "usertype"=>"2"}}}, "commit"=>"Register"}
   (0.2ms)  BEGIN
  User Exists (1.1ms)  SELECT  1 AS one FROM "users" WHERE LOWER("users"."email") = LOWER('test21@example.com') LIMIT 1
   (0.7ms)  SELECT "users"."email" FROM "users"  ORDER BY "users"."username" ASC
  User Exists (0.3ms)  SELECT  1 AS one FROM "users" WHERE LOWER(users"."username") = LOWER('test21') LIMIT 1
  Organization Exists (0.6ms)  SELECT  1 AS one FROM "organizations" WHERE LOWER("organizations"."name") = LOWER('test21') LIMIT 1
  Organization Exists (0.4ms)  SELECT  1 AS one FROM "organizations" WHERE LOWER("organizations"."bag") = LOWER('tes21') LIMIT 1
   (0.2ms)  ROLLBACK

由于出现错误，您的验证无法工作

要申请这份工作，你一定是疯了；但如果你是
疯子，你是不可接受的

ActiveRecord模型在保存时从数据库获取其ID。
但是，在将组织插入数据库之前，对嵌套用户进行验证

您可能会猜测，只需检查
即可验证
的存在，而不会通过：

validates_presence_of :organization, unless: -> { usertype == 1 }

不幸的是，没有。为了验证组织的存在，必须将组织持久化到数据库中。第二十二条军规

为了通过验证，我们需要将创建组织和用户分为两个步骤：

org = Organization.create(name: 'M & M Enterprises')
user = org.users.build(username: 'milo_minderbinder', ...)
user.valid? 

不幸的是，你不能使用
的方法接受了用户的嵌套属性，至少不是一蹴而就的

通过使用，我们可以将组织插入数据库，并在用户无效时回滚

def create
  @organization = Organization.new(new_params.except(:users_attributes))
  @organization.transaction do
    @organization.save!
    if new_params[:users_attributes].any?
      @organization.users.create!(new_params[:users_attributes])
    end
  end
  if @organization.persisted?
    # ...
    if @organization.users.any?
      # send emails ... 
    end
  else
    @organization.users.build if @organization.users.blank? 
    render :new
  end
end

后续问题
我们使用
@organization.persistend？
，因为我们可能希望重定向到新创建的组织，无论是否创建了用户记录

因为电子邮件是发送给用户的？这不重要，因为若并没有创建用户，组织将回滚

如果没有用户创建，则不会回滚事务。仅当用户由于参数无效而无法保存时。这是基于您的要求：

但是一个组织也可以（暂时）没有用户

如果您需要@Organization在没有用户的情况下无效，您可以执行以下操作：

  @organisation.errors.add(:users, 'No users provided') unless new_params[:users_attributes].any?
  @organization.transaction do
    @organization.save!
    if new_params[:users_attributes].any?
      @organization.users.create!(new_params[:users_attributes])
    end
  end

您可以使用
@organization.users.any？
检查是否有任何用户
@organization.users.persisted？
将不起作用，因为
.persisted？
是模型实例上的方法，而不是集合

另一方面，我假设用这种方法（不应该）覆盖/更新现有组织/用户，而不是总是创建新记录是不可能的

对，因为这将始终发出两个SQL insert语句，所以它不会更改现有记录

但是，由您创建验证来保证数据库列的唯一性（即您不希望多个记录具有相同的user.email或organiation.name）

有利的一面是，在更新现有组织时，这些警告都不适用：

def update
  @organisation.update(... params for org and and users ...)
end

因为在验证用户时，您不会遇到完全的鸡或蛋的困境。
您的验证无法工作，因为

要申请这份工作，你一定是疯了；但如果你是
疯子，你是不可接受的

ActiveRecord模型在保存时从数据库获取其ID。
但是，在将组织插入数据库之前，对嵌套用户进行验证

您可能会猜测，只需检查
即可验证
的存在，而不会通过：

validates_presence_of :organization, unless: -> { usertype == 1 }

不幸的是，没有。为了验证组织的存在，必须将组织持久化到数据库中。第二十二条军规

为了通过验证，我们需要将创建组织和用户分为两个步骤：

org = Organization.create(name: 'M & M Enterprises')
user = org.users.build(username: 'milo_minderbinder', ...)
user.valid? 

不幸的是，你不能使用
的方法接受了用户的嵌套属性，至少不是一蹴而就的

通过使用，我们可以将组织插入数据库，并在用户无效时回滚

def create
  @organization = Organization.new(new_params.except(:users_attributes))
  @organization.transaction do
    @organization.save!
    if new_params[:users_attributes].any?
      @organization.users.create!(new_params[:users_attributes])
    end
  end
  if @organization.persisted?
    # ...
    if @organization.users.any?
      # send emails ... 
    end
  else
    @organization.users.build if @organization.users.blank? 
    render :new
  end
end

后续问题
我们使用
@organization.persistend？
，因为我们可能希望重定向到新创建的组织，无论是否创建了用户记录

因为电子邮件是发送给用户的？这不重要，因为若并没有创建用户，组织将回滚

如果没有我们，交易不会回滚