关联模型和带有验证的嵌套表单不起作用
Associated models and a nested form with validation not working
Update2: 我清理了代码，似乎解决了一些问题。我已经 post 将新代码编辑为一个新问题 here。
更新： 组织和用户有 1:many 关系。我的问题涉及一个联合注册表单，其中需要组织和用户。在 maxcal 对原始 post 的帮助之后，我为我的嵌套表单 ("organization has many users") 编写了一个新的 create 方法，如下所示。我还添加了 begin...rescue...end 到 create 方法。 situation/problem现在：
- 已提交所有有效信息，它可以正常工作。
- 提交的组织信息无效（无论用户是否也无效），它会按照我们的意愿呈现包含错误消息的页面，但它仅显示组织详细信息的错误。此外，对于用户详细信息，它随后清空了所有不应清空的字段。
- 只为用户提交了无效信息，它再次呈现表单但没有任何错误消息，并且用户的所有字段都已清空。
有人知道代码有什么问题吗？嵌套用户的问题似乎比组织（父级）更多。此外，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。我 对此提出了一个问题，并过早地断定它已经解决了。它没有。 2) 提交我的嵌套 注册表单 并正确填写所有字段，会生成闪现错误消息 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
由于 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）。
有利的一面是 none 这些注意事项在更新现有组织时适用：
def update
@organisation.update(... params for org and and users ...)
end
因为在验证用户时您不会遇到鸡生蛋还是蛋生鸡的困境。
Update2: 我清理了代码，似乎解决了一些问题。我已经 post 将新代码编辑为一个新问题 here。
更新： 组织和用户有 1:many 关系。我的问题涉及一个联合注册表单，其中需要组织和用户。在 maxcal 对原始 post 的帮助之后，我为我的嵌套表单 ("organization has many users") 编写了一个新的 create 方法，如下所示。我还添加了 begin...rescue...end 到 create 方法。 situation/problem现在：
- 已提交所有有效信息，它可以正常工作。
- 提交的组织信息无效（无论用户是否也无效），它会按照我们的意愿呈现包含错误消息的页面，但它仅显示组织详细信息的错误。此外，对于用户详细信息，它随后清空了所有不应清空的字段。
- 只为用户提交了无效信息，它再次呈现表单但没有任何错误消息，并且用户的所有字段都已清空。
有人知道代码有什么问题吗？嵌套用户的问题似乎比组织（父级）更多。此外，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。我 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
由于 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）。
有利的一面是 none 这些注意事项在更新现有组织时适用：
def update
@organisation.update(... params for org and and users ...)
end
因为在验证用户时您不会遇到鸡生蛋还是蛋生鸡的困境。