From 1abaefce4f1c518b6e22787f9bf9dcb3239bad12 Mon Sep 17 00:00:00 2001
From: ajugo <andrej.jugovic7@gmail.com>
Date: Thu, 18 May 2017 09:19:44 +0200
Subject: [PATCH] At importing large sample file, error flash message appears
 [fixes SCI-903] (#625)

* At importing large sample file, error flash message appears [fixes SCI-903]

* Changing double quotes to single

* Adding animatedSpinner
---
 .../javascripts/samples/samples_importer.js   | 33 ++++++++++++-------
 1 file changed, 21 insertions(+), 12 deletions(-)

diff --git a/app/assets/javascripts/samples/samples_importer.js b/app/assets/javascripts/samples/samples_importer.js
index 463d9342d..2f6afa3c8 100644
--- a/app/assets/javascripts/samples/samples_importer.js
+++ b/app/assets/javascripts/samples/samples_importer.js
@@ -1,41 +1,50 @@
 var previousIndex;
 var disabledOptions;
-$("select").focus(function() {
+var loadingSamples = false;
+$('select').focus(function() {
   previousIndex = $(this)[0].selectedIndex;
-}).change(function () {
+}).change(function() {
   var currSelect = $(this);
   var currIndex = $(currSelect)[0].selectedIndex;
 
-  $("select").each(function() {
+  $('select').each(function() {
     if (currSelect !== $(this) && currIndex > 0) {
-      $(this).children().eq(currIndex).attr("disabled", "disabled");
+      $(this).children().eq(currIndex).attr('disabled', 'disabled');
     }
 
-    $(this).children().eq(previousIndex).removeAttr("disabled");
+    $(this).children().eq(previousIndex).removeAttr('disabled');
   });
 
   previousIndex = currIndex;
 });
 
 //  Create import samples ajax
-$("form#form-import")
+$('form#form-import')
 .submit(function(e) {
+  // Check if we already uploading samples
+  if (loadingSamples) {
+    return false;
+  }
   disabledOptions = $("option[disabled='disabled']");
-  disabledOptions.removeAttr("disabled");
+  disabledOptions.removeAttr('disabled');
+  loadingSamples = true;
+  animateSpinner();
 })
-.on("ajax:success", function(ev, data, status) {
+.on('ajax:success', function(ev, data, status) {
   // Simply reload page to show flash and updated samples list
+  loadingSamples = false;
   location.reload();
 })
-.on("ajax:error", function(ev, data, status) {
+.on('ajax:error', function(ev, data, status) {
+  loadingSamples = false;
   if (_.isUndefined(data.responseJSON.html)) {
     // Simply reload page to show flash
-     location.reload();
+    location.reload();
   } else {
     // Re-disable options
-    disabledOptions.attr("disabled", "disabled");
+    disabledOptions.attr('disabled', 'disabled');
 
     // Populate the errors container
-    $("#import-errors-container").html(data.responseJSON.html);
+    $('#import-errors-container').html(data.responseJSON.html);
   }
 });