From 8b1bfa74d1a8faaef7b7c8894436124c8963c314 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 f69671a4b9b..43906a41fd0 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'