raggedstaff/openfoodnetwork
1
0
Fork 0
mirror of https://github.com/openfoodfoundation/openfoodnetwork synced 2026-03-13 04:00:21 +00:00
Code Issues Packages Projects Releases Wiki Activity

Cache DistributorChangeValidator#all_available_distributors, move Enterprise#available_variants into SQLland - massive render speed improvement

Browse Source
This commit is contained in:
Rohan Mitchell
2013-05-20 21:16:43 +10:00
parent f19ef4ed07
commit c3b2f52b7a
3 changed files with 10 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
app/models/enterprise.rb
View File

@@ -29,7 +29,7 @@ class Enterprise < ActiveRecord::Base
end end
def available_variants def available_variants
ProductDistribution.find_all_by_distributor_id( self.id ).map{ |pd| pd.product.variants + [pd.product.master] }.flatten Spree::Variant.joins(:product => :product_distributions).where('product_distributions.distributor_id=?', self.id)
end end

2
app/models/spree/order_decorator.rb
View File

@@ -39,7 +39,7 @@ Spree::Order.class_eval do
end end
def line_item_variants def line_item_variants
line_items.map{ |li| li.variant } line_items.map { |li| li.variant }
end end

6
lib/open_food_web/distributor_change_validator.rb
View File

@@ -11,7 +11,11 @@ class DistributorChangeValidator
def can_change_to_distributor? distributor def can_change_to_distributor? distributor
# Distributor may not be changed once an item has been added to the cart/order, unless all items are available from the specified distributor # Distributor may not be changed once an item has been added to the cart/order, unless all items are available from the specified distributor
@order.line_items.empty? || (available_distributors(Enterprise.all) || []).include?(distributor) @order.line_items.empty? || all_available_distributors.include?(distributor)
end
def all_available_distributors
@all_available_distributors ||= (available_distributors(Enterprise.all) || [])
end end
def available_distributors enterprises def available_distributors enterprises