spree_core 5.1.4 → 5.1.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 90be1a32290303d4feb6a777d2a5ed2d88232edfa9998e748299df632346543f
-  data.tar.gz: 63b6aebfc73bdb8e56e945471245cb2ac4bff7afb09db4d4377b753ba6bcfe06
+  metadata.gz: 8b7ada6c326700f36766e181d422238f5fc9c8e345b92b37df8d5c2ed5e1b56c
+  data.tar.gz: 86b6aabcab0463033b6a8e045445b8b7b90109f9f8a7d5e3084ffad2d5093bab
 SHA512:
-  metadata.gz: 6f4cd428b66cee1b390d3710ff2d33d71a4d4ffcfa8648672937eb748a6c91993e1ec8d23495b099e25efc8790a8d0a093d849339f5659be6097ffb878bde293
-  data.tar.gz: 5c7186492fe423d5b8862771dd3767ad9f9d9fb5c00bd71a434c19a5b3d938c9a071579aa54594f235ad16e0c9bac64bfbab8d286a0f40ae5522cffe8c453b82
+  metadata.gz: cc6d619e2582cc97f6444f55f49db531cc5adc7dcc9c5349e6401c8d15d26c23b2e46d25219135a2a73841c2ddf207e5ab76d3e6cebcf584128118636bc714a3
+  data.tar.gz: ce1337606b76a87689543518feb7dcac21d7ed3b62bfb3d14a1b703a0a90da357c8d9476228971c01a63b850152311ef68c2fdec3f4ee620b0631c9e845e6bf8

data/app/jobs/spree/products/touch_taxons_job.rb

@@ -6,6 +6,7 @@ module Spree
       def perform(taxon_ids, taxonomy_ids)
         Spree::Taxon.where(id: taxon_ids).update_all(updated_at: Time.current)
         Spree::Taxonomy.where(id: taxonomy_ids).update_all(updated_at: Time.current)
+        Spree::Taxons::TouchFeaturedSections.call(taxon_ids: taxon_ids)
       end
     end
   end

data/app/models/spree/credit_card.rb

@@ -51,6 +51,14 @@ module Spree
     # needed for some of the ActiveMerchant gateways (eg. SagePay)
     alias_attribute :brand, :cc_type
 
+    store_accessor :private_metadata, :wallet
+
+    # Returns the type of wallet the card is associated with, eg. "apple_pay", "google_pay", etc.
+    # @return [String]
+    def wallet_type
+      wallet&.[]('type')
+    end
+
     # ActiveMerchant::Billing::CreditCard added this accessor used by some gateways.
     # More info: https://github.com/spree/spree/issues/6209
     #
@@ -116,6 +124,8 @@ module Spree
       @verification_value = value.to_s.gsub(/\s/, '')
     end
 
+    # Returns the last 4 digits of the card number.
+    # @return [String], eg. "4338"
     def set_last_digits
       self.last_digits ||= number.to_s.length <= 4 ? number : number.to_s.slice(-4..-1)
     end
@@ -130,10 +140,13 @@ module Spree
     end
 
     # Show the card number, with all but last 4 numbers replace with "X". (XXXX-XXXX-XXXX-4338)
+    # @return [String]
     def display_number
       "XXXX-XXXX-XXXX-#{last_digits}"
     end
 
+    # Show the card brand, eg. "VISA", "MASTERCARD", etc.
+    # @return [String]
     def display_brand
       brand.present? ? brand.upcase : Spree.t(:no_cc_type)
     end
@@ -141,14 +154,21 @@ module Spree
     # ActiveMerchant needs first_name/last_name because we pass it a Spree::CreditCard and it calls those methods on it.
     # Looking at the ActiveMerchant source code we should probably be calling #to_active_merchant before passing
     # the object to ActiveMerchant but this should do for now.
+    #
+    # Returns the first name of the cardholder.
+    # @return [String]
     def first_name
       name.to_s.split(/[[:space:]]/, 2)[0]
     end
 
+    # Returns the last name of the cardholder.
+    # @return [String]
     def last_name
       name.to_s.split(/[[:space:]]/, 2)[1]
     end
 
+    # Returns an ActiveMerchant::Billing::CreditCard object.
+    # @return [ActiveMerchant::Billing::CreditCard]
     def to_active_merchant
       ActiveMerchant::Billing::CreditCard.new(
         number: number,

data/app/models/spree/order/digital.rb

@@ -1,6 +1,9 @@
 module Spree
   class Order < Spree.base_class
     module Digital
+      # Returns true if all order line items are digital
+      #
+      # @return [Boolean]
       def digital?
         if line_items.empty?
           false
@@ -9,18 +12,30 @@ module Spree
         end
       end
 
+      # Returns true if any order line item is digital
+      #
+      # @return [Boolean]
       def some_digital?
         line_items.any?(&:digital?)
       end
 
+      # Returns true if any order line item has digital assets
+      #
+      # @return [Boolean]
       def with_digital_assets?
         line_items.any?(&:with_digital_assets?)
       end
 
+      # Returns all line items with digital assets
+      #
+      # @return [Array<Spree::LineItem>]
       def digital_line_items
         line_items.with_digital_assets.distinct
       end
 
+      # Returns all digital links for the order
+      #
+      # @return [Array<Spree::DigitalLink>]
       def digital_links
         digital_line_items.map(&:digital_links).flatten
       end

data/app/models/spree/order_inventory.rb

@@ -84,7 +84,7 @@ module Spree
       end
 
       # adding to this shipment, and removing from stock_location
-      if order.completed?
+      if order.completed? && variant.should_track_inventory?
         shipment.stock_location.unstock(variant, quantity, shipment)
       end
 

data/app/models/spree/page_link.rb

@@ -30,7 +30,7 @@ module Spree
 
     def formatted_url
       return if url.blank?
-      return url if url.start_with?("mailto:")
+      return url if %w[mailto: tel:].any? { |protocol| url.start_with?(protocol) }
 
       @formatted_url ||= url.match(/http:\/\/|https:\/\//) ? url : "http://#{url}"
     end

data/app/models/spree/page_section.rb

@@ -23,11 +23,18 @@ module Spree
     has_one_attached :asset, service: Spree.public_storage_service_name
     alias image asset
 
+    #
+    # Rich Text
+    #
+    has_rich_text :text
+    has_rich_text :description
+
     #
     # Validations
     #
     validates :name, :pageable, presence: true
     validates :asset, content_type: Rails.application.config.active_storage.web_image_content_types
+
     #
     # Callbacks
     #

data/app/models/spree/page_sections/announcement_bar.rb

@@ -1,8 +1,6 @@
 module Spree
   module PageSections
     class AnnouncementBar < Spree::PageSection
-      has_rich_text :text
-
       before_validation :set_default_text, on: :create
 
       BACKGROUND_COLOR_DEFAULT = '#F5F5F4'

data/app/models/spree/page_sections/featured_posts.rb

@@ -1,8 +1,6 @@
 module Spree
   module PageSections
     class FeaturedPosts < Spree::PageSection
-      has_rich_text :description
-
       before_validation :make_heading_size_valid
       before_validation :make_max_posts_to_show_valid
       before_validation :make_alignment_valid

data/app/models/spree/page_sections/featured_taxon.rb

@@ -9,8 +9,6 @@ module Spree
         queries.reduce(:or)
       }
 
-      has_rich_text :description
-
       before_validation :make_heading_size_valid
       before_validation :make_alignment_valid
       before_validation :make_taxon_id_valid
@@ -50,6 +48,11 @@ module Spree
       end
 
       def products(currency)
+        Spree::Deprecation.warn(
+          'FeaturedTaxon#products is deprecated and will be removed in Spree 6.0. ' \
+          'Please use taxon_products(featured_taxon.taxon) helper method in views instead'
+        )
+
         @products ||= begin
           finder_params = {
             store: store,

data/app/models/spree/page_sections/related_products.rb

@@ -1,8 +1,6 @@
 module Spree
   module PageSections
     class RelatedProducts < Spree::PageSection
-      has_rich_text :description
-
       before_validation :make_heading_size_valid
       before_validation :make_alignment_valid
 

data/app/models/spree/product.rb

@@ -38,7 +38,7 @@ module Spree
     MEMOIZED_METHODS = %w[total_on_hand taxonomy_ids taxon_and_ancestors category
                           default_variant_id tax_category default_variant
                           default_image secondary_image
-                          purchasable? in_stock? backorderable? has_variants?]
+                          purchasable? in_stock? backorderable? has_variants? digital?]
 
     STATUS_TO_WEBHOOK_EVENT = {
       'active' => 'activated',
@@ -81,6 +81,7 @@ module Spree
 
     belongs_to :tax_category, class_name: 'Spree::TaxCategory'
     belongs_to :shipping_category, class_name: 'Spree::ShippingCategory', inverse_of: :products
+    has_many :shipping_methods, through: :shipping_category, class_name: 'Spree::ShippingMethod'
 
     has_one :master,
             -> { where is_master: true },
@@ -345,11 +346,11 @@ module Spree
     # Returns default Image for Product
     # @return [Spree::Image]
     def default_image
-      @default_image ||= if images.size.positive?
+      @default_image ||= if images.any?
                            images.first
-                         elsif default_variant.images.size.positive?
+                         elsif default_variant.images.any?
                            default_variant.default_image
-                         elsif variant_images.size.positive?
+                         elsif variant_images.any?
                            variant_images.first
                          end
     end
@@ -376,7 +377,7 @@ module Spree
     end
 
     # Returns tax category for Product
-    # @return [Spree::TaxCategory]
+    # @return [Spree::TaxCategory, nil]
     def tax_category
       @tax_category ||= super || TaxCategory.default
     end
@@ -602,7 +603,7 @@ module Spree
     #
     # @return [Boolean]
     def digital?
-      shipping_category&.shipping_methods&.any? { |method| method.calculator.is_a?(Spree::Calculator::Shipping::DigitalDelivery) }
+      @digital ||= shipping_methods&.digital&.exists?
     end
 
     def auto_match_taxons

data/app/models/spree/shipment.rb

@@ -124,6 +124,10 @@ module Spree
       cost
     end
 
+    def digital?
+      shipping_method&.digital? || false
+    end
+
     def add_shipping_method(shipping_method, selected = false)
       shipping_rates.create(shipping_method: shipping_method, selected: selected, cost: cost)
     end
@@ -200,8 +204,11 @@ module Spree
       with_free_shipping_promotion?
     end
 
+    # Returns true if the shipment has a free shipping promotion applied
+    #
+    # @return [Boolean]
     def with_free_shipping_promotion?
-      adjustments.promotion.any? { |p| p.source.type == 'Spree::Promotion::Actions::FreeShipping' }
+      adjustments.promotion.includes(:source).any? { |p| p.source.respond_to?(:free_shipping?) && p.source.free_shipping? }
     end
 
     def finalize!
@@ -251,10 +258,6 @@ module Spree
       inventory_units.includes(:line_item).map(&:line_item).uniq
     end
 
-    def digital?
-      shipping_method&.calculator&.is_a?(Spree::Calculator::Shipping::DigitalDelivery)
-    end
-
     ManifestItem = Struct.new(:line_item, :variant, :quantity, :states)
 
     def manifest
@@ -358,14 +361,27 @@ module Spree
       self.shipped_at = Time.current
     end
 
+    # Returns the shipping method of the selected shipping rate
+    #
+    # @return [Spree::ShippingMethod]
     def shipping_method
       selected_shipping_rate&.shipping_method || shipping_rates.first&.shipping_method
     end
 
+    # Returns the tax category of the selected shipping rate
+    #
+    # @return [Spree::TaxCategory]
     def tax_category
       selected_shipping_rate.try(:tax_rate).try(:tax_category)
     end
 
+    # Returns the tax category ID of the selected shipping rate
+    #
+    # @return [Integer]
+    def tax_category_id
+      selected_shipping_rate.try(:tax_rate).try(:tax_category_id)
+    end
+
     # Only one of either included_tax_total or additional_tax_total is set
     # This method returns the total of the two. Saves having to check if
     # tax is included or additional.

data/app/models/spree/shipping_method.rb

@@ -10,9 +10,12 @@ module Spree
     if defined?(Spree::VendorConcern)
       include Spree::VendorConcern
     end
+    include Spree::MemoizedData
 
     extend Spree::DisplayMoney
 
+    MEMOIZED_METHODS = %w[display_estimated_price digital?]
+
     # Used for #refresh_rates
     DISPLAY_ON_FRONT_END = 1
     DISPLAY_ON_BACK_END = 2
@@ -118,6 +121,13 @@ module Spree
       end
     end
 
+    # Returns true if the shipping method is digital
+    #
+    # @return [Boolean]
+    def digital?
+      @digital ||= calculator.is_a?(Spree::Calculator::Shipping::DigitalDelivery)
+    end
+
     private
 
     # Some shipping methods are only meant to be set via backend

data/app/models/spree/store.rb

@@ -339,7 +339,7 @@ module Spree
     end
 
     def supported_shipping_zones
-      @supported_shipping_zones ||= if checkout_zone_id.present?
+      @supported_shipping_zones ||= if checkout_zone.present?
                                       [checkout_zone]
                                     else
                                       Spree::Zone.includes(zone_members: :zoneable).all

data/app/models/spree/tax_rate.rb

@@ -33,6 +33,17 @@ module Spree
 
     self.whitelisted_ransackable_attributes = %w[amount zone_id tax_category_id included_in_price name]
 
+    # Virtual attribute for percentage display in admin forms
+    def amount_percentage
+      return nil if amount.nil?
+
+      (amount * 100).round(2)
+    end
+
+    def amount_percentage=(value)
+      self.amount = value.present? ? (value.to_f / 100) : nil
+    end
+
     # Gets the array of TaxRates appropriate for the specified tax zone
     def self.match(order_tax_zone)
       return [] unless order_tax_zone
@@ -61,18 +72,18 @@ module Spree
     # to relevant items.
     def self.adjust(order, items)
       rates = match(order.tax_zone)
-      tax_categories = rates.map(&:tax_category)
+      tax_category_ids = rates.map(&:tax_category_id)
 
       # using destroy_all to ensure adjustment destroy callback fires.
       Spree::Adjustment.where(adjustable: items).tax.destroy_all
 
       relevant_items = items.select do |item|
-        tax_categories.include?(item.tax_category)
+        tax_category_ids.include?(item.tax_category_id)
       end
 
       relevant_items.each do |item|
         relevant_rates = rates.select do |rate|
-          rate.tax_category == item.tax_category
+          rate.tax_category_id == item.tax_category_id
         end
         store_pre_tax_amount(item, relevant_rates)
         relevant_rates.each do |rate|

data/app/models/spree/variant.rb

@@ -223,6 +223,8 @@ module Spree
       self.class.in_stock_or_backorderable.exists?(id: id)
     end
 
+    # Returns tax category for Variant
+    # @return [Spree::TaxCategory]
     def tax_category
       @tax_category ||= if self[:tax_category_id].nil?
                           product.tax_category
@@ -231,6 +233,16 @@ module Spree
                         end
     end
 
+    # Returns tax category ID for Variant
+    # @return [Integer]
+    def tax_category_id
+      @tax_category_id ||= if self[:tax_category_id].nil?
+                             product.tax_category_id
+                           else
+                             self[:tax_category_id]
+                           end
+    end
+
     def options_text
       @options_text ||= Spree::Variants::OptionsPresenter.new(self).to_sentence
     end
@@ -254,7 +266,7 @@ module Spree
     # Returns default Image for Variant
     # @return [Spree::Image]
     def default_image
-      @default_image ||= if images.size.positive?
+      @default_image ||= if images.any?
                            images.first
                          else
                            product.default_image
@@ -480,6 +492,8 @@ module Spree
     end
 
     # Is this variant purely digital? (no physical product)
+    #
+    # @return [Boolean]
     def digital?
       product.digital?
     end

data/app/models/spree/zone.rb

@@ -21,6 +21,7 @@ module Spree
 
     after_save :remove_defunct_members
     after_save :remove_previous_default, if: %i[default_tax? saved_change_to_default_tax?]
+    before_destroy :nullify_checkout_zone
 
     alias members zone_members
     accepts_nested_attributes_for :zone_members, allow_destroy: true, reject_if: proc { |a| a['zoneable_id'].blank? }
@@ -205,5 +206,11 @@ module Spree
         members << member
       end
     end
+
+    def nullify_checkout_zone
+      if id == Spree::Store.current.checkout_zone_id
+        Spree::Store.current.update(checkout_zone_id: nil)
+      end
+    end
   end
 end

data/app/services/spree/products/prepare_nested_attributes.rb

@@ -45,8 +45,6 @@ module Spree
           end
         end
 
-        params.delete(:variants_attributes) if params[:variants_attributes].blank?
-
         # mark resource properties to be removed
         # when value is left blank
         if params[:product_properties_attributes].present?
@@ -62,6 +60,18 @@ module Spree
         # ensure there is at least one store
         params[:store_ids] = [store.id] if params[:store_ids].blank?
 
+        # Add empty list for option_type_ids and mark variants as removed if there are no variants and options
+        if params[:variants_attributes].blank? && variants_to_remove.any? && !params.key?(:option_type_ids)
+          params[:option_type_ids] = []
+          params[:variants_attributes] = {}
+
+          variants_to_remove.each_with_index do |variant_id, index|
+            params[:variants_attributes][index.to_s] = { id: variant_id, _destroy: '1' }
+          end
+
+          params[:variants_attributes].permit!
+        end
+
         params
       end
 

data/app/services/spree/taxons/add_products.rb

@@ -30,7 +30,10 @@ module Spree
 
         # clearing cache
         Spree::Product.where(id: products.pluck(:id)).touch_all
-        Spree::Taxon.where(id: taxons.pluck(:id)).touch_all
+
+        taxon_ids = taxons.pluck(:id)
+        Spree::Taxon.where(id: taxon_ids).touch_all
+        Spree::Taxons::TouchFeaturedSections.call(taxon_ids: taxon_ids)
 
         success(true)
       end

data/app/services/spree/taxons/remove_products.rb

@@ -44,7 +44,10 @@ module Spree
 
         # clear cache
         Spree::Product.where(id: products.pluck(:id)).touch_all
-        Spree::Taxon.where(id: taxons.pluck(:id)).touch_all
+
+        taxon_ids = taxons.pluck(:id)
+        Spree::Taxon.where(id: taxon_ids).touch_all
+        Spree::Taxons::TouchFeaturedSections.call(taxon_ids: taxon_ids)
 
         success(true)
       end

data/app/views/spree/invitation_mailer/invitation_email.html.erb

@@ -10,7 +10,7 @@
 
 <% if spree.respond_to?(:admin_invitation_url) %>
   <p>
-    <%= link_to Spree.t('invitation_mailer.invitation_email.link_text'), spree.admin_invitation_url(@invitation, token: @invitation.token, host: @invitation.store.formatted_url) %>
+    <%= link_to Spree.t(:accept), spree.admin_invitation_url(@invitation, token: @invitation.token, host: @invitation.store.formatted_url) %>
   </p>
 <% end %>
 

data/config/importmap.rb

@@ -3,6 +3,7 @@ pin '@rails/request.js', to: '@rails--request.js.js', preload: ['application-spr
 pin 'tailwindcss-stimulus-components', preload: ['application-spree-storefront', 'application-spree-admin'] # @3.0.4
 pin 'stimulus-reveal-controller', preload: ['application-spree-admin'] # @4.1.0
 pin '@stimulus-components/auto-submit', to: '@stimulus-components--auto-submit.js', preload: ['application-spree-admin'] # @6.0.0
+pin 'stimulus-textarea-autogrow', preload: ['application-spree-admin'] # @4.1.0
 
 pin_all_from Spree::Core::Engine.root.join('app/javascript/spree/core/controllers'),
              under: 'spree/core/controllers',

data/config/locales/en.yml

@@ -181,6 +181,7 @@ en:
         name: Name
         permalink: Permalink
         position: Position
+        sort_order: Sort Order
       spree/taxonomy:
         name: Name
       spree/user:
@@ -590,6 +591,7 @@ en:
     add_country: Add Country
     add_coupon_code: Add Coupon Code
     add_gift_card: Add Gift Card
+    add_link: Add link
     add_new_address: Add new address
     add_new_credit_card: Add a new card
     add_new_header: Add New Header
@@ -874,6 +876,7 @@ en:
     current: Current
     current_password: Current password
     custom_code: Custom code
+    custom_domains: Custom domains
     custom_font_code: Custom font code
     customer: Customer
     customer_details: Customer Details
@@ -1854,6 +1857,7 @@ en:
     select_stores: Select Store(s)
     select_user: Select customer
     selected_quantity_not_available: selected of %{item} is not available.
+    send_consumer_transactional_emails: Send Customer Transactional Emails
     send_copy_of_all_mails_to: Send Copy of All Mails To
     send_mails_as: Send Mails As
     send_message: Send message

data/db/migrate/20250304115943_add_performed_by_to_spree_reimbursements.rb

@@ -1,5 +1,5 @@
 class AddPerformedByToSpreeReimbursements < ActiveRecord::Migration[6.1]
   def change
-    add_column :spree_reimbursements, :performed_by_id, :bigint, index: true, null: true, if_not_exists: true
+    add_reference :spree_reimbursements, :performed_by, index: true, null: true unless column_exists?(:spree_reimbursements, :performed_by_id)
   end
 end

data/lib/generators/spree/dummy/dummy_generator.rb

@@ -39,6 +39,13 @@ module Spree
       opts[:skip_test] = true
       opts[:skip_bootsnap] = true
       opts[:skip_asset_pipeline] = true # skip installing propshaft, we're still using sprockets as a dependency
+      opts[:skip_docker] = true
+      opts[:skip_rubocop] = true
+      opts[:skip_brakeman] = true
+      opts[:skip_ci] = true
+      opts[:skip_kamal] = true
+      opts[:skip_devcontainer] = true
+      opts[:skip_solid] = true
 
       puts 'Generating dummy Rails application...'
       invoke Rails::Generators::AppGenerator,
@@ -73,46 +80,11 @@ module Spree
       end
     end
 
-    def test_dummy_clean
-      inside dummy_path do
-        remove_file '.gitignore'
-        remove_file 'doc'
-        remove_file 'Gemfile'
-        remove_file 'lib/tasks'
-        remove_file 'app/assets/images/rails.png'
-        remove_file 'app/assets/javascripts/application.js'
-        remove_file 'public/index.html'
-        remove_file 'public/robots.txt'
-        remove_file 'README'
-        remove_file 'test'
-        remove_file 'vendor'
-        remove_file 'spec'
-      end
-    end
-
-    def inject_content_security_policy
-      inside dummy_path do
-        inject_into_file 'config/initializers/content_security_policy.rb', %Q[
-  p.script_src  :self, :https, :unsafe_inline, :http, :unsafe_eval
-        ], before: /^end/, verbose: true
-      end
-    end
-
     attr_reader :lib_name
     attr_reader :database
 
     protected
 
-    def inject_require_for(requirement)
-      inject_into_file 'config/application.rb', %Q[
-begin
-  require '#{requirement}'
-rescue LoadError
-  # #{requirement} is not available.
-end
-      ], before: /require '#{@lib_name}'/, verbose: true
-    end
-
     def inject_yaml_permitted_classes
       inside dummy_path do
         inject_into_file 'config/application.rb', %Q[
@@ -140,10 +112,6 @@ end
     end
     alias store_application_definition! application_definition
 
-    def camelized
-      @camelized ||= name.gsub(/\W/, '_').squeeze('_').camelize
-    end
-
     def remove_directory_if_exists(path)
       remove_dir(path) if File.directory?(path)
     end

data/lib/generators/spree/install/install_generator.rb

@@ -46,24 +46,6 @@ module Spree
       template 'config/initializers/spree.rb', 'config/initializers/spree.rb'
     end
 
-    def additional_tweaks
-      return unless File.exist? 'public/robots.txt'
-
-      append_file 'public/robots.txt', <<-ROBOTS.strip_heredoc
-        User-agent: *
-        Disallow: /checkout
-        Disallow: /cart
-        Disallow: /orders
-        Disallow: /user
-        Disallow: /account
-        Disallow: /api
-        Disallow: /password
-        Disallow: /api_tokens
-        Disallow: /cart_link
-        Disallow: /account_link
-      ROBOTS
-    end
-
     # Currently we only support devise, in the future we will also add support for default Rails authentication
     def install_authentication
       if @authentication == 'devise'
@@ -121,7 +103,6 @@ module Spree
       say_status :copying, 'migrations'
       silence_stream(STDOUT) do
         silence_warnings { rake 'active_storage:install:migrations' }
-        silence_warnings { rake 'action_mailbox:install:migrations' }
         silence_warnings { rake 'action_text:install:migrations' }
         silence_warnings { rake 'spree:install:migrations' }
         silence_warnings { rake 'spree_api:install:migrations' }

data/lib/spree/core/configuration.rb

@@ -54,7 +54,7 @@ module Spree
       preference :require_master_price, :boolean, default: false
       preference :restock_inventory, :boolean, default: true # Determines if a return item is restocked automatically once it has been received
       preference :return_eligibility_number_of_days, :integer, default: 365
-      preference :send_core_emails, :boolean, default: true # Default mail headers settings
+      preference :send_core_emails, :boolean, default: true, deprecated: true # Default mail headers settings
       preference :shipping_instructions, :boolean, deprecated: true
       preference :show_only_complete_orders_by_default, :boolean, deprecated: true
       preference :show_variant_full_price, :boolean, default: false # Displays variant full price or difference with product price. Default false to be compatible with older behavior

data/lib/spree/core/engine.rb

@@ -299,7 +299,7 @@ module Spree
       end
 
       initializer 'spree.core.checking_migrations' do
-        Migrations.new(config, engine_name).check
+        Migrations.new(config, engine_name).check unless Rails.env.test?
       end
 
       initializer 'spree.core.assets' do |app|

data/lib/spree/core/version.rb

@@ -1,5 +1,5 @@
 module Spree
-  VERSION = '5.1.4'.freeze
+  VERSION = '5.1.6'.freeze
 
   def self.version
     VERSION

data/lib/spree/core.rb

@@ -91,7 +91,7 @@ module Spree
   end
 
   def self.root_domain
-    @@root_domain ||= Rails.application.routes.default_url_options[:host] || 'lvh.me'
+    @@root_domain
   end
 
   def self.queues

data/lib/spree/permitted_attributes.rb

@@ -257,11 +257,11 @@ module Spree
 
     @@store_credit_category_attributes = [:name]
 
-    @@taxonomy_attributes = [:name]
+    @@taxonomy_attributes = [:name, :position]
 
     @@tax_category_attributes = [:name, :tax_code,:description, :is_default]
 
-    @@tax_rate_attributes = [:name, :amount, :zone_id, :tax_category_id, :included_in_price, :show_rate_in_label, :calculator_type, calculator_attributes: {}]
+    @@tax_rate_attributes = [:name, :amount, :amount_percentage, :zone_id, :tax_category_id, :included_in_price, :show_rate_in_label, :calculator_type, calculator_attributes: {}]
 
     @@taxon_attributes = [
       :name, :parent_id, :position, :icon, :description, :permalink, :hide_from_nav,

data/lib/spree/testing_support/common_rake.rb

@@ -21,16 +21,16 @@ namespace :common do
 
     skip_javascript = ['spree/api', 'spree/core', 'spree/sample', 'spree/emails'].include?(ENV['LIB_NAME'])
 
-    Spree::DummyGenerator.start [
-      "--lib_name=#{ENV['LIB_NAME']}",
-      "--skip_javascript=#{skip_javascript}"
+    dummy_app_args = [
+      "--lib_name=#{ENV['LIB_NAME']}"
     ]
+    if skip_javascript
+      dummy_app_args << '--skip_javascript'
+    end
+    Spree::DummyGenerator.start dummy_app_args
 
-    # install frontend libraries
     unless skip_javascript
-      system('bin/rails importmap:install')
-      system('bin/rails turbo:install')
-      system('bin/rails stimulus:install')
+      system('bin/rails importmap:install turbo:install stimulus:install')
     end
 
     # install devise if it's not the legacy user, useful for testing storefront
@@ -52,21 +52,39 @@ namespace :common do
       "--authentication=#{args[:authentication]}"
     ]
 
-    puts 'Setting up dummy database...'
-    system('bin/rails db:environment:set RAILS_ENV=test > /dev/null 2>&1')
-    system('bundle exec rake db:drop db:create > /dev/null 2>&1')
-    Spree::DummyModelGenerator.start
-    system('bundle exec rake db:migrate > /dev/null 2>&1')
+    if !skip_javascript || ENV['LIB_NAME'] == 'spree/emails'
+      puts 'Precompiling assets...'
+      system('bundle exec rake assets:precompile > /dev/null 2>&1')
+    end
+
+    unless ENV['NO_MIGRATE']
+      puts 'Setting up dummy database...'
+      system('bin/rails db:environment:set RAILS_ENV=test > /dev/null 2>&1')
+      system('bundle exec rake db:drop db:create > /dev/null 2>&1')
+      Spree::DummyModelGenerator.start
+      system('bundle exec rake db:migrate > /dev/null 2>&1')
+    end
 
     begin
       require "generators/#{ENV['LIB_NAME']}/install/install_generator"
       puts 'Running extension installation generator...'
-      "#{ENV['LIB_NAME'].camelize}::Generators::InstallGenerator".constantize.start(['--auto-run-migrations'])
+
+      if ENV['NO_MIGRATE']
+        "#{ENV['LIB_NAME'].camelize}::Generators::InstallGenerator".constantize.start([])
+      else
+        "#{ENV['LIB_NAME'].camelize}::Generators::InstallGenerator".constantize.start(['--auto-run-migrations'])
+      end
     rescue LoadError
       puts 'Skipping installation no generator to run...'
     end
+  end
 
-    system('bundle exec rake assets:precompile > /dev/null 2>&1') if !skip_javascript || ENV['LIB_NAME'] == 'spree/emails'
+  task :db_setup do |_t|
+    puts 'Setting up dummy database...'
+    system('bin/rails db:environment:set RAILS_ENV=test > /dev/null 2>&1')
+    system('bundle exec rake db:drop db:create > /dev/null 2>&1')
+    Spree::DummyModelGenerator.start
+    system('bundle exec rake db:migrate > /dev/null 2>&1')
   end
 
   task :seed do |_t|

data/vendor/javascript/stimulus-textarea-autogrow.js

@@ -0,0 +1,4 @@
+// stimulus-textarea-autogrow@4.1.0 downloaded from https://ga.jspm.io/npm:stimulus-textarea-autogrow@4.1.0/dist/stimulus-textarea-autogrow.mjs
+
+import{Controller as e}from"@hotwired/stimulus";function r(e,t){let i;return(...s)=>{const o=this;clearTimeout(i),i=setTimeout((()=>e.apply(o,s)),t)}}class l extends e{initialize(){this.autogrow=this.autogrow.bind(this)}connect(){this.element.style.overflow="hidden";const e=this.resizeDebounceDelayValue;this.onResize=e>0?r(this.autogrow,e):this.autogrow,this.autogrow(),this.element.addEventListener("input",this.autogrow),window.addEventListener("resize",this.onResize)}disconnect(){window.removeEventListener("resize",this.onResize)}autogrow(){this.element.style.height="auto",this.element.style.height=`${this.element.scrollHeight}px`}}l.values={resizeDebounceDelay:{type:Number,default:100}};export{l as default};
+

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: spree_core
 version: !ruby/object:Gem::Version
-  version: 5.1.4
+  version: 5.1.6
 platform: ruby
 authors:
 - Sean Schofield
@@ -10,7 +10,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-07-24 00:00:00.000000000 Z
+date: 2025-09-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: i18n-tasks
@@ -1375,6 +1375,7 @@ files:
 - vendor/javascript/@rails--request.js.js
 - vendor/javascript/@stimulus-components--auto-submit.js
 - vendor/javascript/stimulus-reveal-controller.js
+- vendor/javascript/stimulus-textarea-autogrow.js
 - vendor/javascript/tailwindcss-stimulus-components.js
 homepage: https://spreecommerce.org
 licenses:
@@ -1382,9 +1383,9 @@ licenses:
 - BSD-3-Clause
 metadata:
   bug_tracker_uri: https://github.com/spree/spree/issues
-  changelog_uri: https://github.com/spree/spree/releases/tag/v5.1.4
+  changelog_uri: https://github.com/spree/spree/releases/tag/v5.1.6
   documentation_uri: https://docs.spreecommerce.org/
-  source_code_uri: https://github.com/spree/spree/tree/v5.1.4
+  source_code_uri: https://github.com/spree/spree/tree/v5.1.6
 post_install_message:
 rdoc_options: []
 require_paths: