From 0c184dcdcf76602fed0def274cdd600ac8050329 Mon Sep 17 00:00:00 2001
From: Jure Grabnar <jgrabnar@biosistemika.com>
Date: Tue, 30 Aug 2016 08:26:59 +0200
Subject: [PATCH] Remove Move button on empty experiments

SCI-328 #close.
---
 app/views/canvas/edit/modal/_move_module.html.erb       | 9 ++++++---
 app/views/canvas/edit/modal/_move_module_group.html.erb | 9 ++++++---
 2 files changed, 12 insertions(+), 6 deletions(-)

diff --git a/app/views/canvas/edit/modal/_move_module.html.erb b/app/views/canvas/edit/modal/_move_module.html.erb
index f77f7c27b..7aa592df9 100644
--- a/app/views/canvas/edit/modal/_move_module.html.erb
+++ b/app/views/canvas/edit/modal/_move_module.html.erb
@@ -6,9 +6,10 @@
         <h4 class="modal-title" id="modal-move-module-label"><%=t "experiments.canvas.edit.modal_move_module.title" %></h4>
       </div>
       <div class="modal-body">
-        <% if @experiment.project.experiments.is_archived(false).count > 1 %>
+        <% experiments = @experiment.project.experiments.is_archived(false) %>
+        <% if experiments.count > 1 %>
           <%= bootstrap_form_tag do |f| %>
-            <%= f.select :experiment_id, @experiment.project.experiments.is_archived(false)
+            <%= f.select :experiment_id, experiments
                                         .select { |e| e != @experiment }
                                         .collect { |e| [ e.name, e.id ] }, {},
                 {class: "form-control selectpicker", "data-role" => "clear"} %>
@@ -23,7 +24,9 @@
       </div>
       <div class="modal-footer">
         <button type="button" class="btn btn-default" data-dismiss="modal"><%=t "general.cancel" %></button>
-        <button type="button" class="btn btn-primary" data-action="confirm"><%=t "experiments.canvas.edit.modal_move_module.confirm" %></button>
+        <% if experiments.count > 1 %>
+          <button type="button" class="btn btn-primary" data-action="confirm"><%=t "experiments.canvas.edit.modal_move_module.confirm" %></button>
+        <% end %>
       </div>
     </div>
   </div>
diff --git a/app/views/canvas/edit/modal/_move_module_group.html.erb b/app/views/canvas/edit/modal/_move_module_group.html.erb
index a9f109178..6be739206 100644
--- a/app/views/canvas/edit/modal/_move_module_group.html.erb
+++ b/app/views/canvas/edit/modal/_move_module_group.html.erb
@@ -6,9 +6,10 @@
         <h4 class="modal-title" id="modal-move-module-group-label"><%=t "experiments.canvas.edit.modal_move_module_group.title" %></h4>
       </div>
       <div class="modal-body">
-        <% if @experiment.project.experiments.is_archived(false).count > 1 %>
+        <% experiments =  @experiment.project.experiments.is_archived(false) %>
+        <% if experiments.count > 1 %>
           <%= bootstrap_form_tag do |f| %>
-            <%= f.select :experiment_id, @experiment.project.experiments.is_archived(false)
+            <%= f.select :experiment_id, experiments
                                         .select { |e| e != @experiment }
                                         .collect { |e| [ e.name, e.id ] }, {},
                 {class: "form-control selectpicker", "data-role" => "clear"} %>
@@ -23,7 +24,9 @@
       </div>
       <div class="modal-footer">
         <button type="button" class="btn btn-default" data-dismiss="modal"><%=t "general.cancel" %></button>
-        <button type="button" class="btn btn-primary" data-action="confirm"><%=t "experiments.canvas.edit.modal_move_module_group.confirm" %></button>
+        <% if experiments.count > 1 %>
+          <button type="button" class="btn btn-primary" data-action="confirm"><%=t "experiments.canvas.edit.modal_move_module_group.confirm" %></button>
+      <% end %>
       </div>
     </div>
   </div>