From 212186e059a0cb7863b60a7871fcc286b577c78c Mon Sep 17 00:00:00 2001 From: Andy Brett Date: Thu, 4 Feb 2021 11:45:36 -0800 Subject: [PATCH] check all variants, not just where we use 'items' --- .../20210202052337_migrate_variant_unit_values.rb | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/db/migrate/20210202052337_migrate_variant_unit_values.rb b/db/migrate/20210202052337_migrate_variant_unit_values.rb index 3dd5da4805..769a431ac4 100644 --- a/db/migrate/20210202052337_migrate_variant_unit_values.rb +++ b/db/migrate/20210202052337_migrate_variant_unit_values.rb @@ -1,16 +1,10 @@ class MigrateVariantUnitValues < ActiveRecord::Migration def up - Spree::Variant.includes(:product).where( - spree_products: { variant_unit: "items" }, - spree_variants: { unit_value: [nil, Float::NAN] } - ).find_each do |variant| + Spree::Variant.where(unit_value: [nil, Float::NAN]).find_each do |variant| variant.unit_value = 1 variant.save end - Spree::Variant.includes(:product).where( - spree_products: { variant_unit: "items" }, - spree_variants: { weight: [nil, Float::NAN] } - ).find_each do |variant| + Spree::Variant.where(weight: [nil, Float::NAN]).find_each do |variant| variant.weight = 0 variant.save end