search

FriendsOfREDAXO / multinewsletter

Newsletter Addon für Redaxo 5
MIT License
16 stars 6 forks source link

Fehlermeldung beim Versand nach Update auf MySQL 8.0.25 #47

Closed helpy closed 3 years ago

helpy commented 3 years ago

Nach dem Update auf MySQL 8.0.25 durch meinen Provider, hat redaxo ohne Probleme funktioniert.

Heute ist der erste Fehler (wohl wegen dem Update auf MySQL 8) beim Versand eines Newsletters aufgetreten.

Fehlermeldung: Error while executing statement "DELETE FROM rex_375_archive WHERE sentdate = '' OR sentdate IS NULL": SQLST

Nach der Änderung von folgender Zeile: https://github.com/TobiasKrais/multinewsletter/blob/c2e3dccb1a12c1a3038299b30a8631c378d410f4/lib/newslettermanager.php#L362

nach: $query_archive = "DELETE FROM " . rex::getTablePrefix() . "375_archive " . "WHERE sentdate = '0000-00-00 00:00:00' OR sentdate IS NULL";

... hat der Versand ohne Probleme funktioniert.

Ich habe nur eine eine schnelle Fehlerbehebung gesucht, nach kurzer Recherche obiges probiert ... und der Newsletter wurde verschickt. Ich habe keine Ahnung, ob diese Änderung evtl. andere Probleme verursachen könnte.

TobiasKrais commented 3 years ago

Danke Guido für die Fehlemeldung. Das Feld wurde vor längerem von int auf datetime umgestellt. Das ist die eigentliche Ursache des Fehlers. Hier der Bugfix: https://github.com/TobiasKrais/multinewsletter/commit/f967e7e9677c11482e26c7dcb2fe6f0249c3882f. Kannst du bitte prüfen, ob das bei dir auch funktioniert?

helpy commented 3 years ago

Danke Tobias, für die schnelle Bugfix-Version :-) Ich habe auf die neue Version aktualisiert und der Versand läuft wieder fehlerlos! Super.

helpy commented 3 years ago

Beim Vorbereiten des Versands kommt nur noch eine Warnung in der Log-Datei: Warning | Undefined array key 0 | redaxo/src/addons/multinewsletter/pages/newsletter.php | 262 https://github.com/TobiasKrais/multinewsletter/blob/8a184f5cdd4a617fbb753fa599520435f30a9f6c/pages/newsletter.php#L262

TobiasKrais commented 3 years ago

Ist entfernt: https://github.com/TobiasKrais/multinewsletter/commit/959a7b181a6760b32d36a6b9be95bd8b0f692e69. Dabei ist mir gerade noch eine weitere Warnung aufgefallen, die auch entfernt ist. Danke fürs aufmerksame melden!