raggedstaff/openfoodnetwork
1
0
Fork 0
mirror of https://github.com/openfoodfoundation/openfoodnetwork synced 2026-02-05 22:26:07 +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

4
app/models/enterprise.rb
View File

@@ -27,9 +27,9 @@ class Enterprise < ActiveRecord::Base
def to_param
"#{id}-#{name.parameterize}"
end
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

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

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

10
lib/open_food_web/distributor_change_validator.rb
View File

@@ -11,12 +11,16 @@ class DistributorChangeValidator
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
@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
def available_distributors enterprises
enterprises.select do |e|
(@order.line_item_variants - e.available_variants).empty?
end
end
end
end