solidus_core 2.3.1 → 2.4.0.beta1

data/db/migrate/20151126063028_convert_habtm_to_hmt_for_taxons_promotion_rules.rb

@@ -1,15 +0,0 @@
-class ConvertHabtmToHmtForTaxonsPromotionRules < ActiveRecord::Migration[4.2]
-  def up
-    add_column :spree_taxons_promotion_rules, :created_at, :datetime
-    add_column :spree_taxons_promotion_rules, :updated_at, :datetime
-
-    rename_table :spree_taxons_promotion_rules, :spree_promotion_rule_taxons
-  end
-
-  def down
-    rename_table :spree_promotion_rule_taxons, :spree_taxons_promotion_rules
-
-    remove_column :spree_taxons_promotion_rules, :created_at, :datetime
-    remove_column :spree_taxons_promotion_rules, :updated_at, :datetime
-  end
-end

data/db/migrate/20151219020209_add_stock_item_unique_index.rb

@@ -1,9 +0,0 @@
-class AddStockItemUniqueIndex < ActiveRecord::Migration[4.2]
-  def change
-    # Add a database-level uniqueness constraint for databases that support it
-    # (postgres and sqlite > 3.8)
-    if connection.supports_partial_index?
-      add_index 'spree_stock_items', ['variant_id', 'stock_location_id'], where: 'deleted_at is null', unique: true
-    end
-  end
-end

data/db/migrate/20160111091912_add_column_available_to_all_on_spree_shipping_method.rb

@@ -1,5 +0,0 @@
-class AddColumnAvailableToAllOnSpreeShippingMethod < ActiveRecord::Migration[4.2]
-  def change
-    add_column :spree_shipping_methods, :available_to_all, :boolean, default: true
-  end
-end

data/db/migrate/20160116133604_add_position_to_spree_payment_methods.rb

@@ -1,5 +0,0 @@
-class AddPositionToSpreePaymentMethods < ActiveRecord::Migration[4.2]
-  def change
-    add_column :spree_payment_methods, :position, :integer, default: 0
-  end
-end

data/db/migrate/20160122182105_add_carrier_and_service_level_to_spree_shipping_methods.rb

@@ -1,6 +0,0 @@
-class AddCarrierAndServiceLevelToSpreeShippingMethods < ActiveRecord::Migration[4.2]
-  def change
-    add_column :spree_shipping_methods, :carrier, :string
-    add_column :spree_shipping_methods, :service_level, :string
-  end
-end

data/db/migrate/20160224201413_create_spree_shipping_rate_taxes.rb

@@ -1,11 +0,0 @@
-class CreateSpreeShippingRateTaxes < ActiveRecord::Migration[4.2]
-  def change
-    create_table :spree_shipping_rate_taxes do |t|
-      t.decimal :amount, precision: 8, scale: 2, default: 0.0, null: false
-      t.references :tax_rate, index: true
-      t.references :shipping_rate, index: true
-
-      t.timestamps null: false
-    end
-  end
-end

data/db/migrate/20160229133259_add_cart_tax_country_iso_to_spree_store.rb

@@ -1,5 +0,0 @@
-class AddCartTaxCountryIsoToSpreeStore < ActiveRecord::Migration[4.2]
-  def change
-    add_column :spree_stores, :cart_tax_country_iso, :string, null: true, default: nil
-  end
-end

data/db/migrate/20160301103333_remove_pre_tax_amount_on_line_item_and_shipment.rb

@@ -1,6 +0,0 @@
-class RemovePreTaxAmountOnLineItemAndShipment < ActiveRecord::Migration[4.2]
-  def change
-    remove_column :spree_line_items, :pre_tax_amount, :decimal, precision: 12, scale: 4, default: 0.0, null: false
-    remove_column :spree_shipments, :pre_tax_amount, :decimal, precision: 12, scale: 4, default: 0.0, null: false
-  end
-end

data/db/migrate/20160308000300_disallow_adjustment_finalized_nulls.rb

@@ -1,19 +0,0 @@
-class DisallowAdjustmentFinalizedNulls < ActiveRecord::Migration[4.2]
-  def up
-    execute <<-SQL
-      update spree_adjustments
-      set finalized = #{ActiveRecord::Base.connection.quoted_false}
-      where finalized is null
-    SQL
-
-    change_table :spree_adjustments do |t|
-      t.change :finalized, :boolean, null: false, default: false
-    end
-  end
-
-  def down
-    change_table :spree_adjustments do |t|
-      t.change :finalized, :boolean, null: true, default: nil
-    end
-  end
-end

data/db/migrate/20160318145302_add_timestamps_to_prices.rb

@@ -1,5 +0,0 @@
-class AddTimestampsToPrices < ActiveRecord::Migration[4.2]
-  def change
-    change_table(:spree_prices) { |t| t.timestamps null: true }
-  end
-end

data/db/migrate/20160330204846_add_missing_timestamp_columns.rb

@@ -1,20 +0,0 @@
-class AddMissingTimestampColumns < ActiveRecord::Migration[4.2]
-  def change
-    # Missing updated_at
-    add_column :friendly_id_slugs, :updated_at, :datetime, null: true
-
-    # Missing created_at
-    add_column :spree_countries, :created_at, :datetime, null: true
-    add_column :spree_states, :created_at, :datetime, null: true
-    add_column :spree_variants, :created_at, :datetime, null: true
-
-    # Missing timestamps
-    add_timestamps(:spree_option_values_variants, null: true)
-    add_timestamps(:spree_products_taxons, null: true)
-    add_timestamps(:spree_promotion_action_line_items, null: true)
-    add_timestamps(:spree_promotion_actions, null: true)
-    add_timestamps(:spree_reimbursement_credits, null: true)
-    add_timestamps(:spree_roles, null: true)
-    add_timestamps(:spree_variant_property_rule_values, null: true)
-  end
-end

data/db/migrate/20160420044191_create_spree_wallet_payment_sources.rb

@@ -1,23 +0,0 @@
-class CreateSpreeWalletPaymentSources < ActiveRecord::Migration[4.2]
-  def change
-    create_table :spree_wallet_payment_sources do |t|
-      t.references(
-        :user,
-        foreign_key: { to_table: Spree.user_class.table_name },
-        index: true,
-        null: false,
-      )
-      t.references :payment_source, polymorphic: true, null: false
-      t.boolean :default, default: false, null: false
-
-      t.timestamps null: false
-    end
-
-    add_index(
-      :spree_wallet_payment_sources,
-      [:user_id, :payment_source_id, :payment_source_type],
-      unique: true,
-      name: 'index_spree_wallet_payment_sources_on_source_and_user',
-    )
-  end
-end

data/db/migrate/20160420181916_migrate_credit_cards_to_wallet_payment_sources.rb

@@ -1,27 +0,0 @@
-class MigrateCreditCardsToWalletPaymentSources < ActiveRecord::Migration[4.2]
-  class CreditCard < ActiveRecord::Base
-    self.table_name = 'spree_credit_cards'
-  end
-  class WalletPaymentSource < ActiveRecord::Base
-    self.table_name = 'spree_wallet_payment_sources'
-  end
-
-  def up
-    credit_cards = CreditCard.
-      where.not(gateway_customer_profile_id: nil).
-      where.not(user_id: nil)
-
-    credit_cards.find_each do |credit_card|
-      WalletPaymentSource.find_or_create_by!(
-        user_id: credit_card.user_id,
-        payment_source_id: credit_card.id,
-        payment_source_type: 'Spree::CreditCard'
-      ) do |wallet_source|
-        wallet_source.default = credit_card.default
-      end
-    end
-  end
-
-  def down
-  end
-end

data/db/migrate/20160509181311_add_country_iso_to_prices.rb

@@ -1,8 +0,0 @@
-class AddCountryIsoToPrices < ActiveRecord::Migration[4.2]
-  def change
-    add_column :spree_prices, :country_iso, :string, null: true, limit: 2
-
-    add_index :spree_prices, :country_iso
-    add_index :spree_countries, :iso
-  end
-end

data/db/migrate/20160527070401_rename_shipment_address_field.rb

@@ -1,7 +0,0 @@
-class RenameShipmentAddressField < ActiveRecord::Migration[4.2]
-  def change
-    change_table :spree_shipments do |t|
-      t.rename :address_id, :deprecated_address_id
-    end
-  end
-end

data/db/migrate/20160608162651_ensure_default_store.rb

@@ -1,13 +0,0 @@
-class EnsureDefaultStore < ActiveRecord::Migration[4.2]
-  class Store < ActiveRecord::Base
-    self.table_name = 'spree_stores'
-  end
-
-  def up
-    unless Store.where(default: true).exists?
-      store = Store.first
-      raise "Database has no stores. One should have been created in a previous migration." unless store
-      store.update_column(:default, true)
-    end
-  end
-end

data/db/migrate/20160608180751_ensure_store_on_orders.rb

@@ -1,12 +0,0 @@
-class EnsureStoreOnOrders < ActiveRecord::Migration[4.2]
-  class Store < ActiveRecord::Base
-    self.table_name = 'spree_stores'
-  end
-  class Order < ActiveRecord::Base
-    self.table_name = 'spree_orders'
-  end
-  def up
-    default_store = Store.find_by(default: true)
-    Order.where(store_id: nil).update_all(store_id: default_store.id)
-  end
-end

data/db/migrate/20160616232103_remove_user_id_from_promotion_rules.rb

@@ -1,11 +0,0 @@
-class RemoveUserIdFromPromotionRules < ActiveRecord::Migration[4.2]
-  def up
-    remove_index :spree_promotion_rules, name: 'index_promotion_rules_on_user_id'
-    remove_column :spree_promotion_rules, :user_id
-  end
-
-  def down
-    add_column :spree_promotion_rules, :user_id, :integer
-    add_index :spree_promotion_rules, [:user_id], name: 'index_promotion_rules_on_user_id'
-  end
-end

data/db/migrate/20160718205341_add_payment_id_index_to_spree_refunds.rb

@@ -1,5 +0,0 @@
-class AddPaymentIdIndexToSpreeRefunds < ActiveRecord::Migration[4.2]
-  def change
-    add_index(:spree_refunds, :payment_id)
-  end
-end

data/db/migrate/20160718205859_add_reimbursement_id_index_to_spree_refunds.rb

@@ -1,5 +0,0 @@
-class AddReimbursementIdIndexToSpreeRefunds < ActiveRecord::Migration[4.2]
-  def change
-    add_index(:spree_refunds, :reimbursement_id)
-  end
-end

data/lib/spree/core/controller_helpers/respond_with.rb

@@ -1,65 +0,0 @@
-require 'spree/responder'
-
-module ActionController
-  class Base
-    def respond_with(*resources, &block)
-      if Spree::BaseController.spree_responders.keys.include?(self.class.to_s.to_sym)
-        # Checkout AS Array#extract_options! and original respond_with
-        # implementation for a better picture of this hack
-        if resources.last.is_a? Hash
-          resources.last[:action_name] = action_name.to_sym
-        else
-          resources.push action_name: action_name.to_sym
-        end
-      end
-
-      super
-    end
-  end
-end
-
-module Spree
-  module Core
-    module ControllerHelpers
-      module RespondWith
-        extend ActiveSupport::Concern
-
-        included do
-          cattr_accessor :spree_responders
-          self.spree_responders = {}
-          self.responder = Spree::Responder
-        end
-
-        module ClassMethods
-          def clear_overrides!
-            self.spree_responders = {}
-          end
-
-          def respond_override(options = {})
-            unless options.blank?
-              action_name = options.keys.first
-              action_value = options.values.first
-
-              if action_name.blank? || action_value.blank?
-                raise ArgumentError, "invalid values supplied #{options.inspect}"
-              end
-
-              format_name = action_value.keys.first
-              format_value = action_value.values.first
-
-              if format_name.blank? || format_value.blank?
-                raise ArgumentError, "invalid values supplied #{options.inspect}"
-              end
-
-              if format_value.is_a?(Proc)
-                options = { action_name.to_sym => { format_name.to_sym => { success: format_value } } }
-              end
-
-              spree_responders.deep_merge!(name.to_sym => options)
-            end
-          end
-        end
-      end
-    end
-  end
-end

data/lib/spree/core/price_migrator.rb

@@ -1,30 +0,0 @@
-module Spree
-  # This class performs a data migration. It's usually run from
-  # the `solidus:migrations:create_vat_prices` rake task.
-  class PriceMigrator
-    # Migrate all variant's prices.
-    def self.migrate_default_vat_prices
-      # We need to tag the exisiting prices as "default", so that the VatPriceGenerator knows
-      # that they include the default zone's VAT.
-      Spree::Config.admin_vat_country_iso = Spree::Zone.default_tax.countries.first.iso
-      Spree::Variant.find_each do |variant|
-        new(variant).migrate_vat_prices
-      end
-      # This line stops all weird code paths that generate refunds from happening.
-      Spree::Zone.default_tax.update(default_tax: false)
-    end
-
-    attr_reader :variant
-
-    def initialize(variant)
-      @variant = variant
-    end
-
-    def migrate_vat_prices
-      # With a default tax zone, all prices include VAT by default. Let's tell them which one!
-      variant.prices.update_all(country_iso: Spree::Config.admin_vat_country_iso)
-      Spree::Variant::VatPriceGenerator.new(variant).run
-      variant.save
-    end
-  end
-end

data/lib/spree/responder.rb

@@ -1,44 +0,0 @@
-module Spree
-  class Responder < ::ActionController::Responder #:nodoc:
-    attr_accessor :on_success, :on_failure
-
-    def initialize(controller, resources, options = {})
-      super
-
-      class_name = controller.class.name.to_sym
-      action_name = options.delete(:action_name)
-
-      if result = Spree::BaseController.spree_responders[class_name].try(:[], action_name).try(:[], format.to_sym)
-        self.on_success = handler(controller, result, :success)
-        self.on_failure = handler(controller, result, :failure)
-      end
-    end
-
-    def to_html
-      super && return if !(on_success || on_failure)
-      has_errors? ? controller.instance_exec(&on_failure) : controller.instance_exec(&on_success)
-    end
-
-    def to_format
-      super && return if !(on_success || on_failure)
-      has_errors? ? controller.instance_exec(&on_failure) : controller.instance_exec(&on_success)
-    end
-
-    private
-
-    def handler(controller, result, status)
-      return result if result.respond_to? :call
-
-      case result
-      when Hash
-        if result[status].is_a? Symbol
-          controller.method(result[status])
-        else
-          result[status]
-        end
-      when Symbol
-        controller.method(result)
-      end
-    end
-  end
-end

data/lib/spree/testing_support/factories/stock_packer_factory.rb

@@ -1,13 +0,0 @@
-require 'spree/testing_support/factories/stock_location_factory'
-
-FactoryGirl.define do
-  # must use build()
-  factory :stock_packer, class: Spree::Stock::Packer do
-    transient do
-      stock_location { build(:stock_location) }
-      contents []
-    end
-
-    initialize_with { new(stock_location, contents) }
-  end
-end

data/lib/tasks/migrations/copy_shipped_shipments_to_cartons.rake

@@ -1,172 +0,0 @@
-namespace 'spree:migrations:copy_shipped_shipments_to_cartons' do
-  # This copies data from shipments into cartons from previous versions of
-  # Spree.
-
-  # Used in the migration CopyShippedShipmentsToCartons and made available as a
-  # rake task to allow running it a second time after deploying the new code, in
-  # case some shipment->carton 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.
-
-  # We're doing these via SQL because for large stores with lots of shipments
-  # and lots of inventory units this would take excessively long to do via
-  # ActiveRecord one at a time. Also, these queries can take a long time for
-  # large stores so we do them in batches.
-
-  task up: :environment do
-    bad_shipping_rate = Spree::ShippingRate.
-      select(:shipment_id).
-      where(selected: true).
-      group(:shipment_id).
-      having("count(0) > 1").
-      limit(1).to_a.first
-
-    if bad_shipping_rate
-      # This would end up generating multiple cartons for a single shipment
-      raise(<<-TEXT.squish)
-        Error: You have shipments with more than one 'selected' shipping rate,
-        such as shipment #{bad_shipping_rate.shipment_id}. This code will not
-        work correctly.
-      TEXT
-    end
-
-    say_with_time 'generating cartons' do
-      last_id = Spree::Shipment.last.try!(:id) || 0
-
-      in_batches(last_id: last_id) do |start_id, end_id|
-        say_with_time "processing shipment #{start_id} to #{end_id}" do
-          Spree::Carton.connection.execute(<<-SQL.strip_heredoc)
-            insert into spree_cartons
-              (
-                number, imported_from_shipment_id, stock_location_id,
-                address_id, shipping_method_id, tracking, shipped_at,
-                created_at, updated_at
-              )
-            select
-              -- create the carton number as 'C'+shipment number:
-              #{db_concat("'C'", 'spree_shipments.number')}, -- number
-              spree_shipments.id, -- imported_from_shipment_id
-              spree_shipments.stock_location_id,
-              spree_orders.ship_address_id,
-              spree_shipping_rates.shipping_method_id,
-              spree_shipments.tracking,
-              spree_shipments.shipped_at,
-              '#{Time.current.to_s(:db)}', -- created_at
-              '#{Time.current.to_s(:db)}' -- updated_at
-            from spree_shipments
-            left join spree_orders
-              on spree_orders.id = spree_shipments.order_id
-            left join spree_shipping_rates
-              on spree_shipping_rates.shipment_id = spree_shipments.id
-              and spree_shipping_rates.selected = #{Spree::Carton.connection.quoted_true}
-            left join spree_inventory_units
-              on spree_inventory_units.shipment_id = spree_shipments.id
-              and spree_inventory_units.carton_id is not null
-            where spree_shipments.shipped_at is not null
-            -- must have at least one inventory unit
-            and exists (
-              select 1
-              from spree_inventory_units iu
-              where iu.shipment_id = spree_shipments.id
-            )
-            -- if *any* inventory units are connected to cartons then we assume
-            -- the entire shipment has been either already migrated or handled
-            -- by the new code
-            and spree_inventory_units.id is null
-            and spree_shipments.id >= #{start_id}
-            and spree_shipments.id <= #{end_id}
-          SQL
-        end
-      end
-    end
-
-    say_with_time 'linking inventory units to cartons' do
-      last_id = Spree::InventoryUnit.last.try!(:id) || 0
-
-      in_batches(last_id: last_id) do |start_id, end_id|
-        say_with_time "processing inventory units #{start_id} to #{end_id}" do
-          Spree::InventoryUnit.connection.execute(<<-SQL.strip_heredoc)
-            update spree_inventory_units
-            set carton_id = (
-              select spree_cartons.id
-              from spree_shipments
-              inner join spree_cartons
-                on spree_cartons.imported_from_shipment_id = spree_shipments.id
-              where spree_shipments.id = spree_inventory_units.shipment_id
-            )
-            where spree_inventory_units.carton_id is null
-            and spree_inventory_units.shipment_id is not null
-            and spree_inventory_units.id >= #{start_id}
-            and spree_inventory_units.id <= #{end_id}
-          SQL
-        end
-      end
-    end
-  end
-
-  task down: :environment do
-    last_id = Spree::InventoryUnit.last.try!(:id) || 0
-
-    say_with_time 'unlinking inventory units from cartons' do
-      in_batches(last_id: last_id) do |start_id, end_id|
-        say_with_time "processing inventory units #{start_id} to #{end_id}" do
-          Spree::InventoryUnit.connection.execute(<<-SQL.strip_heredoc)
-            update spree_inventory_units
-            set carton_id = null
-            where carton_id is not null
-            and exists (
-              select 1
-              from spree_cartons
-              where spree_cartons.id = spree_inventory_units.carton_id
-              and spree_cartons.imported_from_shipment_id is not null
-            )
-            and spree_inventory_units.id >= #{start_id}
-            and spree_inventory_units.id <= #{end_id}
-          SQL
-        end
-      end
-    end
-
-    say_with_time "clearing carton imported_from_shipment_ids" do
-      Spree::Carton.where.not(imported_from_shipment_id: nil).delete_all
-    end
-  end
-
-  def db_concat(*args)
-    case Spree::Shipment.connection.adapter_name
-    when /mysql/i
-      "concat(#{args.join(', ')})"
-    else
-      args.join(' || ')
-    end
-  end
-
-  def say_with_time(message)
-    say message
-    ms = Benchmark.ms { yield }
-    say "(#{ms.round}ms)"
-  end
-
-  def say(message)
-    if Rails.env.test?
-      Rails.logger.info message
-    else
-      puts message
-    end
-  end
-
-  def in_batches(last_id:)
-    start_id = 1
-    batch_size = 10_000
-
-    while start_id <= last_id
-      end_id = start_id + batch_size - 1
-
-      yield start_id, end_id
-
-      start_id += batch_size
-    end
-  end
-end