Feature: Letzter Wochentag im Monat in wiederholenden Events

[BokuNoMaxi]

In der derzeitigen Version der Extension kann man nur sagen 1-5 Woche des Monats, man kann aber nicht definieren den den letzten Freitag des Monats. Da die Monate manchmal 5 Wochen haben und manchmal nur 4 wäre die Funktion sinnvoll, da sonst Events in einem Monat 2x vorkommen können, speziell wenn es an einem Montag ist. Dann muss man diesen Tag manuell ausnehmen, oder?

[sfroemkenjw]

Hi @BokuNoMaxi

interessantes Feature. Das klingt einfach, aber unter der Haube müssen da schon einige Klamotten berücksichtig werden. Das erfordert auch nochmals umfangreichere Tests. Über das Feature muss @jweiland entscheiden. Muss man auch mal schauen, wie man das im Backend realisieren möchte? 5 zusätzliche Checkboxen mit Letzter, Vorletzter, 3. letzter, 4. letzter, 5. letzter? Oder eine zusätzlich Checkbox "umgekehrt"? Ich wähle "erster", aber unter der Haube ist's der letze Tag im Monat. Nope...das versteht ja keiner mehr.

Cooles Feature auf jeden Fall, aber definitiv keine Kleinigkeit

Aktueller Vorschlag:
Markiere in den wiederkehrenden Terminen immer 4ter und 5ter und mittels den Ausschlüssen deaktivierst Du dann für 12 Monate im Voraus die doppelt auftretenden Termine.

Schönen Start ins Wochenende

Stefan

[BokuNoMaxi]

Hi @sfroemkenjw und danke für den reply :)

ich hab sowas letztens sowas schon für einen Kunden implementiert und die Anforderung ist immer "es soll wie Microsoft Teams funktionieren...". Bei der Implementierung habe ich vorher gecheckt, ob es so eine Funktionalität schon gibt und da die Maske von events2 gesehen, hab das mal grob nachgebaut und Termine erstellen lassen, aber ganz zufrieden war ich nicht. Die Option " WIederholung jeden 5 im Monat" hat für mich keinen Sinn gemacht, weil wann wähle ich diese Option überhaupt aus? Nämlich nur als Workaround um den letzten des Monats zu bekommen. Bei der Option "Wiederholung jeden 1 im Monat" muss ich noch testen was passiert wenn der Monat mit einem Sonntag startet, ob dann ein Termin fehlt? Gerade migrieren wir von news_events auf events2 bei einem Kunden und wenn er dieses Feature gerne hätte kann ich ja einen PR schicken, sonst sind mir leider die Hände gebunden ;(

Anyways, ich hab das ganze schon simplified implementiert und ein paar Krisengesprächen mit CoPilot geht das Tool ganz gut:
Ich habe einfach statt "5. im Monat" umbenannt auf "jeden letzten" und funktioniert ganz gut

Und das ist mein Check:
`
$interval = new \DateInterval('P1D');
$period = new \DatePeriod($start, $interval, $end->modify('+1 day'));

  foreach ($period as $date) {
    if ($this->shouldCreateCourse($date, $weeks, $weekdays)) {

...
}
}
/**

• check if a date exists in the special week case
  */
  private function shouldCreateCourse(\DateTime $date, array $weeks, array $weekdays): bool
  {
  $weekday = (int)$date->format('w');
  $dayOfMonth = (int)$date->format('j');
  $lastDayOfMonth = (int)$date->format('t');

$weekOfMonth = (int)ceil($dayOfMonth / 7);
$isFirstWeekday = $this->isFirstOccurrenceOfWeekday($date) && in_array('first', $weeks) && in_array($weekday, $weekdays);
$isSecondWeekday = $weekOfMonth === 2 && in_array('second', $weeks) && in_array($weekday, $weekdays);
$isThirdWeekday = $weekOfMonth === 3 && in_array('third', $weeks) && in_array($weekday, $weekdays);
$isFourthWeekday = $weekOfMonth === 4 && in_array('fourth', $weeks) && in_array($weekday, $weekdays);
$isLastWeekday = $dayOfMonth > ($lastDayOfMonth - 7) && in_array('last', $weeks) && in_array($weekday, $weekdays);

return $isFirstWeekday || $isSecondWeekday || $isThirdWeekday || $isFourthWeekday || $isLastWeekday;

}

/**

• Because a week doesn't always start with either Monday or Sunday
• we have to check if the first friday is in the first or the next part of the current month
  */
  private function isFirstOccurrenceOfWeekday(\DateTime $date): bool
  {
  $weekday = (int)$date->format('w');
  $weekdays = ['Sunday', 'Monday', 'Tuesday', 'Wednesday', 'Thursday', 'Friday', 'Saturday'];
  $firstDayOfMonth = (clone $date)->modify('first day of this month');

if ((int)$firstDayOfMonth->format('w') === $weekday) {
  $firstOccurrence = $firstDayOfMonth;
} else {
  $firstOccurrence = (clone $firstDayOfMonth)->modify("next " . $weekdays[$weekday]);
}

return $date->format('Y-m-d') === $firstOccurrence->format('Y-m-d');

}`

Wünsch dir auch ein schönes Wochenende :)