solidus_core 2.10.5 → 3.0.8

data/lib/tasks/core.rake

@@ -1,104 +0,0 @@
-# frozen_string_literal: true
-
-require 'active_record'
-
-def prompt_for_agree(prompt)
-  print prompt
-  ["y", "yes"].include? STDIN.gets.strip.downcase
-end
-
-namespace :db do
-  desc 'Loads a specified fixture file:
-use rake db:load_file[/absolute/path/to/sample/filename.rb]'
-
-  task :load_file, [:file, :dir] => :environment do |_t, args|
-    Spree::Deprecation.warn("load_file has been deprecated. Please load your own file.")
-    file = Pathname.new(args.file)
-
-    puts "loading ruby #{file}"
-    require file
-  end
-
-  desc "Loads fixtures from the the dir you specify using rake db:load_dir[loadfrom]"
-  task :load_dir, [:dir] => :environment do |_t, args|
-    Spree::Deprecation.warn("rake spree:load_dir has been deprecated and will be removed with Solidus 3.0. Please load your files directly.")
-    dir = args.dir
-    dir = File.join(Rails.root, "db", dir) if Pathname.new(dir).relative?
-
-    ruby_files = {}
-    Dir.glob(File.join(dir, '**/*.{rb}')).each do |fixture_file|
-      ruby_files[File.basename(fixture_file, '.*')] = fixture_file
-    end
-    ruby_files.sort.each do |fixture, ruby_file|
-      # If file is exists within application it takes precendence.
-      if File.exist?(File.join(Rails.root, "db/default/spree", "#{fixture}.rb"))
-        ruby_file = File.expand_path(File.join(Rails.root, "db/default/spree", "#{fixture}.rb"))
-      end
-      # an invoke will only execute the task once
-      Rake::Task["db:load_file"].execute( Rake::TaskArguments.new([:file], [ruby_file]) )
-    end
-  end
-
-  desc "Migrate schema to version 0 and back up again. WARNING: Destroys all data in tables!!"
-  task remigrate: :environment do
-    Spree::Deprecation.warn("remigrate has been deprecated. Please use db:reset or other db: commands instead.")
-
-    if ENV['SKIP_NAG'] || ENV['OVERWRITE'].to_s.casecmp('true') || prompt_for_agree("This task will destroy any data in the database. Are you sure you want to \ncontinue? [y/n] ")
-
-      # Drop all tables
-      ActiveRecord::Base.connection.tables.each { |t| ActiveRecord::Base.connection.drop_table t }
-
-      # Migrate upward
-      Rake::Task["db:migrate"].invoke
-
-      # Dump the schema
-      Rake::Task["db:schema:dump"].invoke
-    else
-      puts "Task cancelled."
-      exit
-    end
-  end
-
-  desc "Bootstrap is: migrating, loading defaults, sample data and seeding (for all extensions) and load_products tasks"
-  task :bootstrap do
-    Spree::Deprecation.warn("rake bootstrap has been deprecated, please run db:setup instead.")
-
-    # remigrate unless production mode (as saftey check)
-    if %w[demo development test].include? Rails.env
-      if ENV['AUTO_ACCEPT'] || prompt_for_agree("This task will destroy any data in the database. Are you sure you want to \ncontinue? [y/n] ")
-        ENV['SKIP_NAG'] = 'yes'
-        Rake::Task["db:create"].invoke
-        Rake::Task["db:remigrate"].invoke
-      else
-        puts "Task cancelled, exiting."
-        exit
-      end
-    else
-      puts "NOTE: Bootstrap in production mode will not drop database before migration"
-      Rake::Task["db:migrate"].invoke
-    end
-
-    ActiveRecord::Base.send(:subclasses).each(&:reset_column_information)
-
-    load_defaults = Spree::Country.count == 0
-    load_defaults ||= prompt_for_agree('Countries present, load sample data anyways? [y/n]: ')
-    if load_defaults
-      Rake::Task["db:seed"].invoke
-    end
-
-    if Rails.env.production? && Spree::Product.count > 0
-      load_sample = prompt_for_agree("WARNING: In Production and products exist in database, load sample data anyways? [y/n]:" )
-    else
-      load_sample = true if ENV['AUTO_ACCEPT']
-      load_sample ||= prompt_for_agree('Load Sample Data? [y/n]: ')
-    end
-
-    if load_sample
-      # Reload models' attributes in case they were loaded in old migrations with wrong attributes
-      ActiveRecord::Base.descendants.each(&:reset_column_information)
-      Rake::Task["spree_sample:load"].invoke
-    end
-
-    puts "Bootstrap Complete.\n\n"
-  end
-end

data/lib/tasks/email.rake

@@ -1,11 +0,0 @@
-# frozen_string_literal: true
-
-namespace :email do
-  desc 'Sends test email to specified address - Example: EMAIL=spree@example.com bundle exec rake email:test'
-  task test: :environment do
-    Spree::Deprecation.warn("rake email:test has been deprecated and will be removed with Solidus 3.0")
-
-    raise ArgumentError, "Must pass EMAIL environment variable. Example: EMAIL=spree@example.com bundle exec rake email:test" unless ENV['EMAIL'].present?
-    Spree::TestMailer.test_email(ENV['EMAIL']).deliver!
-  end
-end

data/lib/tasks/migrations/copy_order_bill_address_to_credit_card.rake

@@ -1,119 +0,0 @@
-# frozen_string_literal: true
-
-namespace 'spree:migrations:copy_order_bill_address_to_credit_card' do
-  # This copies the billing address from the order associated with a
-  # credit card's most recent payment to the credit card.
-
-  # Used in the migration CopyOrderBillAddressToCreditCard and made available as a
-  # rake task to allow running it a second time after deploying the new code, in
-  # case some order->credit card data was missed between the time that the
-  # migration was run and the application servers were restarted with the new
-  # code.
-
-  # This task should be safe to run multiple times.
-
-  task up: :environment do
-    Spree::Deprecation.warn("rake spree:migrations:copy_order_bill_address_to_credit_card:up has been deprecated and will be removed with Solidus 3.0.")
-
-    if Spree::CreditCard.connection.adapter_name =~ /postgres/i
-      postgres_copy
-    else
-      ruby_copy
-    end
-  end
-
-  task down: :environment do
-    Spree::Deprecation.warn("rake spree:migrations:copy_order_bill_address_to_credit_card:down has been deprecated and will be removed with Solidus 3.0.")
-
-    Spree::CreditCard.update_all(address_id: nil)
-  end
-
-  def ruby_copy
-    scope = Spree::CreditCard.where(address_id: nil).includes(payments: :order)
-
-    scope.find_each(batch_size: 500) do |cc|
-      # remove payments that lack a bill address
-      payments = cc.payments.select { |p| p.order.bill_address_id }
-
-      payment = payments.sort_by do |p|
-        [
-          %w(failed invalid).include?(p.state) ? 0 : 1, # prioritize valid payments
-          p.created_at, # prioritize more recent payments
-        ]
-      end.last
-
-      next if payment.nil?
-
-      cc.update_column(:address_id, payment.order.bill_address_id)
-      puts "Successfully associated billing address (#{payment.order.bill_address_id}) with credit card (#{cc.id})"
-    end
-  end
-
-  # This was 20x faster for us but the syntax is postgres-specific. I'm sure
-  # there are equivalent versions for other DBs if someone wants to write them.
-  # I took a quick stab at crafting a cross-db compatible version but it was
-  # slow.
-  def postgres_copy
-    batch_size = 10_000
-
-    current_start_id = 1
-
-    while current_start_id <= last_credit_card_id
-      current_end_id = current_start_id + batch_size
-      puts "updating #{current_start_id} to #{current_end_id}"
-
-      # first try to find a valid payment for each credit card
-      Spree::CreditCard.connection.execute(
-        postgres_sql(
-          start_id: current_start_id,
-          end_id: current_end_id,
-          payment_state: "not in ('failed', 'invalid')"
-        )
-      )
-
-      # fall back to using invalid payments for each credit card
-      Spree::CreditCard.connection.execute(
-        postgres_sql(
-          start_id: current_start_id,
-          end_id: current_end_id,
-          payment_state: "in ('failed', 'invalid')"
-        )
-      )
-
-      current_start_id += batch_size
-    end
-  end
-
-  def postgres_sql(start_id:, end_id:, payment_state:)
-    <<-SQL
-      update spree_credit_cards c
-      set address_id = o.bill_address_id
-      from spree_payments p
-      inner join spree_orders o
-        on  o.id = p.order_id
-        and o.bill_address_id is not null
-      left join (
-        select p2.*
-        from spree_payments p2
-        inner join spree_orders o2
-          on  o2.id = p2.order_id
-          and o2.bill_address_id is not null
-      ) more_recent_payment
-        on  more_recent_payment.source_id = p.source_id
-        and more_recent_payment.source_type = 'Spree::CreditCard'
-        and more_recent_payment.created_at > p.created_at
-        and more_recent_payment.state #{payment_state}
-      where c.address_id is null
-        and p.source_id = c.id
-        and p.source_type = 'Spree::CreditCard'
-        and p.state #{payment_state}
-        and more_recent_payment.id is null
-        and o.bill_address_id is not null
-        and c.id between #{start_id} and #{end_id}
-    SQL
-  end
-
-  def last_credit_card_id
-    Spree::CreditCard.last.try!(:id) || 0
-  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

data/script/rails

@@ -1,10 +0,0 @@
-#!/usr/bin/env ruby
-# frozen_string_literal: true
-
-# This command will automatically be run when you run "rails" with Rails 3 gems installed from the root of your application.
-
-ENGINE_ROOT = File.expand_path('..', __dir__)
-ENGINE_PATH = File.expand_path('../lib/spree/core/engine', __dir__)
-
-require 'rails/all'
-require 'rails/engine/commands'

data/spec/helpers/base_helper_spec.rb

@@ -1,168 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-
-RSpec.describe Spree::BaseHelper, type: :helper do
-  include Spree::BaseHelper
-
-  let(:current_store){ create :store }
-
-  context "available_countries" do
-    let(:country) { create(:country) }
-
-    before do
-      3.times { create(:country) }
-    end
-
-    context "with no checkout zone defined" do
-      before do
-        stub_spree_preferences(checkout_zone: nil)
-      end
-
-      it "return complete list of countries" do
-        expect(available_countries.count).to eq(Spree::Country.count)
-      end
-
-      it "uses locales for country names" do
-        expect(available_countries).to include(having_attributes(name: "United States of America"))
-      end
-    end
-
-    context "with a checkout zone defined" do
-      context "checkout zone is of type country" do
-        before do
-          @country_zone = create(:zone, name: "CountryZone")
-          @country_zone.members.create(zoneable: country)
-          stub_spree_preferences(checkout_zone: @country_zone.name)
-        end
-
-        it "return only the countries defined by the checkout zone" do
-          expect(available_countries).to eq([country])
-        end
-
-        it "returns only the countries defined by the checkout zone passed as parameter" do
-          expect(available_countries(restrict_to_zone: @country_zone.name)).to eq([country])
-        end
-      end
-
-      context "checkout zone is of type state" do
-        before do
-          state_zone = create(:zone, name: "StateZone")
-          state = create(:state, country: country)
-          state_zone.members.create(zoneable: state)
-          stub_spree_preferences(checkout_zone: state_zone.name)
-        end
-
-        it "return complete list of countries" do
-          expect(available_countries.count).to eq(Spree::Country.count)
-        end
-      end
-    end
-  end
-
-  # Regression test for https://github.com/spree/spree/issues/2034
-  context "flash_message" do
-    let(:flash) { { "notice" => "ok", "foo" => "foo", "bar" => "bar" } }
-
-    it "should output all flash content" do
-      flash_messages
-      html = Nokogiri::HTML(helper.output_buffer)
-      expect(html.css(".notice").text).to eq("ok")
-      expect(html.css(".foo").text).to eq("foo")
-      expect(html.css(".bar").text).to eq("bar")
-    end
-
-    it "should output flash content except one key" do
-      flash_messages(ignore_types: :bar)
-      html = Nokogiri::HTML(helper.output_buffer)
-      expect(html.css(".notice").text).to eq("ok")
-      expect(html.css(".foo").text).to eq("foo")
-      expect(html.css(".bar").text).to be_empty
-    end
-
-    it "should output flash content except some keys" do
-      flash_messages(ignore_types: [:foo, :bar])
-      html = Nokogiri::HTML(helper.output_buffer)
-      expect(html.css(".notice").text).to eq("ok")
-      expect(html.css(".foo").text).to be_empty
-      expect(html.css(".bar").text).to be_empty
-      expect(helper.output_buffer).to eq("<div class=\"flash notice\">ok</div>")
-    end
-  end
-
-  context "link_to_tracking" do
-    it "returns tracking link if available" do
-      a = link_to_tracking_html(shipping_method: true, tracking: '123', tracking_url: 'http://g.c/?t=123').css('a')
-
-      expect(a.text).to eq '123'
-      expect(a.attr('href').value).to eq 'http://g.c/?t=123'
-    end
-
-    it "returns tracking without link if link unavailable" do
-      html = link_to_tracking_html(shipping_method: true, tracking: '123', tracking_url: nil)
-      expect(html.css('span').text).to eq '123'
-    end
-
-    it "returns nothing when no shipping method" do
-      html = link_to_tracking_html(shipping_method: nil, tracking: '123')
-      expect(html.css('span').text).to eq ''
-    end
-
-    it "returns nothing when no tracking" do
-      html = link_to_tracking_html(tracking: nil)
-      expect(html.css('span').text).to eq ''
-    end
-
-    def link_to_tracking_html(options = {})
-      node = link_to_tracking(double(:shipment, options))
-      Nokogiri::HTML(node.to_s)
-    end
-  end
-
-  # Regression test for https://github.com/spree/spree/issues/2396
-  context "meta_data_tags" do
-    it "truncates a product description to 160 characters" do
-      # Because the controller_name method returns "test"
-      # controller_name is used by this method to infer what it is supposed
-      # to be generating meta_data_tags for
-      @test = Spree::Product.new(description: "a" * 200)
-      tags = Nokogiri::HTML.parse(meta_data_tags)
-      content = tags.css("meta[name=description]").first["content"]
-      expect(content.length).to be <= 160
-    end
-  end
-
-  describe "#pretty_time" do
-    subject { pretty_time(date) }
-
-    let(:date) { Time.new(2012, 11, 6, 13, 33) }
-
-    it "pretty prints time in long format" do
-      is_expected.to eq "November 06, 2012 1:33 PM"
-    end
-
-    context 'with format set to short' do
-      subject { pretty_time(date, :short) }
-
-      it "pretty prints time in short format" do
-        is_expected.to eq "Nov 6 '12 1:33pm"
-      end
-    end
-  end
-
-  context "plural_resource_name" do
-    let(:plural_config) { Spree::I18N_GENERIC_PLURAL }
-    let(:base_class) { Spree::Product }
-
-    subject { plural_resource_name(base_class) }
-
-    it "should use ActiveModel::Naming module to pluralize model names" do
-      expect(subject).to eq base_class.model_name.human(count: plural_config)
-    end
-
-    it "should use the Spree::I18N_GENERIC_PLURAL constant" do
-      expect(base_class.model_name).to receive(:human).with(hash_including(count: plural_config))
-      subject
-    end
-  end
-end