search

terminal42 / contao-conditionalformfields

Display form fields conditionally in Contao Open Source CMS
MIT License
18 stars 13 forks source link

Not allowed characters #85

Closed zonky2 closed 4 months ago

aschempp commented 7 months ago

Der Hinweis stimmt nicht wirklich. Das Feld muss einen gültigen Namen für eine Javascript-Variable haben. Mittlerweile wird da aber im Backend validiert, ich weiss nicht ob der Hinweis so verständlich ist?

zonky2 commented 7 months ago

Ich hatte das "Phänomen" bei einer aktuellen 4.13, wo Redakteure eben genau solche Feldnamen eingesetzt haben und damit einen JS-Error erzeugten

https://www.w3schools.com/js/js_variables.asp#midcontentadcontainer

Names must begin with a letter.

zonky2 commented 7 months ago

hmm... sehe gerade meinen Text nochmal durch - ein 2foo ging nur 2_foo nicht... komisch

aschempp commented 7 months ago

am Ende hängt es damit zusammen, was in Javascript als Variabeln-Namen erlaubt ist. Wie gesagt wurde das in der neusten Version verbessert, da sollten jetzt keine Fehler mehr kommen. Nicht-unterstützte Felder werden ignoriert, und die Eingabe der Formel validiert dass nur gültige Felder verwendet werden.

zonky2 commented 7 months ago

o.k. - teste ich bei Gelegenheit