Transform event filtered list to filter chain (hitobito/hitobito_sac_cas#2366)

Author: codez

.gitignore

@@ -20,6 +20,7 @@ Gemfile.lock
 # reports
 spec/coverage
 spec/reports
+coverage/
 brakeman-output.tabs
 rubocop-results.xml
 .tool-versions

app/controllers/pbs/event/lists_controller.rb

@@ -7,13 +7,9 @@ module Pbs::Event::ListsController
   extend ActiveSupport::Concern
 
   included do
-    # required to allow api calls
-    protect_from_forgery with: :null_session, only: [:bsv_export]
-
     skip_authorization_check only: :camps
     skip_authorize_resource only: [:camps, :all_camps, :kantonalverband_camps,
       :camps_in_canton, :camps_abroad]
-    alias_method_chain :render_bsv_export, :advanced
   end
 
   # simple redirect action acting as a generic entry point from the main navigation
@@ -67,15 +63,6 @@ def camps_abroad
 
   private
 
-  def render_bsv_export_with_advanced(courses_for_bsv_export)
-    if params[:advanced]
-      send_data(Export::Tabular::Events::AdvancedBsvList.csv(courses_for_bsv_export),
-        type: :csv, filename: "advanced_bsv_export.csv")
-    else
-      render_bsv_export_without_advanced(courses_for_bsv_export)
-    end
-  end
-
   def render_camp_list(camps)
     respond_to do |format|
       format.html { @camps = grouped(camps) }

app/controllers/pbs/events/courses_controller.rb

@@ -0,0 +1,24 @@
+#  Copyright (c) 2012-2015, Pfadibewegung Schweiz. This file is part of
+#  hitobito_pbs and licensed under the Affero General Public License version 3
+#  or later. See the COPYING file at the top-level directory or at
+#  https://github.com/hitobito/hitobito_pbs.
+
+module Pbs::Events::CoursesController
+  extend ActiveSupport::Concern
+
+  prepended do
+    # required to allow api calls
+    protect_from_forgery with: :null_session, only: [:bsv_export]
+  end
+
+  private
+
+  def render_bsv_export(courses_for_bsv_export)
+    if params[:advanced]
+      send_data(Export::Tabular::Events::AdvancedBsvList.csv(courses_for_bsv_export),
+        type: :csv, filename: "advanced_bsv_export.csv")
+    else
+      super(courses_for_bsv_export)
+    end
+  end
+end

app/domain/pbs/event/filter.rb

@@ -0,0 +1,16 @@
+#  Copyright (c) 2012-2018, Pfadibewegung Schweiz. This file is part of
+#  hitobito_pbs and licensed under the Affero General Public License version 3
+#  or later. See the COPYING file at the top-level directory or at
+#  https://github.com/hitobito/hitobito_pbs.
+
+module Pbs::Events::Filter::CourseList
+  private
+
+  def accessible_courses
+    if params[:list_all_courses] == true
+      Event::Course.all
+    else
+      Event::Course.in_hierarchy(user)
+    end
+  end
+end

app/domain/pbs/events/filter/course_list.rb

@@ -0,0 +1,26 @@
+#  Copyright (c) 2012-2018, Pfadibewegung Schweiz. This file is part of
+#  hitobito_pbs and licensed under the Affero General Public License version 3
+#  or later. See the COPYING file at the top-level directory or at
+#  https://github.com/hitobito/hitobito_pbs.
+
+module Pbs::Events::Filter::GroupList
+  private
+
+  def filter_with_selection
+    return super unless range == "canton"
+
+    super.where(canton: cantons)
+  end
+
+  def relevant_group_ids
+    return super unless range == "canton"
+
+    Group.select(:id)
+  end
+
+  def cantons
+    if group.respond_to?(:kantonalverband)
+      group.kantonalverband.cantons
+    end.to_a
+  end
+end

app/domain/pbs/events/filter/group_list.rb

@@ -1,6 +1,6 @@
-= action_button(t('event.lists.advanced_bsv_export_button'),
+= action_button(t('events/courses.advanced_bsv_export_button'),
                 nil,
                 :download,
-                title: t('event.lists.bsv_export_button'),
+                title: t('events/courses.bsv_export_button'),
                 data: { bs_toggle: 'popover',
                         bs_content: render('popover_bsv_export', advanced: true).to_str })

app/domain/pbs/events/filter/groups.rb

@@ -15,12 +15,9 @@ de:
       index:
         no_data_for_this_year: Für dieses Jahr sind keine Angaben vorhanden.
       infos:
-        census_evaluation:
-          Die Bestandesmeldungen für %{year} können ab dem %{start_at} erfasst werden.
-        census_evaluation_active:
-          Die aktuellen Bestandesmeldungen für %{year} können ab dem %{start_at} erfasst werden.
-        census_evaluation_reference_date:
-          Stichtag ist der %{finish_at}
+        census_evaluation: Die Bestandesmeldungen für %{year} können ab dem %{start_at} erfasst werden.
+        census_evaluation_active: Die aktuellen Bestandesmeldungen für %{year} können ab dem %{start_at} erfasst werden.
+        census_evaluation_reference_date: Stichtag ist der %{finish_at}
     abteilung:
       actions_index:
         create_population_data: Aktuellen Bestand erzeugen
@@ -38,7 +35,7 @@ de:
 
   crises:
     update:
-      too_old:  Die Krise ist zu alt um zu quittieren
+      too_old: Die Krise ist zu alt um zu quittieren
 
   devise:
     sessions:
@@ -95,17 +92,16 @@ de:
         camps_abroad: Lager im Ausland
       all_camps:
         title: Bevorstehende Lager in der ganzen Schweiz
-        explanation: 'Hier werden alle Lager der gesamten PBS angezeigt, welche aktuell oder in drei oder weniger Wochen durchgeführt werden angezeigt. Dabei werden nur Lager berücksichtigt, welche eingereicht wurden und die nicht abgesagt sind.'
+        explanation: "Hier werden alle Lager der gesamten PBS angezeigt, welche aktuell oder in drei oder weniger Wochen durchgeführt werden angezeigt. Dabei werden nur Lager berücksichtigt, welche eingereicht wurden und die nicht abgesagt sind."
       kantonalverband_camps:
         title: Alle Lager im Kantonalverband %{group}
-        explanation: 'Hier werden alle Lager, welche innerhalb des entsprechenden Kantonalverbandes organisiert werden angezeigt. Dabei werden nur Lager berücksichtigt, welche eingereicht wurden und deren Status bestätigt oder höher ist und die nicht abgesagt sind.'
+        explanation: "Hier werden alle Lager, welche innerhalb des entsprechenden Kantonalverbandes organisiert werden angezeigt. Dabei werden nur Lager berücksichtigt, welche eingereicht wurden und deren Status bestätigt oder höher ist und die nicht abgesagt sind."
       camps_in_canton:
         title: Alle Lager im Kanton %{canton}
-        explanation: 'Hier werden alle Lager, welche aktuell oder in drei oder weniger Wochen im entsprechenden Kanton stattfinden angezeigt. Dabei werden nur Lager berücksichtigt, welche eingereicht wurden und deren Status bestätigt oder höher ist und die nicht abgesagt sind.'
+        explanation: "Hier werden alle Lager, welche aktuell oder in drei oder weniger Wochen im entsprechenden Kanton stattfinden angezeigt. Dabei werden nur Lager berücksichtigt, welche eingereicht wurden und deren Status bestätigt oder höher ist und die nicht abgesagt sind."
       camps_abroad:
         title: Alle Lager im Ausland
-        explanation: 'Hier werden alle Lager, welche im Ausland stattfinden angezeigt. Dabei werden nur Lager berücksichtigt, welche eingereicht wurden und deren Status bestätigt oder höher ist und die nicht abgesagt sind.'
-      advanced_bsv_export_button: Erweiterter BSV/LKB Export
+        explanation: "Hier werden alle Lager, welche im Ausland stattfinden angezeigt. Dabei werden nur Lager berücksichtigt, welche eingereicht wurden und deren Status bestätigt oder höher ist und die nicht abgesagt sind."
     hierarchy: Lagerhierarchie
 
     participations:
@@ -159,15 +155,15 @@ de:
 
   event/camp:
     al_present_info:
-      'true': Im Lager anwesend
-      'false': Nicht im Lager anwesend
+      "true": Im Lager anwesend
+      "false": Nicht im Lager anwesend
     visiting_info:
       at: Besucht das Lager am %{date}
-      'true': Besucht das Lager
-      'false': Besucht das Lager nicht
+      "true": Besucht das Lager
+      "false": Besucht das Lager nicht
     coach_confirmed_info:
-      'true': Bestätigt
-      'false': Nicht bestätigt
+      "true": Bestätigt
+      "false": Nicht bestätigt
     checkpoint_field_caption: Kann nur von dem*der Lagerleiter*in bestätigt werden
 
   event/course:
@@ -198,8 +194,8 @@ de:
     create_camp_application:
       flash:
         success: Das Lager wurde an die PBS eingereicht. Du erhältst eine Kopie des E-Mails.
-        error: 'Das Lager konnte nicht eingereicht werden:'
-        preview: 'Das Lager kann noch nicht durch den*die Coach eingereicht werden:'
+        error: "Das Lager konnte nicht eingereicht werden:"
+        preview: "Das Lager kann noch nicht durch den*die Coach eingereicht werden:"
         preview_success: Es sind alle Informationen zum Einreichen des Lagers vorhanden.
     expected_participants_fields:
       expected_participants_help_inline: Hier wird die Anzahl TN und die Anzahl der Leiter*innen eingetragen. Sind diese Angaben noch nicht bekannt, wird die erwartete Anzahl angegeben.
@@ -262,8 +258,11 @@ de:
       vd_title: Beachte die Sonderregelung für Lager im Kanton Waadt («Hering» -> Kantonale Sonderregelungen)
       vd_url: http://hering.scouts.ch
 
+  events/courses:
+    advanced_bsv_export_button: Erweiterter BSV/LKB Export
+
   filter_navigation/events:
-    canton: 'durchgeführt im Kanton'
+    canton: "durchgeführt im Kanton"
 
   filter_navigation/camps:
     all: Alle untergeordneten Lager
@@ -276,7 +275,7 @@ de:
     pending_approvals:
       index:
         pending_title: Offene Freigaben
-        pending_info: 'Folgende Personen haben sich für einen Kurs angemeldet und müssen durch dich freigegeben werden:'
+        pending_info: "Folgende Personen haben sich für einen Kurs angemeldet und müssen durch dich freigegeben werden:"
         no_pending_info: Zurzeit sind keine offenen Freigaben vorhanden.
       notified_role_setting:
         title: Benachrichtigungen an

app/views/events/courses/_buttons_pbs.html.haml

@@ -54,8 +54,8 @@ class Wagon < Rails::Engine
       Bsv::Info.leader_roles += [Event::Role::Cook]
       Export::Pdf::Participation.runner = Pbs::Export::Pdf::Participation::Runner
       Event::ParticipantAssigner.include Pbs::Event::ParticipantAssigner
-      Event::Filter.include Pbs::Event::Filter
-      Events::Filter::Groups.include Pbs::Events::Filter::Groups
+      Events::Filter::GroupList.prepend Pbs::Events::Filter::GroupList
+      Events::Filter::CourseList.prepend Pbs::Events::Filter::CourseList
       Event::Qualifier::QualifyAction.include Pbs::Event::Qualifier::QualifyAction
       Export::Tabular::Events::List.include Pbs::Export::Tabular::Events::List
       Export::Tabular::Events::Row.include Pbs::Export::Tabular::Events::Row
@@ -143,6 +143,7 @@ class Wagon < Rails::Engine
       Event::ParticipationsController.include Pbs::Event::ParticipationsController
       Event::QualificationsController.include Pbs::Event::QualificationsController
       Event::RolesController.prepend Pbs::Event::RolesController
+      Events::CoursesController.prepend Pbs::Events::CoursesController
       QualificationsController.include Pbs::QualificationsController
       Person::QueryController.search_columns << :pbs_number
       SubscriptionsController.include Pbs::SubscriptionsController