Kleiner Fehler im holiday-Modul

[chicco]

Stimmt, das geht eleganter:

Code Auswählen Erweitern

    $t += 3600 * ((localtime())[8] - (localtime($t))[8]);
Ich teste später um 23 Uhr...

Getestet, genauso wie gestern, natürlich mit 179 Tagen statt 180.
Auch nach 23 Uhr bekomme ich noch "Tag der Arbeit" als Rückgabe, ab 0 Uhr kommt "none"

[Beta-User]

Auch nach 23 Uhr

Hmm, vermutlich sollte man einfach nicht bei der aktuellen Uhrzeit "starten", sondern jeweils "heute um 12:00 Uhr". Dann spielt $dst keine Rolle mehr...

[chicco]

Hmm, vermutlich sollte man einfach nicht bei der aktuellen Uhrzeit "starten", sondern jeweils "heute um 12:00 Uhr". Dann spielt $dst keine Rolle mehr...

Der Gedanke ist nicht verkehrt.
Aber mit der jetzt gepatchten Version funzt es doch, also welchen Vorteil hätte ein solcher Umbau gegenüber der jetzt gepatchten Version?
Letztendlich muss es Rudi als Maintainer entscheiden...