spree_core 6.0.0.beta4 → 6.0.0.rc2

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

@@ -38,8 +38,17 @@ FactoryBot.define do
       price { 19.99 }
       compare_at_price { nil }
       currency { nil }
+      published_at { nil }
+      unpublished_at { nil }
     end
 
+    # The deprecated +available_on=+/+discontinue_on=+ setters warn, so the
+    # dates go straight onto the legacy columns, which code still reads and
+    # which seed the default-channel publication created below.
+    after(:build) do |product, evaluator|
+      product[:available_on] = evaluator.published_at if evaluator.published_at
+      product[:discontinue_on] = evaluator.unpublished_at if evaluator.unpublished_at
+    end
     before(:create) do |_product|
       create(:stock_location) unless Spree::StockLocation.any?
     end

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

@@ -4,7 +4,7 @@ FactoryBot.define do
   factory :refund, class: Spree::Refund do
     amount         { 100.00 }
     transaction_id { generate(:refund_transaction_id) }
-    association(:payment, state: 'completed')
+    association(:payment, status: 'completed')
     association(:reason, factory: :refund_reason)
   end
 

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

@@ -20,7 +20,7 @@ FactoryBot.define do
 
     after(:build) do |package, evaluator|
       evaluator.variants_contents.each do |variant, count|
-        package.add_multiple build_list(:inventory_unit, count, variant: variant)
+        package.add_multiple build_list(:fulfillment_item, count, variant: variant)
       end
     end
 

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

@@ -45,7 +45,7 @@ FactoryBot.define do
       product { |p| p.association(:product) }
 
       factory :with_image_variant do
-        images { create_list(:image, 1) }
+        images { create_list(:media, 1) }
       end
 
       trait :with_no_price do

data/lib/spree/testing_support/i18n.rb

@@ -1,94 +1,3 @@
-# This file exists solely to test whether or not there are missing translations
-# within the code that Spree's test suite covers.
-#
-# If there is a translation referenced which has no corresponding key within the
-# .yml file, then there will be a message output at the end of the suite showing
-# that.
-#
-# If there is a translation within the locale file which *isn't* used in the
-# test, this will also be shown at the end of the suite run.
-module Spree
-  class << self
-    attr_accessor :used_translations, :missing_translation_messages,
-                  :unused_translations, :unused_translation_messages
-    alias normal_t t
-  end
-
-  def self.t(*args)
-    original_args = args.dup
-    options = args.extract_options!
-    self.used_translations ||= []
-    [*args.first].each do |translation_key|
-      key = ([*options[:scope]] << translation_key).join('.')
-      self.used_translations << key
-    end
-    normal_t(*original_args)
-  end
-
-  def self.check_missing_translations
-    self.missing_translation_messages = []
-    self.used_translations ||= []
-    used_translations.map { |a| a.split('.') }.each do |translation_keys|
-      root = translations
-      processed_keys = []
-      translation_keys.each do |key|
-        root = root.fetch(key.to_sym)
-        processed_keys << key.to_sym
-      rescue KeyError
-        error = "#{(processed_keys << key).join('.')} (#{I18n.locale})"
-        unless Spree.missing_translation_messages.include?(error)
-          Spree.missing_translation_messages << error
-        end
-      end
-    end
-  end
-
-  def self.check_unused_translations
-    self.used_translations ||= []
-    self.unused_translation_messages = []
-    self.unused_translations = []
-    load_translations(translations)
-    translation_diff = unused_translations - used_translations
-    translation_diff.each do |translation|
-      Spree.unused_translation_messages << "#{translation} (#{I18n.locale})"
-    end
-  end
-
-  private
-
-  def self.load_translations(hash, root = [])
-    hash.each do |k, v|
-      if v.is_a?(Hash)
-        load_translations(v, root.dup << k)
-      else
-        key = (root + [k]).join('.')
-        unused_translations << key
-      end
-    end
-  end
-
-  def self.translations
-    @translations ||= I18n.backend.send(:translations)[I18n.locale][:spree]
-  end
-end
-
-RSpec.configure do |config|
-  # Need to check here again because this is used in i18n_spec too.
-  if ENV['CHECK_TRANSLATIONS']
-    config.after :suite do
-      Spree.check_missing_translations
-      if Spree.missing_translation_messages.any?
-        puts "\nThere are missing translations within Spree:"
-        puts Spree.missing_translation_messages.sort
-        exit(1)
-      end
-
-      Spree.check_unused_translations
-      if Spree.unused_translation_messages.any?
-        puts "\nThere are unused translations within Spree:"
-        puts Spree.unused_translation_messages.sort
-        exit(1)
-      end
-    end
-  end
-end
+# Deprecated: Spree's specs now catch missing keys through Rails'
+# config.i18n.raise_on_missing_translations. Removed in Spree 6.1.
+Spree::Deprecation.warn("spree/testing_support/i18n is deprecated and does nothing; set config.i18n.raise_on_missing_translations = true in your test environment instead. It will be removed in Spree 6.1.") if defined?(Spree::Deprecation)

data/lib/spree/testing_support/lifecycle_events.rb

@@ -21,7 +21,7 @@ shared_examples_for 'lifecycle events' do |factory: nil, event_prefix: nil|
         expect(record).to receive(:publish_event).with("#{lifecycle_event_prefix}.updated")
         allow(record).to receive(:publish_event).with(anything)
 
-        # update_attribute, not update!: Spree::Shipment#update! shadows the
+        # update_attribute, not update!: Spree::Fulfillment#update! shadows the
         # Active Record method with the legacy state-recalculation API.
         Timecop.travel(1.minute.from_now) do
           record.update_attribute(:updated_at, Time.current)

data/lib/spree/testing_support/order_walkthrough.rb

@@ -21,7 +21,7 @@ class OrderWalkthrough
 
     # A delivery method must exist for rates to be displayed on checkout page
     unless Spree::DeliveryMethod.exists?
-      FactoryBot.create(:shipping_method).tap do |delivery_method|
+      FactoryBot.create(:delivery_method).tap do |delivery_method|
         delivery_method.calculator.preferred_amount = 10
         delivery_method.calculator.preferred_currency = store.default_currency
         delivery_method.calculator.save

data/lib/spree/testing_support/preferences.rb

@@ -18,11 +18,6 @@ module Spree
         yield(config) if block_given?
       end
 
-      def assert_preference_unset(preference)
-        find("#preferences_#{preference}")['checked'].should be false
-        Spree::Config[preference].should be false
-      end
-
       # Stubs store-scoped commerce settings for the duration of one example.
       #
       #   stub_store_preferences(track_inventory_levels: false)

data/lib/spree/testing_support/store.rb

@@ -29,6 +29,15 @@ RSpec.configure do |config|
     end
   end
 
+  # Reloaded before each example rather than after: the after hooks run
+  # inside the example's transaction, so a reload there still sees the
+  # example's writes, and the next example would start from values the
+  # database no longer holds. Setting a preference to such a stale value is
+  # no change to Rails, so it would never be saved.
+  config.before(:each) do
+    @default_store&.reload unless self.class.metadata[:without_global_store]
+  end
+
   config.after(:each) do
     unless self.class.metadata[:without_global_store]
       @default_store&.products = []
@@ -43,7 +52,6 @@ RSpec.configure do |config|
         @default_store.association(:default_market).reset if @default_store.association_cached?(:default_market)
         @default_store.association(:markets).reset if @default_store.association_cached?(:markets)
         @default_store.remove_instance_variable(:@has_markets) if @default_store.instance_variable_defined?(:@has_markets)
-        @default_store.reload
       end
     end
   end

data/lib/spree/upgrades/5_6_to_6_0/manifest.yml

@@ -35,6 +35,22 @@ steps:
 
       Idempotent, and a no-op on an already-consolidated schema.
 
+  - id: preferences_json
+    name: "Check for preferences still stored as YAML"
+    task: "spree:upgrade:preferences_json"
+    notes: |
+      Spree 6.0 stores preferences as JSON instead of YAML, and keeps secrets
+      (preferences declared as passwords, such as gateway API keys) in a
+      separate secret_preferences column. The migration converts every row and
+      moves every secret itself, so a normal db:migrate needs nothing from this
+      step.
+
+      This is a safety net: it converts any row still holding YAML, and moves
+      the secrets of a class that was not loaded when the migration ran, such
+      as one from an extension gem installed afterwards.
+
+      Idempotent, and a no-op once everything is converted.
+
   - id: migrate_media_class_names
     name: "Rewrite stored Spree::Asset and Spree::Image class names"
     task: "spree:migrate_media_class_names"
@@ -149,7 +165,9 @@ steps:
       that has_secure_password reads directly only when unpeppered; the task
       aborts if Devise.pepper is set. Because Devise is gone in 6.0 a former
       pepper can't be introspected, so when Devise is absent the task aborts until
-      you confirm none was used with CONFIRM_NO_PEPPER=true. Peppered / SSO /
+      you confirm none was used with CONFIRM_NO_PEPPER=true. The check is skipped
+      when there are no passwords to copy (e.g. an empty spree_users table), so
+      new projects pass without the flag. Peppered / SSO /
       non-bcrypt installs keep a custom customer_class instead (see the 5.6-to-6.0
       upgrade guide) and skip this step.
 
@@ -288,6 +306,20 @@ steps:
       with one statement scoped to the rows still missing a code, so an
       interrupted run picks up where it stopped.
 
+  - id: migrate_checkout_zones
+    name: "Convert checkout zones to markets"
+    task: "spree:markets:migrate_checkout_zones"
+    notes: |
+      Markets replace the store's checkout zone in Spree 6. Stores that have
+      no market yet get a default one covering their checkout zone's
+      countries, or their default country when no zone was set, and the
+      checkout zone is cleared.
+
+      Run before migrate_tax_zones, which matches zone price rules against a
+      store's markets, and before backfill_order_markets, which assigns each
+      store's default market to its orders. Idempotent: stores that already
+      have a market are skipped.
+
   - id: migrate_promotion_option_value_rules
     name: "Rewrite Option Value promotion rules to stable option value ids"
     task: "spree:upgrade:migrate_promotion_option_value_rules"
@@ -715,3 +747,37 @@ steps:
       predates the upgrade.
 
       Idempotent, and safe to re-run at any time to repair drift.
+
+  - id: encrypt_secret_preferences
+    name: "Encrypt gateway and integration secrets"
+    task: "spree:upgrade:encrypt_secret_preferences"
+    notes: |
+      The 6.0 migration moves existing secrets into the secret_preferences
+      column in plain text, so it never needs encryption keys. They stay
+      readable and are encrypted the next time their record is saved. This step
+      encrypts them all now.
+
+      Skips with a note when Active Record encryption keys are not configured;
+      see the encryption section of the upgrade guide to add them, then run
+      this step again.
+
+      Idempotent.
+
+  - id: assign_upload_stores
+    name: "Record which store owns every uploaded file"
+    task: "spree:uploads:assign_stores"
+    notes: |
+      Spree 6.0 records the store that owns every uploaded file, so a file from
+      one store cannot be attached in another and storage can be measured and
+      cleaned up per store. New files get their store when they are uploaded;
+      this gives existing files the store of the record they are attached to.
+
+      Files attached in more than one store are listed and left unchanged for
+      you to resolve. Files attached to nothing keep no store and are left
+      alone: the daily purge of never-attached uploads
+      (Spree::Uploads::PurgeUnattachedJob, see the upgrade guide) only
+      deletes files that belong to a store.
+
+      Run AFTER the steps that create or move attachments (media, categories,
+      returns), so their files are covered. Idempotent — files that already
+      have a store are skipped.

data/lib/spree/uploads.rb

@@ -0,0 +1,101 @@
+module Spree
+  # Store ownership of uploaded files (docs/plans/6.0-uploads-and-file-ownership.md).
+  #
+  # Every `ActiveStorage::Blob` records the store that owns it, and attaching a
+  # blob to a record of another store is reported. Spree 6.0 only logs; 6.1
+  # refuses, so one release of logs finds every path that still creates blobs
+  # without a store.
+  module Uploads
+    # How long an upload reference (`signed_id`) stays attachable.
+    SIGNED_ID_EXPIRY = 1.day
+
+    # Mints an expiring upload reference for +blob+.
+    #
+    # @param blob [ActiveStorage::Blob]
+    # @return [String] an opaque reference clients pass back unchanged
+    def self.signed_id_for(blob)
+      blob.signed_id(expires_in: SIGNED_ID_EXPIRY)
+    end
+
+    # The store owning +record+, or nil for records that belong to no store
+    # (admin users, customers).
+    #
+    # @param record [ActiveRecord::Base]
+    # @return [Integer, String, nil]
+    def self.store_id_for(record)
+      return record.id if record.is_a?(Spree::Store)
+      # A processed rendition belongs to whoever owns its original.
+      return record.blob&.store_id if record.is_a?(ActiveStorage::VariantRecord)
+      return record.store_id if record.has_attribute?(:store_id)
+
+      record.try(:store)&.id
+    rescue Module::DelegationError
+      # An owner whose parent is gone (a digital asset on a deleted variant)
+      # has no store to compare against.
+      nil
+    end
+
+    module BlobStoreOwnership
+      extend ActiveSupport::Concern
+
+      included do
+        before_create :assign_current_store
+      end
+
+      private
+
+      # Reads the store a request or job declared, never the default-store
+      # fallback: guessing the default would hide exactly the creation paths
+      # the 6.0 logs are meant to find.
+      def assign_current_store
+        return unless has_attribute?(:store_id)
+
+        self.store_id ||= Spree::Current.attributes[:store]&.id
+      end
+    end
+
+    module AttachmentStoreCheck
+      extend ActiveSupport::Concern
+
+      included do
+        before_validation :adopt_owner_store, on: :create
+        after_create :check_blob_store
+      end
+
+      private
+
+      # A file Spree just made from bytes it holds (an export, a purchased
+      # label) is created in the same call that attaches it, so it simply
+      # takes its owner's store, written with the file itself.
+      def adopt_owner_store
+        return unless blob&.new_record? && blob.has_attribute?(:store_id) && record
+
+        blob.store_id ||= owner_store_id
+      end
+
+      def check_blob_store
+        return unless blob&.has_attribute?(:store_id) && record
+        return if owner_store_id.nil?
+
+        if blob.store_id.nil?
+          report_store_mismatch('has no store')
+        elsif blob.store_id.to_s != owner_store_id.to_s
+          report_store_mismatch("belongs to store #{blob.store_id}")
+        end
+      end
+
+      def owner_store_id
+        return @owner_store_id if defined?(@owner_store_id)
+
+        @owner_store_id = Spree::Uploads.store_id_for(record)
+      end
+
+      def report_store_mismatch(reason)
+        Rails.logger.warn(
+          "[Spree] Attached file #{blob.id} #{reason}, but #{record_type}##{record_id} (#{name}) " \
+          "belongs to store #{owner_store_id}. Spree 6.1 refuses this attachment."
+        )
+      end
+    end
+  end
+end

data/lib/tasks/categories_and_collections.rake

@@ -80,7 +80,7 @@ namespace :spree do
           # otherwise purge/destroy: ActionText description, images, custom fields, slugs.
           move_polymorphic.call(ActionText::RichText, :record_type, :record_id, category, collection)
           move_polymorphic.call(ActiveStorage::Attachment, :record_type, :record_id, category, collection, name: %w[image square_image])
-          move_polymorphic.call(Spree::Metafield, :resource_type, :resource_id, category, collection)
+          move_polymorphic.call(Spree::CustomField, :resource_type, :resource_id, category, collection)
           move_polymorphic.call(FriendlyId::Slug, :sluggable_type, :sluggable_id, category, collection)
 
           # Rules -> collection_rules (remap the STI type; insert_all bypasses the
@@ -169,8 +169,8 @@ namespace :spree do
     # category rows that still hold the pre-6.0 names.
     puts 'Backfilling renamed class-name strings...'
     [
-      [Spree::Metafield, :resource_type, 'Spree::Taxon', 'Spree::Category'],
-      [Spree::MetafieldDefinition, :resource_type, 'Spree::Taxon', 'Spree::Category'],
+      [Spree::CustomField, :resource_type, 'Spree::Taxon', 'Spree::Category'],
+      [Spree::CustomFieldDefinition, :resource_type, 'Spree::Taxon', 'Spree::Category'],
       [ActiveStorage::Attachment, :record_type, 'Spree::Taxon', 'Spree::Category'],
       # description is an ActionText field (translates :description, backend: :action_text);
       # surviving categories keep their rich-text description without this.

data/lib/tasks/core.rake

@@ -26,14 +26,16 @@ namespace :core do
 
   desc 'Migrate newsletter subscribers'
   task migrate_newsletter_subscribers: :environment do |_t, _args|
-    Spree.customer_class.where(accepts_email_marketing: true).in_batches(of: 500) do |user_batch|
-      subscriber_attributes = user_batch.pluck(:id, :email, :updated_at).map do |id, email, updated_at|
+    store_id = Spree::Store.default&.id
+
+    Spree.customer_class.where(accepts_email_marketing: true).in_batches(of: 500) do |customer_batch|
+      subscriber_attributes = customer_batch.pluck(:id, :email, :updated_at).map do |id, email, updated_at|
         {
           email: Spree::NewsletterSubscriber.new(email: email).email, # normalized email
-          user_id: id,
+          customer_id: id,
           verified_at: updated_at,
           verification_token: nil,
-          store_id: Spree::Store.default&.id,
+          store_id: store_id,
           updated_at: DateTime.current,
           created_at: DateTime.current
         }

data/lib/tasks/markets.rake

@@ -31,7 +31,8 @@ namespace :spree do
         market = store.markets.create!(
           name: primary_country&.name || 'Default',
           currency: iso_country&.currency_code || store.read_attribute(:default_currency) || 'USD',
-          default_locale: iso_country&.languages_official&.first || store.read_attribute(:default_locale) || 'en',
+          default_locale: Spree::Locales.listed(iso_country&.languages_official&.first) ||
+                          Spree::Locales.listed(store.read_attribute(:default_locale)) || 'en',
           default: true,
           countries: countries
         )

data/lib/tasks/media.rake

@@ -31,9 +31,9 @@ namespace :spree do
       # Subquery (not pluck) so the product set doesn't materialize in Ruby —
       # important for catalogs with millions of products.
       variant_product_ids = Spree::Variant
-                              .joins("INNER JOIN #{Spree::Asset.table_name} ON " \
-                                     "#{Spree::Asset.table_name}.viewable_id = #{Spree::Variant.table_name}.id " \
-                                     "AND #{Spree::Asset.table_name}.viewable_type = 'Spree::Variant'")
+                              .joins("INNER JOIN #{Spree::Media.table_name} ON " \
+                                     "#{Spree::Media.table_name}.viewable_id = #{Spree::Variant.table_name}.id " \
+                                     "AND #{Spree::Media.table_name}.viewable_type = 'Spree::Variant'")
                               .select(:product_id)
 
       relation = Spree::Product.where(id: variant_product_ids)

data/lib/tasks/migrate_users_to_customers.rake

@@ -16,7 +16,8 @@ namespace :spree do
       configured — Devise's +encrypted_password+ is a plain bcrypt digest, the
       exact format +has_secure_password+ reads. The task aborts if a pepper is
       detected; when Devise is absent (so a former pepper can't be introspected)
-      it aborts until you confirm none was used with CONFIRM_NO_PEPPER=true.
+      it aborts until you confirm none was used with CONFIRM_NO_PEPPER=true. The
+      check is skipped when there are no passwords to copy.
       Source rows with a blank email, or whose email is already taken by a
       different customer, abort the task with their ids — reconcile them or pass
       SKIP_INVALID_ROWS=true to skip and continue. The legacy source table is
@@ -38,10 +39,29 @@ namespace :spree do
         next
       end
 
+      customer_model = Spree.customer_class
+      admin_model    = Spree.admin_user_class
+
+      # Read the legacy table by name only — the Devise-era Spree::User /
+      # Spree::LegacyUser models are gone in 6.0. Inherit from Spree.base_class
+      # so the source shares the customer table's connection (the id-exclusion
+      # subquery below spans both).
+      source = Class.new(Spree.base_class) { self.table_name = source_table }
+
+      pending = source.where.not(id: customer_model.select(:id))
+      admin_pending = if admin_model.column_names.include?('encrypted_password')
+                        admin_model.where(password_digest: [nil, '']).where.not(encrypted_password: [nil, ''])
+                      else
+                        admin_model.none
+                      end
+
       # Peppered digests can't be verified by has_secure_password. Refuse rather
       # than silently produce accounts nobody can sign into. Once Devise is gone
       # a former pepper can't be introspected, so require an explicit all-clear.
-      if defined?(Devise) && Devise.respond_to?(:pepper) && Devise.pepper.present?
+      # With no digest to copy (e.g. an empty legacy table) there is nothing to protect.
+      if pending.none? && admin_pending.none?
+        puts "  No customer or admin passwords to copy — skipping the Devise pepper check."
+      elsif defined?(Devise) && Devise.respond_to?(:pepper) && Devise.pepper.present?
         abort "  Devise.pepper is configured — bcrypt digests can't be copied into password_digest. " \
               "Use the custom-model path (keep the table, alias password_digest) or force password resets."
       elsif !defined?(Devise) && ENV['CONFIRM_NO_PEPPER'] != 'true'
@@ -51,15 +71,6 @@ namespace :spree do
               "used no pepper, or use the custom-model path / force password resets."
       end
 
-      customer_model = Spree.customer_class
-      admin_model    = Spree.admin_user_class
-
-      # Read the legacy table by name only — the Devise-era Spree::User /
-      # Spree::LegacyUser models are gone in 6.0. Inherit from Spree.base_class
-      # so the source shares the customer table's connection (the id-exclusion
-      # subquery below spans both).
-      source = Class.new(Spree.base_class) { self.table_name = source_table }
-
       # Copy only columns both tables share; map the digest column separately.
       direct_columns = customer_model.column_names & source.column_names
       copy_digest_from_encrypted = source.column_names.include?('encrypted_password') &&
@@ -75,7 +86,6 @@ namespace :spree do
       #      the whole batch on the unique-email index.
       # Either kind aborts the task with the offending ids unless the operator
       # passes SKIP_INVALID_ROWS=true to skip them and continue.
-      pending = source.where.not(id: customer_model.select(:id))
       blank_email_ids = pending.where("email IS NULL OR email = ''").pluck(:id)
       # Compare case-insensitively to match Spree::Customer's uniqueness
       # (case_sensitive: false) — emails are stored case-preserved, so a legacy
@@ -140,12 +150,7 @@ namespace :spree do
 
       # Admins stay in spree_admin_users (in place) — backfill the digest from
       # the legacy column when present.
-      admin_backfilled = 0
-      if admin_model.column_names.include?('encrypted_password')
-        admin_backfilled = admin_model.where(password_digest: [nil, '']).
-                           where.not(encrypted_password: [nil, '']).
-                           update_all('password_digest = encrypted_password')
-      end
+      admin_backfilled = admin_pending.update_all('password_digest = encrypted_password')
 
       puts
       puts "  Customers copied: #{copied} (skipped #{invalid_ids.size} invalid row(s): " \

data/lib/tasks/money.rake

@@ -0,0 +1,46 @@
+namespace :spree do
+  namespace :money do
+    desc <<~DESC
+      Lists payment capture events in currencies without two decimals (yen,
+      dinar, ...) and writes nothing (Spree 6.0).
+
+      Before 6.0 a partial or manual capture recorded its amount in the wrong
+      unit for these currencies: a 1000 JPY capture was recorded as 100000,
+      a 1.500 KWD capture as 0.150. A purchase (authorize and capture in one)
+      recorded the right amount, and the two cannot be told apart from the
+      row, so every event is listed with both readings: as recorded, and
+      corrected as if the capture flow wrote it. Compare them with the
+      gateway's own records before changing anything.
+    DESC
+    task audit_capture_events: :environment do
+      listed = 0
+      # The wrong records were written with the Money gem's exponents as they
+      # were before 6.0, which differ from ISO 4217 for a few currencies.
+      legacy_exponents = { 'HUF' => 0, 'MGA' => 1, 'MRO' => 1, 'MRU' => 1 }
+      exponent_for = ->(code) { legacy_exponents.fetch(code) { Spree::Money::Rounding.precision(code) } }
+      codes = (Spree::Money::Rounding::EXPONENTS.keys | legacy_exponents.keys).reject { |code| exponent_for.(code) == 2 }
+      in_codes = [Spree::Order, Spree::Cart, Spree::OrderGroup].map { |owner| owner.arel_table[:currency].in(codes) }.reduce(:or)
+
+      Spree::PaymentCaptureEvent.left_joins(payment: [:order, :cart, :order_group]).where(in_codes)
+                                .includes(payment: [:order, :cart, :order_group]).find_each do |event|
+        payment = event.payment
+        next if payment.nil?
+
+        exponent = exponent_for.(payment.currency.to_s.upcase)
+
+        corrected = event.amount * (10**exponent) / 100
+        puts [
+          event.prefixed_id,
+          payment.number,
+          payment.currency,
+          "recorded=#{Spree::Money::Rounding.format(event.amount, payment.currency)}",
+          "if_from_capture=#{Spree::Money::Rounding.format(corrected, payment.currency)}",
+          "payment_amount=#{Spree::Money::Rounding.format(payment.amount, payment.currency)}"
+        ].join("\t")
+        listed += 1
+      end
+
+      puts "  #{listed} capture event(s) in currencies without two decimals. Nothing was changed."
+    end
+  end
+end

data/lib/tasks/package_types.rake

@@ -43,7 +43,7 @@ namespace :spree do
         end
 
         store.package_types.create!(
-          name: Spree.t('package_types.default_name'),
+          name: I18n.t('spree.package_types.default_name'),
           kind: 'box',
           default: true,
           weight: weight.positive? ? weight : nil,