spree_core 5.4.2 → 5.5.0.rc1

data/db/migrate/20260515000001_add_store_id_to_spree_newsletter_subscribers.rb

@@ -0,0 +1,25 @@
+class AddStoreIdToSpreeNewsletterSubscribers < ActiveRecord::Migration[7.2]
+  def up
+    add_reference :spree_newsletter_subscribers, :store
+
+    # For spree_multi_tenant we need handle the backfill and indices there
+    return if defined?(SpreeMultiTenant)
+
+    default_store = Spree::Store.default
+    Spree::NewsletterSubscriber.update_all(store_id: default_store.id) if default_store&.persisted?
+
+    change_column_null :spree_newsletter_subscribers, :store_id, false
+
+    remove_index :spree_newsletter_subscribers, :email, unique: true, if_exists: true
+    add_index :spree_newsletter_subscribers, [:email, :store_id], unique: true, if_not_exists: true
+  end
+
+  def down
+    unless defined?(SpreeMultiTenant)
+      remove_index :spree_newsletter_subscribers, [:email, :store_id], if_exists: true
+      add_index :spree_newsletter_subscribers, :email, unique: true, if_not_exists: true
+    end
+
+    remove_reference :spree_newsletter_subscribers, :store
+  end
+end

data/db/migrate/20260529000001_add_unique_index_to_spree_price_rules.rb

@@ -0,0 +1,41 @@
+class AddUniqueIndexToSpreePriceRules < ActiveRecord::Migration[7.2]
+  def up
+    # Remove duplicate rules (same type within the same price list),
+    # keeping the oldest record. Mirrors the
+    # `validates :type, uniqueness: { scope: :price_list_id }` model
+    # constraint at the DB level so concurrent writes can't race past
+    # the validation.
+    if ActiveRecord::Base.connection.adapter_name == 'Mysql2'
+      execute <<-SQL
+        DELETE FROM spree_price_rules
+        WHERE id NOT IN (
+          SELECT min_id FROM (
+            SELECT MIN(id) AS min_id
+            FROM spree_price_rules
+            GROUP BY price_list_id, type
+          ) AS keeper_ids
+        )
+      SQL
+    else
+      execute <<-SQL
+        DELETE FROM spree_price_rules
+        WHERE id NOT IN (
+          SELECT MIN(id)
+          FROM spree_price_rules
+          GROUP BY price_list_id, type
+        )
+      SQL
+    end
+
+    remove_index :spree_price_rules, [:price_list_id, :type], if_exists: true
+
+    add_index :spree_price_rules, [:price_list_id, :type], unique: true,
+              name: 'index_spree_price_rules_on_price_list_id_and_type'
+  end
+
+  def down
+    remove_index :spree_price_rules, name: 'index_spree_price_rules_on_price_list_id_and_type', if_exists: true
+
+    add_index :spree_price_rules, [:price_list_id, :type]
+  end
+end

data/db/migrate/20260529000002_add_unique_index_to_spree_promotion_rules.rb

@@ -0,0 +1,37 @@
+class AddUniqueIndexToSpreePromotionRules < ActiveRecord::Migration[7.2]
+  def up
+    # Remove duplicate rules (same type within the same promotion),
+    # keeping the oldest record. Mirrors the
+    # `validates :type, uniqueness: { scope: :promotion_id }` model
+    # constraint at the DB level so concurrent writes can't race past
+    # the validation.
+    if ActiveRecord::Base.connection.adapter_name == 'Mysql2'
+      execute <<-SQL
+        DELETE FROM spree_promotion_rules
+        WHERE id NOT IN (
+          SELECT min_id FROM (
+            SELECT MIN(id) AS min_id
+            FROM spree_promotion_rules
+            GROUP BY promotion_id, type
+          ) AS keeper_ids
+        )
+      SQL
+    else
+      execute <<-SQL
+        DELETE FROM spree_promotion_rules
+        WHERE id NOT IN (
+          SELECT MIN(id)
+          FROM spree_promotion_rules
+          GROUP BY promotion_id, type
+        )
+      SQL
+    end
+
+    add_index :spree_promotion_rules, [:promotion_id, :type], unique: true,
+              name: 'index_spree_promotion_rules_on_promotion_id_and_type'
+  end
+
+  def down
+    remove_index :spree_promotion_rules, name: 'index_spree_promotion_rules_on_promotion_id_and_type', if_exists: true
+  end
+end

data/db/migrate/20260601000001_create_spree_product_publications.rb

@@ -0,0 +1,14 @@
+class CreateSpreeProductPublications < ActiveRecord::Migration[7.2]
+  def change
+    create_table :spree_product_publications do |t|
+      t.references :product, null: false
+      t.references :channel, null: false
+      t.datetime :published_at
+      t.datetime :unpublished_at
+      t.timestamps
+    end
+
+    add_index :spree_product_publications, %i[product_id channel_id], unique: true,
+              name: 'index_spree_product_publications_on_product_and_channel'
+  end
+end

data/db/migrate/20260601000002_add_store_id_to_spree_products.rb

@@ -0,0 +1,16 @@
+class AddStoreIdToSpreeProducts < ActiveRecord::Migration[7.2]
+  # NOTE: After running this migration, existing products will have
+  # +store_id IS NULL+ and be invisible to +Product.for_store+. Operators
+  # upgrading from 5.4 MUST run the backfill task immediately afterwards:
+  #
+  #   bundle exec rake spree:upgrade:populate_publications
+  #
+  # The backfill is also chained from +spree:channels:upgrade+ so the full
+  # 5.4 → 5.5 channel/publication upgrade is one command.
+  def change
+    add_reference :spree_products, :store, null: true
+    add_column :spree_products, :units_sold_count, :integer, default: 0, null: false
+    add_column :spree_products, :revenue, :decimal, precision: 16, scale: 4, default: 0, null: false
+    add_index :spree_products, %i[store_id units_sold_count]
+  end
+end

data/db/migrate/20260602000001_add_default_to_spree_channels.rb

@@ -0,0 +1,14 @@
+class AddDefaultToSpreeChannels < ActiveRecord::Migration[7.2]
+  def change
+    add_column :spree_channels, :default, :boolean, null: false, default: false
+
+    # Partial unique index: one default channel per store. Supported on
+    # Postgres and SQLite; MySQL ignores the +where:+ option, so we fall
+    # back to a model-level uniqueness validation there.
+    unless ActiveRecord::Base.connection.adapter_name.match?(/mysql/i)
+      add_index :spree_channels, :store_id, unique: true,
+                where: '"default" = TRUE',
+                name: 'index_spree_channels_default_per_store'
+    end
+  end
+end

data/db/sample_data/channels.rb

@@ -0,0 +1,12 @@
+# Adds two extra channels alongside the seeded 'Online Store' so sample data
+# exercises the channel-aware code paths (publishing, channel filters,
+# channel-scoped order attribution).
+store = Spree::Store.default
+
+store.channels.find_or_create_by!(code: 'pos') do |channel|
+  channel.name = 'Point of Sale'
+end
+
+store.channels.find_or_create_by!(code: 'wholesale') do |channel|
+  channel.name = 'Wholesale'
+end

data/db/sample_data/orders.rb

@@ -78,7 +78,7 @@ end
 
 orders.each(&:create_proposed_shipments)
 
-Spree::Order.where(id: orders.map(&:id)).update_all(state: :complete, completed_at: Time.current - 1.day)
+Spree::Order.where(id: orders.map(&:id)).update_all(state: :complete, status: 'placed', completed_at: Time.current - 1.day)
 
 # Adjustments (tax)
 tax_rate = Spree::TaxRate.find_by(name: 'California')