From 2e1e7586cc2e7e9b0de571f3597d6b7c5f4b43e7 Mon Sep 17 00:00:00 2001 From: Thomas von Deyen Date: Tue, 24 Oct 2023 09:13:07 +0200 Subject: [PATCH] Do not require active_storage/engine The activestorage adapter is optional and stores might use another attachment adapter (ie. paperclip). Since bundler already requires activestorage if the store decided to use activestorage adapter (either via a direct dependency or the transient rails gem) we do not need to require it in our engine. This removes a LoadError for stores/engines that do not use activestorage. (cherry picked from commit ea0254b13abe237cb56cd79787207bb7691d3173) --- core/lib/spree/core.rb | 1 - 1 file changed, 1 deletion(-) diff --git a/core/lib/spree/core.rb b/core/lib/spree/core.rb index d1909197a3d..ec81fca61e8 100644 --- a/core/lib/spree/core.rb +++ b/core/lib/spree/core.rb @@ -6,7 +6,6 @@ require "active_job/railtie" require "active_model/railtie" require "active_record/railtie" -require "active_storage/engine" require "sprockets/railtie" require 'acts_as_list'