From 0c5ac99e0f2cb4db5f682d1579c4423b1845511c Mon Sep 17 00:00:00 2001
From: Matt-Yorkley <9029026+Matt-Yorkley@users.noreply.github.com>
Date: Mon, 10 Dec 2018 20:08:06 +0000
Subject: [PATCH] Fix multiple results with 'inventory_items_shipment_id_null'
 ransack query

The q['inventory_items_shipment_id_null'] search element was generating duplicates where there were multiple line items on an order that were ready to be shipped, with id = null
---
 app/services/search_orders.rb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/app/services/search_orders.rb b/app/services/search_orders.rb
index 127a53625e..c1120ffb19 100644
--- a/app/services/search_orders.rb
+++ b/app/services/search_orders.rb
@@ -26,11 +26,11 @@ class SearchOrders
     @search = OpenFoodNetwork::Permissions.new(current_user).editable_orders.ransack(params[:q])
 
     return paginated_results if using_pagination?
-    @search.result
+    @search.result(distinct: true)
   end
 
   def paginated_results
-    @search.result
+    @search.result(distinct: true)
       .page(params[:page])
       .per(params[:per_page])
   end