minimal fix for bug

[nchiapol]

In meinem Test eben hat der Change unten gereicht um den Cevi-spezifischen Bug aus https://github.com/hitobito/hitobito/issues/2196 zu beheben.

Da ich die ganzen Abilities aber immer noch nicht wirklich verstehe, kann ich nicht sagen ob allenfalls noch weitere Anpassungen in dieser Datei nötig wären. Ist sicher gut, wenn jemand von euch da noch kurz darüber schaut.

[nchiapol]

Das erlaubt es Eltern auch, die TN in Kursen zu erstellen wo ein Kind eine Leitungsfunktion hat. Kann man so akzeptieren

Ich bin nicht sicher, ich verstehe, wo du genau den problematischen Aspekt siehst. Gemäss meinen Tests eben scheint für mich alles vernünftig zu funktionieren:

• Die Mutter kann ihr Kind auch in einem Kurs anmelden (oder das Kind kann sich selbst anmelden) - das Kind bekommt dann eine Teilnehmer-Rolle.
• Ich kann die Rolle des Kindes dann auf eine Leiter-Rolle ändern (wir verwenden diesen Prozess bereits sonst für die Leiter-Anmeldung bei Kursen)
• Danach kann die Mutter das Kind nicht erneut anmelden (das Kind ist bereits im Anmeldedropdown ausgegraut)
• Auch wenn ich die Leitungsfunktion direkt als Admin erstelle, ist das Kind im Anmeldedropdown ausgegraut.

Falls du einen problematischen Prozess siehst, schaue ich den gerne noch an - sonst denke ich aber wir können das gut so akzeptieren, wie es jetzt ist. Insbesondere da TN die als Leiter in Kursen sind wohl nur in Ausnahmefällen noch einen Verwalter haben.

oder man könnte nur den "her_own" Teil für Eltern erlauben, aber dann könnte man nicht den for_self_or_manageds Helper brauchen

Bevor ich das implementieren könnte, müsste ich mir das ganze wohl noch einmal genauer anschauen - aber eben, ich denke das ist unnötig.

[carlobeltrame]

Ja, ich habe es noch nicht geschafft das "Problem" verständlich schriftlich festzuhalten. Ich probiere es noch einmal.

Eine Person K organisiert einen Kurs, oder hat aus irgendeinem anderen Grund die Cevi-spezifische manage_attendances-Berechtigung auf einem Kurs. Wenn K nun ein Elternteil E zugeordnet hat, dann kann, mit deinem aktuellen Fix, E beliebige neue Anmeldungen im Kurs anlegen. Also E kann dann nicht nur K für den Kurs anmelden (das wäre eh redundant da K ja bereits mitorganisiert), sondern auch beliebige unbeteiligte Drittpersonen.

Wenn ihr sagt, es ist sowieso so selten dass jemand Eltern zugeordnet hat und gleichzeitig Kurse organisiert, dann können wir das ignorieren und dein Fix funktioniert in der Praxis korrekt.

[nchiapol]

Vielen Dank für die Erklärung. Ich glaube, jetzt habe ich dich richtig verstanden. Aber ich sehe das von dir beschriebene Verhalten in meinen Tests nicht:

• Ich habe K als Hauptleiter, als "Aktiver Kursleiter" und als Angestellter der Geschäftsstelle erfasst. In jedem Fall kann K dann neue Anmeldungen im Kurs erfassen.
• trotzdem hat E in keinem Fall Zugriff auf "participations" (was ja nötig wäre um neue Anmeldungen anlegen zu können)

Ich denke also, wir können mit diesem Fix weiterfahren. Falls wir später doch noch einen problematischen Fall finden, wird das sicher ein seltener Edge-Case sein. Entsprechend können wir fixen, wenn er Auftritt.

[nchiapol]

zur Vollständigkeit: Ich hatte eben noch eine Idee für einen Edge-Case: Wenn K Hauptleiter oder Aktiver Kursleiter in einem Event ist, in dem sich Teilnehmer gegenseitig sehen können und E als Teilnehmer angemeldet, dann kann E die Participations sehen.

Aber selbst in diesem Fall habe ich es nicht geschafft einen Situation zu finden, in der E weitere Teilnehmer hinzufügen könnte. (Weder Anlass auf Ortsgruppe noch Anlass oder Kurs auf Stufe Region)