From 63eb0980eb10706d91dbedaa9af9a09b5586fe10 Mon Sep 17 00:00:00 2001 From: Pau Perez Date: Fri, 17 Apr 2020 17:19:51 +0200 Subject: [PATCH] Memoize result of line items query No need to fetch twice what we just loaded from DB. --- app/mailers/producer_mailer.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/mailers/producer_mailer.rb b/app/mailers/producer_mailer.rb index e130fefd1f..153a99d358 100644 --- a/app/mailers/producer_mailer.rb +++ b/app/mailers/producer_mailer.rb @@ -51,7 +51,7 @@ class ProducerMailer < Spree::BaseMailer end def line_items_from(order_cycle, producer) - Spree::LineItem. + @line_items ||= Spree::LineItem. includes(variant: [:product, { option_values: :option_type }]). from_order_cycle(order_cycle). sorted_by_name_and_unit_value.