Don't pluck variant_ids multiple times (once for each line item)

This commit is contained in:
Matt-Yorkley
2021-05-11 01:45:54 +01:00
parent 1fff81c214
commit 0fabb5bc2f

View File

@@ -185,7 +185,8 @@ module Spree
def line_items
if order.complete?
order.line_items.select { |li| inventory_units.pluck(:variant_id).include?(li.variant_id) }
inventory_unit_ids = inventory_units.pluck(:variant_id)
order.line_items.select { |li| inventory_unit_ids.include?(li.variant_id) }
else
order.line_items
end