From 9f7df0af76e9e43fda08292d02b4573f0144a520 Mon Sep 17 00:00:00 2001
From: Jure Grabnar <jgrabnar@biosistemika.com>
Date: Tue, 9 Aug 2016 13:13:50 +0200
Subject: [PATCH] Fix checkbox in experiment reports

---
 .../new/modal/_project_contents.html.erb      |  2 +-
 .../modal/_project_contents_inner.html.erb    | 22 +++++++++----------
 2 files changed, 11 insertions(+), 13 deletions(-)

diff --git a/app/views/reports/new/modal/_project_contents.html.erb b/app/views/reports/new/modal/_project_contents.html.erb
index 56dbce308..f5b9a2193 100644
--- a/app/views/reports/new/modal/_project_contents.html.erb
+++ b/app/views/reports/new/modal/_project_contents.html.erb
@@ -67,7 +67,7 @@
     projectCheckAlls.click(function() {
       var nextUl = $(this).closest("div.checkbox").next();
       if (nextUl.length && nextUl.is("ul.project-contents-list, \
-                                      ul-experiment-contents-list")) {
+                                      ul.experiment-contents-list")) {
         nextUl.find("input:checkbox").prop("checked", this.checked);
       }
     });
diff --git a/app/views/reports/new/modal/_project_contents_inner.html.erb b/app/views/reports/new/modal/_project_contents_inner.html.erb
index 6372ad462..ca5ce46f1 100644
--- a/app/views/reports/new/modal/_project_contents_inner.html.erb
+++ b/app/views/reports/new/modal/_project_contents_inner.html.erb
@@ -9,6 +9,7 @@
   <%= form.check_box :project, label: project.name, class: "project-all-cb" %>
   <ul class="no-style project-contents-list">
     <% project.experiments.is_archived(false).each do |experiment| %>
+      <% next unless experiment.my_modules.is_archived(false).count > 0 %>
       <li>
         <%= form.check_box "experiment_#{experiment.id}", label: experiment.name, class: "project-all-cb" %>
         <ul class="no-style experiment-contents-list">
@@ -21,22 +22,19 @@
                 <% end %>
             <% end %>
           <% end %>
-        </ul>
-      </li>
 
-      <!-- Tasks without groups -->
-      <% modules_without_group = experiment.modules_without_group %>
-      <% if modules_without_group.present? and modules_without_group.count > 0 %>
-        <ul class="no-style experiment-contents-list">
-          <% modules_without_group.each do |my_module| %>
-            <li>
-              <%= form.check_box "modules[#{my_module.id}]", label: my_module.name %>
-            </li>
+          <!-- Tasks without groups -->
+          <% modules_without_group = experiment.modules_without_group %>
+          <% if modules_without_group.present? and modules_without_group.count > 0 %>
+            <% modules_without_group.each do |my_module| %>
+              <li>
+                <%= form.check_box "modules[#{my_module.id}]", label: my_module.name %>
+              </li>
+            <% end %>
           <% end %>
         </ul>
-      <% end %>
+      </li>
     <% end %>
-
   </ul>
 <% else %>
   <div>