From ececbce596a33bcd97f0902636a67eaddbe28fd3 Mon Sep 17 00:00:00 2001 From: Matt-Yorkley <9029026+Matt-Yorkley@users.noreply.github.com> Date: Tue, 7 Apr 2020 14:43:42 +0200 Subject: [PATCH] Only select id in producers query --- app/controllers/api/products_controller.rb | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/app/controllers/api/products_controller.rb b/app/controllers/api/products_controller.rb index 3182c79044..854d820a47 100644 --- a/app/controllers/api/products_controller.rb +++ b/app/controllers/api/products_controller.rb @@ -69,10 +69,10 @@ module Api end def overridable - producers = OpenFoodNetwork::Permissions.new(current_api_user). - variant_override_producers.by_name + producer_ids = OpenFoodNetwork::Permissions.new(current_api_user). + variant_override_producers.by_name.select('enterprises.id') - @products = paged_products_for_producers producers + @products = paged_products_for_producers producer_ids render_paged_products @products, ::Api::Admin::ProductSimpleSerializer end @@ -118,11 +118,11 @@ module Api ] end - def paged_products_for_producers(producers) + def paged_products_for_producers(producer_ids) Spree::Product.scoped. merge(product_scope). includes(variants: [:product, :default_price, :stock_items]). - where(supplier_id: producers). + where(supplier_id: producer_ids). by_producer.by_name. ransack(params[:q]).result. page(params[:page]).per(params[:per_page])