ruby-on-rails - 关联模型和带有验证的嵌套表单不起作用

Question

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

更新:组织和用户是一对多的关系。我的问题涉及一个联合注册表单，其中需要组织和用户。在 maxcal 对原始帖子的帮助之后，我为我的嵌套表单(“组织有很多用户”)编写了一个新的 create 方法，如下所示。我还添加了 begin...rescue...end到 create 方法。现在的情况/问题:

• 已提交所有有效信息，它可以正常工作。
• 提交了无效的组织信息(用户是否也无效无关紧要)，它会按照我们的意愿呈现包含错误消息的页面，但它只显示组织详细信息的错误。此外，对于用户详细信息，它随后清空了所有不应清空的字段。
• 只为用户提交了无效信息，它再次呈现表单但没有任何错误消息，并且用户的所有字段都已清空。

有人知道代码有什么问题吗？嵌套用户的问题似乎比组织(父级)更多。此外，users_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"}

.

  def create
    @organization = Organization.new(new_params.except(:users_attributes))
begin
    if users_attributes.empty?
        @organisation.errors.add(:users, 'No user provided')
    end
    @organization.transaction do
      @organization.save!
      if users_attributes.any?
        @organization.users.create!(users_attributes)
      end
    end
rescue ActiveRecord::RecordInvalid => invalid
    if @organization.persisted?
      if @organization.users.any?
        @organization.users.each do |single_user|
          single_user.send_activation_email
        end
      end
      flash[:success] = "Confirmation email sent."
      redirect_to root_url
    else
      @organization.users.build if @organization.users.blank? 
      render :new
    end
end
  end

private
  # converts the hash of nested attributes hashes to an array
  def users_attributes
     new_params[:users_attributes].values
  end
end

---

---

原始问题:我有两个关联模型和一个带验证的嵌套表单。不幸的是，它不起作用。 1) 在 seeding 上它生成错误 Validation failed: Users organization can't be blank。我 previously posted一个关于这个的问题，并过早地得出结论，它已经解决了。它没有。 2) 提交所有字段均正确填写的嵌套注册表单，会生成闪现错误消息The form contains 1 error。用户组织不能为空。

我应该如何调整我的代码来解决这些问题？

模型文件:

#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

组织 Controller 方法

  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 %>

在我的种子文件中我有:

Organization.create!(name: "Fictious business",
                     address: Faker::Address.street_address,
                     city: Faker::Address.city,
  users_attributes: [email: "helpst@example.com",
                     username: "helpyzghtst", 
                     usertype: 2,
                     password: "foobar", 
                     password_confirmation: "foobar"])

提交注册表单出错的日志:

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

Answer 1

由于 Catch-22，您的验证无效

To apply for this job, you would have to be insane; but if you are insane, you are unacceptable.

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

您可能会猜想，只要检查 validates_presence_of 就可以通过:

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

不幸的是没有。为了使 validates_presence_of :organization 通过，必须将组织持久保存到数据库中。又是第二十二条军规。

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

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

不幸的是，这意味着您不能使用 accepts_nested_attributes_for :users - 至少不能直接使用。

通过使用 transaction我们可以将组织插入数据库，如果用户无效则回滚。

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.persisted? 因为我们可能希望重定向到新创建的组织，无论是否创建了用户记录。

because the emails are sent to users? It shouldn't matter since organization is rolled back if no user is created.

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

But an organization can also (temporarily) have no users.

如果您需要@organisation 在没有用户的情况下无效，您可以这样做:

  @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? 是模型实例上的方法 - 而不是集合。

On a different note, I assume it's not possible to overwrite/update an existing organization/user with this method (which shouldn't be) instead of always creating a new record?

是的，因为这将始终发出两个 SQL 插入语句，所以它不会更改现有记录。

然而，由您创建验证以保证数据库列的唯一性(IE，您不希望多个记录具有相同的 user.email 或 organiation.name)。

有利的一面是，更新现有组织时这些注意事项均不适用:

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

因为在验证用户时您不会遇到鸡生蛋还是蛋生蛋的困境。