From d598831d67faf2905f591abfeffd1746e3187f68 Mon Sep 17 00:00:00 2001
From: luisramos0 <luisramos0@gmail.com>
Date: Mon, 4 Feb 2019 18:59:41 +0000
Subject: [PATCH] Adapt variants auto complete to spree v2 code with shipments

---
 .../directives/variant_autocomplete.js.coffee |  1 +
 .../spree/admin/variants/_autocomplete.js.erb | 39 ++++++++++---------
 app/views/spree/admin/variants/search.rabl    | 10 ++++-
 3 files changed, 30 insertions(+), 20 deletions(-)

diff --git a/app/assets/javascripts/admin/utils/directives/variant_autocomplete.js.coffee b/app/assets/javascripts/admin/utils/directives/variant_autocomplete.js.coffee
index 9b8f12d3dc..c325c568ec 100644
--- a/app/assets/javascripts/admin/utils/directives/variant_autocomplete.js.coffee
+++ b/app/assets/javascripts/admin/utils/directives/variant_autocomplete.js.coffee
@@ -23,6 +23,7 @@ angular.module("admin.utils").directive "variantAutocomplete", ($timeout) ->
               distributor_id: scope.distributor_id
               order_cycle_id: scope.order_cycle_id
             results: (data, page) ->
+              window.variants = data # this is how spree auto complete JS code picks up variants
               results: data
           formatResult: (variant) ->
             if variant["images"][0] != undefined && variant["images"][0].image != undefined
diff --git a/app/views/spree/admin/variants/_autocomplete.js.erb b/app/views/spree/admin/variants/_autocomplete.js.erb
index b958b0a75b..4cecb1f61d 100644
--- a/app/views/spree/admin/variants/_autocomplete.js.erb
+++ b/app/views/spree/admin/variants/_autocomplete.js.erb
@@ -46,26 +46,27 @@
           <th class="actions"></th>
         </thead>
         <tbody>
-          {{#each variant.stock_items}}
-            <tr>
-              <td>{{this.stock_location_name}}</td>
-              {{#if this.available}}
-                <td>
-                  {{this.count_on_hand}}
-                  {{#if this.backorderable}} (backorders allowed) {{/if}}
-                </td>
-                <td>
-                  <input class="quantity" id="stock_item_quantity" data-stock-location-id="{{this.stock_location_id}}" type="number" min="1" value="1">
-                </td>
-                <td class="actions">
-                  <button class="add_variant no-text icon-plus icon_link with-tip" data-stock-location-id="{{this.stock_location_id}}" title="Add" data-action="add"></button>
-                </td>
+          <tr>
+            <td>{{variant.stock_location_name}}</td>
+            {{#if variant.[in_stock?]}}
+              {{#if variant.on_demand}}
+                <td>Spree.t(:on_demand)</td>
               {{else}}
-                <td><%= Spree.t(:out_of_stock) %></td>
-                <td>0</td>
-              {{/if}}
-            </tr>
-          {{/each}}
+                <td>
+                  {{variant.count_on_hand}}
+                </td>
+              {{/if}}                  
+              <td>
+                <input class="quantity" id="stock_item_quantity" data-stock-location-id="{{variant.stock_location_id}}" type="number" min="1" value="1">
+              </td>
+              <td class="actions">
+                <button class="add_variant no-text icon-plus icon_link with-tip" data-stock-location-id="{{variant.stock_location_id}}" title="Add" data-action="add"></button>
+              </td>
+            {{else}}
+              <td><%= Spree.t(:out_of_stock) %></td>
+              <td>0</td>
+            {{/if}}
+          </tr>
         </tbody>
   </fieldset>
 </script>
diff --git a/app/views/spree/admin/variants/search.rabl b/app/views/spree/admin/variants/search.rabl
index afd3f39ce6..8206932c5f 100644
--- a/app/views/spree/admin/variants/search.rabl
+++ b/app/views/spree/admin/variants/search.rabl
@@ -2,7 +2,7 @@
 # overriding spree/core/app/views/spree/admin/variants/search.rabl
 #
 collection @variants
-attributes :sku, :options_text, :count_on_hand, :id, :cost_price
+attributes :sku, :options_text, :in_stock?, :on_demand, :count_on_hand, :id, :cost_price
 
 node(:name) do |v|
   # TODO: when products must have a unit, full_name will always be present
@@ -22,6 +22,14 @@ node(:producer_name) do |v|
   v.product.supplier.name
 end
 
+node(:stock_location_id) do |v|
+  v.stock_items.first.stock_location.id
+end
+
+node(:stock_location_name) do |v|
+  v.stock_items.first.stock_location.name
+end
+
 child(:images => :images) do
   attributes :mini_url
 end