From 271c6869cd1ad8cc64cbf21cd7ee91efaf12f7f4 Mon Sep 17 00:00:00 2001
From: decabeza <alberto@decabeza.es>
Date: Tue, 9 Jan 2018 19:17:19 +0100
Subject: [PATCH] hide budget investment table heading if no results

---
 .../budget_investments/_investments.html.erb  | 136 +++++++++---------
 config/locales/en/admin.yml                   |   1 +
 config/locales/es/admin.yml                   |   1 +
 3 files changed, 73 insertions(+), 65 deletions(-)

diff --git a/app/views/admin/budget_investments/_investments.html.erb b/app/views/admin/budget_investments/_investments.html.erb
index 75bde7002..86cd03e79 100644
--- a/app/views/admin/budget_investments/_investments.html.erb
+++ b/app/views/admin/budget_investments/_investments.html.erb
@@ -1,72 +1,78 @@
 <div class='relative'>
   <%= link_to 'download current selecction',admin_budget_budget_investments_path(csv_params), class: 'advanced-search small' %>
-  <h3><%= page_entries_info @investments %></h3>
 </div>
 
-<table>
-  <thead>
-    <tr>
-      <th><%= t("admin.budget_investments.index.table_id") %></th>
-      <th><%= t("admin.budget_investments.index.table_title") %></th>
-      <th><%= t("admin.budget_investments.index.table_supports") %></th>
-      <th><%= t("admin.budget_investments.index.table_admin") %></th>
-      <th><%= t("admin.budget_investments.index.table_valuator") %></th>
-      <th><%= t("admin.budget_investments.index.table_geozone") %></th>
-      <th><%= t("admin.budget_investments.index.table_feasibility") %></th>
-      <th class="text-center"><%= t("admin.budget_investments.index.table_valuation_finished") %></th>
-      <th class="text-center"><%= t("admin.budget_investments.index.table_selection") %></th>
-      <% if params[:filter] == 'selected' %>
-        <th class="text-center"><%= t("admin.budget_investments.index.table_incompatible") %></th>
-      <% end %>
-    </tr>
-  </thead>
+<% if @investments.any? %>
+  <h3><%= page_entries_info @investments %></h3>
+  <table>
+    <thead>
+      <tr>
+        <th><%= t("admin.budget_investments.index.table_id") %></th>
+        <th><%= t("admin.budget_investments.index.table_title") %></th>
+        <th><%= t("admin.budget_investments.index.table_supports") %></th>
+        <th><%= t("admin.budget_investments.index.table_admin") %></th>
+        <th><%= t("admin.budget_investments.index.table_valuator") %></th>
+        <th><%= t("admin.budget_investments.index.table_geozone") %></th>
+        <th><%= t("admin.budget_investments.index.table_feasibility") %></th>
+        <th class="text-center"><%= t("admin.budget_investments.index.table_valuation_finished") %></th>
+        <th class="text-center"><%= t("admin.budget_investments.index.table_selection") %></th>
+        <% if params[:filter] == 'selected' %>
+          <th class="text-center"><%= t("admin.budget_investments.index.table_incompatible") %></th>
+        <% end %>
+      </tr>
+    </thead>
 
-  <% @investments.each do |investment| %>
-    <tr id="<%= dom_id(investment) %>" class="budget_investment">
-      <td class="text-right">
-        <strong><%= investment.id %></strong>
-      </td>
-      <td>
-        <%= link_to investment.title, admin_budget_budget_investment_path(budget_id: @budget.id, id: investment.id, params: Budget::Investment.filter_params(params)) %>
-      </td>
-      <td class="text-center">
-        <%= investment.total_votes %>
-      </td>
-      <td class="small">
-        <% if investment.administrator.present? %>
-          <span title="<%= t('admin.budget_investments.index.assigned_admin') %>"><%= investment.administrator.name %></span>
-        <% else %>
-          <%= t("admin.budget_investments.index.no_admin_assigned") %>
-        <% end %>
-      </td>
-      <td class="small">
-        <% if investment.valuators.size == 0 %>
-          <%= t("admin.budget_investments.index.no_valuators_assigned") %>
-        <% else %>
-          <%= investment.valuators.collect(&:description_or_name).join(', ') %>
-        <% end %>
-      </td>
-      <td class="small">
-        <%= investment.heading.name %>
-      </td>
-      <td class="small">
-        <%= t("admin.budget_investments.index.feasibility.#{investment.feasibility}",
-              price: investment.formatted_price)
-        %>
-      </td>
-      <td class="small text-center">
-        <%= investment.valuation_finished? ? t('shared.yes'): t('shared.no') %>
-      </td>
-      <td class="small">
-        <%= investment_selected_link(investment) %>
-      </td>
-      <% if params[:filter] == 'selected' %>
-        <td class="small text-center">
-          <%= investment.incompatible? ? t('shared.yes'): t('shared.no') %>
+    <% @investments.each do |investment| %>
+      <tr id="<%= dom_id(investment) %>" class="budget_investment">
+        <td class="text-right">
+          <strong><%= investment.id %></strong>
         </td>
-      <% end %>
-    </tr>
-  <% end %>
-</table>
+        <td>
+          <%= link_to investment.title, admin_budget_budget_investment_path(budget_id: @budget.id, id: investment.id, params: Budget::Investment.filter_params(params)) %>
+        </td>
+        <td class="text-center">
+          <%= investment.total_votes %>
+        </td>
+        <td class="small">
+          <% if investment.administrator.present? %>
+            <span title="<%= t('admin.budget_investments.index.assigned_admin') %>"><%= investment.administrator.name %></span>
+          <% else %>
+            <%= t("admin.budget_investments.index.no_admin_assigned") %>
+          <% end %>
+        </td>
+        <td class="small">
+          <% if investment.valuators.size == 0 %>
+            <%= t("admin.budget_investments.index.no_valuators_assigned") %>
+          <% else %>
+            <%= investment.valuators.collect(&:description_or_name).join(', ') %>
+          <% end %>
+        </td>
+        <td class="small">
+          <%= investment.heading.name %>
+        </td>
+        <td class="small">
+          <%= t("admin.budget_investments.index.feasibility.#{investment.feasibility}",
+                price: investment.formatted_price)
+          %>
+        </td>
+        <td class="small text-center">
+          <%= investment.valuation_finished? ? t('shared.yes'): t('shared.no') %>
+        </td>
+        <td class="small">
+          <%= investment_selected_link(investment) %>
+        </td>
+        <% if params[:filter] == 'selected' %>
+          <td class="small text-center">
+            <%= investment.incompatible? ? t('shared.yes'): t('shared.no') %>
+          </td>
+        <% end %>
+      </tr>
+    <% end %>
+  </table>
 
-<%= paginate @investments %>
+  <%= paginate @investments %>
+<% else %>
+  <div class="callout primary">
+    <%= t("admin.budget_investments.index.no_budget_investments") %>
+  </div>
+<% end %>
diff --git a/config/locales/en/admin.yml b/config/locales/en/admin.yml
index 9601042f1..5e7123335 100644
--- a/config/locales/en/admin.yml
+++ b/config/locales/en/admin.yml
@@ -129,6 +129,7 @@ en:
           selected: Selected
           winners: Winners
           all: All
+        no_budget_investments: "There are no investment projects."
         title: Investment projects
         assigned_admin: Assigned administrator
         no_admin_assigned: No admin assigned
diff --git a/config/locales/es/admin.yml b/config/locales/es/admin.yml
index 54802b121..44d527ff4 100644
--- a/config/locales/es/admin.yml
+++ b/config/locales/es/admin.yml
@@ -130,6 +130,7 @@ es:
           selected: Seleccionadas
           winners: Ganadoras
           all: Todas
+        no_budget_investments: "No hay proyectos de inversión."
         title: Proyectos de inversión
         assigned_admin: Administrador asignado
         no_admin_assigned: Sin admin asignado