solidus_core 4.6.2 → 4.7.1

data/app/models/spree/wallet/add_payment_sources_to_wallet.rb

@@ -16,10 +16,10 @@ class Spree::Wallet::AddPaymentSourcesToWallet
     if !order.temporary_payment_source && order.user
       # select valid sources
       payments = order.payments.valid
-      sources = payments.map(&:source).
-        uniq.
-        compact.
-        select { |payment| payment.try(:reusable?) }
+      sources = payments.map(&:source)
+        .uniq
+        .compact
+        .select { |payment| payment.try(:reusable?) }
 
       # add valid sources to wallet and optionally set a default
       if sources.any?

data/app/models/spree/wallet/default_payment_builder.rb

@@ -17,7 +17,7 @@ class Spree::Wallet::DefaultPaymentBuilder
     if default && order.payments.where(source_type: default.payment_source_type).none?
       Spree::Payment.new(
         payment_method: default.payment_source.payment_method,
-        source: default.payment_source,
+        source: default.payment_source
       )
     end
   end

data/app/models/spree/wallet_payment_source.rb

@@ -2,7 +2,7 @@
 
 module Spree
   class WalletPaymentSource < Spree::Base
-    belongs_to :user, class_name: Spree::UserClassHandle.new, foreign_key: 'user_id', inverse_of: :wallet_payment_sources, optional: true
+    belongs_to :user, class_name: Spree::UserClassHandle.new, foreign_key: "user_id", inverse_of: :wallet_payment_sources, optional: true
     belongs_to :payment_source, polymorphic: true, inverse_of: :wallet_payment_sources, optional: true
 
     validates :user, presence: true
@@ -27,7 +27,7 @@ module Spree
       return unless payment_source.present?
 
       if payment_source.respond_to?(:user_id) &&
-         payment_source.user_id != user_id
+          payment_source.user_id != user_id
         errors.add(:payment_source, :not_owned_by_user)
       end
     end

data/app/models/spree/zone.rb

@@ -13,7 +13,7 @@ module Spree
     has_many :shipping_method_zones, dependent: :destroy
     has_many :shipping_methods, through: :shipping_method_zones
 
-    validates :name, presence: true, uniqueness: { allow_blank: true, case_sensitive: true }
+    validates :name, presence: true, uniqueness: {allow_blank: true, case_sensitive: true}
     after_save :remove_defunct_members
 
     scope :with_member_ids, ->(state_ids, country_ids) do
@@ -22,11 +22,11 @@ module Spree
       else
         spree_zone_members_table = Spree::ZoneMember.arel_table
         matching_state =
-          spree_zone_members_table[:zoneable_type].eq("Spree::State").
-          and(spree_zone_members_table[:zoneable_id].in(state_ids))
+          spree_zone_members_table[:zoneable_type].eq("Spree::State")
+            .and(spree_zone_members_table[:zoneable_id].in(state_ids))
         matching_country =
-          spree_zone_members_table[:zoneable_type].eq("Spree::Country").
-          and(spree_zone_members_table[:zoneable_id].in(country_ids))
+          spree_zone_members_table[:zoneable_type].eq("Spree::Country")
+            .and(spree_zone_members_table[:zoneable_id].in(country_ids))
         joins(:zone_members).where(matching_state.or(matching_country)).distinct
       end
     end
@@ -39,8 +39,8 @@ module Spree
       end
     end
 
-    alias :members :zone_members
-    accepts_nested_attributes_for :zone_members, allow_destroy: true, reject_if: proc { |member| member['zoneable_id'].blank? }
+    alias_method :members, :zone_members
+    accepts_nested_attributes_for :zone_members, allow_destroy: true, reject_if: proc { |member| member["zoneable_id"].blank? }
 
     self.allowed_ransackable_attributes = %w[name description]
 
@@ -74,9 +74,9 @@ module Spree
 
       members.any? do |zone_member|
         case zone_member.zoneable_type
-        when 'Spree::Country'
+        when "Spree::Country"
           zone_member.zoneable_id == address.country_id
-        when 'Spree::State'
+        when "Spree::State"
           zone_member.zoneable_id == address.state_id
         else
           false
@@ -87,10 +87,10 @@ module Spree
     # convenience method for returning the countries contained within a zone
     def country_list
       @countries ||= case kind
-                     when 'country' then zoneables
-                     when 'state' then zoneables.collect(&:country)
-                     else []
-                     end.flatten.compact.uniq
+      when "country" then zoneables
+      when "state" then zoneables.collect(&:country)
+      else []
+      end.flatten.compact.uniq
     end
 
     def <=>(other)
@@ -104,7 +104,7 @@ module Spree
     end
 
     def country_ids
-      if kind == 'country'
+      if kind == "country"
         members.pluck(:zoneable_id)
       else
         []
@@ -112,7 +112,7 @@ module Spree
     end
 
     def state_ids
-      if kind == 'state'
+      if kind == "state"
         members.pluck(:zoneable_id)
       else
         []
@@ -120,18 +120,18 @@ module Spree
     end
 
     def country_ids=(ids)
-      set_zone_members(ids, 'Spree::Country')
+      set_zone_members(ids, "Spree::Country")
     end
 
     def state_ids=(ids)
-      set_zone_members(ids, 'Spree::State')
+      set_zone_members(ids, "Spree::State")
     end
 
     private
 
     def remove_defunct_members
       if zone_members.any?
-        zone_members.where('zoneable_id IS NULL OR zoneable_type != ?', "Spree::#{kind.classify}").destroy_all
+        zone_members.where("zoneable_id IS NULL OR zoneable_type != ?", "Spree::#{kind.classify}").destroy_all
       end
     end
 

data/app/models/spree/zone_member.rb

@@ -2,7 +2,7 @@
 
 module Spree
   class ZoneMember < Spree::Base
-    belongs_to :zone, class_name: 'Spree::Zone', counter_cache: true, inverse_of: :zone_members, optional: true
+    belongs_to :zone, class_name: "Spree::Zone", counter_cache: true, inverse_of: :zone_members, optional: true
     belongs_to :zoneable, polymorphic: true, optional: true
 
     delegate :name, to: :zoneable, allow_nil: true

data/app/subscribers/spree/carton_shipped_mailer_subscriber.rb

@@ -6,8 +6,8 @@ module Spree
     include Omnes::Subscriber
 
     handle :carton_shipped,
-           with: :send_carton_shipped_emails,
-           id: :spree_carton_mailer_send_carton_shipped_email
+      with: :send_carton_shipped_emails,
+      id: :spree_carton_mailer_send_carton_shipped_email
 
     # Sends carton shipped emails to users.
     #

data/app/subscribers/spree/order_cancel_mailer_subscriber.rb

@@ -6,8 +6,8 @@ module Spree
     include Omnes::Subscriber
 
     handle :order_canceled,
-           with: :send_cancel_email,
-           id: :spree_order_mailer_send_cancel_email
+      with: :send_cancel_email,
+      id: :spree_order_mailer_send_cancel_email
 
     # Sends cancellation email to the user.
     #

data/app/subscribers/spree/order_confirmation_mailer_subscriber.rb

@@ -6,8 +6,8 @@ module Spree
     include Omnes::Subscriber
 
     handle :order_finalized,
-           with: :send_confirmation_email,
-           id: :spree_order_mailer_send_confirmation_email
+      with: :send_confirmation_email,
+      id: :spree_order_mailer_send_confirmation_email
 
     # Sends confirmation email to the user.
     #

data/app/subscribers/spree/order_inventory_cancellation_mailer_subscriber.rb

@@ -6,8 +6,8 @@ module Spree
     include Omnes::Subscriber
 
     handle :order_short_shipped,
-           with: :send_inventory_cancellation_email,
-           id: :spree_order_mailer_send_inventory_cancellation_email
+      with: :send_inventory_cancellation_email,
+      id: :spree_order_mailer_send_inventory_cancellation_email
 
     # Sends inventory cancellation email to the user.
     #

data/app/subscribers/spree/reimbursement_mailer_subscriber.rb

@@ -6,8 +6,8 @@ module Spree
     include Omnes::Subscriber
 
     handle :reimbursement_reimbursed,
-           with: :send_reimbursement_email,
-           id: :spree_order_mailer_send_reimbursement_email
+      with: :send_reimbursement_email,
+      id: :spree_order_mailer_send_reimbursement_email
 
     # Sends reimbursement email to the user
     #

data/app/views/layouts/spree/base_mailer.html.erb

@@ -3,6 +3,7 @@
 <head>
 	<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
 	<meta name="viewport" content="width=device-width">
+	<%# The Ink CSS below is MIT licensed; see ink.license.txt in this directory. %>
 	<style>
 /**********************************************
 * Ink v1.0.5 - Copyright 2013 ZURB Inc        *

data/app/views/layouts/spree/ink.license.txt

@@ -0,0 +1,28 @@
+The mailer layout in this directory (base_mailer.html.erb) inlines the
+CSS of Ink v1.0.5, ZURB's responsive email framework, from
+https://github.com/foundation/foundation-emails (formerly zurb/ink).
+
+--------------------------------------------------------------------------------
+
+Copyright (c) 2013 ZURB, inc.
+
+MIT License
+
+Permission is hereby granted, free of charge, to any person obtaining
+a copy of this software and associated documentation files (the
+"Software"), to deal in the Software without restriction, including
+without limitation the rights to use, copy, modify, merge, publish,
+distribute, sublicense, and/or sell copies of the Software, and to
+permit persons to whom the Software is furnished to do so, subject to
+the following conditions:
+
+The above copyright notice and this permission notice shall be
+included in all copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE
+LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
+OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
+WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

data/config/initializers/assets.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 if Rails.application.config.respond_to?(:assets)
-  Rails.application.config.assets.precompile << 'solidus_core_manifest.js'
+  Rails.application.config.assets.precompile << "solidus_core_manifest.js"
 end

data/config/initializers/db_query_matchers.rb

@@ -0,0 +1,9 @@
+# frozen_string_literal: true
+
+require "db-query-matchers"
+
+DBQueryMatchers.configure do |config|
+  config.ignores = [/SHOW TABLES LIKE/]
+  config.ignore_cached = true
+  config.schemaless = true
+end

data/config/initializers/friendly_id.rb

@@ -18,8 +18,8 @@ FriendlyId.defaults do |config|
   # undesirable to allow as slugs. Edit this list as needed for your app.
   config.use :reserved
 
-  config.reserved_words = %w(new edit index session login logout users admin
-                             stylesheets assets javascripts images)
+  config.reserved_words = %w[new edit index session login logout users admin
+    stylesheets assets javascripts images]
 
   #  ## Friendly Finders
   #

data/config/initializers/inflections.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 ActiveSupport::Inflector.inflections(:en) do |inflect|
-  inflect.acronym 'RMA'
+  inflect.acronym "RMA"
 end

data/config/initializers/money.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'money'
+require "money"
 
 Money.locale_backend = :i18n
 Money.rounding_mode = BigDecimal::ROUND_HALF_EVEN

data/config/locales/en.yml

@@ -182,6 +182,7 @@ en:
         meta_title: Meta Title
         name: Name
         on_hand: On Hand
+        option_type_ids: Option Types
         price: Master Price
         primary_taxon: Primary Taxon
         primary_taxon_id: Primary Taxon

data/db/default/spree/countries.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'carmen'
+require "carmen"
 
 # Insert Countries into the spree_countries table, checking to ensure that no
 # duplicates are created, using as few SQL statements as possible (2)
@@ -8,11 +8,11 @@ require 'carmen'
 connection = Spree::Base.connection
 
 country_mapper = ->(country) do
-  name            = connection.quote country.name
-  iso3            = connection.quote country.alpha_3_code
-  iso             = connection.quote country.alpha_2_code
-  iso_name        = connection.quote country.name.upcase
-  numcode         = connection.quote country.numeric_code
+  name = connection.quote country.name
+  iso3 = connection.quote country.alpha_3_code
+  iso = connection.quote country.alpha_2_code
+  iso_name = connection.quote country.name.upcase
+  numcode = connection.quote country.numeric_code
   states_required = connection.quote country.subregions?
 
   [name, iso3, iso, iso_name, numcode, states_required].join(", ")
@@ -32,7 +32,7 @@ country_values = -> do
     .join("), (")
 end
 
-country_columns = %w(name iso3 iso iso_name numcode states_required).join(', ')
+country_columns = %w[name iso3 iso iso_name numcode states_required].join(", ")
 country_vals = country_values.call
 
 if country_vals.present?

data/db/default/spree/return_reasons.rb

@@ -1,13 +1,13 @@
 # frozen_string_literal: true
 
-Spree::ReturnReason.find_or_create_by(name: 'Better price available')
-Spree::ReturnReason.find_or_create_by(name: 'Missed estimated delivery date')
-Spree::ReturnReason.find_or_create_by(name: 'Missing parts or accessories')
-Spree::ReturnReason.find_or_create_by(name: 'Damaged during transport')
-Spree::ReturnReason.find_or_create_by(name: 'Defective within warranty period')
-Spree::ReturnReason.find_or_create_by(name: 'Other defects or damages')
-Spree::ReturnReason.find_or_create_by(name: 'Different from what was ordered')
-Spree::ReturnReason.find_or_create_by(name: 'Different from description')
-Spree::ReturnReason.find_or_create_by(name: 'No longer needed/wanted')
-Spree::ReturnReason.find_or_create_by(name: 'Accidental order')
-Spree::ReturnReason.find_or_create_by(name: 'Unauthorized purchase')
+Spree::ReturnReason.find_or_create_by(name: "Better price available")
+Spree::ReturnReason.find_or_create_by(name: "Missed estimated delivery date")
+Spree::ReturnReason.find_or_create_by(name: "Missing parts or accessories")
+Spree::ReturnReason.find_or_create_by(name: "Damaged during transport")
+Spree::ReturnReason.find_or_create_by(name: "Defective within warranty period")
+Spree::ReturnReason.find_or_create_by(name: "Other defects or damages")
+Spree::ReturnReason.find_or_create_by(name: "Different from what was ordered")
+Spree::ReturnReason.find_or_create_by(name: "Different from description")
+Spree::ReturnReason.find_or_create_by(name: "No longer needed/wanted")
+Spree::ReturnReason.find_or_create_by(name: "Accidental order")
+Spree::ReturnReason.find_or_create_by(name: "Unauthorized purchase")

data/db/default/spree/stock_locations.rb

@@ -1,4 +1,4 @@
 # frozen_string_literal: true
 
 Spree::StockLocation.create_with(backorderable_default: true)
-                    .find_or_create_by!(name: 'default')
+  .find_or_create_by!(name: "default")

data/db/default/spree/store_credit.rb

@@ -1,6 +1,5 @@
 # frozen_string_literal: true
 
-
 Spree::PaymentMethod.create_with(
   name: "Store Credit",
   description: "Store credit",
@@ -14,10 +13,10 @@ Spree::PaymentMethod.create_with(
 Spree::StoreCreditType.create_with(priority: 1).find_or_create_by!(name: Spree::StoreCreditType::EXPIRING)
 Spree::StoreCreditType.create_with(priority: 2).find_or_create_by!(name: Spree::StoreCreditType::NON_EXPIRING)
 
-Spree::ReimbursementType.create_with(name: "Store Credit").find_or_create_by!(type: 'Spree::ReimbursementType::StoreCredit')
-Spree::ReimbursementType.create_with(name: "Original").find_or_create_by!(type: 'Spree::ReimbursementType::OriginalPayment')
+Spree::ReimbursementType.create_with(name: "Store Credit").find_or_create_by!(type: "Spree::ReimbursementType::StoreCredit")
+Spree::ReimbursementType.create_with(name: "Original").find_or_create_by!(type: "Spree::ReimbursementType::OriginalPayment")
 
 Spree::StoreCreditCategory.find_or_create_by!(name: Spree::StoreCreditCategory::GIFT_CARD)
 Spree::StoreCreditCategory.find_or_create_by!(name: Spree::StoreCreditCategory::REIMBURSEMENT)
 
-Spree::StoreCreditReason.find_or_create_by!(name: 'Credit Given In Error')
+Spree::StoreCreditReason.find_or_create_by!(name: "Credit Given In Error")

data/db/default/spree/stores.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-unless Spree::Store.where(code: 'sample-store').exists?
+unless Spree::Store.where(code: "sample-store").exists?
   Spree::Store.create!(
     name: "Sample Store",
     code: "sample-store",

data/db/default/spree/zones.rb

@@ -3,11 +3,11 @@
 eu_vat = Spree::Zone.find_or_create_by!(name: "EU_VAT", description: "Countries that make up the EU VAT zone.")
 north_america = Spree::Zone.find_or_create_by!(name: "North America", description: "USA + Canada")
 
-%w(PL FI PT RO DE FR SK HU SI IE AT ES IT BE SE LV BG GB LT CY LU MT DK NL EE HR CZ GR).
-each do |symbol|
-  eu_vat.zone_members.find_or_create_by!(zoneable: Spree::Country.find_by!(iso: symbol))
-end
+%w[PL FI PT RO DE FR SK HU SI IE AT ES IT BE SE LV BG GB LT CY LU MT DK NL EE HR CZ GR]
+  .each do |symbol|
+    eu_vat.zone_members.find_or_create_by!(zoneable: Spree::Country.find_by!(iso: symbol))
+  end
 
-%w(US CA).each do |symbol|
+%w[US CA].each do |symbol|
   north_america.zone_members.find_or_create_by!(zoneable: Spree::Country.find_by!(iso: symbol))
 end

data/db/migrate/20160101010000_solidus_one_four.rb

@@ -11,27 +11,27 @@ class SolidusOneFour < ActiveRecord::Migration[5.0]
     # but spree_api depends on it existing. This defininition comes from solidus_auth_devise's first migration,
     # and creates a table equivolent to it
     create_table "spree_users", force: true do |t|
-      t.string   "crypted_password", limit: 128
-      t.string   "salt", limit: 128
-      t.string   "email"
-      t.string   "remember_token"
-      t.string   "remember_token_expires_at"
-      t.string   "persistence_token"
-      t.string   "single_access_token"
-      t.string   "perishable_token"
-      t.integer  "login_count",        default: 0, null: false
-      t.integer  "failed_login_count", default: 0, null: false
+      t.string "crypted_password", limit: 128
+      t.string "salt", limit: 128
+      t.string "email"
+      t.string "remember_token"
+      t.string "remember_token_expires_at"
+      t.string "persistence_token"
+      t.string "single_access_token"
+      t.string "perishable_token"
+      t.integer "login_count", default: 0, null: false
+      t.integer "failed_login_count", default: 0, null: false
       t.datetime "last_request_at"
       t.datetime "current_login_at"
       t.datetime "last_login_at"
-      t.string   "current_login_ip"
-      t.string   "last_login_ip"
-      t.string   "login"
-      t.integer  "ship_address_id"
-      t.integer  "bill_address_id"
+      t.string "current_login_ip"
+      t.string "last_login_ip"
+      t.string "login"
+      t.integer "ship_address_id"
+      t.integer "bill_address_id"
       t.datetime "created_at", null: false, precision: 6
       t.datetime "updated_at", null: false, precision: 6
-      t.string   "openid_identifier"
+      t.string "openid_identifier"
     end
 
     create_table "friendly_id_slugs", force: :cascade do |t|

data/db/migrate/20160420044191_create_spree_wallet_payment_sources.rb

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

data/db/migrate/20160420181916_migrate_credit_cards_to_wallet_payment_sources.rb

@@ -2,22 +2,23 @@
 
 class MigrateCreditCardsToWalletPaymentSources < ActiveRecord::Migration[4.2]
   class CreditCard < ActiveRecord::Base
-    self.table_name = 'spree_credit_cards'
+    self.table_name = "spree_credit_cards"
   end
+
   class WalletPaymentSource < ActiveRecord::Base
-    self.table_name = 'spree_wallet_payment_sources'
+    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 = 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'
+        payment_source_type: "Spree::CreditCard"
       ) do |wallet_source|
         wallet_source.default = credit_card.default
       end

data/db/migrate/20161014221052_add_available_to_columns_and_remove_display_on_from_payment_methods.rb

@@ -4,25 +4,25 @@ class AddAvailableToColumnsAndRemoveDisplayOnFromPaymentMethods < ActiveRecord::
   def up
     add_column(:spree_payment_methods, :available_to_users, :boolean, default: true)
     add_column(:spree_payment_methods, :available_to_admin, :boolean, default: true)
-    execute("UPDATE spree_payment_methods "\
-             "SET available_to_users=#{quoted_false} "\
+    execute("UPDATE spree_payment_methods " \
+             "SET available_to_users=#{quoted_false} " \
              "WHERE NOT (display_on='front_end' OR display_on='' OR display_on IS NULL)")
-    execute("UPDATE spree_payment_methods "\
-             "SET available_to_admin=#{quoted_false} "\
+    execute("UPDATE spree_payment_methods " \
+             "SET available_to_admin=#{quoted_false} " \
              "WHERE NOT (display_on='back_end' OR display_on='' OR display_on IS NULL)")
     remove_column(:spree_payment_methods, :display_on)
   end
 
   def down
     add_column(:spree_payment_methods, :display_on, :string)
-    execute("UPDATE spree_payment_methods "\
-            "SET display_on='' "\
+    execute("UPDATE spree_payment_methods " \
+            "SET display_on='' " \
             "WHERE (available_to_users=#{quoted_true} AND available_to_admin=#{quoted_true})")
-    execute("UPDATE spree_payment_methods "\
-            "SET display_on='front_end' "\
+    execute("UPDATE spree_payment_methods " \
+            "SET display_on='front_end' " \
             "WHERE (available_to_users=#{quoted_true} AND NOT available_to_admin=#{quoted_true})")
-    execute("UPDATE spree_payment_methods "\
-            "SET display_on='back_end' "\
+    execute("UPDATE spree_payment_methods " \
+            "SET display_on='back_end' " \
             "WHERE (available_to_admin=#{quoted_true} AND NOT available_to_users=#{quoted_true})")
     remove_column(:spree_payment_methods, :available_to_users)
     remove_column(:spree_payment_methods, :available_to_admin)

data/db/migrate/20161123154034_add_available_to_users_and_remove_display_on_from_shipping_methods.rb

@@ -3,19 +3,19 @@
 class AddAvailableToUsersAndRemoveDisplayOnFromShippingMethods < ActiveRecord::Migration[5.0]
   def up
     add_column(:spree_shipping_methods, :available_to_users, :boolean, default: true)
-    execute("UPDATE spree_shipping_methods "\
-             "SET available_to_users=#{quoted_false} "\
+    execute("UPDATE spree_shipping_methods " \
+             "SET available_to_users=#{quoted_false} " \
              "WHERE display_on='back_end'")
     remove_column(:spree_shipping_methods, :display_on)
   end
 
   def down
     add_column(:spree_shipping_methods, :display_on, :string)
-    execute("UPDATE spree_shipping_methods "\
-            "SET display_on='both' "\
+    execute("UPDATE spree_shipping_methods " \
+            "SET display_on='both' " \
             "WHERE (available_to_users=#{quoted_true})")
-    execute("UPDATE spree_shipping_methods "\
-            "SET display_on='back_end' "\
+    execute("UPDATE spree_shipping_methods " \
+            "SET display_on='back_end' " \
             "WHERE (available_to_users=#{quoted_false})")
     remove_column(:spree_shipping_methods, :available_to_users)
   end

data/db/migrate/20161129035810_add_index_to_spree_payments_number.rb

@@ -2,6 +2,6 @@
 
 class AddIndexToSpreePaymentsNumber < ActiveRecord::Migration[5.0]
   def change
-    add_index 'spree_payments', ['number'], unique: true
+    add_index "spree_payments", ["number"], unique: true
   end
 end

data/db/migrate/20170319191942_remove_order_id_from_inventory_units.rb

@@ -14,10 +14,10 @@ class RemoveOrderIdFromInventoryUnits < ActiveRecord::Migration[5.0]
   end
 
   def up
-    if InventoryUnit.
-        joins(:shipment).
-        where.not(
-          'spree_inventory_units.order_id = spree_shipments.order_id'
+    if InventoryUnit
+        .joins(:shipment)
+        .where.not(
+          "spree_inventory_units.order_id = spree_shipments.order_id"
         ).exists?
       raise InconsistentInventoryUnitError, "You have inventory units with inconsistent order references. Please fix those before running this migration"
     end