Verlässt ein Ansprechpartner eine Gruppe, verbleibt seine Id in der Liste der Organizer-Ids

softwerkskammer / Agora

Die Software für die Plattform der Softwerkskammer

Other

55 stars 58 forks source link

Verlässt ein Ansprechpartner eine Gruppe, verbleibt seine Id in der Liste der Organizer-Ids #1396

Open UrsMetz opened 5 years ago

UrsMetz commented 5 years ago

Beim manuellen Testen von #1390 ist mir gerade aufgefallen: wenn ein Ansprechpartner eine Gruppe verlässt wird die Member-Id nicht aus group.organizers entfernt, so dass es für manchen Code (z. B. den jetzt in #1390 ;-) ) so aussieht als ob die Person noch Ansprechpartner ist (konkret ist mir das aufgefallen, als ich als letzter Ansprechpartner die Gruppe verlassen hatte aber der Button um eine Mail an die Ansprechpartner zu senden noch angezeigt wurde).

leider commented 5 years ago

Hier muss beim Verlassen der Gruppe noch ein Aufräumen der Gruppe erfolgen.

Zusätzlich bei der Umsetzung noch ein Aufräumskript bauen, dass die aktuellen Gruppen diesbezüglich korrigiert.

NicoleRauch commented 5 years ago

Kann ich in Produktion nicht nachvollziehen. Wird ein Ansprechpartner von einer Gruppe entfernt, ist er danach nicht mehr in der Liste der organizers enthalten (in der DB).

@UrsMetz In welchem Szenario ist Dir dies aufgefallen, und wie kann ich das nachvollziehen?

UrsMetz commented 5 years ago

Bei mir konnte ich den Fehler so provozieren:

per Open-ID angemeldeter Nutzer "Organizer" ist einziger Ansprechpartner (local-openid aus dem Verzeichnis docker wurde als Open-ID-Provider verwendet)
"Organizer" klick auf der Gruppenansicht auf "Gruppe verlassen"
Es wird die Erfolgsmeldung angezeigt, dass man nicht mehr Mitglied der Gruppe ist. "Organizer" taucht an der Oberfläche nicht mehr in der Liste der Mitglieder auf. In der Datenbank steht die Id noch im Array organizers der Gruppe in der Collection groupstore.
Tritt "Organizer" jetzt der Gruppe wieder bei, dann ist er sofort wieder Ansprechpartner

Wird hingegen zwischen dem letzten und vorletzten Schritt über das Verwalten-Menü die Ansprechpartner editiert, dann ist die Id von "Organizer"' nicht mehr im Array der organizers der Gruppe vorhanden.

Ich kann das gerne morgen beim Treffen nochmal vorführen :-).

leider commented 5 years ago

Ich kann das lokal reproduzieren

leider commented 5 years ago

Kann das auch in Produktion reproduzieren

leider commented 5 years ago

Status:

Unsubscribe kann von Gruppenseite ausgelöst werden -> Gruppenobjekt (und damit auch die Ansprechpartner) wird noch nicht mal geladen. Kann gefixt erden durch laden der Gruppe, aktualisieren der Ansprechpartner und speichern der Gruppe
Unsubscribe wird IMPLIZIT beim Speichern eines Member aufgerufen -> Hier muss mehr Logik rein: Erkennen, aus welchen Gruppen das Member ausgetreten ist; diese dann laden, aktualisieren und speichern

NicoleRauch commented 5 years ago

Aaaah, jetzt! Sorry, ich hab das mit dem "Abmelden" fehlinterpretiert.

Wir hatten das damals als Feature angesehen, damit jemand, der aus Versehen (oder Experimentierfreude) die Gruppe verlässt, nicht gleich wieder mühsam Ansprechpartner werden muss (was vor allem blöd ist, wenn er der letzte Ansprechpartner ist, denn dann kann er (glaube ich?) nicht selbst wieder Ansprechpartner werden - oder?).

Hier würde ich gern erstmal überlegen, was da feature-mäßig Sinn macht...