solidus_core 4.3.5 → 4.4.0

data/app/models/spree/promotion_handler/coupon.rb

@@ -1,123 +0,0 @@
-# frozen_string_literal: true
-
-module Spree
-  module PromotionHandler
-    class Coupon
-      attr_reader :order, :coupon_code
-      attr_accessor :error, :success, :status_code
-
-      def initialize(order)
-        @order = order
-        @coupon_code = order.coupon_code && order.coupon_code.downcase
-      end
-
-      def apply
-        if coupon_code.present?
-          if promotion.present? && promotion.active? && promotion.actions.exists?
-            handle_present_promotion(promotion)
-          elsif promotion_code&.promotion&.expired?
-            set_error_code :coupon_code_expired
-          else
-            set_error_code :coupon_code_not_found
-          end
-        end
-
-        self
-      end
-
-      def remove
-        if promotion.blank?
-          set_error_code :coupon_code_not_found
-        elsif !promotion_exists_on_order?(order, promotion)
-          set_error_code :coupon_code_not_present
-        else
-          promotion.remove_from(order)
-          order.recalculate
-          set_success_code :coupon_code_removed
-        end
-
-        self
-      end
-
-      def set_success_code(status_code)
-        @status_code = status_code
-        @success = I18n.t(status_code, scope: 'spree')
-      end
-
-      def set_error_code(status_code, options = {})
-        @status_code = status_code
-        @error = options[:error] || I18n.t(status_code, scope: 'spree')
-      end
-
-      def promotion
-        @promotion ||= begin
-          if promotion_code && promotion_code.promotion.active?
-            promotion_code.promotion
-          end
-        end
-      end
-
-      def successful?
-        success.present? && error.blank?
-      end
-
-      private
-
-      def promotion_code
-        @promotion_code ||= Spree::PromotionCode.where(value: coupon_code).first
-      end
-
-      def handle_present_promotion(promotion)
-        return promotion_usage_limit_exceeded if promotion.usage_limit_exceeded? || promotion_code.usage_limit_exceeded?
-        return promotion_applied if promotion_exists_on_order?(order, promotion)
-
-        unless promotion.eligible?(order, promotion_code: promotion_code)
-          set_promotion_eligibility_error_code(promotion)
-          return error || ineligible_for_this_order
-        end
-
-        # If any of the actions for the promotion return `true`,
-        # then result here will also be `true`.
-        result = promotion.activate(order: order, promotion_code: promotion_code)
-        if result
-          order.recalculate
-          set_success_code :coupon_code_applied
-        else
-          set_error_code :coupon_code_unknown_error
-        end
-      end
-
-      def set_promotion_eligibility_error_code(promotion)
-        return unless eligibility_error_code_present?(promotion)
-
-        eligibility_error = promotion.eligibility_errors.details[:base].first
-
-        @status_code = eligibility_error[:error_code]
-        @error = eligibility_error[:error]
-      end
-
-      def promotion_usage_limit_exceeded
-        set_error_code :coupon_code_max_usage
-      end
-
-      def ineligible_for_this_order
-        set_error_code :coupon_code_not_eligible
-      end
-
-      def promotion_applied
-        set_error_code :coupon_code_already_applied
-      end
-
-      def promotion_exists_on_order?(order, promotion)
-        order.promotions.include? promotion
-      end
-
-      def eligibility_error_code_present?(promotion)
-        promotion.eligibility_errors.present? &&
-          promotion.eligibility_errors.details.present? &&
-          promotion.eligibility_errors.details.key?(:base) &&
-          promotion.eligibility_errors.details[:base].first[:error_code].present?
-      end
-    end
-  end
-end

data/app/models/spree/promotion_handler/page.rb

@@ -1,26 +0,0 @@
-# frozen_string_literal: true
-
-module Spree
-  module PromotionHandler
-    class Page
-      attr_reader :order, :path
-
-      def initialize(order, path)
-        @order = order
-        @path = path.gsub(/\A\//, '')
-      end
-
-      def activate
-        if promotion && promotion.eligible?(order)
-          promotion.activate(order: order)
-        end
-      end
-
-      private
-
-      def promotion
-        @promotion ||= Spree::Promotion.active.find_by(path: path)
-      end
-    end
-  end
-end

data/app/models/spree/promotion_handler/shipping.rb

@@ -1,61 +0,0 @@
-# frozen_string_literal: true
-
-module Spree
-  module PromotionHandler
-    # Used for activating promotions with shipping rules
-    class Shipping
-      attr_reader :order
-      attr_accessor :error, :success
-
-      def initialize(order)
-        @order = order
-      end
-
-      def activate
-        connected_promotions.each do |order_promotion|
-          if order_promotion.promotion.eligible?(order)
-            order_promotion.promotion.activate(
-              order: order,
-              promotion_code: order_promotion.promotion_code,
-            )
-          end
-        end
-
-        not_connected_automatic_promotions.each do |promotion|
-          if promotion.eligible?(order)
-            promotion.activate(order: order)
-          end
-        end
-      end
-
-      private
-
-      def not_connected_automatic_promotions
-        automatic_promotions - connected_promotions.map(&:promotion)
-      end
-
-      def automatic_promotions
-        @automatic_promotions ||= active_shipping_promotions.
-          where(apply_automatically: true).
-          to_a.
-          uniq
-      end
-
-      def connected_promotions
-        @connected_promotions ||= order.order_promotions.
-          joins(:promotion).
-          includes(:promotion).
-          merge(active_shipping_promotions).
-          to_a.
-          uniq
-      end
-
-      def active_shipping_promotions
-        Spree::Promotion.all.
-          active.
-          joins(:promotion_actions).
-          merge(Spree::PromotionAction.shipping)
-      end
-    end
-  end
-end

data/app/models/spree/promotion_rule.rb

@@ -1,55 +0,0 @@
-# frozen_string_literal: true
-
-require 'spree/preferences/persistable'
-
-module Spree
-  # Base class for all promotion rules
-  class PromotionRule < Spree::Base
-    include Spree::Preferences::Persistable
-
-    belongs_to :promotion, class_name: 'Spree::Promotion', inverse_of: :promotion_rules, optional: true
-
-    scope :of_type, ->(type) { where(type: type) }
-
-    validates :promotion, presence: true
-    validate :unique_per_promotion, on: :create
-
-    def preload_relations
-      []
-    end
-
-    def applicable?(_promotable)
-      raise NotImplementedError, "applicable? should be implemented in a sub-class of Spree::PromotionRule"
-    end
-
-    def eligible?(_promotable, _options = {})
-      raise NotImplementedError, "eligible? should be implemented in a sub-class of Spree::PromotionRule"
-    end
-
-    # This states if a promotion can be applied to the specified line item
-    # It is true by default, but can be overridden by promotion rules to provide conditions
-    def actionable?(_line_item)
-      true
-    end
-
-    def eligibility_errors
-      @eligibility_errors ||= ActiveModel::Errors.new(self)
-    end
-
-    def to_partial_path
-      "spree/admin/promotions/rules/#{model_name.element}"
-    end
-
-    private
-
-    def unique_per_promotion
-      if Spree::PromotionRule.exists?(promotion_id: promotion_id, type: self.class.name)
-        errors[:base] << "Promotion already contains this rule type"
-      end
-    end
-
-    def eligibility_error_message(key, options = {})
-      I18n.t(key, **{ scope: [:spree, :eligibility_errors, :messages] }.merge(options))
-    end
-  end
-end

data/app/models/spree/promotion_rule_store.rb

@@ -1,10 +0,0 @@
-# frozen_string_literal: true
-
-module Spree
-  class PromotionRuleStore < Spree::Base
-    self.table_name = "spree_promotion_rules_stores"
-
-    belongs_to :promotion_rule, class_name: "Spree::PromotionRule", optional: true
-    belongs_to :store, class_name: "Spree::Store", optional: true
-  end
-end

data/app/models/spree/promotion_rule_taxon.rb

@@ -1,8 +0,0 @@
-# frozen_string_literal: true
-
-module Spree
-  class PromotionRuleTaxon < Spree::Base
-    belongs_to :promotion_rule, optional: true
-    belongs_to :taxon, optional: true
-  end
-end

data/app/models/spree/promotion_rule_user.rb

@@ -1,10 +0,0 @@
-# frozen_string_literal: true
-
-module Spree
-  class PromotionRuleUser < Spree::Base
-    self.table_name = 'spree_promotion_rules_users'
-
-    belongs_to :promotion_rule, class_name: 'Spree::PromotionRule', optional: true
-    belongs_to :user, class_name: Spree::UserClassHandle.new, optional: true
-  end
-end

data/app/views/spree/promotion_code_batch_mailer/promotion_code_batch_errored.text.erb

@@ -1,2 +0,0 @@
-<%= t(".message", error: @promotion_code_batch.error) %>
-<%= @promotion_code_batch.promotion.name %>

data/app/views/spree/promotion_code_batch_mailer/promotion_code_batch_finished.text.erb

@@ -1,2 +0,0 @@
-<%= t(".message", number_of_codes: 10) %>
-<%= @promotion_code_batch.promotion.name %>

data/db/migrate/20161017102621_create_spree_promotion_code_batch.rb

@@ -1,38 +0,0 @@
-# frozen_string_literal: true
-
-class CreateSpreePromotionCodeBatch < ActiveRecord::Migration[5.0]
-  def change
-    create_table :spree_promotion_code_batches do |t|
-      t.references :promotion, null: false, index: true
-      t.string :base_code, null: false
-      t.integer :number_of_codes, null: false
-      t.string :email
-      t.string :error
-      t.string :state, default: "pending"
-      t.timestamps precision: 6
-    end
-
-    add_foreign_key(
-      :spree_promotion_code_batches,
-      :spree_promotions,
-      column: :promotion_id
-    )
-
-    add_column(
-      :spree_promotion_codes,
-      :promotion_code_batch_id,
-      :integer
-    )
-
-    add_foreign_key(
-      :spree_promotion_codes,
-      :spree_promotion_code_batches,
-      column: :promotion_code_batch_id
-    )
-
-    add_index(
-      :spree_promotion_codes,
-      :promotion_code_batch_id
-    )
-  end
-end

data/db/migrate/20180202190713_create_promotion_rule_stores.rb

@@ -1,12 +0,0 @@
-# frozen_string_literal: true
-
-class CreatePromotionRuleStores < ActiveRecord::Migration[5.1]
-  def change
-    create_table :spree_promotion_rules_stores do |t|
-      t.references :store, null: false
-      t.references :promotion_rule, null: false
-
-      t.timestamps
-    end
-  end
-end

data/db/migrate/20180328172631_add_join_characters_to_promotion_code_batch.rb

@@ -1,11 +0,0 @@
-# frozen_string_literal: true
-
-class AddJoinCharactersToPromotionCodeBatch < ActiveRecord::Migration[5.1]
-  def change
-    add_column(:spree_promotion_code_batches,
-               :join_characters,
-               :string,
-               null: false,
-               default: '_')
-  end
-end

data/db/migrate/20190106184413_remove_code_from_spree_promotions.rb

@@ -1,42 +0,0 @@
-# frozen_string_literal: true
-
-require 'solidus/migrations/promotions_with_code_handlers'
-
-class RemoveCodeFromSpreePromotions < ActiveRecord::Migration[5.1]
-  class Promotion < ActiveRecord::Base
-    self.table_name = "spree_promotions"
-    self.ignored_columns = %w(type)
-  end
-
-  def up
-    promotions_with_code = Promotion.where.not(code: [nil, ''])
-
-    if promotions_with_code.any?
-      # You have some promotions with "code" field present! This is not good
-      # since we are going to remove that column.
-      #
-      self.class.promotions_with_code_handler.new(self, promotions_with_code).call
-    end
-
-    remove_index :spree_promotions, name: :index_spree_promotions_on_code
-    remove_column :spree_promotions, :code
-  end
-
-  def down
-    add_column :spree_promotions, :code, :string
-    add_index :spree_promotions, :code, name: :index_spree_promotions_on_code
-  end
-
-  def self.promotions_with_code_handler
-    # We propose different approaches, just pick the one that you prefer or
-    # write your custom one.
-    #
-    # The fist one (raising an exception) is the default but you can
-    # comment/uncomment the one then better fits you needs or use a
-    # custom class or callable object.
-    #
-    Solidus::Migrations::PromotionWithCodeHandlers::RaiseException
-    # Solidus::Migrations::PromotionWithCodeHandlers::MoveToSpreePromotionCode
-    # Solidus::Migrations::PromotionWithCodeHandlers::DoNothing
-  end
-end

data/db/migrate/20220317165036_set_promotions_with_any_policy_to_all_if_possible.rb

@@ -1,20 +0,0 @@
-# frozen_string_literal: true
-
-class SetPromotionsWithAnyPolicyToAllIfPossible < ActiveRecord::Migration[5.2]
-  def up
-    Spree::Promotion.where(match_policy: :any).includes(:promotion_rules).all.find_each do |promotion|
-      if promotion.promotion_rules.length <= 1
-        promotion.update!(match_policy: :all)
-      else
-        raise StandardError, <<~MSG
-          You have promotions with a match policy of any and more than one rule. Please
-          run `bundle exec rake solidus:split_promotions_with_any_match_policy`.
-        MSG
-      end
-    end
-  end
-
-  def down
-    # No-Op
-  end
-end

data/db/migrate/20230322085416_remove_match_policy_from_spree_promotion.rb

@@ -1,5 +0,0 @@
-class RemoveMatchPolicyFromSpreePromotion < ActiveRecord::Migration[5.2]
-  def change
-    remove_column :spree_promotions, :match_policy, :string, default: "all"
-  end
-end

data/db/migrate/20230325132905_remove_unused_columns_from_promotion_rules.rb

@@ -1,6 +0,0 @@
-class RemoveUnusedColumnsFromPromotionRules < ActiveRecord::Migration[5.2]
-  def change
-    remove_column :spree_promotion_rules, :code, :string
-    remove_column :spree_promotion_rules, :product_group_id, :integer
-  end
-end

data/db/migrate/20230325161633_drop_unused_promo_action_line_items.rb

@@ -1,13 +0,0 @@
-class DropUnusedPromoActionLineItems < ActiveRecord::Migration[5.2]
-  def change
-    drop_table :spree_promotion_action_line_items, force: :cascade do |t|
-      t.integer "promotion_action_id"
-      t.integer "variant_id"
-      t.integer "quantity", default: 1
-      t.datetime "created_at", precision: 6
-      t.datetime "updated_at", precision: 6
-      t.index ["promotion_action_id"], name: "index_spree_promotion_action_line_items_on_promotion_action_id"
-      t.index ["variant_id"], name: "index_spree_promotion_action_line_items_on_variant_id"
-    end
-  end
-end

data/db/migrate/20231027084517_add_order_promotions_foreign_key.rb

@@ -1,10 +0,0 @@
-class AddOrderPromotionsForeignKey < ActiveRecord::Migration[7.0]
-  def up
-    Spree::OrderPromotion.left_joins(:order).where(spree_orders: { id: nil }).delete_all
-    add_foreign_key :spree_orders_promotions, :spree_orders, column: :order_id, validate: false, on_delete: :cascade
-  end
-
-  def down
-    remove_foreign_key :spree_orders_promotions, :spree_orders
-  end
-end

data/db/migrate/20231031175215_add_promotion_order_promotion_foreign_key.rb

@@ -1,10 +0,0 @@
-class AddPromotionOrderPromotionForeignKey < ActiveRecord::Migration[7.0]
-  def up
-    Spree::OrderPromotion.left_joins(:promotion).where(spree_promotions: { id: nil }).delete_all
-    add_foreign_key :spree_orders_promotions, :spree_promotions, column: :promotion_id, on_delete: :cascade
-  end
-
-  def down
-    remove_foreign_key :spree_orders_promotions, :spree_promotions
-  end
-end

data/lib/solidus/migrations/promotions_with_code_handlers.rb

@@ -1,66 +0,0 @@
-# frozen_string_literal: true
-
-module Solidus
-  module Migrations
-    module PromotionWithCodeHandlers
-      class PromotionCode < Spree::Base
-        self.table_name = "spree_promotion_codes"
-      end
-
-      class Base
-        attr_reader :migration_context, :promotions
-
-        def initialize(migration_context, promotions)
-          @migration_context = migration_context
-          @promotions = promotions
-        end
-      end
-
-      class RaiseException < Base
-        def call
-          # Please note that this will block the current migration and rollback all
-          # the previous ones run with the same "rails db:migrate" command.
-          #
-          raise StandardError, "You are trying to drop 'code' column from "\
-            "spree_promotions table but you have at least one record with that "\
-            "column filled. Please take care of that or you could lose data. See:" \
-            "\n" \
-            "https://github.com/solidusio/solidus/pull/3028"\
-            "\n"
-        end
-      end
-
-      class MoveToSpreePromotionCode < Base
-        def call
-          # This is another possible approach, it will convert Spree::Promotion#code
-          # to a Spree::PromotionCode before removing the `code` field.
-          #
-          # NOTE: promotion codes will be downcased and stripped
-          promotions.find_each do |promotion|
-            normalized_code = promotion.code.downcase.strip
-
-            PromotionCode.find_or_create_by!(
-              value: normalized_code,
-              promotion_id: promotion.id
-            ) do
-              migration_context.say "Creating Spree::PromotionCode with value "\
-               "'#{normalized_code}' for Spree::Promotion with id '#{promotion.id}'"
-            end
-          end
-        end
-      end
-
-      class DoNothing < Base
-        def call
-          # This approach will delete all codes without taking any action. At
-          # least we could print a message to track what we are deleting.
-          #
-          promotions.find_each do |promotion|
-            migration_context.say "Code '#{promotion.code}' is going to be removed "\
-              "from Spree::Promotion with id '#{promotion.id}'"
-          end
-        end
-      end
-    end
-  end
-end

data/lib/spree/permission_sets/promotion_display.rb

@@ -1,25 +0,0 @@
-# frozen_string_literal: true
-
-module Spree
-  module PermissionSets
-    # Read-only permissions for promotions.
-    #
-    # This permission set allows users to view all related information about
-    # promotions, also from the admin panel, including:
-    #
-    # - Promotions
-    # - Promotion rules
-    # - Promotion actions
-    # - Promotion categories
-    # - Promotion codes
-    class PromotionDisplay < PermissionSets::Base
-      def activate!
-        can [:read, :admin, :edit], Spree::Promotion
-        can [:read, :admin], Spree::PromotionRule
-        can [:read, :admin], Spree::PromotionAction
-        can [:read, :admin], Spree::PromotionCategory
-        can [:read, :admin], Spree::PromotionCode
-      end
-    end
-  end
-end

data/lib/spree/permission_sets/promotion_management.rb

@@ -1,25 +0,0 @@
-# frozen_string_literal: true
-
-module Spree
-  module PermissionSets
-    # Full permissions for promotion management.
-    #
-    # This permission set grants full control over all promotion and related resources,
-    # including:
-    #
-    # - Promotions
-    # - Promotion rules
-    # - Promotion actions
-    # - Promotion categories
-    # - Promotion codes
-    class PromotionManagement < PermissionSets::Base
-      def activate!
-        can :manage, Spree::Promotion
-        can :manage, Spree::PromotionRule
-        can :manage, Spree::PromotionAction
-        can :manage, Spree::PromotionCategory
-        can :manage, Spree::PromotionCode
-      end
-    end
-  end
-end

data/lib/spree/ransack_4_1_patch.rb

@@ -1,16 +0,0 @@
-# frozen_string_literal: true
-
-require "ransack/version"
-
-return unless Ransack::VERSION.start_with?("4.1.")
-
-module RansackNodeConditionPatch
-  private
-
-  # Waiting for https://github.com/activerecord-hackery/ransack/pull/1468
-  def casted_array?(predicate)
-    predicate.is_a?(Arel::Nodes::Casted) && predicate.value.is_a?(Array)
-  end
-
-  Ransack::Nodes::Condition.prepend(self)
-end

data/lib/spree/testing_support/factories/order_promotion_factory.rb

@@ -1,8 +0,0 @@
-# frozen_string_literal: true
-
-FactoryBot.define do
-  factory :order_promotion, class: 'Spree::OrderPromotion' do
-    association :order
-    association :promotion
-  end
-end

data/lib/spree/testing_support/factories/promotion_category_factory.rb

@@ -1,7 +0,0 @@
-# frozen_string_literal: true
-
-FactoryBot.define do
-  factory :promotion_category, class: 'Spree::PromotionCategory' do
-    name { 'Promotion Category' }
-  end
-end

data/lib/spree/testing_support/factories/promotion_code_factory.rb

@@ -1,8 +0,0 @@
-# frozen_string_literal: true
-
-FactoryBot.define do
-  factory :promotion_code, class: 'Spree::PromotionCode' do
-    promotion
-    sequence(:value) { |i| "code#{i}" }
-  end
-end