From e7e208061f554a4c2d540f26ea39eb9bbb420443 Mon Sep 17 00:00:00 2001
From: Matt-Yorkley <9029026+Matt-Yorkley@users.noreply.github.com>
Date: Fri, 21 Dec 2018 20:01:16 +0000
Subject: [PATCH] Assign polling numbers to variables and add comments

---
 .../orders/controllers/bulk_invoice_controller.js.coffee    | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/app/assets/javascripts/admin/orders/controllers/bulk_invoice_controller.js.coffee b/app/assets/javascripts/admin/orders/controllers/bulk_invoice_controller.js.coffee
index 9c8e5e8286..0442564e2e 100644
--- a/app/assets/javascripts/admin/orders/controllers/bulk_invoice_controller.js.coffee
+++ b/app/assets/javascripts/admin/orders/controllers/bulk_invoice_controller.js.coffee
@@ -5,13 +5,15 @@ angular.module("admin.orders").controller "bulkInvoiceCtrl", ($scope, $http, $ti
     $scope.loading = true
     $scope.message = null
     $scope.error = null
+    $scope.poll_wait = 5 # 5 Seconds between each check
+    $scope.poll_retries = 80 # Maximum checks before stopping
 
     $http.post('/admin/orders/invoices', {order_ids: $scope.selected_orders}).success (data) ->
       $scope.invoice_id = data
       $scope.pollBulkInvoice()
 
   $scope.pollBulkInvoice = ->
-    $timeout($scope.nextPoll, 5000)
+    $timeout($scope.nextPoll, $scope.poll_wait * 1000)
 
   $scope.nextPoll = ->
     $http.get('/admin/orders/invoices/'+$scope.invoice_id+'/poll').success (data) ->
@@ -21,7 +23,7 @@ angular.module("admin.orders").controller "bulkInvoiceCtrl", ($scope, $http, $ti
     .error (data) ->
       $scope.poll++
 
-      if $scope.poll > 30
+      if $scope.poll > $scope.poll_retries
         $scope.loading = false
         $scope.error = t('js.admin.orders.index.bulk_invoice_failed')
         return