Question

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

更新：组织和用户之间存在1：很多关系。我的问题涉及一个联合注册表单，其中组织和用户都是必需的。在maxcal对原始帖子的帮助之后，我为我的嵌套表单编写了一个新的create方法（＆＃34;组织有很多用户＆＃34;），如下所示。我还在create方法中添加了begin...rescue...end。现在的情况/问题：

提交所有有效信息，无法正常使用。
提交组织的无效信息（如果用户也无效，则无关紧要），它会按照我们的意愿呈现包含错误消息的页面，但它仅显示组织详细信息的错误。此外，对于用户详细信息，它已经清空了所有字段，它不应该这样。
仅为用户提交无效信息，它再次呈现表单但没有任何错误消息，并且用户的所有字段都已清空。

任何人都知道代码有什么问题？对于嵌套用户而言，问题似乎与组织（父组件）相关。此外，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）在播种上，它会生成错误Validation failed: Users organization can't be blank。我previously posted对此提出了一个问题并提前得出结论它解决了这个问题。它没有。 2）提交我的嵌套注册表单并正确填写所有字段，生成flash错误消息The form contains 1 error. Users organization can't be blank。

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

模型文件：

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

在我的种子档案中，我有：

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

，您的验证无效

申请这份工作，你必须疯了;但如果你是疯了，你是不可接受的。

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

你会猜测只需检查validates_presence_of即可：

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

不幸的是没有。为了使validates_presence_of :organization通过，必须将组织持久保存到数据库中。再次捕获22。

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

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

因为电子邮件是发送给用户的？如果没有创建用户，组织就会回滚，这不重要。

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

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

如果您在没有用户的情况下需要@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?是模型实例上的方法 - 而不是集合。

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

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

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

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

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

因为在验证用户时你不会得到整鸡或鸡蛋的困境。

关联模型和嵌套表单，验证不起作用

1 个答案:

后续问题