solidus_core 2.11.7 → 3.0.1

data/lib/tasks/migrations/migrate_address_names.rake

@@ -1,158 +0,0 @@
-# frozen_string_literal: true
-
-require 'thor'
-
-namespace :solidus do
-  namespace :migrations do
-    namespace :migrate_address_names do
-      desc 'Backfills Spree::Address name attribute using firstname and lastname
-        concatenation in order to retain historical data when upgrading to new
-        address name format'
-      task up: :environment do
-        puts "Combining addresses' firstname and lastname into name ... "
-        class Spree::AddressForMigration < ApplicationRecord
-          self.table_name = 'spree_addresses'
-        end
-
-        records = Spree::AddressForMigration.unscoped.where(name: [nil, ''])
-        count = records.count
-        connection = ActiveRecord::Base.connection
-        adapter_name = connection.adapter_name.downcase
-        shell = Thor::Shell::Basic.new
-        puts "  Your DB contains #{count} addresses that may be affected by this task."
-        # `trim` is not needed for pg or mysql when using `concat_ws`:
-        # select concat_ws('joinstring', 'foo', null);
-        #  concat_ws
-        # -----------
-        #  foo
-        # (1 row)
-        # select concat_ws('joinstring', 'foo', null) = trim(concat_ws('joinstring', 'foo', null));
-        #  ?column?
-        # ----------
-        #  t
-        # (1 row)
-        unless count.zero?
-          concat_statement = begin
-            case adapter_name
-            when /sqlite/
-              "name = TRIM(COALESCE(firstname, '') || ' ' || COALESCE(lastname, ''))"
-            when /postgres/, /mysql2/
-              "name = CONCAT_WS(' ', firstname, lastname)"
-            else
-              abort "  No migration path available for adapter #{adapter_name}. Please write your own."
-            end
-          end
-
-          # The batch size should be limited to avoid locking the table records for too long. These are
-          # the numbers I got with 1_000_000 records in `spree_addresses`, all with different name and
-          # surname, with postgresql:
-          #
-          # Updating 1000000 records in one shot
-          # batch took 178 seconds
-          #
-          # Updating 1000000 addresses in batches of 200000
-          # batch took 36 seconds
-          # batch took 31 seconds
-          # batch took 31 seconds
-          # batch took 31 seconds
-          # batch took 30 seconds
-          #
-          # Updating 1000000 addresses in batches of 150000
-          # batch took 29 seconds
-          # batch took 27 seconds
-          # batch took 27 seconds
-          # batch took 27 seconds
-          # batch took 26 seconds
-          # batch took 26 seconds
-          # batch took 19 seconds
-          #
-          # Updating 1000000 addresses in batches of 100000
-          # batch took 17 seconds
-          # batch took 15 seconds
-          # batch took 17 seconds
-          # batch took 17 seconds
-          # batch took 17 seconds
-          # batch took 17 seconds
-          # batch took 17 seconds
-          # batch took 17 seconds
-          # batch took 17 seconds
-          # batch took 17 seconds
-          #
-          # This is with mysql:
-          # Updating 1000000 records in one shot
-          # batch updated in 153 seconds
-          #
-          # Updating 1000000 records in batches of 200000, this may take a while...
-          # batch took 41 seconds
-          # batch took 37 seconds
-          # batch took 35 seconds
-          # batch took 28 seconds
-          # batch took 27 seconds
-          #
-          # Updating 1000000 records in batches of 150000, this may take a while...
-          # batch took 30 seconds
-          # batch took 29 seconds
-          # batch took 18 seconds
-          # batch took 18 seconds
-          # batch took 17 seconds
-          # batch took 29 seconds
-          # batch took 12 seconds
-          #
-          # Updating 1000000 records in batches of 100000, this may take a while...
-          # batch took 10 seconds
-          # batch took 11 seconds
-          # batch took 12 seconds
-          # batch took 13 seconds
-          # batch took 12 seconds
-          # batch took 12 seconds
-          # batch took 14 seconds
-          # batch took 19 seconds
-          # batch took 20 seconds
-          # batch took 21 seconds
-          #
-          # Please note that the migration will be much faster when there's no index
-          # on the `name` column. For example, with mysql each batch takes exactly
-          # the same time:
-          #
-          # Updating 1000000 records in batches of 200000, this may take a while...
-          # batch took 17 seconds
-          # batch took 17 seconds
-          # batch took 17 seconds
-          # batch took 16 seconds
-          # batch took 17 seconds
-          #
-          # So, if special need arise, one can drop the index added with migration
-          # 20210122110141_add_name_to_spree_addresses.rb and add the index later,
-          # in non blocking ways. For postgresql:
-          # add_index(:spree_addresses, :name, algorithm: :concurrently)
-          #
-          # For mysql 5.6:
-          # sql = "ALTER TABLE spree_addresses ADD INDEX index_spree_addresses_on_name (name), ALGORITHM=INPLACE, LOCK=NONE;"
-          # ActiveRecord::Base.connection.execute sql
-          #
-          puts '  Data migration will happen in batches. The default value is 100_000, which should take less than 20 seconds on mysql or postgresql.'
-          size = shell.ask('  Please enter a different batch size, or press return to confirm the default: ')
-          size = (size.presence || 100_000).to_i
-
-          abort "  Invalid batch size number #{size}, please run the task again." unless size.positive?
-
-          batches_total = (count / size).ceil
-          puts "  We're going to migrate #{count} records in #{batches_total} batches of #{size}."
-
-          answer = shell.ask('  Do you want to proceed?', limited_to: ['Y', 'N'], case_insensitive: true)
-          if answer == 'Y'
-            puts "  Updating #{count} records in batches of #{size}, this may take a while..."
-
-            records.in_batches(of: size).each.with_index(1) do |batch, index|
-              now = Time.zone.now
-              batch.update_all(concat_statement)
-              puts "  Batch #{index}/#{batches_total} done in #{(Time.zone.now - now).to_i} seconds."
-            end
-          else
-            puts "  Database not migrated. Please, make sure to fill address's name field on your own."
-          end
-        end
-      end
-    end
-  end
-end

data/lib/tasks/migrations/migrate_default_billing_addresses_to_address_book.rake

@@ -1,38 +0,0 @@
-# frozen_string_literal: true
-
-namespace :solidus do
-  namespace :migrations do
-    namespace :migrate_default_billing_addresses_to_address_book do
-      task up: :environment do
-        print "Migrating default billing addresses to address book ... "
-        if Spree::UserAddress.where(default_billing: true).any?
-          Spree.user_class.joins(:bill_address).update_all(bill_address_id: nil) # rubocop:disable Rails/SkipsModelValidations
-        end
-        adapter_type = Spree::Base.connection.adapter_name.downcase.to_sym
-        if adapter_type == :mysql2
-          sql = <<~SQL
-            UPDATE spree_user_addresses
-            JOIN spree_users ON spree_user_addresses.user_id = spree_users.id
-              AND spree_user_addresses.address_id = spree_users.bill_address_id
-            SET spree_user_addresses.default_billing = true
-          SQL
-        else
-          sql = <<~SQL
-            UPDATE spree_user_addresses
-            SET default_billing = true
-            FROM spree_users
-            WHERE spree_user_addresses.address_id = spree_users.bill_address_id
-              AND spree_user_addresses.user_id = spree_users.id;
-          SQL
-        end
-        Spree::Base.connection.execute sql
-        puts "Success"
-      end
-
-      task down: :environment do
-        Spree::UserAddress.update_all(default_billing: false) # rubocop:disable Rails/SkipsModelValidations
-        puts "Rolled back default billing address migration to address book"
-      end
-    end
-  end
-end

data/lib/tasks/migrations/migrate_shipping_rate_taxes.rake

@@ -1,22 +0,0 @@
-# frozen_string_literal: true
-
-namespace :solidus do
-  namespace :migrations do
-    namespace :migrate_shipping_rate_taxes do
-      task up: :environment do
-        Spree::Deprecation.warn("rake spree:migrations:migrate_shipping_rate_taxes:up has been deprecated and will be removed with Solidus 3.0.")
-
-        print "Adding persisted tax notes to historic shipping rates ... "
-        Spree::ShippingRate.where.not(tax_rate_id: nil).find_each do |shipping_rate|
-          tax_rate = Spree::TaxRate.unscoped.find(shipping_rate.tax_rate_id)
-          shipping_rate.taxes.find_or_create_by!(
-            tax_rate: tax_rate,
-            amount: tax_rate.compute_amount(shipping_rate)
-          )
-        end
-        Spree::ShippingRate.where.not(tax_rate_id: nil).update_all(tax_rate_id: nil)
-        puts "Success."
-      end
-    end
-  end
-end

data/lib/tasks/migrations/migrate_user_addresses.rake

@@ -1,34 +0,0 @@
-# frozen_string_literal: true
-
-namespace 'spree:migrations:migrate_user_addresses' do
-  # This creates an entry in the user_addresses table for a user's currently
-  # associated shipping and billing addresses.
-
-  # This task should not need to be run more than once. But the de-dup behavior in
-  # UserAddressBook should make it safe.
-  # When the code (in the same PR) is deployed, each new order's addresses will be added
-  # to the user's address book. This will catch up all the historical data.
-
-  task up: :environment do
-    Spree::Deprecation.warn("rake spree:migrations:migrate_user_addresses:up has been deprecated and will be removed with Solidus 3.0.")
-
-    Spree.user_class.find_each(batch_size: 500) do |user|
-      ship_address = Spree::Address.find_by(id: user.ship_address_id)
-      bill_address = Spree::Address.find_by(id: user.bill_address_id)
-
-      current_addresses = [bill_address, ship_address].compact.uniq
-
-      current_addresses.each do |address|
-        # since ship_address is last, it will override bill_address as default when both are present
-        user.save_in_address_book(address.attributes, true)
-      end
-
-      puts "Migrated addresses for user ##{user.id}"
-    end
-  end
-
-  task down: :environment do
-    Spree::Deprecation.warn("rake spree:migrations:migrate_user_addresses:down has been deprecated and will be removed with Solidus 3.0.")
-    Spree::UserAddress.delete_all
-  end
-end

data/lib/tasks/migrations/rename_gateways.rake

@@ -1,23 +0,0 @@
-# frozen_string_literal: true
-
-require 'solidus/migrations/rename_gateways'
-
-namespace 'solidus:migrations:rename_gateways' do
-  task up: :environment do
-    Spree::Deprecation.warn("rake solidus:migrations:rename_gateways:up has been deprecated and will be removed with Solidus 3.0.")
-    count = Solidus::Migrations::RenameGateways.new.up
-
-    unless ENV['VERBOSE'] == 'false' || !verbose
-      puts "Renamed #{count} gateways into payment methods."
-    end
-  end
-
-  task down: :environment do
-    Spree::Deprecation.warn("rake solidus:migrations:rename_gateways:down has been deprecated and will be removed with Solidus 3.0.")
-    count = Solidus::Migrations::RenameGateways.new.down
-
-    unless ENV['VERBOSE'] == 'false' || !verbose
-      puts "Renamed #{count} payment methods into gateways."
-    end
-  end
-end

data/lib/tasks/order_capturing.rake

@@ -1,27 +0,0 @@
-# frozen_string_literal: true
-
-namespace :order_capturing do
-  desc "Looks for orders with inventory that is fully shipped/short-shipped, and captures money for it"
-  task capture_payments: :environment do
-    Spree::Deprecation.warn("rake order_capturing:capture_payments has been deprecated and will be removed with Solidus 3.0.")
-
-    failures = []
-    orders = Spree::Order.complete.where(payment_state: 'balance_due').where('completed_at > ?', Spree::Config[:order_capturing_time_window].days.ago)
-
-    orders.find_each do |order|
-      if order.inventory_units.all? { |iu| iu.canceled? || iu.shipped? }
-        if Spree::OrderCapturing.failure_handler
-          begin
-            Spree::OrderCapturing.new(order).capture_payments
-          rescue StandardError => e
-            failures << { message: "Order #{order.number} unable to capture. #{e.class}: #{e.message}" }
-          end
-        else
-          Spree::OrderCapturing.new(order).capture_payments
-        end
-      end
-    end
-
-    Spree::OrderCapturing.failure_handler.call(failures) if failures.any?
-  end
-end