GRUPPEN: Testing - Internes Gremium

[Michael-Schaer]

Bisher sind mir nur zwei kleine Sachen aufgefallen beim internen Gremium.

• Als Hussein Kohlmann (globaler Admin) kann ich bei Internen Gremien scheinbar Personen hinzufügen. Erst bei der Rollenauswahl scheitert dies (es ist dann aber nicht klar, warum es scheitert). Z.B: hier https://pbs.puzzle.ch/de/groups/834/people. Schöner wäre es, wenn der Button "Personen hinzufügen" ausgeblendet würde in solchen Fällen.

• Interne Gremien können von Personen fusioniert, verschoben und gelöscht werden, die keinen Zugriff darauf haben (z.B. übergeordnete Kantonsleitung). Ich frage mich, ob das nicht auch auf die entsprechende Ebene eingeschränkt werden sollte. Hier bin ich allerdings nicht ganz schlüssig geworden.

[carlobeltrame]

Mir ist zudem noch aufgefallen, dass die JSON API durch diese Änderung kaputt gegangen ist, weil wir die sichtbaren Rollen einer Person neu global einschränken, basierend auf den Ebenen des current_user. current_user ist aber nil wenn man mit einem Service Token daherkommt, und es entsteht ein Folgefehler.

Um das zu fixen müssen wir im Core application_decorator zusätzlich current_service_token an h delegieren, sowie im PBS Wagon person_decorator#layer_group_ids ungefähr folgendermassen implementieren:

def layer_group_ids
  @layer_group_ids ||= current_user&.layer_group_ids.presence ||
      [current_service_token&.layer_group_id]
end

Wir haben dann mal noch ein allgemeines Issue #1083 wo wir die Situation mit current_user vs. current_service_token etc. aufräumen wollen.

EDIT: Den Change im Core hab ich gleich mal vorgenommen, da er sehr ungefährlich ist.

[nchiapol]

Intere Gremien tönen auch für uns spannend. Habt ihr da irgendwo noch mehr Details?

[Vento-Nuenenen]

@nchiapol Da: hitobito/hitobito_pbs#162