Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
UrsMetz opened this issue Jul 7, 2019 · 7 comments

Comments

@UrsMetz
Copy link
Contributor

UrsMetz commented Jul 7, 2019

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 leider added the bug label Jul 8, 2019
@leider
Copy link
Contributor

leider commented Jul 8, 2019

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
Copy link
Member

NicoleRauch commented Jul 8, 2019

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?

@NicoleRauch NicoleRauch removed the bug label Jul 8, 2019
@UrsMetz
Copy link
Contributor Author

UrsMetz commented Jul 9, 2019

Bei mir konnte ich den Fehler so provozieren:

  1. per Open-ID angemeldeter Nutzer "Organizer" ist einziger Ansprechpartner (local-openid aus dem Verzeichnis docker wurde als Open-ID-Provider verwendet)
  2. "Organizer" klick auf der Gruppenansicht auf "Gruppe verlassen"
  3. 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.
  4. 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
Copy link
Contributor

leider commented Jul 9, 2019

Ich kann das lokal reproduzieren

@leider
Copy link
Contributor

leider commented Jul 9, 2019

Kann das auch in Produktion reproduzieren

@leider
Copy link
Contributor

leider commented Jul 9, 2019

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
Copy link
Member

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...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants