Kleiner Bug in class.mailer.php

[bodil]

Wenn man in der System-Konfiguration für den Mail-Versand das Feld Verschlüsselung (leer, ssl oder tls): tatsächlich leer lässt, kommt es beim Mailversand in contenido/classes/class.mailer.php zu einer Fehlermeldung. An die Methode self::constructTransport() wird dann als Encryption NULL übergeben, und das führt zu einem Fatal Error.
Lösen lässt sich das, wenn man in Zeile 614

Code: Alles auswählen

$this->_mailEncryption = null;

ersetzt durch

Code: Alles auswählen

$this->_mailEncryption = '';

Und ja: Mails besser verschüsselt verschicken. Aber wo das nicht geht, sollte das der Contenido-eigene Mail-Mechanismus nicht verhindern.
Grüße aus dem Eis!
bodil

[Faar]

Das NULL macht immer wieder mal Probleme, nicht nur in Contenido, weil NULL nicht gleich null oder "leer" ist.
Tatsächlich ist hier ' ' richtig, weil es wird auf Charakter (ssl oder tls) geprüft und im leeren Fall kommt "Nichts", also NULL an, noch nicht mal irgendwelche Daten, sondern im Zweifel "False".
In der Folgeverarbeitung gibt es dann die Variable nicht, sie wurde ja quasi gelöscht.

[xmurrix]

Danke für den Hinweis, der Fehler wurde behoben.

https://github.com/CONTENIDO/CONTENIDO/issues/518