eventStarted?

[ursbraem]

kursausschreibung/app/framework/date-helpers.js

Line 72 in 28c7887

export function eventStarted(event) {

Wir verstehen hier die Logik nicht ganz.

• Wenn das Flag showStartedEvents gesetzt ist, werden bereits begonnene Events auch angezeigt.
• Wenn Nachmeldungen möglich sind, soll als als Trigger für das Ausblenden des Events nicht das Startdatum des Events, sondern der Anmeldeschluss verwendet werden.
• Ist dann return parseISO(date).getTime() >= now.getTime(); nicht verkehrt?? Der Event gilt als "started" (=nicht dargestellt) wenn das (zusammengebaute) "Startdatum" grösser ist als jetzt. Das ist doch verkehrt oder verstehen wir etwas nicht?

Ist etwa nur der Name der Methode eventStarted falsch, aber die Funktion korrekt?

(Eigentlich müssten die Helpers granularer aufgebaut sein, Started oder "Nachmeldungen möglich" müssten 2 methoden sein.)

Wir stellen uns die Frage in Bezug auf die reverteten Commits vom 22. Jan 21 von Aram Loosman. Wir wissen nicht genau, weshalb diese revertet wurden, bzw ob wir das selber waren?

[schefbi]

@ursbraem und @AramLoosman
Es funktioniert wie im Wiki beschrieben. Mit Issue #116 wurde die Anforderung umgesetzt. Dieser UseCase geht um die Sichtbarkeit der Anlässe.

Ob Anmeldung möglich ist oder nicht wird mit der Funktion isInSubscriptionRange gemacht.

kursausschreibung/app/framework/date-helpers.js

Line 57 in 28c7887

export function isInSubscriptionRange(event) {

[ursbraem]

Unsere Frage ist wegen dem Operator >= auf Zeile 78

return parseISO(date).getTime() >= now.getTime();

bedeutet doch: "wenn das relevante Datum (je nach dem DateFrom oder SubscriptionDateTo) des Anlasses grösser gleich jetzt ist, schicke für eventStarted() true zurück." Aber damit der Event als gestartet gilt, müsste doch sein Datum KLEINER gleich jetzt sein?!

Wo stehen wir auch dem Schlauch?

---

Beispiel:

Angenommen Event hat am Montag um 8h gestartet (DateFrom): Timestamp ist 1635148800.
Jetzt ist aber Dienstag Nachmittag (now): 1635251790.
DateFrom ist in dieser Situation also kleiner als now.
Das würde aber mit return parseISO(date).getTime() >= now.getTime(); false zurückgeben.
Erwarten würde ich aber true (event ist started).

[schefbi]

@ursbraem
Sie haben schon Recht. Das Problem liegt aber auch in der Benutzung der Funktion.

kursausschreibung/app/framework/store.js

Line 282 in 28c7887

events = events.filter(event => eventStarted(event));

Ich werde den Code lesbarer machen in dem ich die function eventStarted() anpasse und alle Referenzen überprüfe.