form_for collection_select在编辑时不显示保存的选择

时间:2019-12-03 12:46:28

标签: ruby-on-rails nested-form-for collection-select

我有一个用于新建和编辑的表格。表单中填写了产品名称,说明等信息。

此外,用户可以从嵌套的下拉列表(collection_select)中选择一个项目,也可以创建一个新项目。在“新”上,该表格可以正常工作-保存所有条目和选择。

当用户去编辑保存的产品时,该表单会预加载该商品的所有已填写条目,但不会在collection_select中预加载其原始选择。

并且,如果用户要编辑项目并决定创建先前选择的collection_select项目的新项目INSTEAD,则会出现错误,指出已经使用该化学基团创建了产品。 对此双重困境的任何帮助将不胜感激。我是RoR的新手,我确定自己在某处缺少任何东西。

这是我的表格

<%= render partial: 'layouts/errors', locals: {object: @product} %>

<%= form_for(@product) do |f| %>
    <div>
        <%= f.label :name %><br>
        <%= f.text_field :name %><br>
    <div>
    <div>
        <%= f.label :active_ingredient %><br>
        <%= f.text_field :active_ingredient %><br>
    <div>
    <div>
        <%= f.label :description %><br>
        <%= f.text_area :description %><br>
    </div>
    <div>
        <%= f.label :image %><br>
        <%= f.file_field :image %><br>
    </div>
    <div>
        <p>Select a Chemical Group:</p>
        <%= f.collection_select :chem_group_id, ChemGroup.all, :id, :name, include_blank: 'Select One', selected: @product.chem_group, value: @product.chem_group.name %>
    </div>
    <div>
        <p>Or, create a new Chemical Group:</p>
        <!-- NESTED FORM! User writing attributes for another object. Use fields_for -->
        <%= f.fields_for :chem_group do |cg| %>
            <%= cg.label :name %>
            <%= cg.text_field :name %>
        <% end %>
    </div>
    <div>
        <p>Select an Application Area:</p>
        <%= f.collection_select :application_area_id, ApplicationArea.all, :id, :area_name, include_blank: 'Select One', selected: @product.application_area, value: @product.application_area.area_name %>
    </div>
    <div>
        <p>Or, create a new Application Area:</p>
        <!-- NESTED FORM! User writing attributes for another object. Use fields_for -->
        <%= f.fields_for :application_area do |aa| %>
            <%= aa.label :area_name %>
            <%=aa.text_field :area_name %>
        <% end %>
    </div>
    <br>
    <%= f.submit "Save" %>
<% end %>

这是我的模特

class Product < ApplicationRecord
  belongs_to :chem_group
  belongs_to :application_area
  belongs_to :user #admin creator
  accepts_nested_attributes_for :chem_group #tells the model to accept chem_group attributes from cg nested form in new product form
  accepts_nested_attributes_for :application_area

  validates :active_ingredient, presence: true
  validates :application_area, presence: true
  validates :description, presence: true
  validates :name, presence: true
  validate :not_a_duplicate #checking for what we DON'T WANT

  def chem_group_attributes=(attributes)
    self.chem_group = ChemGroup.find_or_create_by(attributes) if !attributes['name'].empty?
    self.chem_group
  end

  def application_area_attributes=(attributes)
    self.application_area = ApplicationArea.find_or_create_by(attributes) if !attributes['area_name'].empty?
    self.application_area
  end

  #if there is already a product with that name && chem_group, give error
  def not_a_duplicate
    #calling the instance of the attribute [string/integer: key]
      if Product.find_by(name: name, chem_group_id: chem_group_id)
        errors.add(:name, 'has already been created for that Chemical Group')
      end
    end
end

这是我的控制人

class ProductsController < ApplicationController

    def new
        if logged_in?
            @product = Product.new
            1.times {@product.build_chem_group} #for the nested form. Builds the chem_group attributes
            @product.build_application_area
        else
            flash[:error] = "Sorry, you must be logged in to create a new product."
            redirect_to products_path
        end
    end

    def create
        @product = Product.new(product_params)
        @product.user_id = session[:user_id] #bc product belongs_to user. user_id required from model
        if @product.save #validation
            # @product.image.purge
            # @product.image.attach(params[:product][:image]) # allows image to be replaced if user changes image
            redirect_to product_path(@product)
        else
            @product.build_chem_group
            @product.build_application_area
            render :new
        end
    end

    def edit
        find_product
        1.times {@product.build_chem_group}
        if @product.user != current_user
            flash[:error] = "Sorry, you can only edit your own products"
            redirect_to products_path
        end
    end

    def update
        find_product
        if @product.update(product_params)
            redirect_to product_path(@product)
        else
            render :edit
        end
    end

    private

    def product_params
        params.require(:product).permit(:name, :description, :active_ingredient, :image, :chem_group_id, :application_area_id, chem_group_attributes: [:id, :name], application_area_attributes: [:id, :area_name])
        #chem_group_id and chem_group_attributes [:name] is permitting elements from new product form
    end

    def find_product
        @product = Product.find_by(id: params[:id])
    end

end

1 个答案:

答案 0 :(得分:2)

阅读您提供的表格:

  1. 如果您要编辑产品,则收藏集应显示以前的 选择chem_group,然后填写要创建的填充 它是名称,因为它是相同的关联对象。不是因为你 搞砸了selectedvalue选项。 selected在 像这样的collection_select应该指向id而不是反对。 像:selected: @product.chem_group.id,但这实际上是不必要的 这里。您应该只需跳过这些选项(selectedvalue 因为您正在为@product使用表单构建器。
  2. 有人尝试用填充的新值编辑chem_group后 真正的混乱开始了,因为在选择字段中仍然有一个ID 指向上一个chem_groupchem_group_attributes中 我猜现在只有:name,Rails会保存它 chem_group_id,您的自定义验证将失败,因为 现有对象(它表明此验证永远不会让您 保存对象而不更改chem_group 现有-您正在编辑!)

尝试:

  1. 更改或删除自定义验证。或检查是否在 验证不是您要编辑的验证,而是先前保存的
  2. 默认情况下禁用输入之一(<%= cg.text_field :name, disabled: @product.chem_group.present? %>)并进行切换 根据用户的操作-选择或键入(例如,使用 复选框和JS onchange)。确保您所使用的参数是 过去。
  3. 还:chem_group_attributes: [:id, :name]来吧,你没路过 这里的ID-删除它。
  4. def chem_group_attributes=(attributes)覆盖很危险。一世 知道你想得到什么,但这可能不是最重要的 实现这一目标的美好方式。

通常,您可能还希望看到:https://select2.org/tagging。它仍然是与您这里相同的复杂逻辑,并且还有一些JS可以使标记在创建新的关联对象上正常工作,但它看起来更好,UX更好,而且我去过那里-我通常将它与accepts_nested_attributes_一起使用,通常,可以打我例子。