hitobito / hitobito_cevi

A hitobito wagon defining the organization hierarchy and additional features for Cevi
Other
6 stars 4 forks source link

Fehler bei Teilnehmerliste #113

Open nchiapol opened 1 year ago

nchiapol commented 1 year ago

@carlobeltrame @ThomasEllenberger

Eine Kurshauptleiterin von uns hat ein 500er gemeldet. Ich kann diesen im Produktivsystem reproduzieren. Entsprechend solltet ihr diverse 500er im Log haben von heute 2023-04-10 ab ca. 20:35. Hoffentlich seht ihr dort, was das Problem ist.

Der Fehler tritt auf, wenn die Kurshauptleiterin (bzw. ich via imitieren) auf die Teilnehmerliste ihres Kurses zugreifen möchte. Wenn ich die gleiche Liste als Admin öffne, hat die eine etwas längere Lade-Zeit funktioniert aber ohne 500er (und der 500er kommt bei der Leiterin sofort).

Wenn ich als Admin die anderen möglichen Links auf dem Teilnehmer-Tab kopiere, kann ich die auch als Leiterin öffnen.

Problematisch:

unproblematisch:

Da alle Teilseiten und Page 2 funktionieren, vermute ich das Problem irgendwo beim Paging der ersten Seite. Aber das seht ihr hoffentlich im Log besser. Entsprechend versuche ich im Moment auch nicht das in der Integration nachzustellen.

ThomasEllenberger commented 1 year ago

Ohne ein Imitieren auf eurer Produktivumgebung kann ich dies leider nicht nachstellen (Was wir im Normalfall verhindern, da wir eigentleich nicht aktiv in eure Produktivumgebung eingreifen). Dem WV sind die 500er aufgefallen, sind jedoch ohne grösseres Nachforschen auch nicht einfach zu beheben.

Könnt ihr der Kursleiterin die Teilnehmendendaten als Workaround zustellen, oder für sie Änderungen durchführen? Wir hoffen dass es sich hier um einen Einzelfall handelt und würden vorerst nichts unternehmen. Wenn wir dem prioritär nachgehen sollen, eröffne bitte ein Helpticket.

nchiapol commented 1 year ago

@ThomasEllenberger

Wer ist der WV? Ich finde es gut, wenn ihr nicht in unserer Produktivumgebung eingreift. Meine Hoffnung war einfach, dass das Log zum 500er genügend Info enthält um das Problem zu lokalisieren und gezielt nachzustellen. Falls das nicht der Fall ist können wir hier schon warten ob das noch einmal Auftritt.

Die Kursleiterin kann auch selbst via unproblematische Links auf alle Daten zugreifen - einfach nicht auf alle gleichzeitig. Das und die Unterstützung der Geschäftsstelle sollten für den Moment reichen.

ThomasEllenberger commented 1 year ago

@nchiapol

Wir können vom Log ablesen, dass es mit grosser Wahrscheinlichkeit etwas mit den ausgewählten Spalten zu tun hat. Wenn du uns die Personen ID der Hauptleiterin geben kannst, bei welcher das auftritt können wir die Spaltenauswahl auf das default zurücksetzen, wodurch bei ihr kein Fehler mehr auftreten sollte (ohne gewähr) und kaum Aufwand bedeutet. Dies behebt den Fehler jedoch nicht nachhaltig.

nchiapol commented 1 year ago

@ThomasEllenberger

Vielen Dank für die Infos aus dem Log. Damit konnte ich die problematische Spalte identifizieren: es waren "Zusätzliche Angaben". Wenn ich die bei mir aktiviere sehe ich das gleiche Verhalten. Ich habe die entsprechende Spalte nun bei der Hauptleiterin via Imitieren und eine der funktionierenden Ansichten deaktiviert.

Ich habe zudem getestet wo das Problem sonst noch auftritt und ich bin ziemlich sicher, dass es mit wie vermutet mit dem Paging zu tun hat: Sobald ich "Zusätzliche Angaben" einblende tritt der 500er bei allen Kursen auf die mehr als 50 Personen auf der Teilnehmerliste haben oder genauer bei allen Teilnehmerlisten-Seiten mit 50 Einträgen. Beim Kurs https://db.cevi.ch/groups/2/events/2880 schlägt sowohl https://db.cevi.ch/groups/2/events/2880/participations wie auch https://db.cevi.ch/groups/2/events/2880/participations?filter=participants fehl (da mehr als 50 TN angemeldet sind und deshalb auch der zweite Link paging benötigt). In beiden Fällen tritt der Fehler auf der ersten, vollen Page auf, die zweite Seite ist aber wieder ok.

Schliesslich habe ich jetzt einen Testkurs auf der Integration erstellt, der das Problem ebenfalls zeigt: https://cevi.puzzle.ch/groups/1/events/1195

Ich hoffe, damit könnt ihr das bei Gelegenheit ganz fixen.

ThomasEllenberger commented 1 year ago

Merci fürs aufsetzen des Testcases. Ich konnte das nachstellen und habe entsprechend ein Bugissue in unserem Repo erstellt.