From 1cbd1f9324489c7cf2016ec7a6a37ca7dbf943dd Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Juanjo=20Baz=C3=A1n?= <jjbazan@gmail.com>
Date: Tue, 8 Mar 2016 13:33:03 +0100
Subject: [PATCH] moves assigned valuators info to helper

---
 app/helpers/valuation_helper.rb                   | 15 +++++++++++++++
 .../valuation/spending_proposals/index.html.erb   | 13 +------------
 2 files changed, 16 insertions(+), 12 deletions(-)

diff --git a/app/helpers/valuation_helper.rb b/app/helpers/valuation_helper.rb
index becd29abb..cd70a6732 100644
--- a/app/helpers/valuation_helper.rb
+++ b/app/helpers/valuation_helper.rb
@@ -8,4 +8,19 @@ module ValuationHelper
     end
   end
 
+  def assigned_valuators_info(valuators)
+    case valuators.size
+    when 0
+      t("valuation.spending_proposals.index.no_valuators_assigned")
+    when 1
+      "<span title=\"#{t('valuation.spending_proposals.index.valuators_assigned', count: 1)}\">".html_safe +
+        valuators.first.name +
+      "</span>".html_safe
+    else
+      "<span title=\"".html_safe + valuators.map(&:name).join(', ') + "\">".html_safe +
+        t('valuation.spending_proposals.index.valuators_assigned', count: valuators.size) +
+      "</span>".html_safe
+    end
+  end
+
 end
\ No newline at end of file
diff --git a/app/views/valuation/spending_proposals/index.html.erb b/app/views/valuation/spending_proposals/index.html.erb
index b47348e09..e6c05d6c7 100644
--- a/app/views/valuation/spending_proposals/index.html.erb
+++ b/app/views/valuation/spending_proposals/index.html.erb
@@ -29,18 +29,7 @@
         <%= link_to t("valuation.spending_proposals.index.edit"), edit_valuation_spending_proposal_path(spending_proposal) %>
       </td>
       <td class="small">
-        <% case spending_proposal.valuators.size %>
-        <% when 0 %>
-          <%= t("valuation.spending_proposals.index.no_valuators_assigned") %>
-        <% when 1 %>
-          <span title="<%= t('valuation.spending_proposals.index.valuators_assigned', count: 1) %>">
-            <%= spending_proposal.valuators.first.name %>
-          </span>
-        <% else %>
-          <span title="<%= spending_proposal.valuators.map(&:name).join(', ') %>">
-            <%= t('valuation.spending_proposals.index.valuators_assigned', count: spending_proposal.valuators.size) %>
-          </span>
-        <% end %>
+        <%= assigned_valuators_info(spending_proposal.valuators) %>
       </td>
       <td class="small">
         <%= geozone_name(spending_proposal) %>