spree_core 4.0.0 → 4.6.0

data/app/models/spree/store.rb

@@ -1,30 +1,214 @@
 module Spree
   class Store < Spree::Base
+    include TranslatableResource
+    if defined?(Spree::Webhooks)
+      include Spree::Webhooks::HasWebhooks
+    end
+    if defined?(Spree::Security::Stores)
+      include Spree::Security::Stores
+    end
+
+    TRANSLATABLE_FIELDS = %i[name meta_description meta_keywords seo_title facebook
+                             twitter instagram customer_support_email description
+                             address contact_phone new_order_notifications_email].freeze
+    translates(*TRANSLATABLE_FIELDS)
+
+    self::Translation.class_eval do
+      acts_as_paranoid
+      # deleted translation values still need to be accessible - remove deleted_at scope
+      default_scope { unscope(where: :deleted_at) }
+    end
+
+    typed_store :settings, coder: ActiveRecord::TypedStore::IdentityCoder do |s|
+      # Spree Digital Asset Configurations
+      s.boolean :limit_digital_download_count, default: true, null: false
+      s.boolean :limit_digital_download_days, default: true, null: false
+      s.integer :digital_asset_authorized_clicks, default: 5, null: false # number of times a customer can download a digital file.
+      s.integer :digital_asset_authorized_days, default: 7, null: false # number of days after initial purchase the customer can download a file.
+      s.integer :digital_asset_link_expire_time, default: 300, null: false # 5 minutes in seconds
+    end
+
+    attr_accessor :skip_validate_not_last
+
+    acts_as_paranoid
+
     has_many :orders, class_name: 'Spree::Order'
+    has_many :line_items, through: :orders, class_name: 'Spree::LineItem'
+    has_many :shipments, through: :orders, class_name: 'Spree::Shipment'
+    has_many :payments, through: :orders, class_name: 'Spree::Payment'
+    has_many :return_authorizations, through: :orders, class_name: 'Spree::ReturnAuthorization'
+    # has_many :reimbursements, through: :orders, class_name: 'Spree::Reimbursement' FIXME: we should fetch this via Customer Return
+
+    has_many :store_payment_methods, class_name: 'Spree::StorePaymentMethod'
+    has_many :payment_methods, through: :store_payment_methods, class_name: 'Spree::PaymentMethod'
+
+    has_many :cms_pages, class_name: 'Spree::CmsPage'
+    has_many :cms_sections, through: :cms_pages, class_name: 'Spree::CmsSection'
+
+    has_many :menus, class_name: 'Spree::Menu'
+    has_many :menu_items, through: :menus, class_name: 'Spree::MenuItem'
+
+    has_many :store_products, class_name: 'Spree::StoreProduct'
+    has_many :products, through: :store_products, class_name: 'Spree::Product'
+    has_many :product_properties, through: :products, class_name: 'Spree::ProductProperty'
+    has_many :variants, through: :products, class_name: 'Spree::Variant', source: :variants_including_master
+    has_many :stock_items, through: :variants, class_name: 'Spree::StockItem'
+    has_many :inventory_units, through: :variants, class_name: 'Spree::InventoryUnit'
+    has_many :customer_returns, class_name: 'Spree::CustomerReturn', inverse_of: :store
+
+    has_many :store_credits, class_name: 'Spree::StoreCredit'
+    has_many :store_credit_events, through: :store_credits, class_name: 'Spree::StoreCreditEvent'
+
+    has_many :taxonomies, class_name: 'Spree::Taxonomy'
+    has_many :taxons, through: :taxonomies, class_name: 'Spree::Taxon'
+
+    has_many :store_promotions, class_name: 'Spree::StorePromotion'
+    has_many :promotions, through: :store_promotions, class_name: 'Spree::Promotion'
+
+    has_many :wishlists, class_name: 'Spree::Wishlist'
+
+    has_many :data_feeds, class_name: 'Spree::DataFeed'
+
+    belongs_to :default_country, class_name: 'Spree::Country'
+    belongs_to :checkout_zone, class_name: 'Spree::Zone'
 
     with_options presence: true do
-      validates :code, uniqueness: { case_sensitive: false, allow_blank: true }
-      validates :name, :url, :mail_from_address
+      validates :name, :url, :mail_from_address, :default_currency, :code
     end
 
+    validates :digital_asset_authorized_clicks, numericality: { only_integer: true, greater_than: 0 }
+    validates :digital_asset_authorized_days, numericality: { only_integer: true, greater_than: 0 }
+    validates :code, uniqueness: { case_sensitive: false, conditions: -> { with_deleted } }
+    validates :mail_from_address, email: { allow_blank: false }
+
+    # FIXME: we should remove this condition in v5
+    if !ENV['SPREE_DISABLE_DB_CONNECTION'] &&
+        connected? &&
+        table_exists? &&
+        connection.column_exists?(:spree_stores, :new_order_notifications_email)
+      validates :new_order_notifications_email, email: { allow_blank: true }
+    end
+
+    default_scope { order(created_at: :asc) }
+
+    has_one :logo, class_name: 'Spree::StoreLogo', dependent: :destroy, as: :viewable
+    accepts_nested_attributes_for :logo, reject_if: :all_blank
+
+    has_one :mailer_logo, class_name: 'Spree::StoreMailerLogo', dependent: :destroy, as: :viewable
+    accepts_nested_attributes_for :mailer_logo, reject_if: :all_blank
+
+    has_one :favicon_image, class_name: 'Spree::StoreFaviconImage', dependent: :destroy, as: :viewable
+    accepts_nested_attributes_for :favicon_image, reject_if: :all_blank
+
     before_save :ensure_default_exists_and_is_unique
-    before_destroy :validate_not_default
+    before_save :ensure_supported_currencies, :ensure_supported_locales, :ensure_default_country
+    before_destroy :validate_not_last, unless: :skip_validate_not_last
+    before_destroy :pass_default_flag_to_other_store
 
     scope :by_url, ->(url) { where('url like ?', "%#{url}%") }
 
     after_commit :clear_cache
 
-    def self.current(domain = nil)
-      current_store = domain ? Store.by_url(domain).first : nil
-      current_store || Store.default
+    alias_attribute :contact_email, :customer_support_email
+
+    def self.current(url = nil)
+      ActiveSupport::Deprecation.warn(<<-DEPRECATION, caller)
+        `Spree::Store.current` is deprecated and will be removed in Spree 5.0
+        Please use `Spree::Stores::FindCurrent.new(url: "https://example.com").execute` instead
+      DEPRECATION
+      Stores::FindCurrent.new(url: url).execute
     end
 
+    # FIXME: we need to drop `or_initialize` in v5
+    # this behaviour is very buggy and unpredictable
     def self.default
       Rails.cache.fetch('default_store') do
-        where(default: true).first_or_initialize
+        # workaround for Mobility bug with first_or_initialize
+        if where(default: true).any?
+          where(default: true).first
+        else
+          new(default: true)
+        end
       end
     end
 
+    def self.available_locales
+      Rails.cache.fetch('stores_available_locales') do
+        Spree::Store.all.map(&:supported_locales_list).flatten.uniq
+      end
+    end
+
+    def default_menu(location)
+      menu = menus.find_by(location: location, locale: default_locale) || menus.find_by(location: location)
+
+      menu.root if menu.present?
+    end
+
+    def supported_currencies_list
+      @supported_currencies_list ||= (read_attribute(:supported_currencies).to_s.split(',') << default_currency).sort.map(&:to_s).map do |code|
+        ::Money::Currency.find(code.strip)
+      end.uniq.compact
+    end
+
+    def homepage(requested_locale)
+      cms_pages.by_locale(requested_locale).find_by(type: 'Spree::Cms::Pages::Homepage') ||
+        cms_pages.by_locale(default_locale).find_by(type: 'Spree::Cms::Pages::Homepage') ||
+        cms_pages.find_by(type: 'Spree::Cms::Pages::Homepage')
+    end
+
+    def seo_meta_description
+      if meta_description.present?
+        meta_description
+      elsif seo_title.present?
+        seo_title
+      else
+        name
+      end
+    end
+
+    def supported_locales_list
+      # TODO: add support of multiple supported languages to a single Store
+      @supported_locales_list ||= (read_attribute(:supported_locales).to_s.split(',') << default_locale).compact.uniq.sort
+    end
+
+    def unique_name
+      @unique_name ||= "#{name} (#{code})"
+    end
+
+    def formatted_url
+      return if url.blank?
+
+      @formatted_url ||= if url.match(/http:\/\/|https:\/\//)
+                           url
+                         else
+                           Rails.env.development? || Rails.env.test? ? "http://#{url}" : "https://#{url}"
+                         end
+    end
+
+    def countries_available_for_checkout
+      @countries_available_for_checkout ||= checkout_zone.try(:country_list) || Spree::Country.all
+    end
+
+    def states_available_for_checkout(country)
+      checkout_zone.try(:state_list_for, country) || country.states
+    end
+
+    def checkout_zone_or_default
+      ActiveSupport::Deprecation.warn('Store#checkout_zone_or_default is deprecated and will be removed in Spree 5')
+
+      @checkout_zone_or_default ||= checkout_zone || Spree::Zone.default_checkout_zone
+    end
+
+    def favicon
+      return unless favicon_image&.attachment&.attached?
+
+      favicon_image.attachment.variant(resize_to_limit: [32, 32])
+    end
+
+    def can_be_deleted?
+      self.class.where.not(id: id).any?
+    end
+
     private
 
     def ensure_default_exists_and_is_unique
@@ -35,15 +219,50 @@ module Spree
       end
     end
 
-    def validate_not_default
-      if default
-        errors.add(:base, :cannot_destroy_default_store)
+    def ensure_supported_locales
+      return unless attributes.keys.include?('supported_locales')
+      return if supported_locales.present?
+      return if default_locale.blank?
+
+      self.supported_locales = default_locale
+    end
+
+    def ensure_supported_currencies
+      return unless attributes.keys.include?('supported_currencies')
+      return if supported_currencies.present?
+      return if default_currency.blank?
+
+      self.supported_currencies = default_currency
+    end
+
+    def validate_not_last
+      unless can_be_deleted?
+        errors.add(:base, :cannot_destroy_only_store)
         throw(:abort)
       end
     end
 
+    def pass_default_flag_to_other_store
+      if default? && can_be_deleted?
+        self.class.where.not(id: id).first.update!(default: true)
+        self.default = false
+      end
+    end
+
     def clear_cache
       Rails.cache.delete('default_store')
+      Rails.cache.delete('stores_available_locales')
+    end
+
+    def ensure_default_country
+      return unless has_attribute?(:default_country_id)
+      return if default_country.present? && (checkout_zone.blank? || checkout_zone.country_list.blank? || checkout_zone.country_list.include?(default_country))
+
+      self.default_country = if checkout_zone.present? && checkout_zone.country_list.any?
+                               checkout_zone.country_list.first
+                             else
+                               Country.find_by(iso: 'US') || Country.first
+                             end
     end
   end
 end

data/app/models/spree/store_credit.rb

@@ -1,5 +1,11 @@
 module Spree
   class StoreCredit < Spree::Base
+    include SingleStoreResource
+    include Metadata
+    if defined?(Spree::Webhooks)
+      include Spree::Webhooks::HasWebhooks
+    end
+
     acts_as_paranoid
 
     VOID_ACTION       = 'void'.freeze
@@ -12,13 +18,14 @@ module Spree
 
     DEFAULT_CREATED_BY_EMAIL = 'spree@example.com'.freeze
 
-    belongs_to :user, class_name: Spree.user_class.to_s, foreign_key: 'user_id'
+    belongs_to :user, class_name: "::#{Spree.user_class}", foreign_key: 'user_id'
     belongs_to :category, class_name: 'Spree::StoreCreditCategory'
-    belongs_to :created_by, class_name: Spree.user_class.to_s, foreign_key: 'created_by_id'
+    belongs_to :created_by, class_name: Spree.admin_user_class.to_s, foreign_key: 'created_by_id'
     belongs_to :credit_type, class_name: 'Spree::StoreCreditType', foreign_key: 'type_id'
-    has_many :store_credit_events
+    belongs_to :store, class_name: 'Spree::Store'
+    has_many :store_credit_events, class_name: 'Spree::StoreCreditEvent'
 
-    validates :user, :category, :credit_type, :created_by, :currency, presence: true
+    validates :user, :category, :credit_type, :created_by, :currency, :store, presence: true
     validates :amount, numericality: { greater_than: 0 }
     validates :amount_used, numericality: { greater_than_or_equal_to: 0 }
     validate :amount_used_less_than_or_equal_to_amount
@@ -38,6 +45,9 @@ module Spree
     extend Spree::DisplayMoney
     money_methods :amount, :amount_used
 
+    self.whitelisted_ransackable_attributes = %w[user_id created_by_id amount currency type_id]
+    self.whitelisted_ransackable_associations = %w[type user created_by]
+
     def amount_remaining
       amount - amount_used - amount_authorized
     end
@@ -68,7 +78,7 @@ module Spree
     end
 
     def validate_authorization(amount, order_currency)
-      if amount_remaining.to_d < amount.to_d
+      if BigDecimal(amount_remaining, 3) < BigDecimal(amount, 3)
         errors.add(:base, Spree.t('store_credit_payment_method.insufficient_funds'))
       elsif currency != order_currency
         errors.add(:base, Spree.t('store_credit_payment_method.currency_mismatch'))
@@ -188,7 +198,8 @@ module Spree
         created_by_id: created_by_id,
         currency: currency,
         type_id: type_id,
-        memo: credit_allocation_memo
+        memo: credit_allocation_memo,
+        store: store
       }
     end
 
@@ -242,7 +253,7 @@ module Spree
     def associate_credit_type
       unless type_id
         credit_type_name = category.try(:non_expiring?) ? 'Non-expiring' : 'Expiring'
-        self.credit_type = Spree::StoreCreditType.find_by(name: credit_type_name)
+        self.credit_type = Spree::StoreCreditType.find_or_create_by(name: credit_type_name)
       end
     end
   end

data/app/models/spree/store_credit_category.rb

@@ -7,6 +7,8 @@ module Spree
     GIFT_CARD_CATEGORY_NAME = 'Gift Card'.freeze
     DEFAULT_NON_EXPIRING_TYPES = [GIFT_CARD_CATEGORY_NAME]
 
+    self.whitelisted_ransackable_attributes = %w[name]
+
     def non_expiring?
       non_expiring_category_types.include? name
     end

data/app/models/spree/store_credit_event.rb

@@ -8,15 +8,10 @@ module Spree
     scope :exposed_events, -> { where.not(action: [Spree::StoreCredit::ELIGIBLE_ACTION, Spree::StoreCredit::AUTHORIZE_ACTION]) }
     scope :reverse_chronological, -> { order(created_at: :desc) }
 
-    delegate :currency, to: :store_credit
+    delegate :currency, :store, to: :store_credit
 
-    def display_amount
-      Spree::Money.new(amount, currency: currency)
-    end
-
-    def display_user_total_amount
-      Spree::Money.new(user_total_amount, currency: currency)
-    end
+    extend DisplayMoney
+    money_methods :amount, :user_total_amount
 
     def display_action
       case action
@@ -32,7 +27,7 @@ module Spree
     end
 
     def order
-      Spree::Payment.find_by(response_code: authorization_code).try(:order)
+      store.payments.find_by(response_code: authorization_code).try(:order)
     end
   end
 end

data/app/models/spree/store_credit_type.rb

@@ -2,5 +2,7 @@ module Spree
   class StoreCreditType < Spree::Base
     DEFAULT_TYPE_NAME = 'Expiring'.freeze
     has_many :store_credits, class_name: 'Spree::StoreCredit', foreign_key: 'type_id'
+
+    validates :name, presence: true
   end
 end

data/app/models/spree/store_favicon_image.rb

@@ -0,0 +1,17 @@
+module Spree
+  class StoreFaviconImage < Asset
+    if Spree.public_storage_service_name
+      has_one_attached :attachment, service: Spree.public_storage_service_name
+    else
+      has_one_attached :attachment
+    end
+
+    VALID_CONTENT_TYPES = ['image/png', 'image/x-icon', 'image/vnd.microsoft.icon'].freeze
+
+    validates :attachment,
+              content_type: VALID_CONTENT_TYPES,
+              dimension: { max: 256..256 },
+              aspect_ratio: :square,
+              size: { less_than_or_equal_to: 1.megabyte }
+  end
+end

data/app/models/spree/store_logo.rb

@@ -0,0 +1,9 @@
+module Spree
+  class StoreLogo < Asset
+    if Spree.public_storage_service_name
+      has_one_attached :attachment, service: Spree.public_storage_service_name
+    else
+      has_one_attached :attachment
+    end
+  end
+end

data/app/models/spree/store_mailer_logo.rb

@@ -0,0 +1,13 @@
+module Spree
+  class StoreMailerLogo < Asset
+    if Spree.public_storage_service_name
+      has_one_attached :attachment, service: Spree.public_storage_service_name
+    else
+      has_one_attached :attachment
+    end
+
+    VALID_CONTENT_TYPES = ['image/png', 'image/jpg', 'image/jpeg'].freeze
+
+    validates :attachment, content_type: VALID_CONTENT_TYPES
+  end
+end

data/app/models/spree/store_payment_method.rb

@@ -0,0 +1,11 @@
+module Spree
+  class StorePaymentMethod < Spree::Base
+    self.table_name = 'spree_payment_methods_stores'
+
+    belongs_to :store, class_name: 'Spree::Store', touch: true
+    belongs_to :payment_method, class_name: 'Spree::PaymentMethod', touch: true
+
+    validates :store, :payment_method, presence: true
+    validates :store_id, uniqueness: { scope: :payment_method_id }
+  end
+end

data/app/models/spree/store_product.rb

@@ -0,0 +1,11 @@
+module Spree
+  class StoreProduct < Spree::Base
+    self.table_name = 'spree_products_stores'
+
+    belongs_to :store, class_name: 'Spree::Store', touch: true
+    belongs_to :product, class_name: 'Spree::Product', touch: true
+
+    validates :store, :product, presence: true
+    validates :store_id, uniqueness: { scope: :product_id }
+  end
+end

data/app/models/spree/store_promotion.rb

@@ -0,0 +1,11 @@
+module Spree
+  class StorePromotion < Spree::Base
+    self.table_name = 'spree_promotions_stores'
+
+    belongs_to :store, class_name: 'Spree::Store', touch: true
+    belongs_to :promotion, class_name: 'Spree::Promotion', touch: true
+
+    validates :store, :promotion, presence: true
+    validates :store_id, uniqueness: { scope: :promotion_id }
+  end
+end

data/app/models/spree/tax_category.rb

@@ -1,12 +1,20 @@
 module Spree
   class TaxCategory < Spree::Base
+    if defined?(Spree::Webhooks)
+      include Spree::Webhooks::HasWebhooks
+    end
+
     acts_as_paranoid
-    validates :name, presence: true, uniqueness: { case_sensitive: false, scope: :deleted_at }
+    validates :name, presence: true, uniqueness: { case_sensitive: false, scope: spree_base_uniqueness_scope.push(:deleted_at) }
 
     has_many :tax_rates, dependent: :destroy, inverse_of: :tax_category
+    has_many :products, dependent: :nullify
+    has_many :variants, dependent: :nullify
 
     before_save :set_default_category
 
+    self.whitelisted_ransackable_attributes = %w[name is_default tax_code]
+
     def set_default_category
       # set existing default tax category to false if this one has been marked as default
 

data/app/models/spree/tax_rate.rb

@@ -4,6 +4,10 @@ module Spree
 
     include Spree::CalculatedAdjustments
     include Spree::AdjustmentSource
+    include Spree::Metadata
+    if defined?(Spree::Webhooks)
+      include Spree::Webhooks::HasWebhooks
+    end
 
     with_options inverse_of: :tax_rates do
       belongs_to :zone, class_name: 'Spree::Zone', optional: true
@@ -13,7 +17,7 @@ module Spree
 
     with_options presence: true do
       validates :amount, numericality: { allow_nil: true }
-      validates :tax_category
+      validates :tax_category, :name
     end
 
     scope :by_zone, ->(zone) { where(zone_id: zone.id) }
@@ -27,6 +31,8 @@ module Spree
           ->(category) { where(tax_category_id: category.try(:id)) }
     scope :included_in_price, -> { where(included_in_price: true) }
 
+    self.whitelisted_ransackable_attributes = %w[amount zone_id tax_category_id included_in_price name]
+
     # Gets the array of TaxRates appropriate for the specified tax zone
     def self.match(order_tax_zone)
       return [] unless order_tax_zone
@@ -112,7 +118,8 @@ module Spree
 
       ' ' + ActiveSupport::NumberHelper::NumberToPercentageConverter.convert(
         amount * 100,
-        locale: I18n.locale
+        locale: I18n.locale,
+        strip_insignificant_zeros: true
       )
     end
   end

data/app/models/spree/taxon.rb

@@ -3,6 +3,13 @@ require 'stringex'
 
 module Spree
   class Taxon < Spree::Base
+    include TranslatableResource
+    include TranslatableResourceSlug
+    include Metadata
+    if defined?(Spree::Webhooks)
+      include Spree::Webhooks::HasWebhooks
+    end
+
     extend FriendlyId
     friendly_id :permalink, slug_column: :permalink, use: :history
     before_validation :set_permalink, on: :create, if: :name
@@ -13,28 +20,59 @@ module Spree
     has_many :classifications, -> { order(:position) }, dependent: :delete_all, inverse_of: :taxon
     has_many :products, through: :classifications
 
+    has_many :menu_items, as: :linked_resource
+    has_many :cms_sections, as: :linked_resource
+
     has_many :prototype_taxons, class_name: 'Spree::PrototypeTaxon', dependent: :destroy
     has_many :prototypes, through: :prototype_taxons, class_name: 'Spree::Prototype'
 
     has_many :promotion_rule_taxons, class_name: 'Spree::PromotionRuleTaxon', dependent: :destroy
     has_many :promotion_rules, through: :promotion_rule_taxons, class_name: 'Spree::PromotionRule'
 
-    validates :name, presence: true, uniqueness: { scope: [:parent_id, :taxonomy_id], allow_blank: true, case_sensitive: false }
-    validates :permalink, uniqueness: { case_sensitive: false }
+    validates :name, presence: true, uniqueness: { scope: [:parent_id, :taxonomy_id], case_sensitive: false }
+    validates :taxonomy, presence: true
+    validates :permalink, uniqueness: { case_sensitive: false, scope: [:parent_id, :taxonomy_id] }
+    validates :hide_from_nav, inclusion: { in: [true, false] }
     validates_associated :icon
     validate :check_for_root, on: :create
+    validate :parent_belongs_to_same_taxonomy
     with_options length: { maximum: 255 }, allow_blank: true do
       validates :meta_keywords
       validates :meta_description
       validates :meta_title
     end
 
+    before_validation :copy_taxonomy_from_parent
     after_save :touch_ancestors_and_taxonomy
+    after_update :sync_taxonomy_name
     after_touch :touch_ancestors_and_taxonomy
 
     has_one :icon, as: :viewable, dependent: :destroy, class_name: 'Spree::TaxonImage'
 
+    scope :for_store, ->(store) { joins(:taxonomy).where(spree_taxonomies: { store_id: store.id }) }
+
     self.whitelisted_ransackable_associations = %w[taxonomy]
+    self.whitelisted_ransackable_attributes = %w[name permalink]
+
+    scope :for_stores, ->(stores) { joins(:taxonomy).where(spree_taxonomies: { store_id: stores.ids }) }
+
+    TRANSLATABLE_FIELDS = %i[name description permalink].freeze
+    translates(*TRANSLATABLE_FIELDS)
+
+    self::Translation.class_eval do
+      alias_attribute :slug, :permalink
+
+      before_create :set_permalink
+
+      def set_permalink
+        parent = translated_model.parent
+        if parent.present?
+          self.permalink = [parent.permalink, (permalink.blank? ? name.to_url : permalink.split('/').last)].join('/')
+        else
+          self.permalink = name.to_url if permalink.blank?
+        end
+      end
+    end
 
     # indicate which filters should be used for a taxon
     # this method should be customized to your own site
@@ -48,22 +86,14 @@ module Spree
       fs
     end
 
-    # Return meta_title if set otherwise generates from root name and/or taxon name
+    # Return meta_title if set otherwise generates from taxon name
     def seo_title
-      if meta_title.blank?
-        root? ? name : "#{root.name} - #{name}"
-      else
-        meta_title
-      end
+      meta_title.blank? ? name : meta_title
     end
 
     # Creates permalink base for friendly_id
     def set_permalink
-      if parent.present?
-        self.permalink = [parent.permalink, (permalink.blank? ? name.to_url : permalink.split('/').last)].join('/')
-      else
-        self.permalink = name.to_url if permalink.blank?
-      end
+      translations.each(&:set_permalink)
     end
 
     def active_products
@@ -77,6 +107,12 @@ module Spree
       ancestor_chain + name.to_s
     end
 
+    def cached_self_and_descendants_ids
+      Rails.cache.fetch("#{cache_key_with_version}/descendant-ids") do
+        self_and_descendants.ids
+      end
+    end
+
     # awesome_nested_set sorts by :lft and :rgt. This call re-inserts the child
     # node so that its resulting position matches the observable 0-indexed position.
     # ** Note ** no :position column needed - a_n_s doesn't handle the reordering if
@@ -89,6 +125,14 @@ module Spree
 
     private
 
+    def sync_taxonomy_name
+      if saved_change_to_name? && root?
+        return if taxonomy.name.to_s == name.to_s
+
+        taxonomy.update(name: name)
+      end
+    end
+
     def touch_ancestors_and_taxonomy
       # Touches all ancestors at once to avoid recursive taxonomy touch, and reduce queries.
       ancestors.update_all(updated_at: Time.current)
@@ -101,5 +145,15 @@ module Spree
         errors.add(:root_conflict, 'this taxonomy already has a root taxon')
       end
     end
+
+    def parent_belongs_to_same_taxonomy
+      if parent.present? && parent.taxonomy_id != taxonomy_id
+        errors.add(:parent, 'must belong to the same taxonomy')
+      end
+    end
+
+    def copy_taxonomy_from_parent
+      self.taxonomy = parent.taxonomy if parent.present? && taxonomy.blank?
+    end
   end
 end