accepts_nested_attributes_for链接到现有记录,而不是创建一个新的记录

时间:2022-03-12 00:16:15

I have the following models

我有以下的模型

class Order < AR::Base
  has_many :products

  accepts_nested_attributes_for :products
end

class Product < AR::Base
  belongs_to :order
  has_and_belongs_to_many :stores

  accepts_nested_attributes_for :stores
end

class Store < AR::Base
  has_and_belongs_to_many :products
end

Now I have a order view where I want to update the shops for the product. The thing is that I only want to connect the products to the existing shops in my db, not create new ones.

现在我有一个订单视图,我想要更新产品的商店。问题是,我只想将产品连接到db中的现有商店,而不是创建新的商店。

My form in the order view looks like this (using Formtastic):

我在order视图中的表单是这样的(使用Formtastic):

= semantic_form_for @order do |f|
  = f.inputs :for => :live_products do |live_products_form|
    = live_products_form.inputs :for => :stores do |stores_form|
      = stores_form.input :name, :as => :select, :collection => Store.all.map(&:name)

Although its nested it works fine. The problem is that, when I select a store and try to update the order (and the products and stores with it), Rails tries to create a new store with that name. I want it to just use the existing store and connect the product to that.

虽然它嵌套的很好。问题是,当我选择一个商店并试图更新订单(以及与之相关的产品和商店)时,Rails试图创建一个同名的新商店。我想让它使用现有的存储并将产品连接到那个。

Any help appreciated!

任何帮助表示赞赏!

EDIT 1:

编辑1:

In the end I solved this problem in a very crude way:

最后我以一种非常粗糙的方式解决了这个问题:

# ProductsController

def update
  [...]

  # Filter out stores
  stores_attributes = params[:product].delete(:stores_attributes)

  @product.attributes = params[:product]

  if stores_attributes.present?
    # Set stores
    @product.stores = stores_attributes.map do |store_attributes|
      # This will raise RecordNotFound exception if a store with that name doesn't exist
      Store.find_by_name!(store_attributes[:name])
    end
  end

  @order.save

  [...]
end

EDIT 2:

编辑2:

Pablo's solution is much more elegant and should be preferred over mine.

巴勃罗的解决方案要优雅得多,应该比我的更好。

2 个解决方案

#1


21  

Try to implement a :reject_if that check if the Store already exists and then use it:

尝试实现a:reject_if检查商店是否已经存在,然后使用它:

class Product < AR::Base
  belongs_to :order
  has_and_belongs_to_many :stores

  accepts_nested_attributes_for :stores, :reject_if => :check_store

  protected

    def check_store(store_attr)
      if _store = Store.find(store_attr['id'])
        self.store = _store
        return true
      end
      return false
    end
end

I have this code working fine in a current project.

我让这个代码在当前的项目中正常工作。

Please, let me know if you found a better solution.

如果你找到更好的解决办法,请告诉我。

#2


0  

I had the same problem and solved it by adding :id to the nested parameter list.

我遇到了同样的问题,通过向嵌套参数列表中添加:id来解决它。

def family_params
  params.require(:family).permit(:user_id, :address, people_attributes: [:id, :relation, :first_name, :last_name)
end

#1


21  

Try to implement a :reject_if that check if the Store already exists and then use it:

尝试实现a:reject_if检查商店是否已经存在,然后使用它:

class Product < AR::Base
  belongs_to :order
  has_and_belongs_to_many :stores

  accepts_nested_attributes_for :stores, :reject_if => :check_store

  protected

    def check_store(store_attr)
      if _store = Store.find(store_attr['id'])
        self.store = _store
        return true
      end
      return false
    end
end

I have this code working fine in a current project.

我让这个代码在当前的项目中正常工作。

Please, let me know if you found a better solution.

如果你找到更好的解决办法,请告诉我。

#2


0  

I had the same problem and solved it by adding :id to the nested parameter list.

我遇到了同样的问题,通过向嵌套参数列表中添加:id来解决它。

def family_params
  params.require(:family).permit(:user_id, :address, people_attributes: [:id, :relation, :first_name, :last_name)
end