From f7dce94a451608c0dea70a766b065d63ce1f0acd Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Mar=C3=ADa=20Checa?= <MariaCheca@users.noreply.github.com>
Date: Mon, 23 Oct 2017 15:50:53 +0200
Subject: [PATCH 01/13] Added tests for Poll results and stats views

---
 spec/features/polls/polls_spec.rb | 71 ++++++++++++++++++++++++++++---
 1 file changed, 64 insertions(+), 7 deletions(-)

diff --git a/spec/features/polls/polls_spec.rb b/spec/features/polls/polls_spec.rb
index 7bf51473f..424787468 100644
--- a/spec/features/polls/polls_spec.rb
+++ b/spec/features/polls/polls_spec.rb
@@ -367,17 +367,74 @@ feature 'Polls' do
     end
 
   end
-  
+
   context "Results and stats" do
-    scenario "See polls statistics", :js do
+    scenario "Show poll results and stats if enabled and poll expired" do
+      poll = create(:poll, :expired, results_enabled: true, stats_enabled: true)
       user = create(:user)
-      poll = create(:poll, summary: "Summary", description: "Description")
+
       login_as user
       visit poll_path(poll)
-      
-      click_link "Participation statistics"
-      
-      expect(page).to have_content("Total participation")      
+
+      expect(page).to have_content("Poll results")
+      expect(page).to have_content("Participation statistics")
+
+      visit poll_results_path(poll)
+      expect(page.driver.status_code).to eq(200)
+
+      visit poll_stats_path(poll)
+      expect(page.driver.status_code).to eq(200)
+    end
+
+    scenario "Don't show poll results and stats if not enabled" do
+      poll = create(:poll, :expired, results_enabled: false, stats_enabled: false)
+      user = create(:user)
+
+      login_as user
+      visit poll_path(poll)
+
+      expect(page).to_not have_content("Poll results")
+      expect(page).to_not have_content("Participation statistics")
+
+      visit poll_results_path(poll)
+      expect(page).to have_content("You do not have permission to carry out the action 'results' on poll.")
+
+      visit poll_stats_path(poll)
+      expect(page).to have_content("You do not have permission to carry out the action 'stats' on poll.")
+    end
+
+    scenario "Don't show poll results and stats if is not expired" do
+      poll = create(:poll, :current, results_enabled: true, stats_enabled: true)
+      user = create(:user)
+
+      login_as user
+      visit poll_path(poll)
+
+      expect(page).to_not have_content("Poll results")
+      expect(page).to_not have_content("Participation statistics")
+
+      visit poll_results_path(poll)
+      expect(page).to have_content("You do not have permission to carry out the action 'results' on poll.")
+
+      visit poll_stats_path(poll)
+      expect(page).to have_content("You do not have permission to carry out the action 'stats' on poll.")
+    end
+
+    scenario "Show poll results and stats if user is administrator" do
+      poll = create(:poll, :current, results_enabled: false, stats_enabled: false)
+      user = create(:administrator).user
+
+      login_as user
+      visit poll_path(poll)
+
+      expect(page).to have_content("Poll results")
+      expect(page).to have_content("Participation statistics")
+
+      visit poll_results_path(poll)
+      expect(page.driver.status_code).to eq(200)
+
+      visit poll_stats_path(poll)
+      expect(page.driver.status_code).to eq(200)
     end
   end
 end

From da00ac947e38e926b26aacd4e8909ccf57f042de Mon Sep 17 00:00:00 2001
From: decabeza <alberto@decabeza.es>
Date: Mon, 23 Oct 2017 19:56:26 +0200
Subject: [PATCH 02/13] replaces link on poll title if results or stats are
 enabled

---
 app/views/polls/_poll_group.html.erb | 20 ++++++++++++++++++--
 1 file changed, 18 insertions(+), 2 deletions(-)

diff --git a/app/views/polls/_poll_group.html.erb b/app/views/polls/_poll_group.html.erb
index 529e9a99e..9d83d5233 100644
--- a/app/views/polls/_poll_group.html.erb
+++ b/app/views/polls/_poll_group.html.erb
@@ -19,11 +19,27 @@
         <div class="dates"></div>
         <% if poll.questions.count == 1 %>
           <% poll.questions.each do |question| %>
-            <h4><%= link_to question.title, poll %></h4>
+            <h4>
+              <% if poll.results_enabled? %>
+                <%= link_to question.title, poll_results_path(poll) %>
+              <% elsif poll.stats_enabled? %>
+                <%= link_to question.title, poll_stats_path(poll) %>
+              <% else %>
+                <%= link_to question.title, poll %>
+              <% end %>
+            </h4>
             <%= poll_dates(poll) %>
           <% end %>
         <% else %>
-          <h4><%= link_to poll.name, poll %></h4>
+          <h4>
+            <% if poll.results_enabled? %>
+              <%= link_to poll.name, poll_results_path(poll) %>
+            <% elsif poll.stats_enabled? %>
+                <%= link_to poll.name, poll_stats_path(poll) %>
+            <% else %>
+              <%= link_to poll.name, poll %>
+            <% end %>
+          </h4>
           <%= poll_dates(poll) %>
           <ul class="margin-top">
             <% poll.questions.each do |question| %>

From e11957a91cb070e6eb86d4202135470a3c5221bc Mon Sep 17 00:00:00 2001
From: decabeza <alberto@decabeza.es>
Date: Mon, 23 Oct 2017 19:56:39 +0200
Subject: [PATCH 03/13] adds specs for poll title link

---
 spec/features/polls/polls_spec.rb | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

diff --git a/spec/features/polls/polls_spec.rb b/spec/features/polls/polls_spec.rb
index 424787468..8fe23d2ea 100644
--- a/spec/features/polls/polls_spec.rb
+++ b/spec/features/polls/polls_spec.rb
@@ -59,6 +59,22 @@ feature 'Polls' do
       expect(page).to have_link('Incoming')
       expect(page).to_not have_link('Expired')
     end
+
+    scenario "Poll title link to stats if enabled" do
+      poll = create(:poll, name: "Poll with stats", stats_enabled: true)
+
+      visit polls_path
+
+      expect(page).to have_link("Poll with stats", href: poll_stats_path(poll))
+    end
+
+    scenario "Poll title link to results if enabled" do
+      poll = create(:poll, name: "Poll with results", stats_enabled: true, results_enabled: true)
+
+      visit polls_path
+
+      expect(page).to have_link("Poll with results", href: poll_results_path(poll))
+    end
   end
 
   context 'Show' do

From 81e4978747190b27ed8668d95c2f428477e2c42b Mon Sep 17 00:00:00 2001
From: decabeza <alberto@decabeza.es>
Date: Mon, 23 Oct 2017 19:56:49 +0200
Subject: [PATCH 04/13] removes blank space

---
 app/views/polls/_poll_subnav.html.erb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/views/polls/_poll_subnav.html.erb b/app/views/polls/_poll_subnav.html.erb
index 67325fa3a..76d9b4508 100644
--- a/app/views/polls/_poll_subnav.html.erb
+++ b/app/views/polls/_poll_subnav.html.erb
@@ -1,4 +1,4 @@
-<% if current_user && current_user.administrator? || 
+<% if current_user && current_user.administrator? ||
   (@poll.expired? && (@poll.results_enabled? || @poll.stats_enabled?)) %>
   <div class="row margin-top">
     <div class="small-12 column">

From aa7dbaa9c497d02446bf11874ad1be6837bd45c6 Mon Sep 17 00:00:00 2001
From: decabeza <alberto@decabeza.es>
Date: Mon, 23 Oct 2017 20:13:11 +0200
Subject: [PATCH 05/13] replace eq200 to text content on specs

---
 spec/features/polls/polls_spec.rb | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/spec/features/polls/polls_spec.rb b/spec/features/polls/polls_spec.rb
index 8fe23d2ea..82180cb23 100644
--- a/spec/features/polls/polls_spec.rb
+++ b/spec/features/polls/polls_spec.rb
@@ -396,10 +396,10 @@ feature 'Polls' do
       expect(page).to have_content("Participation statistics")
 
       visit poll_results_path(poll)
-      expect(page.driver.status_code).to eq(200)
+      expect(page).to have_content("Questions")
 
       visit poll_stats_path(poll)
-      expect(page.driver.status_code).to eq(200)
+      expect(page).to have_content("Participation data")
     end
 
     scenario "Don't show poll results and stats if not enabled" do
@@ -447,10 +447,10 @@ feature 'Polls' do
       expect(page).to have_content("Participation statistics")
 
       visit poll_results_path(poll)
-      expect(page.driver.status_code).to eq(200)
+      expect(page).to have_content("Questions")
 
       visit poll_stats_path(poll)
-      expect(page.driver.status_code).to eq(200)
+      expect(page).to have_content("Participation data")
     end
   end
 end

From 2081114628f8558aafb12870e8cefb518da32fee Mon Sep 17 00:00:00 2001
From: decabeza <alberto@decabeza.es>
Date: Mon, 23 Oct 2017 20:14:21 +0200
Subject: [PATCH 06/13] adds missing i18n

---
 app/views/polls/results.html.erb | 2 +-
 config/locales/en/general.yml    | 1 +
 config/locales/es/general.yml    | 1 +
 3 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/app/views/polls/results.html.erb b/app/views/polls/results.html.erb
index 07ca75bc3..5f2b973bd 100644
--- a/app/views/polls/results.html.erb
+++ b/app/views/polls/results.html.erb
@@ -7,7 +7,7 @@
 
   <div class="row margin">
     <div class="small-12 medium-3 column sidebar">
-      <h4>Preguntas</h4>
+      <h4><%= t("polls.show.results.title") %></h4>
       <ul class="menu vertical">
         <%- @poll.questions.each do |question| %>
           <li><%=link_to question.title, "##{question.title.parameterize}" %></li>
diff --git a/config/locales/en/general.yml b/config/locales/en/general.yml
index 586a90a45..926410b51 100644
--- a/config/locales/en/general.yml
+++ b/config/locales/en/general.yml
@@ -514,6 +514,7 @@ en:
         white: "White votes"
         null_votes: "Invalid"
       results:
+        title: "Questions"
         most_voted_answer: "Most voted answer: "
   poll_questions:
     create_question: "Create question"
diff --git a/config/locales/es/general.yml b/config/locales/es/general.yml
index 08fe25e1e..f82ba9ed7 100644
--- a/config/locales/es/general.yml
+++ b/config/locales/es/general.yml
@@ -514,6 +514,7 @@ es:
         white: "En blanco"
         null_votes: "Nulos"
       results:
+        title: "Preguntas"
         most_voted_answer: "Respuesta más votada: "
   poll_questions:
     create_question: "Crear pregunta para votación"

From 39a0d814274ebcc823f111950c6f1912b3e61ae3 Mon Sep 17 00:00:00 2001
From: Bertocq <bertocq@gmail.com>
Date: Mon, 23 Oct 2017 20:44:27 +0200
Subject: [PATCH 07/13] Fix current_user access when not anonymous user visits

---
 app/views/polls/_poll_subnav.html.erb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/app/views/polls/_poll_subnav.html.erb b/app/views/polls/_poll_subnav.html.erb
index 76d9b4508..779c5c8d5 100644
--- a/app/views/polls/_poll_subnav.html.erb
+++ b/app/views/polls/_poll_subnav.html.erb
@@ -3,7 +3,7 @@
   <div class="row margin-top">
     <div class="small-12 column">
       <ul class="menu simple clear">
-        <% if current_user.administrator? || @poll.results_enabled? %>
+        <% if current_user && current_user.administrator? || @poll.results_enabled? %>
           <% if controller_name == "polls" && action_name == "results" %>
             <li class="active">
               <h2><%= t("polls.show.results_menu") %></h2>
@@ -13,7 +13,7 @@
           <% end %>
         <% end %>
 
-        <% if current_user.administrator? || @poll.stats_enabled? %>
+        <% if current_user && current_user.administrator? || @poll.stats_enabled? %>
           <% if controller_name == "polls" && action_name == "stats" %>
             <li class="active">
               <h2><%= t("polls.show.stats_menu") %></h2>

From 608ca48b32bfc3648f4e18f9cf87e99d09d8428f Mon Sep 17 00:00:00 2001
From: Bertocq <bertocq@gmail.com>
Date: Mon, 23 Oct 2017 21:02:23 +0200
Subject: [PATCH 08/13] Add Recounting and Expired with stats&results polls to
 dev seeds

---
 db/dev_seeds.rb | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

diff --git a/db/dev_seeds.rb b/db/dev_seeds.rb
index 8c09923d8..ba6ad3962 100644
--- a/db/dev_seeds.rb
+++ b/db/dev_seeds.rb
@@ -546,6 +546,13 @@ poll = Poll.create(name: "Upcoming Poll",
                    starts_at: 1.month.from_now,
                    ends_at:   2.months.from_now)
 
+puts " ✅"
+print "Recounting Poll"
+poll = Poll.create(name: "Recounting Poll",
+                   # slug: "recounting-poll",
+                   starts_at: 1.months.ago,
+                   ends_at:   5.days.ago)
+
 puts " ✅"
 print "Expired Poll"
 poll = Poll.create(name: "Expired Poll",
@@ -553,6 +560,15 @@ poll = Poll.create(name: "Expired Poll",
                    starts_at: 2.months.ago,
                    ends_at:   1.month.ago)
 
+puts " ✅"
+print "Expired Poll with Stats & Results"
+poll = Poll.create(name: "Expired Poll with Stats & Results",
+                   # slug: "expired-poll-with-stats-and-results",
+                   starts_at: 2.months.ago,
+                   ends_at:   1.month.ago,
+                   results_enabled: true,
+                   stats_enabled: true)
+
 puts " ✅"
 print "Creating Poll Questions"
 

From ca63e4998f2e5b71f968cde76d62a8d2bddffab2 Mon Sep 17 00:00:00 2001
From: rgarcia <voodoorai2000@gmail.com>
Date: Mon, 23 Oct 2017 21:34:13 +0200
Subject: [PATCH 09/13] adds permissions for poll's results and stats

---
 app/models/abilities/everyone.rb | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/app/models/abilities/everyone.rb b/app/models/abilities/everyone.rb
index 32f948a5a..dd692c269 100644
--- a/app/models/abilities/everyone.rb
+++ b/app/models/abilities/everyone.rb
@@ -7,8 +7,12 @@ module Abilities
       can [:read, :map, :summary, :share], Proposal
       can :read, Comment
       can :read, Poll
-      cannot :results, Poll, results_enabled: false
-      cannot :stats, Poll, stats_enabled: false
+      can :results, Poll do |poll|
+        poll.expired? && poll.results_enabled?
+      end
+      can :stats, Poll do |poll|
+        poll.expired? && poll.stats_enabled?
+      end
       can :read, Poll::Question
       can [:read, :welcome], Budget
       can :read, SpendingProposal

From 21c7ff15fed56543026b1df363b3416eb0314aa8 Mon Sep 17 00:00:00 2001
From: rgarcia <voodoorai2000@gmail.com>
Date: Mon, 23 Oct 2017 21:34:28 +0200
Subject: [PATCH 10/13] fixes routes for poll's results and stats

---
 app/views/polls/_poll_group.html.erb  |  8 ++++----
 app/views/polls/_poll_subnav.html.erb |  4 ++--
 config/routes.rb                      |  6 ++++--
 spec/features/polls/polls_spec.rb     | 20 ++++++++++----------
 4 files changed, 20 insertions(+), 18 deletions(-)

diff --git a/app/views/polls/_poll_group.html.erb b/app/views/polls/_poll_group.html.erb
index 9d83d5233..93174e157 100644
--- a/app/views/polls/_poll_group.html.erb
+++ b/app/views/polls/_poll_group.html.erb
@@ -21,9 +21,9 @@
           <% poll.questions.each do |question| %>
             <h4>
               <% if poll.results_enabled? %>
-                <%= link_to question.title, poll_results_path(poll) %>
+                <%= link_to question.title, results_poll_path(poll) %>
               <% elsif poll.stats_enabled? %>
-                <%= link_to question.title, poll_stats_path(poll) %>
+                <%= link_to question.title, stats_poll_path(poll) %>
               <% else %>
                 <%= link_to question.title, poll %>
               <% end %>
@@ -33,9 +33,9 @@
         <% else %>
           <h4>
             <% if poll.results_enabled? %>
-              <%= link_to poll.name, poll_results_path(poll) %>
+              <%= link_to poll.name, results_poll_path(poll) %>
             <% elsif poll.stats_enabled? %>
-                <%= link_to poll.name, poll_stats_path(poll) %>
+                <%= link_to poll.name, stats_poll_path(poll) %>
             <% else %>
               <%= link_to poll.name, poll %>
             <% end %>
diff --git a/app/views/polls/_poll_subnav.html.erb b/app/views/polls/_poll_subnav.html.erb
index 779c5c8d5..112f267ee 100644
--- a/app/views/polls/_poll_subnav.html.erb
+++ b/app/views/polls/_poll_subnav.html.erb
@@ -9,7 +9,7 @@
               <h2><%= t("polls.show.results_menu") %></h2>
             </li>
           <% else %>
-            <li><%= link_to t("polls.show.results_menu"), poll_results_path(@poll) %></li>
+            <li><%= link_to t("polls.show.results_menu"), results_poll_path(@poll) %></li>
           <% end %>
         <% end %>
 
@@ -19,7 +19,7 @@
               <h2><%= t("polls.show.stats_menu") %></h2>
             </li>
           <% else %>
-            <li><%= link_to t("polls.show.stats_menu"), poll_stats_path(@poll) %></li>
+            <li><%= link_to t("polls.show.stats_menu"), stats_poll_path(@poll) %></li>
           <% end %>
         <% end %>
 
diff --git a/config/routes.rb b/config/routes.rb
index fe38eb9b4..5d54a7c3b 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -113,8 +113,10 @@ Rails.application.routes.draw do
   end
 
   resources :polls, only: [:show, :index] do
-    get :stats
-    get :results
+    member do
+      get :stats
+      get :results
+    end
     resources :questions, controller: 'polls/questions', shallow: true do
       post :answer, on: :member
     end
diff --git a/spec/features/polls/polls_spec.rb b/spec/features/polls/polls_spec.rb
index 82180cb23..8629a1087 100644
--- a/spec/features/polls/polls_spec.rb
+++ b/spec/features/polls/polls_spec.rb
@@ -65,7 +65,7 @@ feature 'Polls' do
 
       visit polls_path
 
-      expect(page).to have_link("Poll with stats", href: poll_stats_path(poll))
+      expect(page).to have_link("Poll with stats", href: stats_poll_path(poll))
     end
 
     scenario "Poll title link to results if enabled" do
@@ -73,7 +73,7 @@ feature 'Polls' do
 
       visit polls_path
 
-      expect(page).to have_link("Poll with results", href: poll_results_path(poll))
+      expect(page).to have_link("Poll with results", href: results_poll_path(poll))
     end
   end
 
@@ -395,10 +395,10 @@ feature 'Polls' do
       expect(page).to have_content("Poll results")
       expect(page).to have_content("Participation statistics")
 
-      visit poll_results_path(poll)
+      visit results_poll_path(poll)
       expect(page).to have_content("Questions")
 
-      visit poll_stats_path(poll)
+      visit stats_poll_path(poll)
       expect(page).to have_content("Participation data")
     end
 
@@ -412,10 +412,10 @@ feature 'Polls' do
       expect(page).to_not have_content("Poll results")
       expect(page).to_not have_content("Participation statistics")
 
-      visit poll_results_path(poll)
+      visit results_poll_path(poll)
       expect(page).to have_content("You do not have permission to carry out the action 'results' on poll.")
 
-      visit poll_stats_path(poll)
+      visit stats_poll_path(poll)
       expect(page).to have_content("You do not have permission to carry out the action 'stats' on poll.")
     end
 
@@ -429,10 +429,10 @@ feature 'Polls' do
       expect(page).to_not have_content("Poll results")
       expect(page).to_not have_content("Participation statistics")
 
-      visit poll_results_path(poll)
+      visit results_poll_path(poll)
       expect(page).to have_content("You do not have permission to carry out the action 'results' on poll.")
 
-      visit poll_stats_path(poll)
+      visit stats_poll_path(poll)
       expect(page).to have_content("You do not have permission to carry out the action 'stats' on poll.")
     end
 
@@ -446,10 +446,10 @@ feature 'Polls' do
       expect(page).to have_content("Poll results")
       expect(page).to have_content("Participation statistics")
 
-      visit poll_results_path(poll)
+      visit results_poll_path(poll)
       expect(page).to have_content("Questions")
 
-      visit poll_stats_path(poll)
+      visit stats_poll_path(poll)
       expect(page).to have_content("Participation data")
     end
   end

From b8dd93748f6a4c50e6b54978144e4aac47bfa3e9 Mon Sep 17 00:00:00 2001
From: rgarcia <voodoorai2000@gmail.com>
Date: Mon, 23 Oct 2017 21:34:49 +0200
Subject: [PATCH 11/13] removes unnecessary code

---
 app/controllers/polls_controller.rb | 8 --------
 1 file changed, 8 deletions(-)

diff --git a/app/controllers/polls_controller.rb b/app/controllers/polls_controller.rb
index aa93581cc..708f68abb 100644
--- a/app/controllers/polls_controller.rb
+++ b/app/controllers/polls_controller.rb
@@ -3,8 +3,6 @@ class PollsController < ApplicationController
 
   load_and_authorize_resource
 
-  before_filter :set_poll, only: [:stats, :results]
-
   has_filters %w{current expired incoming}
   has_orders %w{most_voted newest oldest}, only: :show
 
@@ -36,10 +34,4 @@ class PollsController < ApplicationController
   def results
   end
 
-  private
-
-  def set_poll
-    @poll = Poll.find(params[:poll_id])
-  end
-
 end

From 1259d473c6bded544fd66dabf26d7fb6b0cc625e Mon Sep 17 00:00:00 2001
From: rgarcia <voodoorai2000@gmail.com>
Date: Mon, 23 Oct 2017 22:13:53 +0200
Subject: [PATCH 12/13] fixes spec

---
 spec/features/polls/results_spec.rb | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/spec/features/polls/results_spec.rb b/spec/features/polls/results_spec.rb
index b17fb463a..326344dd9 100644
--- a/spec/features/polls/results_spec.rb
+++ b/spec/features/polls/results_spec.rb
@@ -6,7 +6,7 @@ feature 'Poll Results' do
     user2 = create(:user, :level_two)
     user3 = create(:user, :level_two)
 
-    poll = create(:poll)
+    poll = create(:poll, results_enabled: true)
     question1 = create(:poll_question, poll: poll)
     answer1 = create(:poll_question_answer, question: question1, title: 'Yes')
     answer2 = create(:poll_question_answer, question: question1, title: 'No')
@@ -34,7 +34,9 @@ feature 'Poll Results' do
     expect(Poll::Voter.count).to eq(3)
     logout
 
-    visit poll_results_path(poll)
+    poll.update(ends_at: 1.day.ago)
+
+    visit results_poll_path(poll)
 
     expect(page).to have_content(question1.title)
     expect(page).to have_content(question2.title)

From f548f0a1dbc9f430c6a9cd0935233a7a100ce947 Mon Sep 17 00:00:00 2001
From: Bertocq <bertocq@gmail.com>
Date: Mon, 23 Oct 2017 22:25:36 +0200
Subject: [PATCH 13/13] Fix voting_polls variable typo on shift creation

---
 app/views/admin/poll/shifts/_form.html.erb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/views/admin/poll/shifts/_form.html.erb b/app/views/admin/poll/shifts/_form.html.erb
index 230864218..7e2930751 100644
--- a/app/views/admin/poll/shifts/_form.html.erb
+++ b/app/views/admin/poll/shifts/_form.html.erb
@@ -24,7 +24,7 @@
     <div class="small-12 medium-3 column">
       <label><%= t("admin.poll_shifts.new.date") %></label>
       <%= select 'shift[date]', 'vote_collection_date',
-                  options_for_select(shift_vote_collection_dates(@booth, @vote_collection_polls)),
+                  options_for_select(shift_vote_collection_dates(@booth, @voting_polls)),
                   { prompt: @voting_polls.present? ? t("admin.poll_shifts.new.select_date") : t("admin.poll_shifts.new.no_voting_days"),
                   label: false },
                   class: 'js-shift-vote-collection-dates' %>