Merge branch 'fix/alert-before-ending-a-volunteer' into 'develop'

Alert before ending a volunteer

See merge request !618

app/views/application/_index_actions.html.slim

@@ -2,6 +2,6 @@ td.index-action-cell.hidden-print
   = button_link navigation_glyph(:show), subject, title: 'Anzeigen'
   - if policy(subject).edit?
     = button_link navigation_glyph(:edit), edit_polymorphic_path(subject), title: 'Bearbeiten'
-  - if policy(subject.class).destroy? && controller_name != 'clients'
+  - if policy(subject.class).destroy? && !['clients', 'volunteers'].include?(controller_name)
     = button_link navigation_glyph(:delete), subject, nil,
         confirm_deleting(subject, 'btn btn-default').merge(title: 'Löschen')

app/views/volunteers/_buttons.html.slim

@@ -7,8 +7,9 @@ nav.navbar.section-navigation
     - if policy(@volunteer).terminate?
       li.button-acceptance = link_to t("volunteer_applications.acceptance.#{@volunteer.acceptance}"), '#',
         class: "btn btn-acceptance-#{@volunteer.acceptance}"
-      li= link_to navigation_glyph(:terminate), terminate_volunteer_path(@volunteer), class: 'btn btn-default',
-        title: 'Beenden', method: :put unless @volunteer.resigned?
+      li= link_to navigation_glyph(:terminate), terminate_volunteer_path(@volunteer),
+        class: 'btn btn-default', title: 'Beenden', method: :put,
+        data: { confirm: t('messages.confirm_terminate_volunteer') } unless @volunteer.resigned?
     - if @volunteer.internal_and_started_assignments?
       li = button_link navigation_glyph(:hours), volunteer_hours_path(@volunteer), title: t('volunteer_applications.show.hour_reports')
     - if policy(Volunteer).superadmin_privileges? && @volunteer.seeking_clients?

config/locales/de.yml

@@ -164,6 +164,7 @@ de:
     confirm_delete: "Wirklich %{model} %{record} löschen?"
     confirm_record_delete: '%{model} wirklich löschen?'
     select_prompt: "%{model} wählen"
+    confirm_terminate_volunteer: "Wollen Sie den Freiwilligen wirklich beenden?"
   rejection_type: &id-rejection-types
     her: von Freiwillige/r vor / nach Erstgespräch
     other: Anderer Grund

test/system/terminate_volunteers_test.rb

@@ -51,7 +51,10 @@ class TerminateVolunteersTest < ApplicationSystemTestCase
 
   test 'Volunteer with active assignments can not be terminated' do
     visit volunteer_path(@volunteer_aa)
-    first(:link, 'Beenden').click
+
+    accept_confirm do
+      first(:link, 'Beenden').click
+    end
 
     assert page.has_text?
       'Beenden fehlgeschlagen. Freiwillige/r kann nicht beendet werden, solange noch laufende Einsätze existieren.'
@@ -60,7 +63,9 @@ class TerminateVolunteersTest < ApplicationSystemTestCase
     @active_assignment.update(period_end: 2.days.ago)
     @volunteer_aa.reload
     visit current_url
-    first(:link, 'Beenden').click
+    accept_confirm do
+      first(:link, 'Beenden').click
+    end
     assert page.has_text?
       'Beenden fehlgeschlagen. Freiwillige/r kann nicht beendet werden, solange noch laufende Einsätze existieren.'
     assert page.has_link? 'Begleitung bearbeiten'
@@ -68,7 +73,9 @@ class TerminateVolunteersTest < ApplicationSystemTestCase
     @active_group_assignment.update(period_end: 2.days.ago)
     @volunteer_aa.reload
     visit current_url
-    first(:link, 'Beenden').click
+    accept_confirm do
+      first(:link, 'Beenden').click
+    end
     assert page.has_text?
       'Beenden fehlgeschlagen. Freiwillige/r kann nicht beendet werden, solange noch laufende Einsätze existieren.'
     assert page.has_link? 'Begleitung bearbeiten'
@@ -76,7 +83,9 @@ class TerminateVolunteersTest < ApplicationSystemTestCase
 
   test 'Volunteer with ended but not submitted assignments can not be terminated' do
     visit volunteer_path(@volunteer_ua)
-    first(:link, 'Beenden').click
+    accept_confirm do
+      first(:link, 'Beenden').click
+    end
 
     assert page.has_text?
       'Beenden fehlgeschlagen. Freiwillige/r kann nicht beendet werden, solange noch laufende Einsätze existieren.'
@@ -85,7 +94,9 @@ class TerminateVolunteersTest < ApplicationSystemTestCase
       termination_submitted_by: @volunteer_sa.user, period_end_set_by: @superadmin)
     @volunteer_ua.reload
     visit current_url
-    first(:link, 'Beenden').click
+    accept_confirm do
+      first(:link, 'Beenden').click
+    end
     assert page.has_text?
       'Beenden fehlgeschlagen. Freiwillige/r kann nicht beendet werden, solange noch laufende Einsätze existieren.'
     assert page.has_link? 'Begleitung bearbeiten'
@@ -94,7 +105,9 @@ class TerminateVolunteersTest < ApplicationSystemTestCase
       termination_submitted_by: @volunteer_ua.user, period_end_set_by: @superadmin)
     @volunteer_ua.reload
     visit current_url
-    first(:link, 'Beenden').click
+    accept_confirm do
+      first(:link, 'Beenden').click
+    end
     assert page.has_text?
       'Beenden fehlgeschlagen. Freiwillige/r kann nicht beendet werden, solange noch laufende Einsätze existieren.'
     assert page.has_link? 'Begleitung bearbeiten'
@@ -102,7 +115,9 @@ class TerminateVolunteersTest < ApplicationSystemTestCase
 
   test 'Volunteer with ended, submitted but not verified assignments can not be terminated' do
     visit volunteer_path(@volunteer_sa)
-    first(:link, 'Beenden').click
+    accept_confirm do
+      first(:link, 'Beenden').click
+    end
 
     assert page.has_text?
       'Beenden fehlgeschlagen. Freiwillige/r kann nicht beendet werden, solange noch laufende Einsätze existieren.'
@@ -112,7 +127,9 @@ class TerminateVolunteersTest < ApplicationSystemTestCase
       termination_verified_at: 2.days.ago, termination_verified_by: @superadmin)
     @volunteer_sa.reload
     visit current_url
-    first(:link, 'Beenden').click
+    accept_confirm do
+      first(:link, 'Beenden').click
+    end
     assert page.has_text?
       'Beenden fehlgeschlagen. Freiwillige/r kann nicht beendet werden, solange noch laufende Einsätze existieren.'
     assert page.has_link? 'Begleitung bearbeiten'
@@ -122,7 +139,9 @@ class TerminateVolunteersTest < ApplicationSystemTestCase
       termination_verified_at: 2.days.ago, termination_verified_by: @superadmin)
     @volunteer_sa.reload
     visit current_url
-    first(:link, 'Beenden').click
+    accept_confirm do
+      first(:link, 'Beenden').click
+    end
     assert page.has_text? 'Freiwillige/r wurde erfolgreich beendet.'
   end
 end