Advancedform_XH 2.1

Ein CMSimple Support Forum für deutsch sprechende Nutzer und Entwickler
manu
Posts: 955
Joined: Wed Jun 04, 2008 12:05 pm
Location: St. Gallen - Schweiz
Contact:

Re: Advancedform_XH 2.1

Post by manu » Tue May 18, 2021 8:05 am

cmb wrote:
Thu May 06, 2021 10:10 pm
...
frase wrote:
Thu May 06, 2021 5:56 am
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.
Das ist ein Bug.
Hier ein Quickfix.
Ich würde auch die Version 2.2 als kritisch einstufen. Wenn ich mit einer Version <=2.0 rumgondle, ist jetzt im Updatecheck der Update gar nicht mehr kritisch, obschon mich mein Provider darauf hinweist.

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

Re: Advancedform_XH 2.1

Post by cmb » Tue May 18, 2021 9:00 pm

manu wrote:
Tue May 18, 2021 8:05 am
Ich würde auch die Version 2.2 als kritisch einstufen. Wenn ich mit einer Version <=2.0 rumgondle, ist jetzt im Updatecheck der Update gar nicht mehr kritisch, obschon mich mein Provider darauf hinweist.
Ja, sorry, dass ich noch nicht zur 2.2 kam – in den letzten Wochen stand wieder anderes auf dem Programm. Aber ich finde diesen Fehler weit weniger schlimm als die vermurkste 2.0, und der Fix sollte noch gründlich getestet werden (inklusive Unit-Test). Und ich glaube, dass es noch ein paar Probleme unter PHP 8.0 gibt.
Christoph M. Becker – Plugins for CMSimple_XH

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

Re: Advancedform_XH 2.1

Post by frase » Wed May 19, 2021 6:50 am

manu wrote:
Tue May 18, 2021 8:05 am
Hier ein Quickfix.
In der aktuellen Version scheint dieser Quickfix jetzt zu funktionieren. :D
cmb wrote:
Tue May 18, 2021 9:00 pm
Und ich glaube, dass es noch ein paar Probleme unter PHP 8.0 gibt.
Das befürchte ich auch - konnte aber noch nicht testen.
Das ist auch gar nicht so einfach, solange XH selbst nicht vollständig PHP8-kompatibel ist.
Wir sollten das also möglichst bald erledigen um anschließend die Plugins testen zu können.
PHP8 ist mittlerweile schon beinahe Standard bei den Providern.

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

Re: Advancedform_XH 2.1

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

Eine Kleinigkeit ist mir noch aufgefallen, im
Beispielformular Confirmation:
Config - Focus - Form: 1 (angehakt)

Bei Aufruf der Seite liegt der Fokus im Feld "Name".
Darüber befinden sich aber noch die Radio-Buttons (Title), die ebenfalls "required" sind.
Eigentlich müsste also der Focus hier auf dem ersten Radio-Button liegen. :?:

manu
Posts: 955
Joined: Wed Jun 04, 2008 12:05 pm
Location: St. Gallen - Schweiz
Contact:

Re: Advancedform_XH 2.1

Post by manu » Wed May 19, 2021 10:14 am

frase wrote:
Wed May 19, 2021 6:50 am
manu wrote:
Tue May 18, 2021 8:05 am
Hier ein Quickfix.
In der aktuellen Version scheint dieser Quickfix jetzt zu funktionieren. :D
Achtung! Christoph hat noch was entdeckt. Ich habs aktualisiert (~8:00 Uhr). Hast Du evtl. schon.
Die Korrekturen müssen so aussehen:

Code: Select all

$res = $this->checkMail($form, $field)] && $res;
Das boolsche UND am Ende ist wichtig.

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

Re: Advancedform_XH 2.1

Post by frase » Wed May 19, 2021 10:52 am

manu wrote:
Wed May 19, 2021 10:14 am
Achtung! Christoph hat noch was entdeckt. Ich habs aktualisiert (~8:00 Uhr). Hast Du evtl. schon.
:D
Klar. mein Post war von 8:50 - deshalb sprach ich auch von "der aktuellen Version".

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

Re: Advancedform_XH 2.1

Post by cmb » Wed May 19, 2021 11:24 pm

frase wrote:
Wed May 19, 2021 6:50 am
Das ist auch gar nicht so einfach, solange XH selbst nicht vollständig PHP8-kompatibel ist.
Wir sollten das also möglichst bald erledigen um anschließend die Plugins testen zu können.
Da kann ich nur voll zustimmen. Ich bitte alle mitzuhelfen, die offenen Issues zu lösen.
frase wrote:
Wed May 19, 2021 6:50 am
PHP8 ist mittlerweile schon beinahe Standard bei den Providern.
Soweit ich weiß, wird es noch dauern bis WordPress PHP 8.0 unterstützt; das dürfte die meisten Provider "zwingen", zumindest PHP 7.4 ebenfalls anzubieten.
frase wrote:
Wed May 19, 2021 7:05 am
Eine Kleinigkeit ist mir noch aufgefallen, im
Beispielformular Confirmation:
Config - Focus - Form: 1 (angehakt)

Bei Aufruf der Seite liegt der Fokus im Feld "Name".
Darüber befinden sich aber noch die Radio-Buttons (Title), die ebenfalls "required" sind.
Eigentlich müsste also der Focus hier auf dem ersten Radio-Button liegen. :?:
Interessant! Im Code werden Checkboxen, Radio-Buttons und Ausgabefelder explizit vom Fokussieren ausgeschlossen. Letzteres ist klar, aber warum die ersten beiden?

Ich bin sehr froh, dass ihr viele Probleme findet. Danke! :) Ich hoffe aber, dass es nicht zu weit geht. ;)
Christoph M. Becker – Plugins for CMSimple_XH

olape
Posts: 1778
Joined: Fri Mar 13, 2015 8:47 am
Contact:

Re: Advancedform_XH 2.1

Post by olape » Thu May 20, 2021 5:59 am

cmb wrote:
Wed May 19, 2021 11:24 pm
Da kann ich nur voll zustimmen. Ich bitte alle mitzuhelfen, die offenen Issues zu lösen.
Jetzt mal so, auf einen schnellen Blick, scheint es mir gar nicht mehr so viel.
Einiges ist zumindest schon begonnen. Anderes, wie das Template "richu172_2" ist nur eine Löschaktion.

Ich muss mir mal den aktuellen Stand ziehen.

Ergänzen würde ich vielleicht noch:

https://github.com/cmsimple-xh/cmsimple ... -782860424
viele 301 im content
Es wäre gut, wenn das auch korrigiert würde. Ist allerdings nicht lebensnotwendig.

doczimut
Posts: 73
Joined: Thu Dec 25, 2008 1:10 pm

Advancedform_XH 2.2 (accented characters)

Post by doczimut » Thu Jun 03, 2021 3:54 pm

Hello
It seems to me that accented characters are not accepted in the Name field (label_field), annoying!
Cannot create a First name control named "Prénom" in French.

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

Re: Advancedform_XH 2.1

Post by cmb » Thu Jun 03, 2021 5:43 pm

doczimut wrote:
Thu Jun 03, 2021 3:54 pm
It seems to me that accented characters are not accepted in the Name field (label_field), annoying!
Cannot create a First name control named "Prénom" in French.
The name fields (first input) only allow ASCII letters and digits, because these are used to create HTML IDs etc. However, the labels (second input) should accept all characters, and only these are shown to users.
Christoph M. Becker – Plugins for CMSimple_XH

Post Reply