WBCE Home | WBCE Hilfe | WBCE Addon Repository | Impressum | Datenschutz

WBCE Forum

WBCE CMS – Way Better Content Editing.

You are not logged in.

#26 09.12.2016 09:07:52

colinax
Member

Re: mpForm

mrbaseman wrote:
colinax wrote:

Hi, ich habe zwei Bugs entdeckt:
1. Beim Upgrade von 1.2.x auf 1.3.x werden die Allgemeinen Einstellungen fehlerhaft übertragen.

In wiefern? was geht verloren?

Alles was in dem Bereich "E-Mail-Einstellungen (Mail an Sitebetreiber)" drinnen ist/war, geht verloren.

mrbaseman wrote:

2. mpForm hat Probleme beim versenden von längeren Texten die auf mehrere Absätze aufgeteilt sind. Der Absender bekommt die Bestätigungsmail aber der Siitenbetreiber bekommt die Mail nicht.

welche Felder sind im Formular, und kommen denn kürzere Texte an?

Kurze Texte kommen an, Feldertypen die vorhanden sind: Kurztext, Langtext, E-Mail-Adresse und E-Mail-Betreff.


Mein Herz schlägt für WBCE

WBCE = best community ever

Offline

#27 12.12.2016 12:09:02

mrbaseman
Developer

Re: mpForm

colinax wrote:
mrbaseman wrote:

2. mpForm hat Probleme beim versenden von längeren Texten die auf mehrere Absätze aufgeteilt sind. Der Absender bekommt die Bestätigungsmail aber der Siitenbetreiber bekommt die Mail nicht.

welche Felder sind im Formular, und kommen denn kürzere Texte an?

Kurze Texte kommen an, Feldertypen die vorhanden sind: Kurztext, Langtext, E-Mail-Adresse und E-Mail-Betreff.

danke, damit konnte ich es reproduzieren, aber ich bekomme keine Fehlermeldung. Ich sehe sogar eine unvollständige Multipart-Message, die vom phpmailer verschickt wird (Mail ist nach der ersten Boundary abgeschnitten, wo eigentlich der Plaintext-Body anfangen sollte). Ich vermute, dass das Problem entweder im phpmailer oder in der umgebenden Funktion vom Core irgendwo liegt, aber genau konnte ich es noch nicht lokalisieren.

Offline

#28 15.12.2016 11:50:00

colinax
Member

Re: mpForm

colinax wrote:

1. Beim Upgrade von 1.2.x auf 1.3.x werden die Allgemeinen Einstellungen fehlerhaft übertragen.

Update bezüglich des zitierten Fehlers:

Gestern habe ich auf einer zweisprachigen Webseite mpForm von Version 1.3.1.1 auf 1.3.2 aktualisiert.


Leider waren im englischsprachigen Formular wieder diverse Einstellungen verschwunden, und Funktionen wie "Formular exportieren" funktionieren nicht.

Kurios: Nach dem Aufrufen der Einstellungen des deutschsprachigen Formulars und speichern ohne eine Änderung zu machen sind die Einstellungen im englischsprachigen Formular wieder vorhanden, und die Funktionen wie "Formular exportieren" funktionieren immer noch nicht.


Leider hat das englischsprachige Formular den Betrieb komplett eingestellt, soll heißen in BE (ACP) ist alles vorhanden, im FE wird aber nichts angezeigt.

Auch ein löschen und neu erstellen des Formulars hilft nicht.


Das Formular im deutschsprachigen Bereich funktioniert ganz normal, auch die anderen Funktionen wie "Formular exportieren" funktionieren.

Diesen Effekt konnte ich auf einer anderen Seite reproduzieren.


Wie es aussieht verträgt, mpForm nur mehr ein Formular pro WBCE Installation.


LG Colinax


Mein Herz schlägt für WBCE

WBCE = best community ever

Offline

#29 15.12.2016 13:05:08

mrbaseman
Developer

Re: mpForm

colinax wrote:

Leider hat das englischsprachige Formular den Betrieb komplett eingestellt, soll heißen in BE (ACP) ist alles vorhanden, im FE wird aber nichts angezeigt.
[...]
Wie es aussieht verträgt, mpForm nur mehr ein Formular pro WBCE Installation.

Hi colinax,

nein. Hier handelt es sich jetzt um einen einfachen Typo in der englischen Sprachdatei (languages/EN.php). Dort fehlt in Zeile 109 das schließende Anführezeichen. Ich korrigiere das in der nächsten Version.

Das davor gemeldete Verhalten, dass Einstellungen bein Update verloren gehen, muss ich noch näher untersuchen. Da kann ich noch nichts dazu sagen.

Viele Grüße,
Martin

Offline

Liked by:

#30 15.12.2016 14:26:11

mrbaseman
Developer

Re: mpForm

ich seh gerade, dass mir der Lapsus mit dem fehlenden Single-Quote auch in den meisten Sprachdateien passiert ist. Die englische habe ich als Vorlage für die anderen Fremdsprachen genommen und nur den Text geändert, während das "Drumherum" gleich (falsch) geblieben ist. Sorry. Ein Hotfix mit korrigierten Sprachdateien ist angehängt. Auf dem Github kommt die Korrektur mit der nächsten Version dann auch rein.

Offline

Liked by:

#31 15.12.2016 17:06:45

colinax
Member

Re: mpForm

mrbaseman wrote:

ich seh gerade, dass mir der Lapsus mit dem fehlenden Single-Quote auch in den meisten Sprachdateien passiert ist. Die englische habe ich als Vorlage für die anderen Fremdsprachen genommen und nur den Text geändert, während das "Drumherum" gleich (falsch) geblieben ist. Sorry. Ein Hotfix mit korrigierten Sprachdateien ist angehängt. Auf dem Github kommt die Korrektur mit der nächsten Version dann auch rein.

Danke für deine schnelle Reaktion, ich werde den Hotfix morgen installieren.  thumb_up  thumb_up  smile


Mein Herz schlägt für WBCE

WBCE = best community ever

Offline

#32 21.12.2016 23:23:13

mrbaseman
Developer

Re: mpForm

I have just updated mpform to Version 1.3.3 on Github

       *** 1.3.3 (Martin Hecht: 19-Dec-2016) ***
        fix the local template so that indeed all place holders are allowed there
        introduce {TEMPLATE1} ... {TEMPLATE9} for a single line of the local template
        correctly decode html special characters for html emails (thanks to astrid)
        allow to specify success_email_from by field or by logged on user (astrid)
        for consistency also allow to specify success_email_fromname by field
        fix missing single quote in most of the language files (thanks to colinax)
        allow to select multiple name fields for the sender name (astrid)
        allow place holders {FIELD...} in subject lines (request by astrid)
        fixed default form footer and correctly implement place holder {SUBMIT}
        correct the html code in the backend template for the basic settings
        offer a setting for user defined classes inside the fields and
        introduce place holder {FORMATTED_FIELD} in the field loop for that (florian, gottfried)
        avoid collision of the {TEMPLATE} place holder with the template for heading fields
        correctly detect if we use table layout or not for the output of headings
        repair update script to add the column for the field template (colinax)
        avoid multiply including the department in notification mails
        trim email_to when saving basic settings
        update documentation and example forms

Offline

#33 22.12.2016 00:19:30

mrbaseman
Developer

Re: mpForm

colinax wrote:

Hi, ich habe zwei Bugs entdeckt:

1. Beim Upgrade von 1.2.x auf 1.3.x werden die Allgemeinen Einstellungen fehlerhaft übertragen.

2. mpForm hat Probleme beim versenden von längeren Texten die auf mehrere Absätze aufgeteilt sind. Der Absender bekommt die Bestätigungsmail aber der Siitenbetreiber bekommt die Mail nicht.

ich hoffe, dass die beiden Dinge behoben sind. Die Sprachdateien sind jedenfalls in 1.3.3 gefixt.

Auch im Update-Skript habe ich einen Bug gefunden und behoben. WIe das aber zu dem beschriebenen Verhalten führen kann, verstehe ich nicht so ganz. Vielleicht im Zusammenspiel mit den fehlenden Quotes in der Sprachdatei?

Mit den längeren Texten bin ich nicht schlau geworden. Das kuriose: Die Mail an den Sitebetreiber wird zuerst generiert und verschickt und nur wenn diese Routine erfolgreich zurückkehrt, wird eine Bestätigungsmail an den Benutzer verschickt. Die Mail an den Sitebetreiber geht dann wohl irgendwo verloren, wo sie vom phpmailer weitergereicht wird, und zwar ohne dass dieser einen Fehlercode zurückliefert. Wenn es an der Länge hängt, dann gibt es mittlerweile die Möglichkeit, bei den Textareas auch eine maxlength anzugeben, wenn die Mail an den Sitebetreiber in jedem Fall rausgehen soll. Wie groß man den Wert einstellen muss, muss man vermutlich im Einzelfall ausprobieren.

Offline

#34 23.12.2016 19:56:15

colinax
Member

Re: mpForm

mrbaseman wrote:

ich hoffe, dass die beiden Dinge behoben sind. Die Sprachdateien sind jedenfalls in 1.3.3 gefixt.

Auch im Update-Skript habe ich einen Bug gefunden und behoben. WIe das aber zu dem beschriebenen Verhalten führen kann, verstehe ich nicht so ganz. Vielleicht im Zusammenspiel mit den fehlenden Quotes in der Sprachdatei?

Seitdem die Sprachdateien wieder korrekt funktionieren, gab es keine Probleme mit den Allgemeinen Einstellungen nach dem Upgrade auf 1.3.3

mrbaseman wrote:

Mit den längeren Texten bin ich nicht schlau geworden. Das kuriose: Die Mail an den Sitebetreiber wird zuerst generiert und verschickt und nur wenn diese Routine erfolgreich zurückkehrt, wird eine Bestätigungsmail an den Benutzer verschickt. Die Mail an den Sitebetreiber geht dann wohl irgendwo verloren, wo sie vom phpmailer weitergereicht wird, und zwar ohne dass dieser einen Fehlercode zurückliefert. Wenn es an der Länge hängt, dann gibt es mittlerweile die Möglichkeit, bei den Textareas auch eine maxlength anzugeben, wenn die Mail an den Sitebetreiber in jedem Fall rausgehen soll. Wie groß man den Wert einstellen muss, muss man vermutlich im Einzelfall ausprobieren.

Ich werde mir über die Feiertage eine Testumgebung einrichten und testen ob und abwann es eine maximale Länge gibt (auch wenn ich mir das nicht vorstellen kann). Erledigt!

Edit:
Ich habe in meiner Testumgebung ca. 30 Mails, mit längen von 50 - 900 Zeichen, übers Formular gesendet. Soweit hat alles gut funktioniert, bis auf zwei Mails, wo ich mir nicht sicher bin.

Denn nachdem die zwei Mails nicht angekommen sind, habe ich den Verlauf gelöscht und die folgenden Mails kamen wieder an.

Last edited by colinax (26.12.2016 13:42:50)


Mein Herz schlägt für WBCE

WBCE = best community ever

Offline

#35 26.12.2016 17:39:21

colinax
Member

Re: mpForm

Vielleicht liegt das Problem an der uralten PHPMailer Version aus dem Jahre 2011.

Ich hab es mal als Issue auf Github vermerkt:

https://github.com/WBCE/WebsiteBaker_Co … issues/168

LG Colinax


Mein Herz schlägt für WBCE

WBCE = best community ever

Offline

Board footer

Powered by FluxBB