spree_core 5.3.4 → 5.4.0.beta2

data/db/sample_data/promotions.rb

@@ -0,0 +1,8 @@
+promotion = Spree::Promotion.where(
+  name: 'Free Shipping',
+  code: 'FREESHIP'
+).first_or_create! do |promo|
+  promo.stores = Spree::Store.all
+end
+
+Spree::Promotion::Actions::FreeShipping.where(promotion: promotion).first_or_create!

data/db/sample_data/shipping_methods.rb

@@ -0,0 +1,39 @@
+begin
+  north_america = Spree::Zone.find_by!(name: 'North America')
+rescue ActiveRecord::RecordNotFound
+  puts "Couldn't find 'North America' zone. Did you run `rake db:seed` first?"
+  exit
+end
+
+europe_vat = Spree::Zone.find_by!(name: 'EU_VAT')
+shipping_category = Spree::ShippingCategory.find_or_create_by!(name: 'Default')
+
+shipping_methods = [
+  { name: 'UPS Ground (USD)', zones: [north_america], display_on: 'both', shipping_categories: [shipping_category] },
+  { name: 'UPS Two Day (USD)', zones: [north_america], display_on: 'both', shipping_categories: [shipping_category] },
+  { name: 'UPS One Day (USD)', zones: [north_america], display_on: 'both', shipping_categories: [shipping_category] },
+  { name: 'UPS Ground (EU)', zones: [europe_vat], display_on: 'both', shipping_categories: [shipping_category] },
+  { name: 'UPS Ground (EUR)', zones: [europe_vat], display_on: 'both', shipping_categories: [shipping_category] }
+]
+
+shipping_methods.each do |attributes|
+  Spree::ShippingMethod.where(name: attributes[:name]).first_or_create! do |shipping_method|
+    shipping_method.calculator = Spree::Calculator::Shipping::FlatRate.create!
+    shipping_method.zones = attributes[:zones]
+    shipping_method.display_on = attributes[:display_on]
+    shipping_method.shipping_categories = attributes[:shipping_categories]
+  end
+end
+
+{
+  'UPS Ground (USD)' => [5, 'USD'],
+  'UPS Ground (EU)' => [5, 'USD'],
+  'UPS One Day (USD)' => [15, 'USD'],
+  'UPS Two Day (USD)' => [10, 'USD'],
+  'UPS Ground (EUR)' => [8, 'EUR']
+}.each do |shipping_method_name, (price, currency)|
+  shipping_method = Spree::ShippingMethod.find_by!(name: shipping_method_name)
+  shipping_method.calculator.preferences = { amount: price, currency: currency }
+  shipping_method.calculator.save!
+  shipping_method.save!
+end

data/lib/generators/spree/authentication/devise/devise_generator.rb

@@ -55,7 +55,7 @@ module Spree
             inject_into_file admin_user_class_file, after: "class #{Spree.admin_user_class.name} < ApplicationRecord\n" do
               <<-RUBY
     # Spree modules
-    include Spree::UserMethods
+    include Spree::AdminUserMethods
               RUBY
             end
             gsub_file admin_user_class_file, "< ApplicationRecord", "< Spree.base_class"
@@ -65,7 +65,7 @@ module Spree
             say "Could not locate admin user model file at #{admin_user_class_file}. Please add these lines manually:", :red
             say <<~RUBY
               # Spree modules
-              include Spree::UserMethods
+              include Spree::AdminUserMethods
             RUBY
 
             say "Please replace < ApplicationRecord with < Spree.base_class in #{admin_user_class_file}"

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

@@ -0,0 +1,54 @@
+require 'rails/generators'
+
+module Spree
+  module Authentication
+    class DummyGenerator < Rails::Generators::Base
+      desc 'Set up a Spree installation with dummy authentication for testing. Creates a separate spree_admin_users table.'
+
+      def self.source_paths
+        paths = superclass.source_paths
+        paths << File.expand_path('templates', __dir__)
+        paths.flatten
+      end
+
+      def create_authentication_helpers
+        template 'authentication_helpers.rb.tt', 'lib/spree/authentication_helpers.rb'
+      end
+
+      def configure_authentication_helpers
+        file_action = File.exist?('config/initializers/spree.rb') ? :append_file : :create_file
+        send(file_action, 'config/initializers/spree.rb') do
+          <<~RUBY
+
+            Rails.application.config.to_prepare do
+              require_dependency 'spree/authentication_helpers'
+            end
+          RUBY
+        end
+      end
+
+      def create_admin_users_migration
+        # Use a fixed early timestamp (20210913) to ensure this migration runs BEFORE
+        # any Spree core migrations that reference spree_admin_users table.
+        # The earliest such migration is 20250122113708_add_first_and_last_name_to_spree_admin_class.rb
+        # We use a timestamp before the main Spree migration (20210914000000_spree_four_three.rb)
+        migration_file = File.join('db', 'migrate', '20210913000000_create_spree_admin_users.rb')
+
+        # Skip if migration already exists
+        return if File.exist?(migration_file) || migration_exists?('create_spree_admin_users')
+
+        template 'create_spree_admin_users.rb.tt', migration_file
+      end
+
+      private
+
+      def migration_exists?(name)
+        Dir.glob(File.join('db', 'migrate', "*_#{name}.rb")).any?
+      end
+
+      def migration_version
+        "[#{Rails::VERSION::MAJOR}.#{Rails::VERSION::MINOR}]"
+      end
+    end
+  end
+end

data/lib/generators/spree/authentication/dummy/templates/authentication_helpers.rb.tt

@@ -0,0 +1,52 @@
+# Dummy authentication helpers for testing purposes.
+# These provide stub implementations that don't require actual routes.
+module Spree
+  module AuthenticationHelpers
+    def self.included(receiver)
+      receiver.helper_method(
+        :spree_current_user,
+        :spree_login_path,
+        :spree_signup_path,
+        :spree_logout_path,
+        :spree_forgot_password_path,
+        :spree_edit_password_path,
+        :spree_admin_login_path,
+        :spree_admin_logout_path
+      )
+    end
+
+    def spree_current_user
+      @spree_current_user
+    end
+
+    def spree_login_path(_opts = {})
+      '/login'
+    end
+
+    def spree_signup_path(_opts = {})
+      '/signup'
+    end
+
+    def spree_logout_path(_opts = {})
+      '/logout'
+    end
+
+    def spree_forgot_password_path(_opts = {})
+      '/forgot_password'
+    end
+
+    def spree_edit_password_path(_opts = {})
+      '/edit_password'
+    end
+
+    def spree_admin_login_path(_opts = {})
+      '/admin/login'
+    end
+
+    def spree_admin_logout_path(_opts = {})
+      '/admin/logout'
+    end
+  end
+end
+
+ApplicationController.include Spree::AuthenticationHelpers if defined?(ApplicationController)

data/lib/generators/spree/authentication/dummy/templates/create_spree_admin_users.rb.tt

@@ -0,0 +1,33 @@
+class CreateSpreeAdminUsers < ActiveRecord::Migration<%= migration_version %>
+  def change
+    create_table :spree_admin_users do |t|
+      t.string :encrypted_password, limit: 128
+      t.string :password_salt, limit: 128
+      t.string :email
+      t.string :remember_token
+      t.string :persistence_token
+      t.string :reset_password_token
+      t.string :perishable_token
+      t.integer :sign_in_count, default: 0, null: false
+      t.integer :failed_attempts, default: 0, null: false
+      t.datetime :last_request_at
+      t.datetime :current_sign_in_at
+      t.datetime :last_sign_in_at
+      t.string :current_sign_in_ip
+      t.string :last_sign_in_ip
+      t.string :login
+      t.string :authentication_token
+      t.string :unlock_token
+      t.datetime :locked_at
+      t.datetime :remember_created_at
+      t.datetime :reset_password_sent_at
+      t.string :first_name
+      t.string :last_name
+      t.string :selected_locale
+
+      t.timestamps
+    end
+
+    add_index :spree_admin_users, :email, unique: true
+  end
+end

data/lib/generators/spree/cursor_rules/templates/spree_rules.mdc

@@ -77,9 +77,7 @@ class Spree::Product < Spree.base_class
   include Spree::Metafields
   
   has_many :variants, class_name: 'Spree::Variant', dependent: :destroy
-  has_many :product_properties, class_name: 'Spree::ProductProperty', dependent: :destroy
-  has_many :properties, through: :product_properties, source: :property
-  
+
   scope :available, -> { where(available_on: ..Time.current) }
   
   validates :name, presence: true

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

@@ -4,7 +4,7 @@ require 'spree/core/version'
 
 module Spree
   class DummyGenerator < Rails::Generators::Base
-    SPREE_GEMS = %w(spree_admin spree_storefront spree_api spree_emails).freeze
+    SPREE_GEMS = %w(spree_admin spree_api spree_emails).freeze
 
     desc 'Creates blank Rails application, installs Spree and all sample data'
 

data/lib/spree/core/configuration.rb

@@ -15,7 +15,6 @@
 # a.get :color
 # a.preferred_color
 #
-require 'spree/core/search/base'
 require 'spree/core/preferences/configuration' # for compatibility reasons
 require 'spree/core/preferences/runtime_configuration'
 require 'spree/core/preferences/preferable'
@@ -40,6 +39,7 @@ module Spree
       preference :company, :boolean, default: false, deprecated: 'Use the company_field_enabled preference in the Spree::Store model' # Request company field for billing and shipping addr
       preference :currency, :string, default: 'USD', deprecated: true
       preference :credit_to_new_allocation, :boolean, default: false
+      preference :disable_migration_check, :boolean, default: false # when turned on disables the startup warning about missing engine migrations
       preference :disable_sku_validation, :boolean, default: false # when turned off disables the built-in SKU uniqueness validation
       preference :disable_store_presence_validation, :boolean, default: false # when turned off disables Store presence validation for Products and Payment Methods
       preference :events_log_enabled, :boolean, default: true # Log all Spree events to Rails logger
@@ -82,7 +82,6 @@ module Spree
       preference :mailer_logo, :string, deprecated: true
       preference :max_level_in_taxons_menu, :integer, deprecated: true
       preference :non_expiring_credit_types, :array, default: []
-      preference :product_properties_enabled, :boolean, default: false # enable legacy product properties
       preference :products_per_page, :integer, default: 12
       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
@@ -111,7 +110,6 @@ module Spree
       preference :gift_card_batch_web_limit, :integer, default: 500 # number of gift card codes to be generated in the web process, more than this will be generated in a background job
       preference :gift_card_batch_limit, :integer, default: 50_000
 
-      attr_writer :searcher_class
     end
   end
 end

data/lib/spree/core/controller_helpers/common.rb

@@ -1,7 +1,13 @@
 module Spree
   module Core
     module ControllerHelpers
+      # @deprecated This module is deprecated and will be removed in Spree 5.5.
       module Common
+        def self.included(base)
+          Spree::Deprecation.warn(
+            'Spree::Core::ControllerHelpers::Common is deprecated and will be removed in Spree 5.5.'
+          )
+        end
       end
     end
   end

data/lib/spree/core/controller_helpers/currency.rb

@@ -39,8 +39,13 @@ module Spree
         end
 
         # Returns the list of supported currencies for all stores.
+        # @deprecated This method will be removed in Spree 5.5.
         # @return [Array<String>] the list of supported currencies, eg. `["USD", "EUR"]`
         def supported_currencies_for_all_stores
+          Spree::Deprecation.warn(
+            'supported_currencies_for_all_stores is deprecated and will be removed in Spree 5.5.'
+          )
+
           @supported_currencies_for_all_stores ||= begin
             (
               Spree::Store.pluck(:supported_currencies).map { |c| c&.split(',') }.flatten + Spree::Store.pluck(:default_currency)

data/lib/spree/core/controller_helpers/order.rb

@@ -15,8 +15,12 @@ module Spree
           @order_token ||= cookies.signed[:token] || params[:order_token]
         end
 
-        # Used in the link_to_cart helper.
+        # @deprecated Use `current_order` instead. This method will be removed in Spree 5.5.
         def simple_current_order
+          Spree::Deprecation.warn(
+            'simple_current_order is deprecated and will be removed in Spree 5.5. Use current_order instead.'
+          )
+
           return @simple_current_order if @simple_current_order
 
           @simple_current_order = find_order_by_token_or_user

data/lib/spree/core/controller_helpers/store.rb

@@ -61,7 +61,7 @@ module Spree
 
         def current_tax_zone
           @current_tax_zone ||= begin
-            zone = @current_order&.tax_zone || Spree::Zone.default_tax || current_store.checkout_zone
+            zone = @current_order&.tax_zone || Spree::Zone.default_tax
             Spree::Current.zone = zone
             zone
           end

data/lib/spree/core/controller_helpers/strong_parameters.rb

@@ -34,8 +34,7 @@ module Spree
         def permitted_product_attributes
           permitted_attributes.product_attributes + [
             variants_attributes: permitted_variant_attributes + ['id', :_destroy],
-            master_attributes: permitted_variant_attributes + ['id'],
-            product_properties_attributes: permitted_product_properties_attributes + ['id', :_destroy]
+            master_attributes: permitted_variant_attributes + ['id']
           ]
         end
 

data/lib/spree/core/dependencies.rb

@@ -56,9 +56,8 @@ module Spree
         # sorter
         collection_sorter: 'Spree::BaseSorter',
         order_sorter: 'Spree::BaseSorter',
+        posts_sorter: nil,
         products_sorter: 'Spree::Products::Sort',
-        posts_sorter: 'Spree::Posts::Sort',
-
         # paginator
         collection_paginator: 'Spree::Shared::Paginate',
 
@@ -87,24 +86,17 @@ module Spree
 
         payment_create_service: 'Spree::Payments::Create',
 
-        # data feeds
-        data_feeds_google_rss_service: 'Spree::DataFeeds::Google::Rss',
-        data_feeds_google_optional_attributes_service: 'Spree::DataFeeds::Google::OptionalAttributes',
-        data_feeds_google_required_attributes_service: 'Spree::DataFeeds::Google::RequiredAttributes',
-        data_feeds_google_optional_sub_attributes_service: 'Spree::DataFeeds::Google::OptionalSubAttributes',
-        data_feeds_google_products_list: 'Spree::DataFeeds::Google::ProductsList',
-
         # finders
         address_finder: 'Spree::Addresses::Find',
         country_finder: 'Spree::Countries::Find',
         cms_page_finder: nil, # LEGACY
         menu_finder: nil, # LEGACY
         current_order_finder: 'Spree::Orders::FindCurrent',
-        current_store_finder: 'Spree::Stores::FindCurrent',
+        current_store_finder: 'Spree::Stores::FindDefault',
         completed_order_finder: 'Spree::Orders::FindComplete',
         credit_card_finder: 'Spree::CreditCards::Find',
+        posts_finder: nil,
         products_finder: 'Spree::Products::Find',
-        posts_finder: 'Spree::Posts::Find',
         taxon_finder: 'Spree::Taxons::Find',
         line_item_by_variant_finder: 'Spree::LineItems::FindByVariant',
         variant_finder: 'Spree::Variants::Find'

data/lib/spree/core/engine.rb

@@ -14,7 +14,6 @@ module Spree
                                :promotions,
                                :pricing,
                                :line_item_comparison_hooks,
-                               :data_feed_types,
                                :export_types,
                                :import_types,
                                :taxon_rules,
@@ -29,7 +28,9 @@ module Spree
                                :analytics_events,
                                :analytics_event_handlers,
                                :integrations,
-                               :subscribers)
+                               :subscribers,
+                               :store_authentication_strategies,
+                               :admin_authentication_strategies)
       SpreeCalculators = Struct.new(:shipping_methods, :tax_rates, :promotion_actions_create_adjustments, :promotion_actions_create_item_adjustments)
       PromoEnvironment = Struct.new(:rules, :actions)
       PricingEnvironment = Struct.new(:rules)
@@ -41,10 +42,6 @@ module Spree
       # Add app/subscribers to autoload paths
       config.paths.add 'app/subscribers', eager_load: true
 
-      rake_tasks do
-        load File.join(root, 'lib', 'tasks', 'exchanges.rake')
-      end
-
       initializer 'spree.environment', before: :load_config_initializers do |app|
         app.config.spree = Environment.new(SpreeCalculators.new, SpreeValidators.new, Spree::Core::Configuration.new, Spree::Core::Dependencies.new)
 
@@ -125,7 +122,6 @@ module Spree
 
         Rails.application.config.spree.payment_methods = [
           Spree::Gateway::Bogus,
-          Spree::Gateway::BogusSimple,
           Spree::Gateway::CustomPaymentSourceMethod,
           Spree::PaymentMethod::Check,
           Spree::PaymentMethod::StoreCredit
@@ -166,6 +162,7 @@ module Spree
 
         Rails.application.config.spree.pricing.rules.concat [
           Spree::PriceRules::ZoneRule,
+          Spree::PriceRules::MarketRule,
           Spree::PriceRules::UserRule,
           Spree::PriceRules::CustomerGroupRule,
           Spree::PriceRules::VolumeRule
@@ -178,10 +175,6 @@ module Spree
           Promotion::Actions::FreeShipping
         ]
 
-        Rails.application.config.spree.data_feed_types = [
-          Spree::DataFeed::Google
-        ]
-
         Rails.application.config.spree.export_types = [
           Spree::Exports::Products,
           Spree::Exports::Orders,
@@ -192,6 +185,7 @@ module Spree
 
         Rails.application.config.spree.import_types = [
           Spree::Imports::Products,
+          Spree::Imports::Customers,
         ]
 
         Rails.application.config.spree.taxon_rules = [
@@ -208,7 +202,6 @@ module Spree
         Rails.application.config.spree.translatable_resources = [
           Spree::OptionType,
           Spree::Product,
-          Spree::Property,
           Spree::Taxon,
           Spree::Taxonomy,
           Spree::Store,
@@ -228,7 +221,6 @@ module Spree
           Spree::Address,
           Spree::Asset,
           Spree::CreditCard,
-          Spree::CustomDomain,
           Spree::CustomerReturn,
           Spree::GiftCard,
           Spree::Image,
@@ -240,8 +232,6 @@ module Spree
           Spree::Payment,
           Spree::PaymentMethod,
           Spree::PaymentSource,
-          Spree::Post,
-          Spree::PostCategory,
           Spree::Product,
           Spree::Promotion,
           Spree::Refund,
@@ -304,6 +294,14 @@ module Spree
           Spree::InvitationEmailSubscriber,
           Spree::ProductMetricsSubscriber
         ]
+
+        # Pre-load authentication strategy classes to avoid reflection at request time
+        Rails.application.config.spree.store_authentication_strategies = {
+          email: Spree::Authentication::Strategies::EmailPasswordStrategy
+        }
+        Rails.application.config.spree.admin_authentication_strategies = {
+          email: Spree::Authentication::Strategies::EmailPasswordStrategy
+        }
       end
 
       initializer 'spree.promo.register.promotions.actions' do |app|
@@ -322,8 +320,10 @@ module Spree
         ]
       end
 
-      initializer 'spree.core.checking_migrations' do
-        Migrations.new(config, engine_name).check unless Rails.env.test?
+      initializer 'spree.core.checking_migrations' do |app|
+        app.config.after_initialize do
+          Migrations.new(config, engine_name).check unless Rails.env.test? || Spree::Config.disable_migration_check
+        end
       end
 
       initializer 'spree.core.assets' do |app|

data/lib/spree/core/pricing/context.rb

@@ -1,22 +1,24 @@
 module Spree
   module Pricing
     class Context
-      attr_reader :variant, :currency, :store, :zone, :user, :quantity, :date, :order
+      attr_reader :variant, :currency, :store, :zone, :market, :user, :quantity, :date, :order
 
       # Initializes the context
       # @param variant [Spree::Variant]
       # @param currency [String]
       # @param store [Spree::Store]
       # @param zone [Spree::Zone]
+      # @param market [Spree::Market]
       # @param user [Spree::User]
       # @param quantity [Integer]
       # @param date [Time]
       # @param order [Spree::Order]
-      def initialize(variant: nil, currency:, store: nil, zone: nil, user: nil, quantity: nil, date: nil, order: nil)
+      def initialize(variant: nil, currency:, store: nil, zone: nil, market: nil, user: nil, quantity: nil, date: nil, order: nil)
         @variant = variant
         @currency = currency
         @store = store || Spree::Current.store
         @zone = zone || Spree::Current.zone
+        @market = market || Spree::Current.market
         @user = user
         @quantity = quantity
         @date = date || Time.current
@@ -36,7 +38,7 @@ module Spree
           variant: variant,
           currency: order.currency,
           store: order.store,
-          zone: order.tax_zone || order.store.checkout_zone,
+          zone: order.tax_zone || Spree::Zone.default_tax,
           user: order.user,
           quantity: quantity || order.line_items.find_by(variant: variant)&.quantity,
           order: order
@@ -53,6 +55,7 @@ module Spree
           currency,
           store&.id,
           zone&.id,
+          market&.id,
           user&.id,
           quantity,
           date&.to_i

data/lib/spree/core/pricing/resolver.rb

@@ -12,9 +12,7 @@ module Spree
       # Returns the best price for the variant
       # @return [Spree::Price]
       def resolve
-        Rails.cache.fetch(cache_key, expires_in: 15.minutes) do
-          find_best_price
-        end
+        find_best_price
       end
 
       private
@@ -118,12 +116,6 @@ module Spree
           price_list_id: nil
         )
       end
-
-      # Returns the cache key for the resolver
-      # @return [String]
-      def cache_key
-        "#{context.cache_key}/resolved_price"
-      end
     end
   end
 end

data/lib/spree/core/version.rb

@@ -1,5 +1,5 @@
 module Spree
-  VERSION = '5.3.4'.freeze
+  VERSION = '5.4.0.beta2'.freeze
 
   def self.version
     VERSION