C3 BETA - Newsletter Fehler / ModuleSubscribe.php / ModuleUnsubscribe.php

[n30xxx]

Hallo liebe Gemeinde,
da ich momentan die Contao Beta 3 nutze und eine Newsletter Seite aufsetzen wollte, ist mir ein Fehler beim abonnieren eines Newsletterverteilers aufgefallen.
Dieser Fehler machte es unmöglich eine Email Adresse in den Verteiler eintragen zu lassen.
Die ganze Zeit wurde eine Exception angezeigt - wo drin stand in der DB das Feld 'email' nicht NULL sein darf.
Ich habe mich auf Fehler Suche gemacht und die ModuleSubscribe.php aus der Beta 3 mit der aus der 2.10 Version verglichen.
Mir ist dabei aufgefallen das nach Zeile 214 in der ModuleSubscribe.php (Beta3) der folgende Eintrag fehlt.

$varInput = $this->idnaEncodeEmail($this->Input->post('email', true));

Habe diesen Eintrag eingefügt und schwupps war der Fehler beseitigt!

So sollte es dann aussehen:

        // Check the selection
        if (!is_array($arrChannels) || empty($arrChannels))
        {
            $_SESSION['SUBSCRIBE_ERROR'] = $GLOBALS['TL_LANG']['ERR']['noChannels'];
            $this->reload();
        }
        
        $varInput = $this->idnaEncodeEmail($this->Input->post('email', true));


        // Validate the e-mail address
        if (!\Validator::isEmail(\Input::post('email', true)))
        {
            $_SESSION['SUBSCRIBE_ERROR'] = $GLOBALS['TL_LANG']['ERR']['email'];
            $this->reload();
        } 

Mit freundlichen Grüßen
Marvin


EDIT:
Selbiger Fehler beim kündigen eines Abonnoments, in der ModuleUnsubscribe.php (Beta3) der folgende Eintrag fehlt nach Zeile 146:

$varInput = $this->idnaEncodeEmail($this->Input->post('email', true));

Sollte dann so aussehen:

        // Check the selection
        if (!is_array($arrChannels) || empty($arrChannels))
        {
            $_SESSION['UNSUBSCRIBE_ERROR'] = $GLOBALS['TL_LANG']['ERR']['noChannels'];
            $this->reload();
        }

$varInput = $this->idnaEncodeEmail($this->Input->post('email', true));

        // Validate e-mail address
        if (!\Validator::isEmail(\Input::post('email', true)))
        {
            $_SESSION['UNSUBSCRIBE_ERROR'] = $GLOBALS['TL_LANG']['ERR']['email'];
            $this->reload();
        } 

[andreas.schempp]

Ich sehe keine Unterschied zwischen vorher/nachher, ausser den Validierungszeilen? Die hast du wohl nicht hinzugefügt?

[Flex]

Ist auch kein Code Vergleich, sondern zwei geänderte Code Stellen aus 2 Dateien...

n30xxx meint, es fehlt die komplette Zuweisung und Enkodierung in der 3.0 Beta...

$varInput = $this->idnaEncodeEmail($this->Input->post('email', true));

https://github.com/contao/core/blob/...eSubscribe.php

Und ich würde sagen das stimmt...

Zeile 226 greift auf $varInput zu, die Variable wird in der Funktion addRecipient aber nicht erstellt...

[andreas.schempp]

Könntest du das als Bug Report stellen? Oder besser gleich als Pull Request?

https://github.com/contao/core/issues/new

[pmmueller]

Ist da ein Ticket erstellt worden?

Ich kann auf Github nichts dergleichen finden, habe aber im RC1 dieselbe Fehlermeldung beim Versuch, einen Newsletter zu abonnieren:
https://www.contao-community.de/show...045#post217045

/edit
Nachdem ich die Variablendefinition in ModuleSubscribe.php nach Zeile 214 hinzugefügt habe, ändert sich die Fehlermeldung:
Fatal error: Namespace declaration statement has to be the very first statement in the script in /home/www/c3/system/modules/newsletter/modules/ModuleSubscribe.php on line 17

Jedenfalls ist die fragliche Zeile anscheinend nicht im Skript und ich mache dann mal ein Ticket auf:
https://github.com/contao/core/issues/4660