solidus_core 1.3.0 → 1.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: e46ab6204fe773f2c5a9eca4e58848739f4a4086
-  data.tar.gz: 5cf71d6898cf7b9990fda8396cb07f609d883267
+  metadata.gz: 4c7402622fc30f6fef2fc3cad846e07adaeac3bf
+  data.tar.gz: 808c7fc752dbe3298afc6131a49509df4086202d
 SHA512:
-  metadata.gz: 97f2fa3bd1d807f5310878b3eb08b3718bd53c896e09dcc533362a89c7ab2f6eae4e61eb864669b3a22a547e6e0f74338f2bab12f02a3ccb22ee03e4d49caf2e
-  data.tar.gz: db37f8907d4d04c010a26253ffca83494c440bca20d4c32d0c6a7e481ac65f77c0cb20956ca6ca6708d4f11d0f681f4b00d887ba892b263c161f1447bf72cbca
+  metadata.gz: ff019cdef35791c28148b68397ac795b78ec86f2eed1d0d7cd33c34383b7bc5cd24342d79fe0e7c7fb95a3c77a3242df739419d821aca59ac6f5b5007d786250
+  data.tar.gz: ceabe9026cfb4469481120ab7ad9152fc02fe20e03435950f42b92149a3347eb64538d14d822760821204fd211830ad05b0a8afba6dbe76258bc454ec5f0f44b

data/lib/spree/core/version.rb

@@ -5,7 +5,7 @@ module Spree
   end
 
   def self.solidus_version
-    "1.3.0"
+    "1.3.1"
   end
 
   def self.solidus_gem_version

data/lib/tasks/upgrade.rake

@@ -2,7 +2,6 @@ namespace :solidus do
   namespace :upgrade do
     desc "Upgrade Solidus to version 1.3"
     task one_point_three: [
-        'solidus:migrations:ensure_store_on_orders:up',
         'solidus:migrations:migrate_shipping_rate_taxes:up',
         'solidus:migrations:create_vat_prices:up'
       ] do

data/spec/lib/tasks/upgrade_spec.rb

@@ -8,7 +8,6 @@ describe 'solidus:upgrade:one_point_three' do
   )
 
   before do
-    load Spree::Core::Engine.root.join('lib/tasks/migrations/ensure_store_on_orders.rake')
     load Spree::Core::Engine.root.join('lib/tasks/migrations/migrate_shipping_rate_taxes.rake')
     load Spree::Core::Engine.root.join('lib/tasks/migrations/create_vat_prices.rake')
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: solidus_core
 version: !ruby/object:Gem::Version
-  version: 1.3.0
+  version: 1.3.1
 platform: ruby
 authors:
 - Solidus Team
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-06-22 00:00:00.000000000 Z
+date: 2016-07-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activemerchant
@@ -1054,7 +1054,6 @@ files:
 - lib/tasks/migrations/copy_order_bill_address_to_credit_card.rake
 - lib/tasks/migrations/copy_shipped_shipments_to_cartons.rake
 - lib/tasks/migrations/create_vat_prices.rake
-- lib/tasks/migrations/ensure_store_on_orders.rake
 - lib/tasks/migrations/migrate_shipping_rate_taxes.rake
 - lib/tasks/migrations/migrate_user_addresses.rake
 - lib/tasks/order_capturing.rake
@@ -1159,7 +1158,6 @@ files:
 - spec/lib/tasks/exchanges_spec.rb
 - spec/lib/tasks/migrations/copy_shipped_shipments_to_cartons_spec.rb
 - spec/lib/tasks/migrations/create_vat_prices_spec.rb
-- spec/lib/tasks/migrations/ensure_store_on_orders_spec.rb
 - spec/lib/tasks/migrations/migrate_shipping_rate_taxes_spec.rb
 - spec/lib/tasks/order_capturing_spec.rb
 - spec/lib/tasks/upgrade_spec.rb

data/lib/tasks/migrations/ensure_store_on_orders.rake

@@ -1,38 +0,0 @@
-namespace :solidus do
-  namespace :migrations do
-    namespace :ensure_store_on_orders do
-      desc "Makes sure every order in the system has a store attached"
-      task up: :environment do
-
-        attach_store_to_all_orders
-      end
-
-      def attach_store_to_all_orders
-        orders_without_store_count = Spree::Order.where(store_id: nil).count
-        if orders_without_store_count == 0
-          puts "Everything is good, all orders in your database have a store attached."
-          return
-        end
-
-        spree_store_count = Spree::Store.count
-        if spree_store_count == 0
-          raise "You do not have a store set up. Please create a store instance for your installation."
-        elsif spree_store_count > 1
-          raise(<<-TEXT.squish)
-            You have more than one store set up. We can not be sure which store to attach your
-            orders to. Please attach store ids to all your orders, and run this task again
-            when you're finished.
-          TEXT
-        end
-
-        default_store = Spree::Store.where(default: true).first
-        unless default_store
-          raise "Your store is not marked as default. Please mark your one store as the default store and run this task again."
-        end
-
-        Spree::Order.where(store_id: nil).update_all(store_id: Spree::Store.default.id)
-        puts "All orders updated with the default store."
-      end
-    end
-  end
-end

data/spec/lib/tasks/migrations/ensure_store_on_orders_spec.rb

@@ -1,80 +0,0 @@
-require 'spec_helper'
-
-describe 'solidus:migrations:ensure_store_on_orders' do
-  describe 'up' do
-    include_context(
-      'rake',
-      task_path: Spree::Core::Engine.root.join('lib/tasks/migrations/ensure_store_on_orders.rake'),
-      task_name: 'solidus:migrations:ensure_store_on_orders:up',
-    )
-
-    context 'with no orders' do
-      it 'succeeds' do
-        expect { task.invoke }.to output(
-          "Everything is good, all orders in your database have a store attached.\n"
-        ).to_stdout
-      end
-    end
-
-    context 'when all orders have store_ids' do
-      let!(:order) { create(:order, store: create(:store)) }
-
-      it 'succeeds' do
-        expect { task.invoke }.to output(
-          "Everything is good, all orders in your database have a store attached.\n"
-        ).to_stdout
-      end
-    end
-
-    context 'when some orders do not have store_ids' do
-      let!(:order_with_store) { create(:order, store: store) }
-      let!(:order_without_store) do
-        # due to a before_validation that adds a store when one is missing,
-        # we can't simply specify `store: nil`
-        create(:order, store: store).tap do |o|
-          o.update_columns(store_id: nil)
-        end
-      end
-      let!(:store) { create(:store) }
-
-      it 'succeeds' do
-        expect { task.invoke }.to output(
-          "All orders updated with the default store.\n"
-        ).to_stdout
-      end
-
-      context 'when there are no stores' do
-        before do
-          order_with_store.update_columns(store_id: nil)
-          # due to a before_validation that adds a store when one is missing,
-          # we can't simply specify `store: nil`
-          store.destroy
-        end
-
-        it 'raises' do
-          expect { task.invoke }.to raise_error(/You do not have a store set up/)
-        end
-      end
-
-      context 'when there are multiple stores' do
-        let!(:extra_store) { create(:store) }
-
-        it 'raises' do
-          expect { task.invoke }.to raise_error(/You have more than one store set up/)
-        end
-      end
-
-      context 'when there is no default store' do
-        before do
-          # The before_save 'ensure_default_exists_and_is_unique' means that we
-          # need to use update_columns to set up this scenario.
-          store.update_columns(default: false)
-        end
-
-        it 'raises' do
-          expect { task.invoke }.to raise_error(/Your store is not marked as default/)
-        end
-      end
-    end
-  end
-end