raggedstaff/openfoodnetwork
1
0
Fork 0
mirror of https://github.com/openfoodfoundation/openfoodnetwork synced 2026-02-27 01:43:22 +00:00
Code Issues Packages Projects Releases Wiki Activity

Make ProductsController#bulk_update work by making ProductSet#create_variant not mass-assigning the provided on_hand and on_demand values and set them after each variant is created

Browse Source
This commit is contained in:
luisramos0
2018-12-08 15:11:41 +00:00
parent 1819fc8493
commit 4b74e50353
1 changed files with 14 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
app/models/spree/product_set.rb
View File

@@ -63,16 +63,16 @@ class Spree::ProductSet < ModelSet
def update_product_master(product, attributes)
return true unless attributes[:master_attributes]
update_variant(product, attributes[:master_attributes])
create_or_update_variant(product, attributes[:master_attributes])
end
def update_variants_attributes(product, variants_attributes)
variants_attributes.each do |attributes|
update_variant(product, attributes)
create_or_update_variant(product, attributes)
end
end
def update_variant(product, variant_attributes)
def create_or_update_variant(product, variant_attributes)
found_variant = product.variants_including_master.find do |variant|
variant.id.to_s == variant_attributes[:id].to_s && variant.persisted?
end
@@ -80,10 +80,20 @@ class Spree::ProductSet < ModelSet
if found_variant.present?
found_variant.update_attributes(variant_attributes.except(:id))
else
product.variants.create(variant_attributes)
create_variant(product, variant_attributes)
end
end
def create_variant(product, variant_attributes)
on_hand = variant_attributes.delete(:on_hand)
on_demand = variant_attributes.delete(:on_demand)
variant = product.variants.create(variant_attributes)
variant.on_demand = on_demand if on_demand.present?
variant.on_hand = on_hand.to_i if on_hand.present?
end
def collection_attributes=(attributes)
@collection = Spree::Product
.where(id: attributes.each_value.map { |product| product[:id] })