FriendsOfREDAXO / multinewsletter

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

Platzhalter werden nicht ersetzt: +++GRAD+++ +++FIRSTNAME+++ +++LASTNAME+++ +++EMAIL+++ #53

Closed steri-rex closed 1 year ago

steri-rex commented 1 year ago

Die Platzhalter +++GRAD+++ +++FIRSTNAME+++ +++LASTNAME+++ +++EMAIL+++ werden im Textkörper des Newsletters nicht ersetzt - stattdessen wird gar kein Text ausgegeben. Alle anderen Platzhalter wie +++TITLE++ +++NEWSLETTERLINK+++ usw. werden jedoch richtig ersetzt.

Multinewsletter v. 3.5.4

TobiasKrais commented 1 year ago

Ich kann das Verhalten nicht nachvollziehen. Kannst du mal versuchen die aktuelle Git Version zu ziehen und auf den Server zu laden. Ist das dann immernoch so?

steri-rex commented 1 year ago

Danke für die Antwort - hab die aktuelle Git Version versucht aber auch hier das gleiche. Es funktioniert nur dieser erste Teil

$replaces = [
            '+++GRAD+++' => $user->grad,
            '+++FIRSTNAME+++' => $user->firstname,
            '+++LASTNAME+++' => $user->lastname,
            '+++EMAIL+++' => $user->email,
        ];

von dieser Funktion nicht https://github.com/TobiasKrais/multinewsletter/blob/44118e8675fa31967b0381742b7cf0635ac88841/lib/MultinewsletterNewsletter.php#L208 nicht - die Ersetzungen darunter mit +++TITLE+++ funktionieren.

Ich verwende PHP 8.1 und Redaxo 5.15.1

TobiasKrais commented 1 year ago

Da bist du an der richtigen Methode dran. Kannst du das bitte weiter debuggen? Ich kann die Situation nicht nachstellen.

TobiasKrais commented 1 year ago

Noch eine Frage: ändert sich das Verhalten, wenn du in den Einstellungen des Addons die Methode wie der Artikel gelesen wird änderst?

steri-rex commented 1 year ago

ja tatsächlich bei "redaxo intern" statt "socket" funktioniert es

TobiasKrais commented 1 year ago

Kannst du den Commit von eben bitte testen? (https://github.com/TobiasKrais/multinewsletter/commit/0db08d43728f5a444369369c48cf462e5552ab2e). Damit müsste der Fehler behoben sein.

steri-rex commented 1 year ago

ja funktioniert nun auch mit "Socket" - vielen Dank!

TobiasKrais commented 1 year ago

Danke fürs Testen. Ich mach gleich die Version 3.5.5 fertig.