solidus_core 1.2.3 → 1.3.0.beta1

data/db/migrate/20121017010007_remove_not_null_constraint_from_products_on_hand.rb

@@ -1,11 +1,11 @@
 class RemoveNotNullConstraintFromProductsOnHand < ActiveRecord::Migration
   def up
-    change_column :spree_products, :count_on_hand, :integer, :null => true
-    change_column :spree_variants, :count_on_hand, :integer, :null => true
+    change_column :spree_products, :count_on_hand, :integer, null: true
+    change_column :spree_variants, :count_on_hand, :integer, null: true
   end
 
   def down
-    change_column :spree_products, :count_on_hand, :integer, :null => false
-    change_column :spree_variants, :count_on_hand, :integer, :null => false
+    change_column :spree_products, :count_on_hand, :integer, null: false
+    change_column :spree_variants, :count_on_hand, :integer, null: false
   end
 end

data/db/migrate/20121031162139_split_prices_from_variants.rb

@@ -1,16 +1,16 @@
 class SplitPricesFromVariants < ActiveRecord::Migration
   def up
     create_table :spree_prices do |t|
-      t.integer :variant_id, :null => false
-      t.decimal :amount, :precision => 8, :scale => 2, :null => false
+      t.integer :variant_id, null: false
+      t.decimal :amount, precision: 8, scale: 2, null: false
       t.string :currency
     end
 
     Spree::Variant.all.each do |variant|
       Spree::Price.create!(
-        :variant_id => variant.id,
-        :amount => variant[:price],
-        :currency => Spree::Config[:currency]
+        variant_id: variant.id,
+        amount: variant[:price],
+        currency: Spree::Config[:currency]
       )
     end
 
@@ -19,13 +19,13 @@ class SplitPricesFromVariants < ActiveRecord::Migration
 
   def down
     prices = ActiveRecord::Base.connection.execute("select variant_id, amount from spree_prices")
-    add_column :spree_variants, :price, :decimal, :after => :sku, :scale => 2, :precision => 8
+    add_column :spree_variants, :price, :decimal, after: :sku, scale: 2, precision: 8
 
     prices.each do |price|
       ActiveRecord::Base.connection.execute("update spree_variants set price = #{price['amount']} where id = #{price['variant_id']}")
     end
-    
-    change_column :spree_variants, :price, :decimal, :after => :sku, :scale => 2, :precision => 8, :null => false
+
+    change_column :spree_variants, :price, :decimal, after: :sku, scale: 2, precision: 8, null: false
     drop_table :spree_prices
   end
 end

data/db/migrate/20121107003422_remove_not_null_from_spree_prices_amount.rb

@@ -1,9 +1,9 @@
 class RemoveNotNullFromSpreePricesAmount < ActiveRecord::Migration
   def up
-    change_column :spree_prices, :amount, :decimal, :precision => 8, :scale => 2, :null => true
+    change_column :spree_prices, :amount, :decimal, precision: 8, scale: 2, null: true
   end
 
   def down
-    change_column :spree_prices, :amount, :decimal, :precision => 8, :scale => 2, :null => false
+    change_column :spree_prices, :amount, :decimal, precision: 8, scale: 2, null: false
   end
 end

data/db/migrate/20121109173623_add_cost_currency_to_variants.rb

@@ -1,5 +1,5 @@
 class AddCostCurrencyToVariants < ActiveRecord::Migration
   def change
-    add_column :spree_variants, :cost_currency, :string, :after => :cost_price
+    add_column :spree_variants, :cost_currency, :string, after: :cost_price
   end
 end

data/db/migrate/20121124203911_add_position_to_taxonomies.rb

@@ -1,5 +1,5 @@
 class AddPositionToTaxonomies < ActiveRecord::Migration
   def change
-  	add_column :spree_taxonomies, :position, :integer, :default => 0
+    add_column :spree_taxonomies, :position, :integer, default: 0
   end
 end

data/db/migrate/20130120201805_add_position_to_product_properties.spree.rb

@@ -1,6 +1,5 @@
 class AddPositionToProductProperties < ActiveRecord::Migration
   def change
-    add_column :spree_product_properties, :position, :integer, :default => 0
+    add_column :spree_product_properties, :position, :integer, default: 0
   end
 end
-

data/db/migrate/20130211190146_create_spree_stock_items.rb

@@ -9,6 +9,6 @@ class CreateSpreeStockItems < ActiveRecord::Migration
       t.timestamps null: true
     end
     add_index :spree_stock_items, :stock_location_id
-    add_index :spree_stock_items, [:stock_location_id, :variant_id], :name => 'stock_item_by_loc_and_var_id'
+    add_index :spree_stock_items, [:stock_location_id, :variant_id], name: 'stock_item_by_loc_and_var_id'
   end
 end

data/db/migrate/20130213191427_create_default_stock.rb

@@ -1,34 +1,25 @@
 class CreateDefaultStock < ActiveRecord::Migration
+  class Variant < ActiveRecord::Base
+    self.table_name = 'spree_variants'
+  end
+  class StockLocation < ActiveRecord::Base
+    self.table_name = 'spree_stock_locations'
+  end
+  class StockItem < ActiveRecord::Base
+    self.table_name = 'spree_stock_items'
+  end
+
   def up
     unless column_exists? :spree_stock_locations, :default
       add_column :spree_stock_locations, :default, :boolean, null: false, default: false
     end
 
-    Spree::StockLocation.skip_callback(:create, :after, :create_stock_items)
-    Spree::StockLocation.skip_callback(:save, :after, :ensure_one_default)
-    Spree::StockItem.skip_callback(:save, :after, :process_backorders)
-    location = Spree::StockLocation.new(name: 'default')
-    location.save(validate: false)
+    location = StockLocation.create!(name: 'default')
 
     Spree::Variant.find_each do |variant|
-      stock_item = Spree::StockItem.unscoped.build(stock_location: location, variant: variant)
-      stock_item.send(:count_on_hand=, variant.count_on_hand)
-      # Avoid running default_scope defined by acts_as_paranoid, related to https://github.com/spree/spree/issues/3805,
-      # validations would run a query with a delete_at column that might not be present yet
-      stock_item.save! validate: false
+      StockItem.create!(stock_location: location, variant: variant, count_on_hand: variant.count_on_hand)
     end
 
     remove_column :spree_variants, :count_on_hand
   end
-
-  def down
-    add_column :spree_variants, :count_on_hand, :integer
-
-    Spree::StockItem.find_each do |stock_item|
-      stock_item.variant.update_column :count_on_hand, stock_item.count_on_hand
-    end
-
-    Spree::StockLocation.delete_all
-    Spree::StockItem.delete_all
-  end
 end

data/db/migrate/20130226032817_change_meta_description_on_spree_products_to_text.rb

@@ -1,5 +1,5 @@
 class ChangeMetaDescriptionOnSpreeProductsToText < ActiveRecord::Migration
   def change
-    change_column :spree_products, :meta_description, :text, :limit => nil
+    change_column :spree_products, :meta_description, :text, limit: nil
   end
 end

data/db/migrate/20130227143905_add_pending_to_inventory_unit.rb

@@ -1,6 +1,6 @@
 class AddPendingToInventoryUnit < ActiveRecord::Migration
   def change
-    add_column :spree_inventory_units, :pending, :boolean, :default => true
-    Spree::InventoryUnit.update_all(:pending => false)
+    add_column :spree_inventory_units, :pending, :boolean, default: true
+    Spree::InventoryUnit.update_all(pending: false)
   end
 end

data/db/migrate/20130228210442_create_shipping_method_zone.rb

@@ -3,7 +3,7 @@ class CreateShippingMethodZone < ActiveRecord::Migration
     self.table_name = 'shipping_methods_zones'
   end
   def up
-    create_table :shipping_methods_zones, :id => false do |t|
+    create_table :shipping_methods_zones, id: false do |t|
       t.integer :shipping_method_id
       t.integer :zone_id
     end

data/db/migrate/20130301162924_create_shipping_method_categories.rb

@@ -1,8 +1,8 @@
 class CreateShippingMethodCategories < ActiveRecord::Migration
   def change
     create_table :spree_shipping_method_categories do |t|
-      t.integer :shipping_method_id, :null => false
-      t.integer :shipping_category_id, :null => false
+      t.integer :shipping_method_id, null: false
+      t.integer :shipping_category_id, null: false
 
       t.timestamps null: true
     end

data/db/migrate/20130304162240_create_spree_shipping_rates.rb

@@ -3,13 +3,13 @@ class CreateSpreeShippingRates < ActiveRecord::Migration
     create_table :spree_shipping_rates do |t|
       t.belongs_to :shipment
       t.belongs_to :shipping_method
-      t.boolean :selected, :default => false
-      t.decimal :cost, :precision => 8, :scale => 2
+      t.boolean :selected, default: false
+      t.decimal :cost, precision: 8, scale: 2
       t.timestamps null: true
     end
     add_index(:spree_shipping_rates, [:shipment_id, :shipping_method_id],
-              :name => 'spree_shipping_rates_join_index',
-              :unique => true)
+              name: 'spree_shipping_rates_join_index',
+              unique: true)
 
     # Spree::Shipment.all.each do |shipment|
     #   shipping_method = Spree::ShippingMethod.find(shipment.shipment_method_id)

data/db/migrate/20130306181701_add_address_fields_to_stock_location.rb

@@ -11,12 +11,11 @@ class AddAddressFieldsToStockLocation < ActiveRecord::Migration
     add_column :spree_stock_locations, :zipcode, :string
     add_column :spree_stock_locations, :phone, :string
 
-
-    usa = Spree::Country.where(:iso => 'US').first
+    usa = Spree::Country.where(iso: 'US').first
     # In case USA isn't found.
     # See https://github.com/spree/spree/issues/3115
     country = usa || Spree::Country.first
     Spree::Country.reset_column_information
-    Spree::StockLocation.update_all(:country_id => country)
+    Spree::StockLocation.update_all(country_id: country)
   end
 end

data/db/migrate/20130306191917_add_active_field_to_stock_locations.rb

@@ -1,5 +1,5 @@
 class AddActiveFieldToStockLocations < ActiveRecord::Migration
   def change
-    add_column :spree_stock_locations, :active, :boolean, :default => true
+    add_column :spree_stock_locations, :active, :boolean, default: true
   end
 end

data/db/migrate/20130306195650_add_backorderable_to_stock_item.rb

@@ -1,5 +1,5 @@
 class AddBackorderableToStockItem < ActiveRecord::Migration
   def change
-    add_column :spree_stock_items, :backorderable, :boolean, :default => true
+    add_column :spree_stock_items, :backorderable, :boolean, default: true
   end
 end

data/db/migrate/20130307161754_add_default_quantity_to_stock_movement.rb

@@ -1,5 +1,5 @@
 class AddDefaultQuantityToStockMovement < ActiveRecord::Migration
   def change
-    change_column :spree_stock_movements, :quantity, :integer, :default => 0
+    change_column :spree_stock_movements, :quantity, :integer, default: 0
   end
 end

data/db/migrate/20130319062004_change_orders_total_precision.rb

@@ -1,8 +1,8 @@
 class ChangeOrdersTotalPrecision < ActiveRecord::Migration
    def change
-    change_column :spree_orders, :item_total,  :decimal, :precision => 10, :scale => 2, :default => 0.0, :null => false
-    change_column :spree_orders, :total,  :decimal, :precision => 10, :scale => 2, :default => 0.0, :null => false
-    change_column :spree_orders, :adjustment_total,  :decimal, :precision => 10, :scale => 2, :default => 0.0, :null => false
-    change_column :spree_orders, :payment_total,  :decimal, :precision => 10, :scale => 2, :default => 0.0                                
+    change_column :spree_orders, :item_total, :decimal, precision: 10, scale: 2, default: 0.0, null: false
+    change_column :spree_orders, :total, :decimal, precision: 10, scale: 2, default: 0.0, null: false
+    change_column :spree_orders, :adjustment_total, :decimal, precision: 10, scale: 2, default: 0.0, null: false
+    change_column :spree_orders, :payment_total, :decimal, precision: 10, scale: 2, default: 0.0
    end
 end

data/db/migrate/20130319063911_change_spree_payments_amount_precision.rb

@@ -1,7 +1,5 @@
 class ChangeSpreePaymentsAmountPrecision < ActiveRecord::Migration
   def change
-   
-    change_column :spree_payments, :amount,  :decimal, :precision => 10, :scale => 2, :default => 0.0, :null => false
-                                   
+    change_column :spree_payments, :amount, :decimal, precision: 10, scale: 2, default: 0.0, null: false
   end
 end

data/db/migrate/20130319064308_change_spree_return_authorization_amount_precision.rb

@@ -1,7 +1,5 @@
 class ChangeSpreeReturnAuthorizationAmountPrecision < ActiveRecord::Migration
-   def change
-   
-    change_column :spree_return_authorizations, :amount,  :decimal, :precision => 10, :scale => 2, :default => 0.0, :null => false
-                                   
+  def change
+    change_column :spree_return_authorizations, :amount, :decimal, precision: 10, scale: 2, default: 0.0, null: false
   end
 end

data/db/migrate/20130319082943_change_adjustments_amount_precision.rb

@@ -1,7 +1,5 @@
 class ChangeAdjustmentsAmountPrecision < ActiveRecord::Migration
   def change
-   
-    change_column :spree_adjustments, :amount,  :decimal, :precision => 10, :scale => 2
-                                   
+    change_column :spree_adjustments, :amount, :decimal, precision: 10, scale: 2
   end
 end

data/db/migrate/20130319190507_drop_source_and_destination_from_stock_movement.rb

@@ -1,8 +1,8 @@
 class DropSourceAndDestinationFromStockMovement < ActiveRecord::Migration
   def up
     change_table :spree_stock_movements do |t|
-      t.remove_references :source, :polymorphic => true
-      t.remove_references :destination, :polymorphic => true
+      t.remove_references :source, polymorphic: true
+      t.remove_references :destination, polymorphic: true
     end
   end
 

data/db/migrate/20130325163316_migrate_inventory_unit_sold_to_on_hand.rb

@@ -1,9 +1,9 @@
 class MigrateInventoryUnitSoldToOnHand < ActiveRecord::Migration
   def up
-    Spree::InventoryUnit.where(:state => 'sold').update_all(:state => 'on_hand')
+    Spree::InventoryUnit.where(state: 'sold').update_all(state: 'on_hand')
   end
 
   def down
-    Spree::InventoryUnit.where(:state => 'on_hand').update_all(:state => 'sold')
+    Spree::InventoryUnit.where(state: 'on_hand').update_all(state: 'sold')
   end
 end

data/db/migrate/20130417120035_update_adjustment_states.rb

@@ -1,14 +1,14 @@
 class UpdateAdjustmentStates < ActiveRecord::Migration
   def up
     Spree::Order.complete.find_each do |order|
-      order.adjustments.update_all(:state => 'closed')
+      order.adjustments.update_all(state: 'closed')
     end
 
     Spree::Shipment.shipped.includes(:adjustment).find_each do |shipment|
       shipment.adjustment.update_column(:state, 'finalized') if shipment.adjustment
     end
 
-    Spree::Adjustment.where(:state => nil).update_all(:state => 'open')
+    Spree::Adjustment.where(state: nil).update_all(state: 'open')
   end
 
   def down

data/db/migrate/20130417123427_add_shipping_rates_to_shipments.rb

@@ -1,9 +1,9 @@
 class AddShippingRatesToShipments < ActiveRecord::Migration
   def up
     Spree::Shipment.find_each do |shipment|
-      shipment.shipping_rates.create(:shipping_method_id => shipment.shipping_method_id,
-                                     :cost => shipment.cost,
-                                     :selected => true)
+      shipment.shipping_rates.create(shipping_method_id: shipment.shipping_method_id,
+                                     cost: shipment.cost,
+                                     selected: true)
     end
 
     remove_column :spree_shipments, :shipping_method_id

data/db/migrate/20130628021056_add_unique_index_to_permalink_on_spree_products.rb

@@ -1,5 +1,5 @@
 class AddUniqueIndexToPermalinkOnSpreeProducts < ActiveRecord::Migration
   def change
-    add_index "spree_products", ["permalink"], :name => "permalink_idx_unique", :unique => true
+    add_index "spree_products", ["permalink"], name: "permalink_idx_unique", unique: true
   end
 end

data/db/migrate/20130628022817_add_unique_index_to_orders_shipments_and_stock_transfers.rb

@@ -1,7 +1,7 @@
 class AddUniqueIndexToOrdersShipmentsAndStockTransfers < ActiveRecord::Migration
   def add
-    add_index "spree_orders", ["number"], :name => "number_idx_unique", :unique => true
-    add_index "spree_shipments", ["number"], :name => "number_idx_unique", :unique => true
-    add_index "spree_stock_transfers", ["number"], :name => "number_idx_unique", :unique => true
+    add_index "spree_orders", ["number"], name: "number_idx_unique", unique: true
+    add_index "spree_shipments", ["number"], name: "number_idx_unique", unique: true
+    add_index "spree_stock_transfers", ["number"], name: "number_idx_unique", unique: true
   end
 end

data/db/migrate/20130718042445_add_cost_price_to_line_item.rb

@@ -1,5 +1,5 @@
 class AddCostPriceToLineItem < ActiveRecord::Migration
   def change
-    add_column :spree_line_items, :cost_price, :decimal, :precision => 8, :scale => 2
+    add_column :spree_line_items, :cost_price, :decimal, precision: 8, scale: 2
   end
 end

data/db/migrate/20130718233855_set_backorderable_to_default_to_false.rb

@@ -1,6 +1,6 @@
 class SetBackorderableToDefaultToFalse < ActiveRecord::Migration
   def change
-    change_column :spree_stock_items, :backorderable, :boolean, :default => false
-    change_column :spree_stock_locations, :backorderable_default, :boolean, :default => false
+    change_column :spree_stock_items, :backorderable, :boolean, default: false
+    change_column :spree_stock_locations, :backorderable_default, :boolean, default: false
   end
 end

data/db/migrate/20130807024301_upgrade_adjustments.rb

@@ -5,7 +5,7 @@ class UpgradeAdjustments < ActiveRecord::Migration
       belongs_to :originator, polymorphic: true
     end
     # Shipping adjustments are now tracked as fields on the object
-    Spree::Adjustment.where(:source_type => "Spree::Shipment").find_each do |adjustment|
+    Spree::Adjustment.where(source_type: "Spree::Shipment").find_each do |adjustment|
       # Account for possible invalid data
       next if adjustment.source.nil?
       adjustment.source.update_column(:cost, adjustment.amount)
@@ -13,14 +13,14 @@ class UpgradeAdjustments < ActiveRecord::Migration
     end
 
     # Tax adjustments have their sources altered
-    Spree::Adjustment.where(:originator_type => "Spree::TaxRate").find_each do |adjustment|
+    Spree::Adjustment.where(originator_type: "Spree::TaxRate").find_each do |adjustment|
       adjustment.source_id = adjustment.originator_id
       adjustment.source_type = "Spree::TaxRate"
       adjustment.save!
     end
 
     # Promotion adjustments have their source altered also
-    Spree::Adjustment.where(:originator_type => "Spree::PromotionAction").find_each do |adjustment|
+    Spree::Adjustment.where(originator_type: "Spree::PromotionAction").find_each do |adjustment|
       next if adjustment.originator.nil?
       adjustment.source = adjustment.originator
       begin

data/db/migrate/20130813004002_add_shipment_total_to_spree_orders.rb

@@ -1,5 +1,5 @@
 class AddShipmentTotalToSpreeOrders < ActiveRecord::Migration
   def change
-    add_column :spree_orders, :shipment_total, :decimal, :precision => 10, :scale => 2, :default => 0.0, :null => false
+    add_column :spree_orders, :shipment_total, :decimal, precision: 10, scale: 2, default: 0.0, null: false
   end
 end

data/db/migrate/20130813140619_expand_order_number_size.rb

@@ -1,9 +1,9 @@
 class ExpandOrderNumberSize < ActiveRecord::Migration
   def up
-    change_column :spree_orders, :number, :string, :limit => 32
+    change_column :spree_orders, :number, :string, limit: 32
   end
 
   def down
-    change_column :spree_orders, :number, :string, :limit => 15
+    change_column :spree_orders, :number, :string, limit: 15
   end
 end

data/db/migrate/20130815000406_add_adjustment_total_to_line_items.rb

@@ -1,5 +1,5 @@
 class AddAdjustmentTotalToLineItems < ActiveRecord::Migration
   def change
-    add_column :spree_line_items, :adjustment_total, :decimal, :precision => 10, :scale => 2, :default => 0.0
+    add_column :spree_line_items, :adjustment_total, :decimal, precision: 10, scale: 2, default: 0.0
   end
 end

data/db/migrate/20130815024413_add_adjustment_total_to_shipments.rb

@@ -1,5 +1,5 @@
 class AddAdjustmentTotalToShipments < ActiveRecord::Migration
   def change
-    add_column :spree_shipments, :adjustment_total, :decimal, :precision => 10, :scale => 2, :default => 0.0
+    add_column :spree_shipments, :adjustment_total, :decimal, precision: 10, scale: 2, default: 0.0
   end
 end

data/db/migrate/20130826062534_add_depth_to_spree_taxons.rb

@@ -5,7 +5,7 @@ class AddDepthToSpreeTaxons < ActiveRecord::Migration
 
       say_with_time 'Update depth on all taxons' do
         Spree::Taxon.reset_column_information
-        Spree::Taxon.all.each { |t| t.save }
+        Spree::Taxon.all.each(&:save)
       end
     end
   end

data/db/migrate/20130830001033_add_shipping_category_to_shipping_methods_and_products.rb

@@ -1,7 +1,7 @@
 class AddShippingCategoryToShippingMethodsAndProducts < ActiveRecord::Migration
   def up
     default_category = Spree::ShippingCategory.first
-    default_category ||= Spree::ShippingCategory.create!(:name => "Default")
+    default_category ||= Spree::ShippingCategory.create!(name: "Default")
 
     Spree::ShippingMethod.all.each do |method|
       method.shipping_categories << default_category if method.shipping_categories.blank?

data/db/migrate/20130830001159_migrate_old_shipping_calculators.rb

@@ -3,7 +3,7 @@ class MigrateOldShippingCalculators < ActiveRecord::Migration
     Spree::ShippingMethod.all.each do |shipping_method|
       old_calculator = shipping_method.calculator
       next if old_calculator.class < Spree::ShippingCalculator # We don't want to mess with new shipping calculators
-      new_calculator = eval(old_calculator.class.name.sub("::Calculator::", "::Calculator::Shipping::")).new
+      new_calculator = old_calculator.class.name.sub("::Calculator::", "::Calculator::Shipping::").constantize
       new_calculator.preferences.keys.each do |pref|
         # Preferences can't be read/set by name, you have to prefix preferred_
         pref_method = "preferred_#{pref}"

data/db/migrate/20131001013410_remove_unused_credit_card_fields.rb

@@ -4,6 +4,7 @@ class RemoveUnusedCreditCardFields < ActiveRecord::Migration
     remove_column :spree_credit_cards, :start_year if column_exists?(:spree_credit_cards, :start_year)
     remove_column :spree_credit_cards, :issue_number if column_exists?(:spree_credit_cards, :issue_number)
   end
+
   def down
     add_column :spree_credit_cards, :start_month,  :string
     add_column :spree_credit_cards, :start_year,   :string

data/db/migrate/20131026154747_add_track_inventory_to_variant.rb

@@ -1,5 +1,5 @@
 class AddTrackInventoryToVariant < ActiveRecord::Migration
   def change
-    add_column :spree_variants, :track_inventory, :boolean, :default => true
+    add_column :spree_variants, :track_inventory, :boolean, default: true
   end
 end

data/db/migrate/20131118043959_add_included_to_adjustments.rb

@@ -1,5 +1,5 @@
 class AddIncludedToAdjustments < ActiveRecord::Migration
   def change
-    add_column :spree_adjustments, :included, :boolean, :default => false unless Spree::Adjustment.column_names.include?("included")
+    add_column :spree_adjustments, :included, :boolean, default: false unless Spree::Adjustment.column_names.include?("included")
   end
 end

data/db/migrate/20131118183431_add_line_item_id_to_spree_inventory_units.rb

@@ -9,7 +9,6 @@ class AddLineItemIdToSpreeInventoryUnits < ActiveRecord::Migration
 
       shipments.find_each do |shipment|
         shipment.inventory_units.group_by(&:variant_id).each do |variant_id, units|
-
           line_item = shipment.order.line_items.find_by(variant_id: variant_id)
           next unless line_item
 

data/db/migrate/20131211112807_create_spree_orders_promotions.rb

@@ -1,6 +1,6 @@
 class CreateSpreeOrdersPromotions < ActiveRecord::Migration
   def change
-    create_table :spree_orders_promotions, :id => false do |t|
+    create_table :spree_orders_promotions, id: false do |t|
       t.references :order
       t.references :promotion
     end

data/db/migrate/20131218054603_add_item_count_to_spree_orders.rb

@@ -1,5 +1,5 @@
 class AddItemCountToSpreeOrders < ActiveRecord::Migration
   def change
-    add_column :spree_orders, :item_count, :integer, :default => 0
+    add_column :spree_orders, :item_count, :integer, default: 0
   end
 end

data/db/migrate/20140106065820_remove_value_type_from_spree_preferences.rb

@@ -2,6 +2,7 @@ class RemoveValueTypeFromSpreePreferences < ActiveRecord::Migration
   def up
     remove_column :spree_preferences, :value_type
   end
+
   def down
     raise ActiveRecord::IrreversableMigration
   end

data/db/migrate/20140219060952_add_considered_risky_to_orders.rb

@@ -1,5 +1,5 @@
 class AddConsideredRiskyToOrders < ActiveRecord::Migration
   def change
-    add_column :spree_orders, :considered_risky, :boolean, :default => false
+    add_column :spree_orders, :considered_risky, :boolean, default: false
   end
 end

data/db/migrate/20140309023735_migrate_old_preferences.rb

@@ -11,7 +11,8 @@ class MigrateOldPreferences < ActiveRecord::Migration
   end
 
   private
-  def migrate_preferences klass
+
+  def migrate_preferences(klass)
     klass.reset_column_information
     klass.find_in_batches do |batch|
       ActiveRecord::Base.transaction do
@@ -36,7 +37,7 @@ class MigrateOldPreferences < ActiveRecord::Migration
     end
   end
 
-  def cache_key model, key
+  def cache_key(model, key)
     [
       ENV["RAILS_CACHE_ID"],
       class_underscore_cache[model.type],
@@ -46,6 +47,6 @@ class MigrateOldPreferences < ActiveRecord::Migration
   end
 
   def class_underscore_cache
-    @class_underscore_cache ||= Hash.new{ |h,k| h[k] = k.underscore }
+    @class_underscore_cache ||= Hash.new{ |h, k| h[k] = k.underscore }
   end
 end

data/db/migrate/20140309033438_create_store_from_preferences.rb

@@ -12,10 +12,10 @@ class CreateStoreFromPreferences < ActiveRecord::Migration
     else
       # we set defaults for the things we now require
       Store.new do |s|
-        s.name              = preference_store.get 'spree/app_configuration/site_name' do
+        s.name = preference_store.get 'spree/app_configuration/site_name' do
           'Sample Store'
         end
-        s.url               = preference_store.get 'spree/app_configuration/site_url' do
+        s.url = preference_store.get 'spree/app_configuration/site_url' do
           'example.com'
         end
         s.mail_from_address = preference_store.get 'spree/app_configuration/mails_from' do