spree_core 4.2.5 → 4.3.0.rc1

data/config/routes.rb

@@ -1,6 +1 @@
-Spree::Core::Engine.add_routes do
-  get '/forbidden', to: 'errors#forbidden', as: :forbidden
-  get '/unauthorized', to: 'errors#unauthorized', as: :unauthorized
-end
-
 Spree::Core::Engine.draw_routes

data/db/migrate/20201023152810_add_filterable_to_spree_properties.rb

@@ -0,0 +1,8 @@
+class AddFilterableToSpreeProperties < ActiveRecord::Migration[5.2]
+  def change
+    unless column_exists?(:spree_properties, :filterable)
+      add_column :spree_properties, :filterable, :boolean, default: false, null: false
+      add_index :spree_properties, :filterable
+    end
+  end
+end

data/db/migrate/20210407200948_create_spree_menus.rb

@@ -0,0 +1,16 @@
+class CreateSpreeMenus < ActiveRecord::Migration[5.2]
+  def change
+    create_table :spree_menus do |t|
+      t.column :name, :string
+      t.column :location, :string
+      t.column :locale, :string
+
+      t.belongs_to :store
+
+      t.timestamps
+    end
+
+    add_index :spree_menus, :locale
+    add_index :spree_menus, [:store_id, :location, :locale], unique: true
+  end
+end

data/db/migrate/20210408092939_create_spree_menu_items.rb

@@ -0,0 +1,31 @@
+class CreateSpreeMenuItems < ActiveRecord::Migration[5.2]
+  def change
+    create_table :spree_menu_items do |t|
+      t.column :name, :string, null: false
+      t.column :subtitle, :string
+      t.column :destination, :string
+      t.column :new_window, :boolean, default: false
+      t.column :item_type, :string
+      t.column :linked_resource_type, :string, default: 'URL'
+      t.column :linked_resource_id, :integer
+      t.column :code, :string
+
+      t.column :parent_id, :integer
+      t.column :lft, :integer, null: false
+      t.column :rgt, :integer, null: false
+      t.column :depth, :integer, null: false, default: 0
+
+      t.belongs_to :menu
+
+      t.timestamps
+    end
+
+    add_index :spree_menu_items, :lft
+    add_index :spree_menu_items, :rgt
+    add_index :spree_menu_items, :depth
+    add_index :spree_menu_items, :parent_id
+    add_index :spree_menu_items, :item_type
+    add_index :spree_menu_items, :code
+    add_index :spree_menu_items, [:linked_resource_type, :linked_resource_id], name: 'index_spree_menu_items_on_linked_resource'
+  end
+end

data/db/migrate/20210504163720_add_filter_param_to_spree_product_properties.rb

@@ -0,0 +1,8 @@
+class AddFilterParamToSpreeProductProperties < ActiveRecord::Migration[5.2]
+  def change
+    unless column_exists?(:spree_product_properties, :filter_param)
+      add_column :spree_product_properties, :filter_param, :string
+      add_index :spree_product_properties, :filter_param
+    end
+  end
+end

data/db/migrate/20210505114659_add_filter_param_to_spree_properties.rb

@@ -0,0 +1,8 @@
+class AddFilterParamToSpreeProperties < ActiveRecord::Migration[5.2]
+  def change
+    unless column_exists?(:spree_properties, :filter_param)
+      add_column :spree_properties, :filter_param, :string
+      add_index :spree_properties, :filter_param
+    end
+  end
+end

data/db/migrate/20210512191732_create_spree_cms_pages.rb

@@ -0,0 +1,24 @@
+class CreateSpreeCmsPages < ActiveRecord::Migration[5.2]
+  def change
+    create_table :spree_cms_pages do |t|
+      t.column :title, :string, null: false
+      t.column :meta_title, :string
+      t.column :content, :text
+      t.column :meta_description, :text
+      t.column :visible, :boolean, default: true, index: true
+      t.column :slug, :string
+      t.column :type, :string
+      t.column :locale, :string
+      t.column :deleted_at, :datetime, index: true
+
+      t.belongs_to :store
+
+      t.timestamps
+    end
+
+    add_index :spree_cms_pages, [:slug, :store_id], unique: true
+    add_index :spree_cms_pages, [:slug, :store_id, :deleted_at]
+    add_index :spree_cms_pages, [:title, :type, :store_id]
+    add_index :spree_cms_pages, [:store_id, :locale, :type]
+  end
+end

data/db/migrate/20210514204251_create_spree_cms_sections.rb

@@ -0,0 +1,22 @@
+class CreateSpreeCmsSections < ActiveRecord::Migration[5.2]
+  def change
+    create_table :spree_cms_sections do |t|
+      t.column :name, :string, null: false
+      t.column :content, :text
+      t.column :settings, :text
+      t.column :fit, :string
+      t.column :destination, :string
+      t.column :type, :string
+      t.column :position, :integer
+
+      t.references :linked_resource, polymorphic: true, index: { name: 'index_spree_cms_sections_on_linked_resource' }
+
+      t.belongs_to :cms_page
+
+      t.timestamps
+    end
+
+    add_index :spree_cms_sections, :type
+    add_index :spree_cms_sections, :position
+  end
+end

data/db/migrate/20210527094055_create_spree_products_stores.rb

@@ -0,0 +1,29 @@
+class CreateSpreeProductsStores < ActiveRecord::Migration[5.2]
+  def up
+    unless table_exists?(:spree_products_stores)
+      create_table :spree_products_stores do |t|
+        t.references :product, index: true
+        t.references :store,  index: true
+        t.timestamps
+
+        t.index [:product_id, :store_id], unique: true
+      end
+
+      stores = Spree::Store.all
+      product_ids = Spree::Product.with_deleted.order(:id).ids
+
+      stores.find_each do |store|
+        prepared_values = product_ids.map { |id| "(#{id}, #{store.id}, '#{Time.current.to_s(:db)}', '#{Time.current.to_s(:db)}')" }.join(', ')
+        next if prepared_values.empty?
+
+        begin
+          execute "INSERT INTO spree_products_stores (product_id, store_id, created_at, updated_at) VALUES #{prepared_values};"
+        rescue ActiveRecord::RecordNotUnique; end
+      end
+    end
+  end
+
+  def down
+    drop_table :spree_products_stores
+  end
+end

data/db/migrate/20210608045519_ensure_store_default_country_is_set.rb

@@ -0,0 +1,5 @@
+class EnsureStoreDefaultCountryIsSet < ActiveRecord::Migration[5.2]
+  def change
+    Spree::Store.find_each(&:save)
+  end
+end

data/db/migrate/20210702112334_add_missing_timestamp_columns.rb

@@ -0,0 +1,46 @@
+class AddMissingTimestampColumns < ActiveRecord::Migration[5.2]
+  def change
+    # Add missing created_at columns
+    %i[
+      spree_countries
+      spree_option_type_prototypes
+      spree_option_value_variants
+      spree_order_promotions
+      spree_product_promotion_rules
+      spree_products_taxons
+      spree_promotion_action_line_items
+      spree_promotion_actions
+      spree_promotion_rule_taxons
+      spree_promotion_rule_users
+      spree_property_prototypes
+      spree_prototype_taxons
+      spree_reimbursement_credits
+      spree_role_users
+      spree_roles
+      spree_shipping_method_zones
+      spree_states
+    ].each do |table|
+      add_column table, :created_at, :datetime unless column_exists?(table, :created_at)
+    end
+    # Add missing updated_at columns
+    %i[
+      spree_option_type_prototypes
+      spree_option_value_variants
+      spree_order_promotions
+      spree_product_promotion_rules
+      spree_products_taxons
+      spree_promotion_action_line_items
+      spree_promotion_actions
+      spree_promotion_rule_taxons
+      spree_promotion_rule_users
+      spree_property_prototypes
+      spree_prototype_taxons
+      spree_reimbursement_credits
+      spree_role_users
+      spree_roles
+      spree_shipping_method_zones
+    ].each do |table|
+      add_column table, :updated_at, :datetime unless column_exists?(table, :updated_at)
+    end
+  end
+end

data/db/migrate/20210713131614_add_unique_index_on_property_id_and_product_id_to_product_properties.rb

@@ -0,0 +1,29 @@
+class AddUniqueIndexOnPropertyIdAndProductIdToProductProperties < ActiveRecord::Migration[5.2]
+  def up
+    klass   = Spree::ProductProperty
+    columns = %w[property_id product_id]
+
+    unless index_exists?(klass.table_name, columns)
+      scope   = klass.unscoped
+
+      say "Find duplicate #{klass} records"
+      duplicates = scope.
+        select((columns + %w[COUNT(*)]).join(',')).
+        group("#{columns.join(',')}").
+        having('COUNT(*) > 1').
+        map { |row| row.attributes.slice(*columns) }
+
+      say "Delete all but the oldest duplicate #{klass} record"
+      duplicates.each do |conditions|
+        scope.where(conditions).order(:created_at).drop(1).each(&:destroy)
+      end
+
+      say "Add unique index to #{klass.table_name} for #{columns.inspect}"
+      add_index klass.table_name, columns, unique: true
+    end
+  end
+
+  def down
+    remove_index :spree_product_properties, [:property_id, :product_id]
+  end
+end

data/db/migrate/20210715091956_add_store_id_to_spree_store_credits.rb

@@ -0,0 +1,10 @@
+class AddStoreIdToSpreeStoreCredits < ActiveRecord::Migration[5.2]
+  def change
+    unless column_exists?(:spree_store_credits, :store_id)
+      add_column :spree_store_credits, :store_id, :bigint
+      add_index :spree_store_credits, :store_id
+      Spree::StoreCredit.reset_column_information
+      Spree::StoreCredit.update_all(store_id: Spree::Store.default.id)
+    end
+  end
+end

data/db/migrate/20210716093151_add_store_id_to_spree_taxonomies.rb

@@ -0,0 +1,11 @@
+class AddStoreIdToSpreeTaxonomies < ActiveRecord::Migration[5.2]
+  def change
+    unless column_exists?(:spree_taxonomies, :store_id)
+      add_column :spree_taxonomies, :store_id, :bigint
+      add_index :spree_taxonomies, :store_id
+      add_index :spree_taxonomies, [:name, :store_id], unique: true
+      Spree::Taxonomy.reset_column_information
+      Spree::Taxonomy.update_all(store_id: Spree::Store.default.id)
+    end
+  end
+end

data/db/migrate/20210716104141_add_index_on_name_parent_id_and_taxonomy_id_on_spree_taxons.rb

@@ -0,0 +1,31 @@
+class AddIndexOnNameParentIdAndTaxonomyIdOnSpreeTaxons < ActiveRecord::Migration[5.2]
+  def up
+    klass   = Spree::Taxon
+    columns = %w[name parent_id taxonomy_id]
+
+    unless index_exists?(klass.table_name, columns)
+      remove_index klass.table_name, columns if index_exists?(klass.table_name, columns)
+
+      say "Find duplicate #{klass} records"
+      duplicates = klass.
+        select((columns + %w[COUNT(*)]).join(',')).
+        group("#{columns.join(',')}").
+        having('COUNT(*) > 1').
+        map { |row| row.attributes.slice(*columns) }
+
+      say "Delete all but the oldest duplicate #{klass} record"
+      duplicates.each do |conditions|
+        klass.where(conditions).order(:created_at).drop(1).each(&:destroy)
+      end
+
+      duplicates.each do |conditions| puts klass.where(conditions).order(:created_at).drop(1) end
+
+      say "Add unique index to #{klass.table_name} for #{columns.inspect}"
+      add_index klass.table_name, columns, unique: true
+    end
+  end
+
+  def down
+    remove_index :spree_taxons, [:name, :parent_id, :taxonomy_id]
+  end
+end

data/db/migrate/20210721120857_add_index_on_permalink_parent_id_and_taxonomy_id_on_spree_taxons.rb

@@ -0,0 +1,31 @@
+class AddIndexOnPermalinkParentIdAndTaxonomyIdOnSpreeTaxons < ActiveRecord::Migration[5.2]
+  def up
+    klass   = Spree::Taxon
+    columns = %w[permalink parent_id taxonomy_id]
+
+    unless index_exists?(klass.table_name, columns)
+      remove_index klass.table_name, columns if index_exists?(klass.table_name, columns)
+
+      say "Find duplicate #{klass} records"
+      duplicates = klass.
+        select((columns + %w[COUNT(*)]).join(',')).
+        group("#{columns.join(',')}").
+        having('COUNT(*) > 1').
+        map { |row| row.attributes.slice(*columns) }
+
+      say "Delete all but the oldest duplicate #{klass} record"
+      duplicates.each do |conditions|
+        klass.where(conditions).order(:created_at).drop(1).each(&:destroy)
+      end
+
+      duplicates.each do |conditions| puts klass.where(conditions).order(:created_at).drop(1) end
+
+      say "Add unique index to #{klass.table_name} for #{columns.inspect}"
+      add_index klass.table_name, columns, unique: true
+    end
+  end
+
+  def down
+    remove_index :spree_taxons, [:permalink, :parent_id, :taxonomy_id]
+  end
+end

data/db/migrate/20210721125657_create_spree_promotions_stores.rb

@@ -0,0 +1,29 @@
+class CreateSpreePromotionsStores < ActiveRecord::Migration[5.2]
+  def up
+    unless table_exists?(:spree_promotions_stores)
+      create_table :spree_promotions_stores do |t|
+        t.references :promotion, index: true
+        t.references :store,  index: true
+        t.timestamps
+
+        t.index [:promotion_id, :store_id], unique: true
+      end
+
+      stores = Spree::Store.all
+      promotion_ids = Spree::Promotion.order(:id).ids
+
+      stores.find_each do |store|
+        prepared_values = promotion_ids.map { |id| "(#{id}, #{store.id}, '#{Time.current.to_s(:db)}', '#{Time.current.to_s(:db)}')" }.join(', ')
+        next if prepared_values.empty?
+
+        begin
+          execute "INSERT INTO spree_promotions_stores (promotion_id, store_id, created_at, updated_at) VALUES #{prepared_values};"
+        rescue ActiveRecord::RecordNotUnique; end
+      end
+    end
+  end
+
+  def down
+    drop_table :spree_promotions_stores
+  end
+end

data/db/migrate/20210722090705_add_store_id_to_spree_customer_returns.rb

@@ -0,0 +1,11 @@
+class AddStoreIdToSpreeCustomerReturns < ActiveRecord::Migration[5.2]
+  def change
+    unless column_exists?(:spree_customer_returns, :store_id)
+      add_column :spree_customer_returns, :store_id, :bigint
+      add_index :spree_customer_returns, :store_id
+      Spree::CustomerReturn.reset_column_information
+      default_store_id = Spree::Store.default.id
+      Spree::CustomerReturn.find_each { |cr| cr.update_column(:store_id, cr.order&.store_id || default_store_id) }
+    end
+  end
+end

data/db/migrate/20210726065456_change_integer_id_columns_into_bigint.rb

@@ -0,0 +1,305 @@
+class ChangeIntegerIdColumnsIntoBigint < ActiveRecord::Migration[5.2]
+  def change
+    change_table(:active_storage_attachments) do |t|
+      t.change :record_id, :bigint
+      t.change :blob_id, :bigint
+    end
+
+    change_column :friendly_id_slugs, :sluggable_id, :bigint
+
+    change_table(:spree_addresses) do |t|
+      t.change :state_id, :bigint
+      t.change :country_id, :bigint
+      t.change :user_id, :bigint
+    end
+
+    change_table(:spree_adjustments) do |t|
+      t.change :source_id, :bigint
+      t.change :adjustable_id, :bigint
+      t.change :order_id, :bigint
+    end
+
+    change_column :spree_assets, :viewable_id, :bigint
+
+    change_column :spree_calculators, :calculable_id, :bigint
+
+    change_table(:spree_credit_cards) do |t|
+      t.change :address_id, :bigint
+      t.change :user_id, :bigint
+      t.change :payment_method_id, :bigint
+    end
+
+    change_column :spree_customer_returns, :stock_location_id, :bigint
+
+    change_table(:spree_inventory_units) do |t|
+      t.change :variant_id, :bigint
+      t.change :order_id, :bigint
+      t.change :shipment_id, :bigint
+      t.change :line_item_id, :bigint
+      t.change :original_return_item_id, :bigint
+    end
+
+    change_table(:spree_line_items) do |t|
+      t.change :variant_id, :bigint
+      t.change :order_id, :bigint
+      t.change :tax_category_id, :bigint
+    end
+
+    change_column :spree_log_entries, :source_id, :bigint
+
+    change_table(:spree_menu_items) do |t|
+      t.change :linked_resource_id, :bigint
+      t.change :parent_id, :bigint
+      t.change :lft, :bigint
+      t.change :rgt, :bigint
+      t.change :menu_id, :bigint
+    end
+
+    change_column :spree_menus, :store_id, :bigint
+
+    change_table(:spree_option_type_prototypes) do |t|
+      t.change :prototype_id, :bigint
+      t.change :option_type_id, :bigint
+    end
+
+    change_table(:spree_option_value_variants) do |t|
+      t.change :variant_id, :bigint
+      t.change :option_value_id, :bigint
+    end
+
+    change_column :spree_option_values, :option_type_id, :bigint
+
+    change_table(:spree_order_promotions) do |t|
+      t.change :order_id, :bigint
+      t.change :promotion_id, :bigint
+    end
+
+    change_table(:spree_orders) do |t|
+      t.change :user_id, :bigint
+      t.change :bill_address_id, :bigint
+      t.change :ship_address_id, :bigint
+      t.change :created_by_id, :bigint
+      t.change :approver_id, :bigint
+      t.change :canceler_id, :bigint
+      t.change :store_id, :bigint
+    end
+
+    change_column :spree_payment_capture_events, :payment_id, :bigint
+
+    change_table(:spree_payment_methods_stores) do |t|
+      t.change :payment_method_id, :bigint
+      t.change :store_id, :bigint
+    end
+
+    change_table(:spree_payments) do |t|
+      t.change :order_id, :bigint
+      t.change :source_id, :bigint
+      t.change :payment_method_id, :bigint
+    end
+
+    change_column :spree_prices, :variant_id, :bigint
+
+    change_table(:spree_product_option_types) do |t|
+      t.change :product_id, :bigint
+      t.change :option_type_id, :bigint
+    end
+
+    change_table(:spree_product_promotion_rules) do |t|
+      t.change :product_id, :bigint
+      t.change :promotion_rule_id, :bigint
+    end
+
+    change_table(:spree_product_properties) do |t|
+      t.change :product_id, :bigint
+      t.change :property_id, :bigint
+    end
+
+    change_table(:spree_products) do |t|
+      t.change :tax_category_id, :bigint
+      t.change :shipping_category_id, :bigint
+    end
+
+    change_table(:spree_products_stores) do |t|
+      t.change :product_id, :bigint
+      t.change :store_id, :bigint
+    end
+
+    change_table(:spree_products_taxons) do |t|
+      t.change :product_id, :bigint
+      t.change :taxon_id, :bigint
+    end
+
+    change_table(:spree_promotion_action_line_items) do |t|
+      t.change :promotion_action_id, :bigint
+      t.change :variant_id, :bigint
+    end
+
+    change_column :spree_promotion_actions, :promotion_id, :bigint
+
+    change_table(:spree_promotion_rule_taxons) do |t|
+      t.change :taxon_id, :bigint
+      t.change :promotion_rule_id, :bigint
+    end
+
+    change_table(:spree_promotion_rule_users) do |t|
+      t.change :user_id, :bigint
+      t.change :promotion_rule_id, :bigint
+    end
+
+    change_table(:spree_promotion_rules) do |t|
+      t.change :promotion_id, :bigint
+      t.change :user_id, :bigint
+      t.change :product_group_id, :bigint
+    end
+
+    change_column :spree_promotions, :promotion_category_id, :bigint
+
+    change_table(:spree_promotions_stores) do |t|
+      t.change :promotion_id, :bigint
+      t.change :store_id, :bigint
+    end
+
+    change_table(:spree_property_prototypes) do |t|
+      t.change :prototype_id, :bigint
+      t.change :property_id, :bigint
+    end
+
+    change_table(:spree_prototype_taxons) do |t|
+      t.change :taxon_id, :bigint
+      t.change :prototype_id, :bigint
+    end
+
+    change_table(:spree_refunds) do |t|
+      t.change :payment_id, :bigint
+      t.change :refund_reason_id, :bigint
+      t.change :reimbursement_id, :bigint
+    end
+
+    change_table(:spree_reimbursement_credits) do |t|
+      t.change :reimbursement_id, :bigint
+      t.change :creditable_id, :bigint
+    end
+
+    change_table(:spree_reimbursements) do |t|
+      t.change :customer_return_id, :bigint
+      t.change :order_id, :bigint
+    end
+
+    change_table(:spree_return_authorizations) do |t|
+      t.change :order_id, :bigint
+      t.change :stock_location_id, :bigint
+      t.change :return_authorization_reason_id, :bigint
+    end
+
+    change_table(:spree_return_items) do |t|
+      t.change :return_authorization_id, :bigint
+      t.change :inventory_unit_id, :bigint
+      t.change :exchange_variant_id, :bigint
+      t.change :customer_return_id, :bigint
+      t.change :reimbursement_id, :bigint
+      t.change :preferred_reimbursement_type_id, :bigint
+      t.change :override_reimbursement_type_id, :bigint
+    end
+
+    change_table(:spree_role_users) do |t|
+      t.change :role_id, :bigint
+      t.change :user_id, :bigint
+    end
+
+    change_table(:spree_shipments) do |t|
+      t.change :order_id, :bigint
+      t.change :address_id, :bigint
+      t.change :stock_location_id, :bigint
+    end
+
+    change_table(:spree_shipping_method_categories) do |t|
+      t.change :shipping_method_id, :bigint
+      t.change :shipping_category_id, :bigint
+    end
+
+    change_table(:spree_shipping_method_zones) do |t|
+      t.change :shipping_method_id, :bigint
+      t.change :zone_id, :bigint
+    end
+
+    change_column :spree_shipping_methods, :tax_category_id, :bigint
+
+    change_table(:spree_shipping_rates) do |t|
+      t.change :shipment_id, :bigint
+      t.change :shipping_method_id, :bigint
+      t.change :tax_rate_id, :bigint
+    end
+
+    change_table(:spree_state_changes) do |t|
+      t.change :stateful_id, :bigint
+      t.change :user_id, :bigint
+    end
+
+    change_column :spree_states, :country_id, :bigint
+
+    change_table(:spree_stock_items) do |t|
+      t.change :stock_location_id, :bigint
+      t.change :variant_id, :bigint
+    end
+
+    change_table(:spree_stock_locations) do |t|
+      t.change :state_id, :bigint
+      t.change :country_id, :bigint
+    end
+
+    change_table(:spree_stock_movements) do |t|
+      t.change :stock_item_id, :bigint
+      t.change :originator_id, :bigint
+    end
+
+    change_table(:spree_stock_transfers) do |t|
+      t.change :source_location_id, :bigint
+      t.change :destination_location_id, :bigint
+    end
+
+    change_table(:spree_store_credit_events) do |t|
+      t.change :store_credit_id, :bigint
+      t.change :originator_id, :bigint
+    end
+
+    change_table(:spree_store_credits) do |t|
+      t.change :user_id, :bigint
+      t.change :category_id, :bigint
+      t.change :created_by_id, :bigint
+      t.change :originator_id, :bigint
+      t.change :type_id, :bigint
+    end
+
+    change_table(:spree_stores) do |t|
+      t.change :default_country_id, :bigint
+      t.change :checkout_zone_id, :bigint
+    end
+
+    change_table(:spree_tax_rates) do |t|
+      t.change :zone_id, :bigint
+      t.change :tax_category_id, :bigint
+    end
+
+    change_table(:spree_taxons) do |t|
+      t.change :parent_id, :bigint
+      t.change :taxonomy_id, :bigint
+      t.change :lft, :bigint
+      t.change :rgt, :bigint
+    end
+
+    change_table(:spree_users) do |t|
+      t.change :ship_address_id, :bigint
+      t.change :bill_address_id, :bigint
+    end
+
+    change_table(:spree_variants) do |t|
+      t.change :product_id, :bigint
+      t.change :tax_category_id, :bigint
+    end
+
+    change_table(:spree_zone_members) do |t|
+      t.change :zoneable_id, :bigint
+      t.change :zone_id, :bigint
+    end
+  end
+end