solidus_core 4.6.2 → 4.7.0

data/app/models/spree/shipping_rate.rb

@@ -5,14 +5,15 @@ module Spree
   # method has been selected to deliver the shipment.
   #
   class ShippingRate < Spree::Base
-    belongs_to :shipment, class_name: 'Spree::Shipment', touch: true, optional: true
-    belongs_to :shipping_method, -> { with_discarded }, class_name: 'Spree::ShippingMethod', inverse_of: :shipping_rates, optional: true
+    belongs_to :shipment, class_name: "Spree::Shipment", touch: true, optional: true, inverse_of: :shipping_rates
+    belongs_to :shipping_method, -> { with_discarded }, class_name: "Spree::ShippingMethod", inverse_of: :shipping_rates, optional: true
 
     has_many :taxes,
-             class_name: "Spree::ShippingRateTax",
-             foreign_key: "shipping_rate_id",
-             inverse_of: :shipping_rate,
-             dependent: :destroy
+      class_name: "Spree::ShippingRateTax",
+      foreign_key: "shipping_rate_id",
+      inverse_of: :shipping_rate,
+      dependent: :destroy,
+      autosave: true
 
     delegate :order, :currency, to: :shipment
     delegate :name, :tax_category, :tax_category_id, to: :shipping_method
@@ -21,6 +22,7 @@ module Spree
     alias_attribute :total_before_tax, :cost
 
     extend DisplayMoney
+
     money_methods :amount
 
     def display_price
@@ -30,16 +32,16 @@ module Spree
 
       tax_explanations = taxes.map(&:label).join(tax_label_separator)
 
-      I18n.t 'spree.shipping_rate.display_price.display_price_with_explanations',
-             price:,
-             explanations: tax_explanations
+      I18n.t "spree.shipping_rate.display_price.display_price_with_explanations",
+        price:,
+        explanations: tax_explanations
     end
     alias_method :display_cost, :display_price
 
     private
 
     def tax_label_separator
-      I18n.t 'spree.shipping_rate.display_price.tax_label_separator'
+      I18n.t "spree.shipping_rate.display_price.tax_label_separator"
     end
   end
 end

data/app/models/spree/shipping_rate_tax.rb

@@ -11,13 +11,14 @@ module Spree
     belongs_to :tax_rate, class_name: "Spree::TaxRate", optional: true
 
     extend DisplayMoney
+
     money_methods :absolute_amount
 
     delegate :currency, to: :shipping_rate, allow_nil: true
 
     def label
       I18n.t translation_key,
-        scope: 'spree.shipping_rate_tax.label',
+        scope: "spree.shipping_rate_tax.label",
         amount: display_absolute_amount,
         tax_rate_name: tax_rate.name
     end
@@ -30,9 +31,9 @@ module Spree
 
     def translation_key
       if tax_rate.included_in_price?
-         :vat
-       else
-         :sales_tax
+        :vat
+      else
+        :sales_tax
       end
     end
   end

data/app/models/spree/simple_order_contents.rb

@@ -53,7 +53,7 @@ module Spree
 
     def approve(user: nil, name: nil)
       if user.blank? && name.blank?
-        raise ArgumentError, 'user or name must be specified'
+        raise ArgumentError, "user or name must be specified"
       end
 
       order.update!(
@@ -86,7 +86,7 @@ module Spree
       )
 
       permitted_attributes = Spree::PermittedAttributes.line_item_attributes.dup
-      permitted_attributes << { admin_metadata: {} } if options[:admin_metadata].present?
+      permitted_attributes << {admin_metadata: {}} if options[:admin_metadata].present?
 
       line_item.quantity += quantity.to_i
       line_item.options = ActionController::Parameters.new(options).permit(permitted_attributes).to_h

data/app/models/spree/state.rb

@@ -2,7 +2,7 @@
 
 module Spree
   class State < Spree::Base
-    belongs_to :country, class_name: 'Spree::Country'
+    belongs_to :country, class_name: "Spree::Country"
     has_many :addresses, dependent: :nullify, inverse_of: :state
 
     validates :name, presence: true

data/app/models/spree/stock/availability.rb

@@ -52,22 +52,22 @@ module Spree
 
       def counts_on_hand
         @counts_on_hand ||=
-          stock_item_scope.
-            group(:variant_id, :stock_location_id).
-            sum(:count_on_hand)
+          stock_item_scope
+            .group(:variant_id, :stock_location_id)
+            .sum(:count_on_hand)
       end
 
       def backorderables
         @backorderables ||=
-          stock_item_scope.
-            where(backorderable: true).
-            pluck(:variant_id, :stock_location_id)
+          stock_item_scope
+            .where(backorderable: true)
+            .pluck(:variant_id, :stock_location_id)
       end
 
       def stock_item_scope
-        Spree::StockItem.
-          where(variant_id: @variants).
-          where(stock_location_id: @stock_locations)
+        Spree::StockItem
+          .where(variant_id: @variants)
+          .where(stock_location_id: @stock_locations)
       end
 
       def restore_location_order(quantities_by_location_id)

data/app/models/spree/stock/availability_validator.rb

@@ -12,7 +12,7 @@ module Spree
           display_name += %{ (#{variant.options_text})} unless variant.options_text.blank?
 
           line_item.errors.add(:quantity, I18n.t(
-            'spree.selected_quantity_not_available',
+            "spree.selected_quantity_not_available",
             item: display_name.inspect
           ))
           false
@@ -23,11 +23,11 @@ module Spree
 
       def is_valid?(line_item)
         if line_item.inventory_units.empty?
-          Stock::Quantifier.new(line_item.variant).can_supply?(line_item.quantity)
+          Spree::Config.stock.quantifier_class.new(line_item.variant).can_supply?(line_item.quantity)
         else
           quantity_by_stock_location_id = line_item.inventory_units.pending.joins(:shipment).group(:stock_location_id).count
           quantity_by_stock_location_id.all? do |stock_location_id, quantity|
-            Stock::Quantifier.new(line_item.variant, stock_location_id).can_supply?(quantity)
+            Spree::Config.stock.quantifier_class.new(line_item.variant, stock_location_id).can_supply?(quantity)
           end
         end
       end

data/app/models/spree/stock/differentiator.rb

@@ -22,8 +22,8 @@ module Spree
       def build_missing
         @missing = Hash.new(0)
         required.keys.each do |variant|
-           missing = required[variant] - packed[variant]
-           @missing[variant] = missing if missing > 0
+          missing = required[variant] - packed[variant]
+          @missing[variant] = missing if missing > 0
         end
       end
 

data/app/models/spree/stock/estimator.rb

@@ -60,10 +60,10 @@ module Spree
           .includes(:calculator)
           .to_a
           .select do |ship_method|
-          calculator = ship_method.calculator
-          calculator.available?(package) &&
-            (calculator.preferences[:currency].blank? ||
-             calculator.preferences[:currency] == package.shipment.order.currency)
+            calculator = ship_method.calculator
+            calculator.available?(package) &&
+              (calculator.preferences[:currency].blank? ||
+               calculator.preferences[:currency] == package.shipment.order.currency)
         end
       end
     end

data/app/models/spree/stock/inventory_validator.rb

@@ -6,7 +6,7 @@ module Spree
       def validate(line_item)
         if line_item.inventory_units.count != line_item.quantity
           line_item.errors.add(:inventory, I18n.t(
-            'spree.inventory_not_available',
+            "spree.inventory_not_available",
             item: line_item.variant.name
           ))
         end

data/app/models/spree/stock/package.rb

@@ -109,9 +109,9 @@ module Spree
       # @return [ActiveRecord::Relation] the [Spree::ShippingMethod]s available
       #   for this pacakge based on the stock location and shipping categories.
       def shipping_methods
-        Spree::ShippingMethod.
-          with_all_shipping_category_ids(shipping_category_ids).
-          available_in_stock_location(stock_location)
+        Spree::ShippingMethod
+          .with_all_shipping_category_ids(shipping_category_ids)
+          .available_in_stock_location(stock_location)
       end
 
       # @return [Spree::Shipment] a new shipment containing this package's

data/app/models/spree/stock_item.rb

@@ -4,13 +4,13 @@ module Spree
   class StockItem < Spree::Base
     include Spree::SoftDeletable
 
-    belongs_to :stock_location, class_name: 'Spree::StockLocation', inverse_of: :stock_items, optional: true
-    belongs_to :variant, -> { with_discarded }, class_name: 'Spree::Variant', inverse_of: :stock_items, optional: true
+    belongs_to :stock_location, class_name: "Spree::StockLocation", inverse_of: :stock_items, optional: true
+    belongs_to :variant, -> { with_discarded }, class_name: "Spree::Variant", inverse_of: :stock_items, optional: true
     has_many :stock_movements, inverse_of: :stock_item
 
     validates :stock_location, :variant, presence: true
-    validates :variant_id, uniqueness: { scope: [:stock_location_id, :deleted_at] }, allow_blank: true, unless: :deleted_at
-    validates :count_on_hand, numericality: { greater_than_or_equal_to: 0 }, unless: :backorderable?
+    validates :variant_id, uniqueness: {scope: [:stock_location_id, :deleted_at]}, allow_blank: true, unless: :deleted_at
+    validates :count_on_hand, numericality: {greater_than_or_equal_to: 0}, unless: :backorderable?
 
     delegate :weight, :should_track_inventory?, to: :variant
 
@@ -20,7 +20,7 @@ module Spree
     after_save :conditional_variant_touch, if: :saved_changes?
     after_touch { variant.touch }
 
-    self.allowed_ransackable_attributes = ['count_on_hand', 'stock_location_id']
+    self.allowed_ransackable_attributes = ["count_on_hand", "stock_location_id"]
     self.allowed_ransackable_associations = %w[variant]
 
     # @return [Array<Spree::InventoryUnit>] the backordered inventory units

data/app/models/spree/stock_location.rb

@@ -17,14 +17,14 @@ module Spree
     has_many :users, through: :user_stock_locations
     has_many :customer_returns, inverse_of: :stock_location, dependent: :restrict_with_error
 
-    belongs_to :state, class_name: 'Spree::State', optional: true
-    belongs_to :country, class_name: 'Spree::Country', optional: true
+    belongs_to :state, class_name: "Spree::State", optional: true
+    belongs_to :country, class_name: "Spree::Country", optional: true
 
     has_many :shipping_method_stock_locations, dependent: :destroy
     has_many :shipping_methods, through: :shipping_method_stock_locations
 
     validates :name, presence: true
-    validates :code, uniqueness: { allow_blank: true, case_sensitive: false }
+    validates :code, uniqueness: {allow_blank: true, case_sensitive: false}
 
     scope :active, -> { where(active: true) }
     scope :order_default, -> { order(default: :desc, position: :asc) }
@@ -100,14 +100,14 @@ module Spree
 
     def move(variant, quantity, originator = nil)
       if quantity < 1 && !stock_item(variant)
-        raise InvalidMovementError.new(I18n.t('spree.negative_movement_absent_item'))
+        raise InvalidMovementError.new(I18n.t("spree.negative_movement_absent_item"))
       end
       stock_item_or_create(variant).stock_movements.create!(quantity:,
-                                                            originator:)
+        originator:)
     end
 
     def fill_status(variant, quantity)
-      if item = stock_item(variant)
+      if (item = stock_item(variant))
         item.fill_status(quantity)
       else
         [0, 0]

data/app/models/spree/stock_movement.rb

@@ -2,7 +2,7 @@
 
 module Spree
   class StockMovement < Spree::Base
-    belongs_to :stock_item, class_name: 'Spree::StockItem', inverse_of: :stock_movements, optional: true
+    belongs_to :stock_item, class_name: "Spree::StockItem", inverse_of: :stock_movements, optional: true
     belongs_to :originator, polymorphic: true, optional: true
     has_one :variant, through: :stock_item
 
@@ -14,7 +14,7 @@ module Spree
     scope :recent, -> { order(created_at: :desc) }
 
     self.allowed_ransackable_associations = %w[variant]
-    self.allowed_ransackable_attributes = ['quantity']
+    self.allowed_ransackable_attributes = ["quantity"]
 
     def readonly?
       !new_record?

data/app/models/spree/stock_quantities.rb

@@ -8,8 +8,8 @@ module Spree
 
     # @param quantities [Hash<Spree::Variant=>Numeric>]
     def initialize(quantities = {})
-      raise ArgumentError unless quantities.keys.all?{ |value| value.is_a?(Spree::Variant) }
-      raise ArgumentError unless quantities.values.all?{ |value| value.is_a?(Numeric) }
+      raise ArgumentError unless quantities.keys.all? { |value| value.is_a?(Spree::Variant) }
+      raise ArgumentError unless quantities.values.all? { |value| value.is_a?(Numeric) }
 
       @quantities = quantities
     end

data/app/models/spree/store.rb

@@ -17,7 +17,7 @@ module Spree
 
     has_many :orders, class_name: "Spree::Order"
 
-    validates :code, presence: true, uniqueness: { allow_blank: true, case_sensitive: true }
+    validates :code, presence: true, uniqueness: {allow_blank: true, case_sensitive: true}
     validates :name, presence: true
     validates :url, presence: true
     validates :mail_from_address, presence: true
@@ -34,9 +34,9 @@ module Spree
     }, prefix: true
 
     def available_locales
-      locales = super()
+      locales = super
       if locales
-        super().split(",").map(&:to_sym)
+        super.split(",").map(&:to_sym)
       else
         Spree.i18n_available_locales
       end

data/app/models/spree/store_credit.rb

@@ -3,31 +3,31 @@
 class Spree::StoreCredit < Spree::PaymentSource
   include Spree::SoftDeletable
 
-  VOID_ACTION       = 'void'
-  CREDIT_ACTION     = 'credit'
-  CAPTURE_ACTION    = 'capture'
-  ELIGIBLE_ACTION   = 'eligible'
-  AUTHORIZE_ACTION  = 'authorize'
-  ALLOCATION_ACTION = 'allocation'
-  ADJUSTMENT_ACTION = 'adjustment'
-  INVALIDATE_ACTION = 'invalidate'
+  VOID_ACTION = "void"
+  CREDIT_ACTION = "credit"
+  CAPTURE_ACTION = "capture"
+  ELIGIBLE_ACTION = "eligible"
+  AUTHORIZE_ACTION = "authorize"
+  ALLOCATION_ACTION = "allocation"
+  ADJUSTMENT_ACTION = "adjustment"
+  INVALIDATE_ACTION = "invalidate"
 
   belongs_to :user, class_name: Spree::UserClassHandle.new, optional: true
   belongs_to :created_by, class_name: Spree::UserClassHandle.new, optional: true
   belongs_to :category, class_name: "Spree::StoreCreditCategory", optional: true
-  belongs_to :credit_type, class_name: 'Spree::StoreCreditType', foreign_key: 'type_id', optional: true
+  belongs_to :credit_type, class_name: "Spree::StoreCreditType", foreign_key: "type_id", optional: true
   has_many :store_credit_events
 
   validates :user_id, :category_id, :type_id, :created_by_id, :currency, presence: true
-  validates :amount, numericality: { greater_than: 0 }
-  validates :amount_used, numericality: { greater_than_or_equal_to: 0 }
+  validates :amount, numericality: {greater_than: 0}
+  validates :amount_used, numericality: {greater_than_or_equal_to: 0}
   validate :amount_used_less_than_or_equal_to_amount
   validate :amount_authorized_less_than_or_equal_to_amount
 
   delegate :name, to: :category, prefix: true
   delegate :email, to: :created_by, prefix: true
 
-  scope :order_by_priority, -> { includes(:credit_type).order('spree_store_credit_types.priority ASC') }
+  scope :order_by_priority, -> { includes(:credit_type).order("spree_store_credit_types.priority ASC") }
 
   after_save :store_event
   before_validation :associate_credit_type
@@ -38,6 +38,7 @@ class Spree::StoreCredit < Spree::PaymentSource
   attr_accessor :action, :action_amount, :action_originator, :action_authorization_code, :store_credit_reason
 
   extend Spree::DisplayMoney
+
   money_methods :amount, :amount_used, :amount_authorized
 
   alias_method :display_number, :category_name
@@ -51,7 +52,7 @@ class Spree::StoreCredit < Spree::PaymentSource
   end
 
   def amount_remaining
-    return 0.0.to_d if invalidated?
+    return BigDecimal("0.0") if invalidated?
     amount - amount_used - amount_authorized
   end
 
@@ -83,9 +84,9 @@ class Spree::StoreCredit < Spree::PaymentSource
 
   def validate_authorization(amount, order_currency)
     if amount_remaining.to_d < amount.to_d
-      errors.add(:base, I18n.t('spree.store_credit.insufficient_funds'))
+      errors.add(:base, I18n.t("spree.store_credit.insufficient_funds"))
     elsif currency != order_currency
-      errors.add(:base, I18n.t('spree.store_credit.currency_mismatch'))
+      errors.add(:base, I18n.t("spree.store_credit.currency_mismatch"))
     end
     errors.blank?
   end
@@ -96,7 +97,7 @@ class Spree::StoreCredit < Spree::PaymentSource
 
     if amount <= auth_event.amount
       if currency != order_currency
-        errors.add(:base, I18n.t('spree.store_credit.currency_mismatch'))
+        errors.add(:base, I18n.t("spree.store_credit.currency_mismatch"))
         false
       else
         update!({
@@ -111,13 +112,13 @@ class Spree::StoreCredit < Spree::PaymentSource
         authorization_code
       end
     else
-      errors.add(:base, I18n.t('spree.store_credit.insufficient_authorized_amount'))
+      errors.add(:base, I18n.t("spree.store_credit.insufficient_authorized_amount"))
       false
     end
   end
 
   def void(authorization_code, options = {})
-    if auth_event = store_credit_events.find_by(action: AUTHORIZE_ACTION, authorization_code:)
+    if (auth_event = store_credit_events.find_by(action: AUTHORIZE_ACTION, authorization_code:))
       update!({
         action: VOID_ACTION,
         action_amount: auth_event.amount,
@@ -128,7 +129,7 @@ class Spree::StoreCredit < Spree::PaymentSource
       })
       true
     else
-      errors.add(:base, I18n.t('spree.store_credit.unable_to_void', auth_code: authorization_code))
+      errors.add(:base, I18n.t("spree.store_credit.unable_to_void", auth_code: authorization_code))
       false
     end
   end
@@ -138,7 +139,7 @@ class Spree::StoreCredit < Spree::PaymentSource
     capture_event = store_credit_events.find_by(action: CAPTURE_ACTION, authorization_code:)
 
     if currency != order_currency # sanity check to make sure the order currency hasn't changed since the auth
-      errors.add(:base, I18n.t('spree.store_credit.currency_mismatch'))
+      errors.add(:base, I18n.t("spree.store_credit.currency_mismatch"))
       false
     elsif capture_event && amount <= capture_event.amount
       action_attributes = {
@@ -150,7 +151,7 @@ class Spree::StoreCredit < Spree::PaymentSource
       create_credit_record(amount, action_attributes)
       true
     else
-      errors.add(:base, I18n.t('spree.store_credit.unable_to_credit', auth_code: authorization_code))
+      errors.add(:base, I18n.t("spree.store_credit.unable_to_credit", auth_code: authorization_code))
       false
     end
   end
@@ -162,10 +163,10 @@ class Spree::StoreCredit < Spree::PaymentSource
   def generate_authorization_code
     [
       id,
-      'SC',
-      Time.current.utc.strftime('%Y%m%d%H%M%S%6N'),
+      "SC",
+      Time.current.utc.strftime("%Y%m%d%H%M%S%6N"),
       SecureRandom.uuid
-    ].join('-')
+    ].join("-")
   end
 
   def editable?
@@ -198,7 +199,7 @@ class Spree::StoreCredit < Spree::PaymentSource
       self.invalidated_at = Time.current
       save
     else
-      errors.add(:invalidated_at, I18n.t('spree.store_credit.errors.cannot_invalidate_uncaptured_authorization'))
+      errors.add(:invalidated_at, I18n.t("spree.store_credit.errors.cannot_invalidate_uncaptured_authorization"))
       false
     end
   end
@@ -258,25 +259,25 @@ class Spree::StoreCredit < Spree::PaymentSource
     return true if amount_used.nil?
 
     if amount_used > amount
-      errors.add(:amount_used, I18n.t('spree.admin.store_credits.errors.amount_used_cannot_be_greater'))
+      errors.add(:amount_used, I18n.t("spree.admin.store_credits.errors.amount_used_cannot_be_greater"))
     end
   end
 
   def amount_authorized_less_than_or_equal_to_amount
     if (amount_used + amount_authorized) > amount
-      errors.add(:amount_authorized, I18n.t('spree.admin.store_credits.errors.amount_authorized_exceeds_total_credit'))
+      errors.add(:amount_authorized, I18n.t("spree.admin.store_credits.errors.amount_authorized_exceeds_total_credit"))
     end
   end
 
   def validate_category_unchanged
     if category_id_changed?
-      errors.add(:category, I18n.t('spree.admin.store_credits.errors.cannot_be_modified'))
+      errors.add(:category, I18n.t("spree.admin.store_credits.errors.cannot_be_modified"))
     end
   end
 
   def validate_no_amount_used
     if amount_used > 0
-      errors.add(:amount_used, 'is greater than zero. Can not delete store credit')
+      errors.add(:amount_used, "is greater than zero. Can not delete store credit")
       throw :abort
     end
   end

data/app/models/spree/store_credit_category.rb

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
 class Spree::StoreCreditCategory < Spree::Base
-  GIFT_CARD = 'Gift Card'
-  REIMBURSEMENT = 'Reimbursement'
+  GIFT_CARD = "Gift Card"
+  REIMBURSEMENT = "Reimbursement"
 
   class_attribute :non_expiring_credit_types
   self.non_expiring_credit_types = [Spree::StoreCreditType::NON_EXPIRING]

data/app/models/spree/store_credit_event.rb

@@ -7,15 +7,15 @@ module Spree
 
     belongs_to :store_credit, optional: true
     belongs_to :originator, polymorphic: true, optional: true
-    belongs_to :store_credit_reason, class_name: 'Spree::StoreCreditReason', inverse_of: :store_credit_events, optional: true
+    belongs_to :store_credit_reason, class_name: "Spree::StoreCreditReason", inverse_of: :store_credit_events, optional: true
 
-    validates :store_credit_reason, presence: { if: :action_requires_reason? }
+    validates :store_credit_reason, presence: {if: :action_requires_reason?}
 
     NON_EXPOSED_ACTIONS = [Spree::StoreCredit::ELIGIBLE_ACTION, Spree::StoreCredit::AUTHORIZE_ACTION]
 
     scope :exposed_events, -> { exposable_actions.not_invalidated }
     scope :exposable_actions, -> { where.not(action: NON_EXPOSED_ACTIONS) }
-    scope :not_invalidated, -> { joins(:store_credit).where(spree_store_credits: { invalidated_at: nil }) }
+    scope :not_invalidated, -> { joins(:store_credit).where(spree_store_credits: {invalidated_at: nil}) }
     scope :chronological, -> { order(:created_at) }
     scope :reverse_chronological, -> { order(created_at: :desc) }
 
@@ -34,15 +34,15 @@ module Spree
     end
 
     def display_amount
-      Spree::Money.new(amount, { currency: })
+      Spree::Money.new(amount, {currency:})
     end
 
     def display_user_total_amount
-      Spree::Money.new(user_total_amount, { currency: })
+      Spree::Money.new(user_total_amount, {currency:})
     end
 
     def display_remaining_amount
-      Spree::Money.new(amount_remaining, { currency: })
+      Spree::Money.new(amount_remaining, {currency:})
     end
 
     def display_event_date

data/app/models/spree/store_credit_reason.rb

@@ -4,7 +4,7 @@ class Spree::StoreCreditReason < Spree::Base
   scope :active, -> { where(active: true) }
   default_scope -> { order(arel_table[:name].lower) }
 
-  validates :name, presence: true, uniqueness: { case_sensitive: false, allow_blank: true }
+  validates :name, presence: true, uniqueness: {case_sensitive: false, allow_blank: true}
 
   has_many :store_credit_events, inverse_of: :store_credit_reason
 

data/app/models/spree/store_credit_type.rb

@@ -2,9 +2,9 @@
 
 module Spree
   class StoreCreditType < Spree::Base
-    EXPIRING = 'Expiring'
-    NON_EXPIRING = 'Non-expiring'
+    EXPIRING = "Expiring"
+    NON_EXPIRING = "Non-expiring"
     DEFAULT_TYPE_NAME = EXPIRING
-    has_many :store_credits, class_name: 'Spree::StoreCredit', foreign_key: 'type_id'
+    has_many :store_credits, class_name: "Spree::StoreCredit", foreign_key: "type_id"
   end
 end

data/app/models/spree/store_selector/by_server_name.rb

@@ -17,7 +17,7 @@ module Spree
       # Chooses the current store based on a request.
       # @return [Spree::Store]
       def store
-        server_name = @request.env['SERVER_NAME']
+        server_name = @request.env["SERVER_NAME"]
 
         # We select a store which either matches our server name, or is default.
         # We sort by `default ASC` so that a store matching SERVER_NAME will come

data/app/models/spree/store_selector/legacy.rb

@@ -43,7 +43,7 @@ module Spree
       private
 
       def store_key
-        @request.headers['HTTP_SPREE_STORE'] || @request.env['SERVER_NAME']
+        @request.headers["HTTP_SPREE_STORE"] || @request.env["SERVER_NAME"]
       end
     end
   end

data/app/models/spree/tax/item_tax.rb

@@ -17,6 +17,7 @@ module Spree
     #   in the items price, or additional tax.
     class ItemTax
       include ActiveModel::Model
+
       attr_accessor :item_id, :label, :tax_rate, :amount, :included_in_price
     end
   end

data/app/models/spree/tax/order_tax.rb

@@ -16,6 +16,7 @@ module Spree
     #   tax data for the order's shipments
     class OrderTax
       include ActiveModel::Model
+
       attr_accessor :order_id, :order_taxes, :line_item_taxes, :shipment_taxes
     end
   end

data/app/models/spree/tax_category.rb

@@ -11,15 +11,15 @@ module Spree
     end
 
     validates :name, presence: true
-    validates :name, uniqueness: { case_sensitive: true, unless: :deleted_at }
+    validates :name, uniqueness: {case_sensitive: true, unless: :deleted_at}
 
     has_many :tax_rate_tax_categories,
-      class_name: 'Spree::TaxRateTaxCategory',
+      class_name: "Spree::TaxRateTaxCategory",
       dependent: :destroy,
       inverse_of: :tax_category
     has_many :tax_rates,
       through: :tax_rate_tax_categories,
-      class_name: 'Spree::TaxRate',
+      class_name: "Spree::TaxRate",
       inverse_of: :tax_categories
 
     after_save :ensure_one_default