terminal42 / contao-leads

Leads extension for Contao Open Source CMS; Store and manage form data with ease!
GNU Lesser General Public License v3.0
36 stars 24 forks source link

Platzhalter wird nicht ersetzt #75

Closed akroii closed 4 years ago

akroii commented 5 years ago

Hallo Terminal42, ich habe ein recht großes Eingabeformular mit vielen Eingabefeldern. Aus irgend einem Grund werden aber zwei davon nicht ersetzt. Formulargenerator: Bildschirmfoto 2019-09-20 um 09 23 38

Notification Center: Bildschirmfoto 2019-09-20 um 09 27 29

Email: Bildschirmfoto 2019-09-20 um 09 29 00

Habt ihr eventuell eine Idee woran das liegen könnte?

aschempp commented 4 years ago

Ich glaube nicht dass das etwas mit Leads zu tun hat?

christianromeni commented 4 years ago

Ich hab das gleiche in nem ganz Normalen Formular, ohne alles:

image

image

image

Und wenn ich in den Formular-Einstellungen den Simple Token änder, wird er in der Ansicht auch so angezeigt:

image

In der Datenbank in tl_lead steht alles drin:

image

Contao 4.8 mit

        "codefog/contao-cookiebar": "^2.1",
        "contao/calendar-bundle": "^4.8",
        "contao/comments-bundle": "^4.8",
        "contao/conflicts": "*@dev",
        "contao/faq-bundle": "^4.8",
        "contao/listing-bundle": "^4.8",
        "contao/manager-bundle": "4.8.*",
        "contao/news-bundle": "^4.8",
        "contao/newsletter-bundle": "^4.8",
        "derhaeuptling/contao-seo-serp-preview": "^3.0",
        "heimrichhannot/contao-filename-sanitizer-bundle": "^1.3",
        "heimrichhannot/contao-notification_center_plus": "^1.0",
        "jrgregory/m17-sticky-backend-footer": "^2.0",
        "romeniwebdesign/contao-custom-articles-bundle": "dev-master",
        "terminal42/contao-ajaxform": "^1.1",
        "terminal42/contao-changelanguage": "^3.1",
        "terminal42/contao-folderpage": "^2.0",
        "terminal42/contao-leads": "^1.4",
        "terminal42/contao-node": "^1.1",
        "terminal42/notification_center": "^1.5"
rabauss commented 4 years ago

Wir hatten schon ähnliche Probleme mit dem Notification-Center, wenn wir diesen mit der Notification-Queue betrieben haben. Dann war nämlich das Datenbank-Feld zum Speichern der Daten zu klein - vielleicht könntet ihr das mal prüfen. Tatsächlich kann blob für die Tokens in der Queue ggf. zu klein sein!

martin-netprofit commented 4 years ago

Hab mit Contao 4.9.1 und 4.9.2 das gleiche Problem, die Platzhalter werden nicht mehr ersetzt. Aktuell funktioniert nur der Platzhalter ##created##. Das Notifications-Center ist mir nicht installiert.

"contao/calendar-bundle": "4.9.*", "contao/comments-bundle": "4.9.*", "contao/conflicts": "*@dev", "contao/faq-bundle": "4.9.*", "contao/listing-bundle": "4.9.*", "contao/manager-bundle": "4.9.*", "contao/news-bundle": "4.9.*", "contao/newsletter-bundle": "4.9.*", "jrgregory/m17-sticky-backend-footer": "^2.0", "kmielke/calendar_extended": "^3.7", "madeyourday/contao-rocksolid-antispam": "^2.0", "madeyourday/contao-rocksolid-custom-elements": "^2.2", "terminal42/contao-changelanguage": "^3.1", "terminal42/contao-leads": "^1.4"

aschempp commented 4 years ago

Ich verstehs noch immer nicht.

  1. Leads schickt keine E-Mails
  2. das Blob-Feld in tl_lead wir nicht verwendet, es ist nur als „Backup“ gedacht (enthält alle Rohdaten).

Generell ersetzt Contao die Simple Tokens nur dann, wenn auch ein entsprechender Wert vorhanden ist. Was steht denn in der Datenbank zu ##first_name##?

herrweiss77 commented 4 years ago

Seit einem Update von Contao 4.8 auf 4.9 und Update auf Leads Version 1.4.18 habe ich das gleiche Problem. Die Simple-Tokens werden nicht mehr ersetzt, außer ##created##.

image

Gibt es hierfür schon eine Lösung?

P.S. Danke für die tollen und nützlichen terminal42 Erweiterungen :-)

richardhj commented 4 years ago

https://github.com/terminal42/contao-leads/blob/94c36cbcd362ea7ff9512ff14e4c1b9369eacf8d/dca/tl_lead.php#L245

Hier werden die Tokens generiert. Das erklärt, warum ##created## immer geht.

Könnte man nicht auf das query verzichten und einfach $row nehmen? /cc @aschempp

math-GH commented 4 years ago

Contao 4.9.3 und Leads 1.4.18 funktionieren bei mir bzgl. der Platzhalter super

aschempp commented 4 years ago

Könnte man nicht auf das query verzichten und einfach $row nehmen? /cc @aschempp

$row ist nach meinem Verständnis der Datensatz aus tl_lead, die Zeilen stehen aber in tl_lead_data. Das ersetzen der Platzhalter funktioniert bei mir problemlos. Die Tokens werden aus tl_lead_data.name generiert, könnt ihr mal nachschauen was dort drin steht?

RalfGebhard commented 4 years ago

nun mit verschiedenen Installationen Contao 4.4, 4.9, 4.10 versucht. Ich habe das gleiche Phänomen. tl_lead_data bleibt leer, obwohl tl_lead befüllt wird. Im blob von tl_lead stehen die Daten drin, aber die Detailtabelle tl_lead_data bleibt leer, daher auch keine Tokens.

aschempp commented 4 years ago

Dann hast du die Felder nicht korrekt als "zu speichern in Leads" konfiguriert 😉