From d8d54232c31609fb0fc14e1caf051d173f467a9e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Matej=20Zrim=C5=A1ek?= <mzrimsek@biosistemika.com>
Date: Fri, 2 Jun 2017 19:08:51 +0200
Subject: [PATCH] After repository renaming user is redirected to the same
 repository. Removed the unimplemented repository copy dropdown option.
 [SCI-1269]

---
 app/assets/javascripts/repositories/index.js |  4 ++--
 app/controllers/repositories_controller.rb   |  4 +++-
 app/views/repositories/index.html.erb        | 17 +++++++----------
 config/locales/en.yml                        |  1 -
 4 files changed, 12 insertions(+), 14 deletions(-)

diff --git a/app/assets/javascripts/repositories/index.js b/app/assets/javascripts/repositories/index.js
index 19eb54b94..e9f3e0084 100644
--- a/app/assets/javascripts/repositories/index.js
+++ b/app/assets/javascripts/repositories/index.js
@@ -42,9 +42,9 @@
     if ($modal) {
       var form = $modal.find('form');
       form
-      .on('ajax:success', function() {
+      .on('ajax:success', function(ev, data) {
         animateSpinner(form, true);
-        location.reload();
+        $(location).attr('href', data.url);
       })
       .on('ajax:error', function(e, error) {
         var msg = JSON.parse(error.responseText);
diff --git a/app/controllers/repositories_controller.rb b/app/controllers/repositories_controller.rb
index 8cf47b7d1..365656061 100644
--- a/app/controllers/repositories_controller.rb
+++ b/app/controllers/repositories_controller.rb
@@ -52,7 +52,9 @@ class RepositoriesController < ApplicationController
         if @repository.save
           flash[:success] = t('repositories.index.rename_flash',
                               old_name: old_name, new_name: @repository.name)
-          render json: {}, status: :ok
+          render json: {
+            url: team_repositories_path(@team, repository: @repository)
+          }, status: :ok
         else
           render json: @repository.errors, status: :unprocessable_entity
         end
diff --git a/app/views/repositories/index.html.erb b/app/views/repositories/index.html.erb
index e82b3f870..b1ea121f2 100644
--- a/app/views/repositories/index.html.erb
+++ b/app/views/repositories/index.html.erb
@@ -6,9 +6,12 @@
 
   <!-- Nav tabs -->
   <% if @repositories.present? %>
+    <% active_repo = @repositories.find_by_id(params[:repository]) %>
+    <% active_repo = @repositories.first if !active_repo %>
+
     <ul class="nav nav-tabs nav-settings" role="tablist" id="tabs">
-      <% @repositories.each.with_index do |repo, i| %>
-        <li role="presentation" class="<%= 'active' if i == 0 %>">
+      <% @repositories.each do |repo| %>
+        <li role="presentation" class="<%= 'active' if repo == active_repo %>">
           <a class="repository-nav-tab"
              href="#custom_repo_<%= repo.id %>"
              data-toggle="tab"
@@ -19,8 +22,8 @@
     </ul>
     <!-- Tab panes -->
     <div class="tab-content">
-      <% @repositories.each.with_index do |repo, i| %>
-        <div class="tab-pane tab-pane-settings <%= 'active' if i == 0 %>" id="custom_repo_<%= repo.id %>">
+      <% @repositories.each do |repo| %>
+        <div class="tab-pane tab-pane-settings <%= 'active' if repo == active_repo %>" id="custom_repo_<%= repo.id %>">
           <!-- Tab Content -->
 
           <div id="repository-toolbar">
@@ -45,12 +48,6 @@
                                 class: "rename-repo-option",
                                 remote: true %>
                   </li>
-                  <li>
-                    <!-- TODO -->
-                    <a href="#">
-                      <%= t("repositories.index.options_dropdown.copy") %>
-                    </a>
-                  </li>
                   <li role="separator" class="divider"></li>
                   <li>
                     <%= link_to t('repositories.index.modal_delete.delete'),
diff --git a/config/locales/en.yml b/config/locales/en.yml
index ac085c98b..5e1852e16 100644
--- a/config/locales/en.yml
+++ b/config/locales/en.yml
@@ -161,7 +161,6 @@ en:
       options_dropdown:
         header: "Edit repository"
         rename: "Rename"
-        copy: "Copy"
         delete: "Delete"
       modal_delete:
         title_html: "Delete repository <em>%{name}</em>"