Advancedform_XH 2.1

Ein CMSimple Support Forum für deutsch sprechende Nutzer und Entwickler
knollsen
Posts: 364
Joined: Wed Nov 06, 2013 2:28 pm

Advancedform_XH 2.1

Post by knollsen » Tue May 04, 2021 12:27 am

Ich habe gerade das Update gezogen und getestet
Alle Files unter templates

Code: Select all

<?php php
if (!isset($this)) {
    header('HTTP/1.0 404 Not Found');
muss das 2. php entfernt werden

Dann ist mir aufgefallen, wenn man Dateien nicht versenden will, kommt die Meldung:
Die Datei von Feld 'Datei' konnte nicht angehängt werden!
Der Dateityp '' kann nicht an Feld 'Datei' angehängt werden!
Bei dieser Meldung funktioniert auch nicht mehr der Formularzurücksetzbutton. Wenn ein Datumsfeld integriert ist, ist man gezwungen dies aus zu wählen, obwohl es kein Pflichtfeld ist (Das Feld 'Datum' enthält kein gültiges Datum!)
Bitte überprüft das! - sowas darf nicht passieren.

frase
Posts: 3994
Joined: Thu Apr 21, 2016 6:32 am
Location: Saxony
Contact:

Re: Advancedform_XH 2.1

Post by frase » Tue May 04, 2021 8:16 am

knollsen wrote:
Tue May 04, 2021 12:27 am
Alle Files unter templates

Code: Select all

<?php php
if (!isset($this)) {
    header('HTTP/1.0 404 Not Found');
muss das 2. php entfernt werden
Ja, das stimmt.
Bei Aufruf der Seite "E-Mail-Formulare" kommt eine weiße Seite mit:
syntax error, unexpected 'if' (T_IF) in ... \plugins\advancedform\templates\forms-admin.php on line 2
Dieser Fehler muss sich beim Erzeugen der ZIP-Datei eingeschlichen haben. Im Master auf Github ist das zweite "php" nicht drin.

frase
Posts: 3994
Joined: Thu Apr 21, 2016 6:32 am
Location: Saxony
Contact:

Re: Advancedform_XH 2.1

Post by frase » Tue May 04, 2021 9:09 am

knollsen wrote:
Tue May 04, 2021 12:27 am
Bei dieser Meldung funktioniert auch nicht mehr der Formularzurücksetzbutton.
Der Reset-Button funktioniert generell nicht mehr, sobald eine Fehlermeldung ausgegeben wurde.

Kuriosum:
Im mitgelieferten Formular "Calc" muss man Zahlen eingeben, deren Summe errechnet wird.
Enter numbers to the green and blue field. Their sum will be displayed in the red field.
Und darunter steht kurioserweise:
BTW: today is Tuesday!
Das kommt aus dem zugehörigen Template, ist aber doch wohl ein Versehen - oder?

frase
Posts: 3994
Joined: Thu Apr 21, 2016 6:32 am
Location: Saxony
Contact:

Re: Advancedform_XH 2.1

Post by frase » Tue May 04, 2021 9:20 am

knollsen wrote:
Tue May 04, 2021 12:27 am
Dann ist mir aufgefallen, wenn man Dateien nicht versenden will, kommt die Meldung:
Die Datei von Feld 'Datei' konnte nicht angehängt werden!
Der Dateityp '' kann nicht an Feld 'Datei' angehängt werden!
Bei dieser Meldung funktioniert auch nicht mehr der Formularzurücksetzbutton. Wenn ein Datumsfeld integriert ist, ist man gezwungen dies aus zu wählen, obwohl es kein Pflichtfeld ist (Das Feld 'Datum' enthält kein gültiges Datum!)
Auch das ist richtig.
Es gibt keine Möglichkeit die Felder "Datei" und "Datum" optional zu lassen.

frase
Posts: 3994
Joined: Thu Apr 21, 2016 6:32 am
Location: Saxony
Contact:

Re: Advancedform_XH 2.1

Post by frase » Tue May 04, 2021 9:34 am

frase wrote:
Tue May 04, 2021 9:20 am
Es gibt keine Möglichkeit die Felder "Datei" und "Datum" optional zu lassen.
Das trifft übrigens auch auf Felder vom Typ "E-Mail" zu.

frase
Posts: 3994
Joined: Thu Apr 21, 2016 6:32 am
Location: Saxony
Contact:

Re: Advancedform_XH 2.1

Post by frase » Tue May 04, 2021 9:54 am

E-Mail-Felder werden offensichtlich nicht mehr auf Plausibilität geprüft - oder?
Die Eingabe von z.B. "xxx" wird akzeptiert.
Früher kam in solchen Fällen:
Das Feld 'EMail' enthält keine gültige E-Mail-Adresse!

cmb
Posts: 13675
Joined: Tue Jun 21, 2011 11:04 am
Location: Mü-Sa, RLP, DE
Contact:

Re: Advancedform_XH 2.1

Post by cmb » Tue May 04, 2021 11:08 pm

frase wrote:
Tue May 04, 2021 8:16 am
Dieser Fehler muss sich beim Erzeugen der ZIP-Datei eingeschlichen haben. Im Master auf Github ist das zweite "php" nicht drin.
Stimmt. Da hatte ich eine ziemlich bescheuerte Idee, die mir ganz toll vorkam.
knollsen wrote:
Tue May 04, 2021 12:27 am
Dann ist mir aufgefallen, wenn man Dateien nicht versenden will, kommt die Meldung:
Die Datei von Feld 'Datei' konnte nicht angehängt werden!
Der Dateityp '' kann nicht an Feld 'Datei' angehängt werden!
Bei dieser Meldung funktioniert auch nicht mehr der Formularzurücksetzbutton. Wenn ein Datumsfeld integriert ist, ist man gezwungen dies aus zu wählen, obwohl es kein Pflichtfeld ist (Das Feld 'Datum' enthält kein gültiges Datum!)
Bitte überprüft das! - sowas darf nicht passieren.
Oh ja, die ganze Validierung passt überhaupt nicht. Ich habe das Release als unbrauchbar markiert, und versuche schnellstmöglich eine neue Version freizugeben. Ich hoffe, dass niemandem Schaden dadurch entstanden ist, oder entstehen wird. Das hätte zumindest bei einer Nebenversion nicht passieren dürfen.
frase wrote:
Tue May 04, 2021 9:09 am
knollsen wrote:
Tue May 04, 2021 12:27 am
Bei dieser Meldung funktioniert auch nicht mehr der Formularzurücksetzbutton.
Der Reset-Button funktioniert generell nicht mehr, sobald eine Fehlermeldung ausgegeben wurde.
Das ist Standard-Verhalten von HTML-Formularen; Reset setzt auf den Zustand zurück, der im Quellcode steht; und da stehen eben die User eingetragenen Werte, damit diese nicht alle noch einmal eingegeben werden müssen.
frase wrote:
Tue May 04, 2021 9:09 am
Kuriosum:
Im mitgelieferten Formular "Calc" muss man Zahlen eingeben, deren Summe errechnet wird.
Enter numbers to the green and blue field. Their sum will be displayed in the red field.
Und darunter steht kurioserweise:
BTW: today is Tuesday!
Das kommt aus dem zugehörigen Template, ist aber doch wohl ein Versehen - oder?
Nicht wirklich. Das Formular sollte halt zeigen, was man mit JS so alles anstellen kann. Der ganze JS-Form-Support ist aus heutiger Sicht völlig unsinnig, und sollte entfernt werden. Bis dahin kann das witzige Demoformular gerne bleiben, denke ich.
Christoph M. Becker – Plugins for CMSimple_XH

frase
Posts: 3994
Joined: Thu Apr 21, 2016 6:32 am
Location: Saxony
Contact:

Re: Advancedform_XH 2.1

Post by frase » Wed May 05, 2021 7:02 am

cmb wrote:
Tue May 04, 2021 11:08 pm
Nicht wirklich. Das Formular sollte halt zeigen, was man mit JS so alles anstellen kann. Der ganze JS-Form-Support ist aus heutiger Sicht völlig unsinnig, und sollte entfernt werden. Bis dahin kann das witzige Demoformular gerne bleiben, denke ich.
Autsch, da habe ich wohl etwas witzig Gemeintes zu ernst genommen. Den Wochentag habe ich als Beispiel für die Eingabe (numbers) angesehen - sorry.

Das Formular als Demo ist aber in Ordnung.

Was den JS-Form-Support angeht, das kann ich nicht beurteilen. Aber, dass man benötigtes JS auch direkt ins Template reinschreiben kann, das hatten wir ja früher schonmal.

cmb
Posts: 13675
Joined: Tue Jun 21, 2011 11:04 am
Location: Mü-Sa, RLP, DE
Contact:

Re: Advancedform_XH 2.1

Post by cmb » Thu May 06, 2021 12:35 am

cmb wrote:
Tue May 04, 2021 11:08 pm
[…], und versuche schnellstmöglich eine neue Version freizugeben.
Was ich gerade getan habe. (Kleine Ursache, große Wirkung.)
Christoph M. Becker – Plugins for CMSimple_XH

frase
Posts: 3994
Joined: Thu Apr 21, 2016 6:32 am
Location: Saxony
Contact:

Re: Advancedform_XH 2.1

Post by frase » Thu May 06, 2021 5:56 am

cmb wrote:
Thu May 06, 2021 12:35 am
cmb wrote:
Tue May 04, 2021 11:08 pm
[…], und versuche schnellstmöglich eine neue Version freizugeben.
Was ich gerade getan habe. (Kleine Ursache, große Wirkung.)
Sehr schön. Vielen Dank.

Es sieht soweit ganz gut aus.
Im Unterschied zu früheren Versionen, werden nicht mehr alle Fehlermeldugen insgesamt ausgegeben, sondern immer nur der erste gefundene Fehler. Den muss man korrigieren und es dann nochmals versuchen. Dann kommt der nächste ... usw.
Befindet sich ein Captcha-Feld im Formular und wurde es nicht korrekt ausgefüllt, wird dieser Fehler immer mit ausgegeben (wie früher).

Hat man also ein sehr umfangreiches Formular und hat darin man mehrere Fehleingaben, dann kann das ganz schön lange dauern bis man fertig ist. Früher konnte man die Fehlerliste abarbeiten und gut war's.

Post Reply