Merge pull request #5867 from mamhoff/performance-remove-includes

Performance: Remove includes from Spree::Variant#options_text
solidusio · Oct 18, 2024 · 01a15b5 · 01a15b5
2 parents 2f05527 + 3cb9d79
commit 01a15b5
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/core/app/models/spree/variant.rb b/core/app/models/spree/variant.rb
@@ -194,7 +194,7 @@ def is_backorderable?
     #
     # @return [String] a sentence-ified string of option values.
     def options_text
-      values = option_values.includes(:option_type).sort_by do |option_value|
+      values = option_values.sort_by do |option_value|
         option_value.option_type.position
       end