ROLLEN: Anpassungen Rollen Ortsparteien

[mtnstar]

Dieses Ticket wurde auf Basis von https://help.puzzle.ch/#ticket/zoom/2781 erstellt.

Daraus resultiert die Anforderung das ein Kassier in Präsidium (Group::OrtPraesidium) Abos und damit auch Rechnungsbriefe in der übergeordneten Ebene (Layer Ort) verwalten kann.

Ausserdem sollen folgende Berechtigungen/Rolle angepasst werden:

• Gruppe Präsident in Präsidium: Alle Berechtigungen auf der Ebene Ortspartei
• Gruppe Präsidium Adressverwalter: Alles Lesen und Schreiben auf der Ebene Ortspartei
• Gruppe Präsidium Kassier: Rechnungen und Lesen und Schreiben auf der Ebene Ortspartei (Hier die Bemerkung: wenn jemand Rechnungen erstellen kann, muss die Person ja alle Personen sehen können und Abos erstellen können, sonst kann auch keine Rechung gemacht werden)
• Gruppe Sekretariat Leitung: Alle Berechtigungen auf der Ortspartei
• Gruppe Sekretariat Mitarbeiter: Passt so wie es eingestellt ist.

Offene Punkte:

• Ist es technisch überhaupt möglich über eine Rolle in einer Child Group Berechtigungen im darüberliegenden Layer zu erhalten?

Eine einfache Lösung wäre anstatt einer Untergruppe 'Praesidium' einfach Rollen wie Kassier, Leitung usw. direkt auf dem Layer Ort zu haben.

Tech-Spec

• https://github.com/hitobito/hitobito_die_mitte/blob/master/app/models/group/ort.rb

[olibrian]

Rücksprache mit dem Kunde:

Ist:

• Ortspartei
  [...]
  • Praesidium
    • PräsidentIn: [:layer_and_below_full, :contact_data, :finance]
    • VizepräsidentIn: []
    • KassierIn: [:finance]
    • Mitglied: []
    • AdressverwalterIn: [:layer_and_below_full]
    • Gast: []
  • Sekretariat
    • Leitung: [:layer_full, :contact_data, :finance]
    • MitarbeiterIn: [:layer_full]
    • WebadministratorIn: []

Soll:

• Ortspartei
  [...]
  • Praesidium
    • PräsidentIn: [:layer_and_below_full, :contact_data, :finance]
    • VizepräsidentIn: []
    • KassierIn: [:layer_and_below_full, :contact_data, :finance]
    • Mitglied: []
    • AdressverwalterIn: [:layer_and_below_full, :contact_data]
    • Gast: []
  • Sekretariat
    • Leitung: [:layer_and_below_full, :contact_data, :finance]
    • MitarbeiterIn: [:layer_and_below_full ]
    • WebadministratorIn: []

[carlobeltrame]

Da die Konzeption in diesem Ticket noch nicht komplett über alle Zweifel erhaben ist (an den Berechtigungen der Präsidenten wird gemäss Kommentar von @olibrian z.B. gar nichts geändert) möchte ich dazu einladen, nochmals über die Rollenstruktur und die speziell abgeänderte Layer-Personen-Liste der Mitte nachzudenken.

Ausschliesslich bei der Mitte zeigen wir auf dem Layer alle Personen an, die in Untergruppen des Layers Mitglied sind. Bei anderen Wagons ist das nur der Fall, wenn der spezielle Filter "Gesamte Ebene" gewählt wird. Auf dem Layer selber gibt es bei der Mitte gar keine Rollen. Das hat nun bereits mehr als eine Handvoll Mal zu Verwirrung bei den Nutzenden geführt:

• Issues Abos: Inkonsistente Abo Berechtigungen #161, MESSAGE: Berechtigung Briefe, Rechnungsbriefe und SMS anpassen #168
• Help-Tickets:
  • https://help.puzzle.ch/#ticket/zoom/2968
  • https://help.puzzle.ch/#ticket/zoom/2959
  • https://help.puzzle.ch/#ticket/zoom/2957
  • https://help.puzzle.ch/#ticket/zoom/2781
  • möglicherweise https://help.puzzle.ch/#ticket/zoom/3047 und https://help.puzzle.ch/#ticket/zoom/3084
• Die Auswahl von Gruppen+Rollen bei den Abo-Empfängern und bei Personenlisten-Filtern stimmen nicht damit überein, was in der Personenliste angezeigt wird
• Die API wurde immerhin konsistent mitgeändert, aber eigentlich wäre derselbe Effekt auch mit einem Query-Parameter ?range=layer in der URL erreichbar.

Stefan war es nach eigener Aussage bisher nicht bewusst, dass die Mitte hier eine Ausnahme ist. Diese Situation macht auch zukünftige Änderungen an Berechtigungen oder API im Core komplexer, da wir immer an den Spezialfall der Mitte denken müssen. Berechtigungsfragen sind ohnehin schon kompliziert genug. Daher möchte ich vorschlagen, den Spezialfall abzuschaffen, und stattdessen Rollen direkt auf dem Layer auch bei der Mitte einzuführen.

Der Spezialfall wurde in #58 konzipiert und in b27e065 eingeführt.