FriendsOfREDAXO
MForm 7.1.1: Selects erhalten falsches Attribut
Diese Zeile https://github.com/FriendsOfREDAXO/mform/blob/40ef8f4d5554ae54ee80235408d47fcbea173ce6/lib/MForm/Parser/MFormParser.php#L295 sorgt dafür, dass data-toggle="collapse" an jedes Select-Element gehängt wird, unabhängig davon, ob es benötigt wird, oder nicht. Das hat zur Folge, dass hier https://github.com/FriendsOfREDAXO/mform/blob/40ef8f4d5554ae54ee80235408d47fcbea173ce6/assets/mform.js#L48-L53 alle Event-Listener, die evtl. zusätzlich zu denen von MForm verwendet werden, auch vom Select entfernt werden.
Aus meiner Sicht sollte das data-toggle-Attribut nur vergeben werden, wenn tatsächlich Toggle-Optionen existieren.
Evtl. ist es auch nicht verkehrt, für MForm-Event-Listener generell einen eigenen Namespace zu verwenden und nur den zu löschen, und nicht pauschal alle Event-Listener zu entfernen.
@DanielWeitenauer das Attribute wird doch nur geschrieben wenn getToggleOptions >= 0 ist.
@joachimdoerr das ist nicht ganz richtig. Die Zeile wird ausgeführt, wenn die Größe (sizeof) von getToggleOptions >= 0 ist, also immer.
@DanielWeitenauer kannst du mir bitte ein Modul Beispiel geben oder nochmal nach prüfen. Ich kann keine ungewünschten data-toggle="collapse" ausmachen.
@joachimdoerr Die fehlerhafte Zeile wurde in diesem Commit angepasst: 90d62d943099ae8a1c07add6c4f7f1e0c38e30bf. Das Attribut wird nun nur noch gesetzt, wenn tatsächlich Toggle-Options vorhanden sind. Das Problem sollte also erledigt sein.