[gelöst] Erweiterung funktioniert nicht mehr mit PHP ^8.0

Nach einem Upgrade von 4.9.x auf 4.13.15 funktioniert der Code nicht mehr bei PHP 8.0 oder höher.
Nach Entfernen der Calender Extended läuft Contao wieder normal.

Hat sich schon jemand gefunden diese Erweiterung weiter zu pflegen? Oder kann mir bei diesem Fehler weiterhelfen?

Folgender Fehler wird protokolliert:
[2023-02-02T08:29:43.929879+01:00] request.CRITICAL: Uncaught PHP Exception TypeError: "date(): Argument #2 ($timestamp) must be of type ?int, string given" at /homepages/32/d650190606/htdocs/echt/vendor/kmielke/calendar-extended-bundle/src/Resources/contao/classes/EventsExt.php line 421 {"exception":"[object] (TypeError(code: 0): date(): Argument #2 ($timestamp) must be of type ?int, string given at /homepages/32/d650190606/htdocs/echt/vendor/kmielke/calendar-extended-bundle/src/Resources/contao/classes/EventsExt.php:421)

"cgoit/calendar-extended-bundle" ist als 8.0 definiert und installiert auch.
In ca. 2-3 Stunden weiß ich ob die alten "kmielke/calendar-extended-bundle" Tabellen freundlich mit denen neuen "cgoit" sprechen. Ich hoffe ja sehr ;)

@Kester: vielen Dank für die galante Extension die unseren Tangokalender lange Jahre begleitet hat :D
@cgolt: vielen Dank für die Übername der Wartung der Extension von Kester:D

Vielen Dank für die Info ....

bei https://www.muetterzentrum-march-reute.de/ funktioniert es auch mit PHP 8.1. Ist aber eine komplette Neuinstallation. Kann es daran liegen?

Problem ist bei https://www.hscf.de/ . Beide bei 1&1 Ionos.
Wer hat noch eine Idee?

Du musst cgoit/calendar-extended-bundle installieren.

Habe cgoit/calendar-extended-bundle installiert.

Es kommt der gleiche Fehler wie bei kmielke.

[2023-02-03T09:21:43.941817+01:00] request.INFO: Matched route "tl_page.131.root". {"route":"tl_page.131.root","route_parameters":{"_ token_check":true,"_controller":"Contao\\FrontendI ndex::renderPage","_scope":"frontend","_locale":"d e","_format":"html","_canonical_route":"tl_page.13 1","parameters":"","pageModel":{"Contao\\PageModel ":[]},"_route":"tl_page.131.root","_route_object":{"Sy mfony\\Component\\Routing\\Route":[]}},"request_uri":"https://test1.hscf.de/","method":"GET"} []
[2023-02-03T09:21:43.959667+01:00] security.INFO: Populated the TokenStorage with an anonymous Token. [] []
[2023-02-03T09:21:44.203759+01:00] request.CRITICAL: Uncaught PHP Exception TypeError: "date(): Argument #2 ($timestamp) must be of type ?int, string given" at /homepages/32/d650190606/htdocs/test1/vendor/cgoit/calendar-extended-bundle/src/Resources/contao/classes/EventsExt.php line 428 {"exception":"[object] (TypeError(code: 0): date(): Argument #2 ($timestamp) must be of type ?int, string given at /homepages/32/d650190606/htdocs/test1/vendor/cgoit/calendar-extended-bundle/src/Resources/contao/classes/EventsExt.php:428)"} []

Mit einer leeren Installationsdatenbank funkioniert die Erweiterung und das System, erst nach Importieren meiner EchtDB von Version 4.9.38 läuft das System nicht mehr.
Wenn ich die Erweiterung wieder deinstalliere ohne DB Anpassung, läuft das System wie gewohnt.
Irgendwie passt EventsExt.php nicht zu meinen DB Einträgen?!

Hallo ...,

es muss an meinen eigenen Events in der Datenbank liegen. Wenn ich bei Events diesen Bereich komplett lösche, funktioniert alles wie gewünscht!
Jetzt gilt es nur noch den korrupten Datensatz zu finden. Hat da jemand eine gute Idee?

Das muss in der Extension angepasst werden. Die Extension muss sicherstellen, dass die date Funktion einen integer bekommt - auch wenn aus der Datenbank bspw. ein String returned wird.

Habe den Fehler bei einem Event gefunden. Es konnte ohne Datum gespeichert werden. Warum auch immer. Das war das Problem!

Anhang 26193

Zitat:

---

Zitat von Spooky

Das muss in der Extension angepasst werden. Die Extension muss sicherstellen, dass die date Funktion einen integer bekommt - auch wenn aus der Datenbank bspw. ein String returned wird.

---

Habe nun auch einmal die cgoit/calendar-extended-bundle installiert.
Mit eben dem selben Fehler wie oben schon einmal von jemanden beschrieben:

PHP-Code:

---

2023-09-21T15:35:16.590737+02:00] request.CRITICAL: Uncaught PHP Exception TypeError: "date(): Argument #2 ($timestamp) must be of type ?int, string given" at /var/www/vhosts/vendor/cgoit/calendar-extended-bundle/src/Resources/contao/classes/EventsExt.php line 428 {"exception":"[object] (TypeError(code: 0): date(): Argument #2 ($timestamp) must be of type ?int, string given at /var/www/vhosts/vendor/cgoit/calendar-extended-bundle/src/Resources/contao/classes/EventsExt.php:428)"} [] 


---

dieser Fehler tritt ja auch bei calendar_extended auch

Gibt es dafür noch keinen Fix, den man verwenden kann?
Als entweder für die cgoit-Variante oder die calendar_extended_bundle.

Und löst man das direkt in der Extension? Dann sind alle Änderungen nach einem Update wieder weg?

Schwierig. Die calendar_extended wird ja offenbar auch oft verwendet.

Danke für Tipps


Thomas

Zumindestens für das Original kmielke /calendar-extended-bundle wurde ja ein neuer Maintainer gesucht https://community.contao.org/de/show...uer-Maintainer . Da wird wohl nicht mehr allzuviel passieren. Vielleicht ein PR zum Thema machen.

Ja, ich meine auch eher, dass Leute, die die Erweiterung im Einsatz haben, dass die Ihre Änderungen evtl. zur Verfügung stellen könnten.

Die calendar_extended_bundle hatte ich soweit schonmal im Test, dass ich mir die EventsExt.php auch schonmal angepasst hatte, allerdings hatte ich den Eindruck, dass die Erweiterung zum einen auch im Backend manchmal "zickte" - da gingen einzelne Bereiche für die Wiederholungen nicht sofort auf, wenn man auf die Checkbox klckte.

Zum anderen hatte ich nach der Übernahme aus einem Contao 3 Projekt bei einigen EventListen das Problem, dass plötzlich gar keine Events mehr aufgelistet worden sind, trotz der gleichen Modul-Einstellungen.
Das geht dann schon sehr tief in die Analyse und da muss man dann evtl. auch die SQL Statements anschauen, ob die richtig erstellt werden oder ob da was fehlt ...

Die Erweiterung ist halt schon ziemlich gut und auch komplex und momentan geht aber weder diese noch die von cgoit, leider

--------------------

ich ändere mir die EventsExt.php bei der cgoit Variante mal ab und schau mal, ob das bei mir Veränderungen im FE bei den betreffenden Listen zeigt.
Ansonsten weiss ich da gerade nicht weiter, eim komplettes Fixen erscheint bei mir gerade zeitlich nicht möglich.

Aber falls jemand eine von beiden Erweiterungen ohne viel EigenCode funktionsfähig im Einsatz hat, würde ich das gerne einmal mit so einer angepassten Erweiterung testen.

Vielen Dank


Thomas

Die Anzeige von Terminen krieg ich gelöst, aber wie in einem anderen Thread schon kurz angerissen, die Option im Module:

Verkürzte Darstellung (Wiederholungen)
Events nur einmal anzeigen, auch wenn wiederholt werden.

wird m.E. im calender_extended_bundle in kmielke und im cgoit falsch interpretiert.
Wenn die Option ausgewählt ist, wird gar kein Event in der Eventliste angezeigt.

Wenn die Option nicht ausgewählt ist, dann kommen die Events, aber dann halt auch doppelt und dreifach, wegen der Wiederholungen.

Und ich weiss z.B. nicht, wo die Option aus dem FE Module im "Holen" der Events angewandt wird, dann könnte ich das evtl. sogar anpassen.
Obwohl eine "leere" Liste bei ausgewählter Option sicher auch noch auf einen anderen Bug hinweist.

Gibt es niemanden, der die calendar_extended selbst gefixt im Einsatz hat (und zur Verfügung stellen kann)

Bzw. kann mir jemand sagen, ob ich hier mit meiner Vermutung richtig liege?

Zitat:

---

Verkürzte Darstellung (Wiederholungen)
Events nur einmal anzeigen, auch wenn wiederholt werden.

wird m.E. im calender_extended_bundle in kmielke und im cgoit falsch interpretiert.
Wenn die Option ausgewählt ist, wird gar kein Event in der Eventliste angezeigt.

---

Ich habe cgoit/calendar-extended-bundle bei Webgo mit PHP 8.1 am laufen. Vorher war auch kmielke/calendar-extended-bundle im Einsatz.

Bei mir in dem Projekt war das Problem das Datensätze im Bereich "Unregelmäßige Wiederholungen" neu gespeichert werden mussten.

Teste das doch mal mit einen neuen Archiv und 1-3 frisch angelegten Events.

Klappt bei mir leider nicht, ich bin wieder auf kmielke umgestiegen.

Was mir aber aufgefallen ist:

eine Event-Liste aus "allen zukünfigen Events" + der Option "Verkürzte Darstellung (Wiederholungen)" klappt nach wie vor nicht.

1. Die Gruppierung für "verkürzte Darstellung" ist nicht drin
2. werden bei vielen Events mit auch sehr vielen Events offenbar in der DB Abfrage so viele Events geladen, dass es absolut ewig dauert.

Eine einschränkung auf "zukünftige Events + 2 Jahre" bringt schon etwas, aber die Funktionalität der Event-Liste ist m.E. nicht korrekt.

Ich konnte sämtliche Probleme mit "Undefined array key" mit ?? null in der Extension lösen.

Soll ich dafür einen Pull-Request machen machen?

Moin zusammen,

ich habe mal eine neue Version mit zwei PRs veröffentlicht. Ich hoffe, es hilt dem Einen oder Anderem. ;)

Vielen Dank für die Änderungen! Damit läuft die Erweiterung bei mir auch unter PHP 8.1


Bei der Version 1.0.24 gibt es wie in der alten 1.0.22 noch einen Bug: Erstellt man ein Event und richtet Unregelmäßige Wiederholungen ein (z.B. für morgen als Ausgangsevent oben und dann vier aufeinander folgende Tage bei den Unregelmäßigen Wiederholungen weiter unten) und löscht dann das erste Element der Wiederholungen, dann verschwindet im Frontend auch das letzte Element der Wiederholungen, obwohl es im Backend noch steht. Ähnliches passiert, wenn man andere Elemente aus der Liste löscht.

@Kester: Vielen Dank für die Entwicklung der Kalender-Erweiterung! Hat uns viele Jahre gute Dienste geleistet.

Mein erster Versuch mit cgoit/calendar-extended-bundle hat nicht funktioniert. Da muss ich noch mal ran.

So, ich habe jetzt die cgoit/calendar-extended-bundle installiert und gleichzeitig kmielke/calendar-extended-bundle deinstalliert. Installation hat ohne Probleme funktioniert, die bestehenden Kalender werden (soweit ich das sehe) korrekt angezeigt.

Um den Bug, den ich im vorherigen Beitrag beschrieben habe, mit cgoit zu überprüfen, habe ich das bestehende Test-Event (ein Grund-Event + 4 unregelmäßige Wiederholungen) geöffnet und die erste der Wiederholungen gelöscht. Beim Speichern kommt eine Fehlerseite "Internal Server Error" und
date(): Argument #2 ($timestamp) must be of type ?int, string given.

Um auszuschließen, dass es am bestehenden Event liegt, habe ich ein neues Event mit der gleichen Struktur angelegt. Auch da gibt das Löschen der ersten Wiederholung und Abspeichern diese Fehlermeldung.

Löschen der letzten Wiederholung und Abspeichern funktioniert. Löschen der zweiten und abspeichern funktioniert nicht - ich vermute, es geht nur bei der letzten.

Wieso installierst du zwei Extensions die das gleiche tuen?

Ich weiß nicht wo im Einzelnen die Unterschiede der beiden Extensions sind... die haben aber teilweise den gleichen Unterbau ;)

@Grossvater

Zitat:

---

Zitat von JvH

So, ich habe jetzt die cgoit/calendar-extended-bundle installiert und gleichzeitig kmielke/calendar-extended-bundle deinstalliert.

---

Hi @JvH,

Zitat:

---

Zitat von JvH

So, ich habe jetzt die cgoit/calendar-extended-bundle installiert und gleichzeitig kmielke/calendar-extended-bundle deinstalliert. Installation hat ohne Probleme funktioniert, die bestehenden Kalender werden (soweit ich das sehe) korrekt angezeigt.

Um den Bug, den ich im vorherigen Beitrag beschrieben habe, mit cgoit zu überprüfen, habe ich das bestehende Test-Event (ein Grund-Event + 4 unregelmäßige Wiederholungen) geöffnet und die erste der Wiederholungen gelöscht. Beim Speichern kommt eine Fehlerseite "Internal Server Error" und
date(): Argument #2 ($timestamp) must be of type ?int, string given.

Um auszuschließen, dass es am bestehenden Event liegt, habe ich ein neues Event mit der gleichen Struktur angelegt. Auch da gibt das Löschen der ersten Wiederholung und Abspeichern diese Fehlermeldung.

Löschen der letzten Wiederholung und Abspeichern funktioniert. Löschen der zweiten und abspeichern funktioniert nicht - ich vermute, es geht nur bei der letzten.

---

ich weiß, dass es schon lange her ist, als du das geschrieben hast. Ich habe, nach Rücksprache mit @kmielke, angefangen die Erweiterung etwas auf links zu drehen und in dem Zuge auch für Contao 5 fit zu machen. Das wird noch etwas dauern, aber ich werde mir dabei auch mal genau den von dir beschriebenen Fall anschauen.

lg
oetzi

Hi @all,

ich habe mich nun - wie angekündigt - einmal der Erweiterung angenommen und sie dabei ziemlich refactored. Daher und weil auch einige Funktionen nun nicht mehr unterstützt werden, ist es eine Version 2.0.0 geworden. Diese sollte sich unter Contao 4 und 5 nutzen lassen. Da sich im Hintergrund der Ansatz, wie mit den erweiterten Einstellungen zu Wiederholungen umgegangen werden soll, komplett geändert hat, empfehle ich einen Test der neuen Version in einer Testumgebung. Details zu den Änderungen finden sich auf Github in der README.md.

Bei Fragen, Anmerkungen oder Fehlern am Besten direkt ein Issue auf GitHub aufmachen. Da bekomme ich es am ehesten mit.

lg
oetzi

Erstmal vielen Dank, dass du dich der tollen Erweiterung angenommen hast.
Bin gerade am testen und habe festgestellt das endDate wenn leer immer als startDate gespeichert wird. Ist das so gewollt?

Hi,

Zitat:

---

Zitat von marcohe

Erstmal vielen Dank, dass du dich der tollen Erweiterung angenommen hast.
Bin gerade am testen und habe festgestellt das endDate wenn leer immer als startDate gespeichert wird. Ist das so gewollt?

---

ich vermute mal, du meinst damit, dass für das endDate immer der Wert aus dem Feld startDate gesetzt wird, oder? Wenn ja... es war zumindest in der alten Implementierung auch so. Man kann das sicherlich auch rausnehmen, aber dann müsste ich mal schauen, wo man im Code entsprechende Prüfungen einbauen muss. Das wird durchaus häufig verwendet.

Zitat:

---

Zitat von oetzi

ich vermute mal, du meinst damit, dass für das endDate immer der Wert aus dem Feld startDate gesetzt wird, oder?

---

Genau

Zitat:

---

Zitat von oetzi

Wenn ja... es war zumindest in der alten Implementierung auch so

---

Nein zumindest nicht in der Version die ich verwendet habe.

Aber im Prinzip schadet es auch nicht wenn endDate = startDate automatisch gesetzt wird :D