Event wird nicht gespeichert, weil kein Haken bei "Event wiederholen"

[OppaBredow]

Wenn ich einen Termin bearbeite und aus Versehen den Haken bei "Event wiederholen" setze und sofort wieder entferne, wird mir das Speichern verweigert. Bilder dazu

und wenn ich dann den Haken wieder setze:

Viele Grüße
Oppa

[Aybee]

Sowas ähnliches hatten wir schon mal. Oben links im Browser kommt dann "Bitte füllen sie dieses Feld aus".

Demo:

1. Event erstellen
2. Titel eingeben "test"
3. "Repeat event" aufklappen und zuklappen
4. Speichern

[kubjo]

Habe das Gleiche Problem innerhalb von isotope, beim Anlegen von Varianten!
contao 3.5.6.
isotope 2.3.2.

[kubjo]

Das Problem bleibt auch nach dem isotope Update auf 2.3.3 bestehen. Ich denke, dass es ein contao Problem ist.

[leofeyer]

Wie lässt sich das in der Onlinedemo reproduzieren?

[OppaBredow]

Ich habe das gleiche Problem gestern bei einem normalen Artikel gehabt. War nicht zu lösen. Ich musste einen anderen Artikel kopieren und den bearbeiten. Ich fand auch nichts, was ungewöhnlich war und was auf eine Reproduzierbarkeit hingewiesen hätte.

[Aybee]

@leofeyer - Wie lässt sich das in der Onlinedemo reproduzieren?

Siehe meine Anleitung in diesem Ticket.

@OppaBredow - Ich habe das gleiche Problem gestern bei einem normalen Artikel gehabt.

Ich hatte diese Woche auch so einen Fall, aber zum ersten Mal.

[leofeyer]

Das Problem ist, dass durch das Nachladen der Palette "Event wiederholen" zwei Pflichtfelder mit required-Attribut nachgeladen werden, die beim Ausblenden der Palette nicht entfernt sondern nur auf display:none gesetzt werden. Da die Felder aber noch im DOM sind, kommt es zu einem An invalid form control with name='repeatEach[value]' is not focusable.-Fehler.

@contao/developers Wie beheben wir das am besten? Sollten wir das required-Attribut beim Zuklappen der Palette entfernen? Wie würden wir es dann beim Aufklappen wieder setzen?

[aschempp]

<form novalidate>

[leofeyer]

Das bringt gar nichts, denn dann werden ja auch die anderen Felder nicht validiert.

[ausi]

Ich denke anstatt den Teil auf display: none zu setzen, sollte das JS den Bereich aus dem DOM entfernen. Man könnten den Bereich ja trodzdem im JS zwischenspeichern und beim erneuten aktivieren des Bereichs das Element wieder ins DOM einfügen.

[mwitzmann]

Interessant wäre für Entwickler auch die Möglichkeit "novalidate" für DCA's aktivieren/deaktivieren zu können.

[Toflar]

Ich denke anstatt den Teil auf display: none zu setzen, sollte das JS den Bereich aus dem DOM entfernen. Man könnten den Bereich ja trodzdem im JS zwischenspeichern und beim erneuten aktivieren des Bereichs das Element wieder ins DOM einfügen.

+1. @leofeyer imho ein ziemlich nerviger Bug. 3.5.13? :)

[leofeyer]

Ich habe in #8366 ein PoC hochgeladen, damit wir besprechen können, ob der Weg richtig ist. Der PoC arbeitet mit der Navigation, aber das Prinzip ist ja überall gleich.

[leofeyer]

Die Diskussion geht in #8366 weiter.

[leofeyer]

Ich habe in 01ab76c erstmal die einfache Lösung implementiert.