From 96d582a35ed798815d6553634a4059c896d1923e Mon Sep 17 00:00:00 2001
From: Heitor Reis <marcheing@gmail.com>
Date: Tue, 9 Jun 2015 12:21:19 -0300
Subject: [PATCH] Fixed repeated request for the same address, prevents multiple requests

---
 app/views/repositories/_form.html.erb | 47 +++++++++++++++++++++++++++++++++++------------
 1 file changed, 35 insertions(+), 12 deletions(-)

diff --git a/app/views/repositories/_form.html.erb b/app/views/repositories/_form.html.erb
index 5e8505e..66158ea 100644
--- a/app/views/repositories/_form.html.erb
+++ b/app/views/repositories/_form.html.erb
@@ -125,26 +125,49 @@
 </script>
 
 <script>
+  var branches = {};
+  var request = null;
   function fetch_branches (address_field) {
+    cancel_previous_request();
     var address = address_field.value;
+
+    var el = $("#repository_branch");
+    el.empty(); // remove old options
+
     console.log(address);
-    var scm_type = $("#repository_scm_type option:selected").text();
+    if(branches[address] != undefined) {
+      fill_options(branches[address], el);
+      return;
+    }
 
-    var $el = $("#repository_branch");
-    $el.empty(); // remove old options
+    var scm_type = $("#repository_scm_type option:selected").text();
 
-    $.ajax({
+    request = ($.ajax({
       url: "<%= repository_branches_path() %>",
       data: {'url': address, 'scm_type': scm_type},
       type: 'GET',
-      complete: function (data){
-        var newOptions = JSON.parse(data["responseText"])["branches"];
-
-        $.each(newOptions, function(index, value) {
-          $el.append($("<option></option>")
-             .attr("value", value).text(value));
-        });
+      timeout: 30000,
+      success: function (data){
+        options = data["branches"];
+        if (options != null) {
+          branches[address] = options;
+          fill_options(options, el);
+        }
+       }
+    }));
+
+    function cancel_previous_request() {
+      if (request != null) {
+        request.abort();
+        request = null;
       }
-    })
+    }
+  }
+
+  function fill_options (options, el) {
+    $.each(options, function(index, value) {
+      el.append($("<option></option>")
+          .attr("value", value).text(value));
+    });
   }
 </script>
--
libgit2 0.21.2