From cdaac6a01bd2ed8c4a7e21f59e2d5ac8858ca3db Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Matej=20Zrim=C5=A1ek?= <mzrimsek@biosistemika.com>
Date: Tue, 30 May 2017 15:11:28 +0200
Subject: [PATCH] Added remove repository modal logic. [SCI-1269]

---
 app/assets/javascripts/repositories/index.js  | 19 +++++++++++++++++++
 app/controllers/repositories_controller.rb    | 13 +++++++++++++
 .../_delete_repository_modal.html.erb         |  8 ++++----
 app/views/repositories/index.html.erb         |  6 ++++--
 config/routes.rb                              |  4 +++-
 5 files changed, 43 insertions(+), 7 deletions(-)
 create mode 100644 app/assets/javascripts/repositories/index.js

diff --git a/app/assets/javascripts/repositories/index.js b/app/assets/javascripts/repositories/index.js
new file mode 100644
index 000000000..59c99641a
--- /dev/null
+++ b/app/assets/javascripts/repositories/index.js
@@ -0,0 +1,19 @@
+(function() {
+  'use strict';
+
+  // Show modal for repository deletion
+  $(document).on('click', '#delete-repo-option', function() {
+    var url = $(this).attr('href');
+    $.ajax({
+      method: 'GET',
+      url: url,
+      dataType: 'json'
+    }).done(function(xhr, settings, data) {
+      $('body').append($.parseHTML(data.responseJSON.html));
+      $('#delete-repo-modal').modal('show', {
+        backdrop: true,
+        keyboard: false
+      });
+    });
+  });
+})();
diff --git a/app/controllers/repositories_controller.rb b/app/controllers/repositories_controller.rb
index e70eb914f..b784e2d82 100644
--- a/app/controllers/repositories_controller.rb
+++ b/app/controllers/repositories_controller.rb
@@ -6,6 +6,19 @@ class RepositoriesController < ApplicationController
     render('repositories/index')
   end
 
+  def destroy_modal
+    @repository = Repository.find(params[:repository_id])
+    respond_to do |format|
+      format.json do
+        render json: {
+          html: render_to_string(
+            partial: 'delete_repository_modal.html.erb'
+          )
+        }
+      end
+    end
+  end
+
   def destroy
     @repo = Repository.find(params[:id])
     @repo.destroy if @repo
diff --git a/app/views/repositories/_delete_repository_modal.html.erb b/app/views/repositories/_delete_repository_modal.html.erb
index 728ac182b..589e27c81 100644
--- a/app/views/repositories/_delete_repository_modal.html.erb
+++ b/app/views/repositories/_delete_repository_modal.html.erb
@@ -1,12 +1,12 @@
-<div class="modal fade" id="deleteRepoModal" tabindex="-1" role="dialog">
+<div class="modal fade" id="delete-repo-modal" tabindex="-1" role="dialog">
   <div class="modal-dialog" role="document">
     <div class="modal-content">
       <div class="modal-header">
         <button type="button" class="close" data-dismiss="modal" aria-label="Close"><span aria-hidden="true">&times;</span></button>
-        <h4 class="modal-title"><%= t("repositories.index.modal_delete.title_html", name: "ASD") %></h4>
+        <h4 class="modal-title"><%= t("repositories.index.modal_delete.title_html", name: @repository.name) %></h4>
       </div>
       <div class="modal-body">
-        <p><%= t("repositories.index.modal_delete.message_html", name: "ASD") %></p>
+        <p><%= t("repositories.index.modal_delete.message_html", name: @repository.name) %></p>
         <div class="alert alert-danger" role="alert">
           <span class="glyphicon glyphicon-exclamation-sign"></span>
           &nbsp;
@@ -19,7 +19,7 @@
       </div>
       <div class="modal-footer">
         <%= link_to t('repositories.index.modal_delete.delete'),
-                    team_repository_path(id: -1),
+                    team_repository_path(id: @repository),
                     id: "confirm-repo-delete",
                     method: :delete,
                     remote: true,
diff --git a/app/views/repositories/index.html.erb b/app/views/repositories/index.html.erb
index 6dbdaf4d1..bb9a5fc79 100644
--- a/app/views/repositories/index.html.erb
+++ b/app/views/repositories/index.html.erb
@@ -1,7 +1,6 @@
 <% provide(:head_title, t("repositories.index.head_title")) %>
 
 <% if current_team %>
-  <%= render partial: "repositories/delete_repository_modal" %>
   <%= render partial: "repositories/breadcrumbs.html.erb", locals: { teams: @teams, current_team: current_team, type: @type } %>
 
   <!-- Nav tabs -->
@@ -49,7 +48,10 @@
                 </li>
                 <li role="separator" class="divider"></li>
                 <li>
-                  <%= link_to t('repositories.index.options_dropdown.delete'), "#deleteRepoModal", class: "delete-repo-option", data: {toggle: "modal", href: team_repository_path(id: repo)} %>
+                  <%= link_to t('repositories.index.modal_delete.delete'),
+                              team_repository_destroy_modal_path(repository_id: repo),
+                              id: "delete-repo-option",
+                              remote: true %>
                 </li>
               </ul>
             </div>
diff --git a/config/routes.rb b/config/routes.rb
index 57be0fde6..60f921eca 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -123,7 +123,9 @@ Rails.application.routes.draw do
         as: 'file_expired'
 
     resources :teams do
-      resources :repositories, only: [:index, :edit, :destroy]
+      resources :repositories, only: [:index, :destroy] do
+        get 'destroy_modal', to: 'repositories#destroy_modal'
+      end
       resources :samples, only: [:new, :create]
       resources :sample_types, except: [:show, :new] do
         get 'sample_type_element', to: 'sample_types#sample_type_element'