search

SE-TINF22B2 / G4-Get2Gether

Eine Web-App zum Planen von privaten Events.
0 stars 0 forks source link

Bug #153 - Eventpage Funktionalitäten fixen #153

Closed ebauersandoval closed 4 months ago

ebauersandoval commented 5 months ago

Beschreibung

Einige Buttons haben noch keine richtige Funktionalität implementiert, diese müssen zum Teil auch im Backend erweitert werden.

Aufgaben

Flugschnitzel commented 4 months ago

Hat die doppelte Negation bei der leave Event Schnittstelle im Controller und im Service einen Sinn?

Flugschnitzel commented 4 months ago

Abgesehen davon wäre Code QA durch

Drumber commented 4 months ago

Hat die doppelte Negation bei der leave Event Schnittstelle im Controller und im Service einen Sinn?

Auf dem ersten Blick dachte ich gerad auch, dass das ein Denkfehler sein muss, aber ja es hat einen Sinn.

Im Service überprüfe ich nochmal vorsichtshalber, ob der Benutzer auch wirklich nicht mehr in den participantIds enthalten ist. Als Rückgabe wert erwarte ich hier true wenn der User nicht mehr in den participantIds enthalten ist und somit erfolgreich das Event verlassen hat.

return !eventRepository.save(event).getParticipantIds().contains(user.getId());
// simpler ausgedrückt: !savedEvent.getParticipantIds().contains(user.getId())
Drumber commented 4 months ago

@Flugschnitzel mir fällt aber gerade ein, dass wir den Nutzer ja nicht einfach löschen können, wegen den ganzen Referenzierungen in den Widgets.

Wir hatten in Discord ja auch mal darüber gesprochen. Somit müsste man diese Story nochmal überarbeiten...

Hatte ich komplett vergessen (Upps 😅)