From a59fc1eee37447d3b7ceb35fb7d5214f675167d5 Mon Sep 17 00:00:00 2001 From: Andy Brett Date: Fri, 9 Apr 2021 10:28:39 -0700 Subject: [PATCH] Revert "Switch from map, that does not exist in ActionController::Parameters, to each" This reverts commit 69a15d4f041b692ca02241239760af231aa318c7. --- app/services/cart_service.rb | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/app/services/cart_service.rb b/app/services/cart_service.rb index c4f3b5a5c0..35dcb720f0 100644 --- a/app/services/cart_service.rb +++ b/app/services/cart_service.rb @@ -106,15 +106,13 @@ class CartService end def read_variants_hash(data) - variants_array = [] - (data[:variants] || []).each do |variant_id, quantity| - if quantity.is_a?(ActionController::Parameters) - variants_array.push({ variant_id: variant_id, quantity: quantity[:quantity], max_quantity: quantity[:max_quantity] }) + (data[:variants] || []).map do |variant_id, quantity| + if quantity.is_a?(Hash) + { variant_id: variant_id, quantity: quantity[:quantity], max_quantity: quantity[:max_quantity] } else - variants_array.push({ variant_id: variant_id, quantity: quantity }) + { variant_id: variant_id, quantity: quantity } end end - variants_array end def cart_remove(variant_id)