Ungültige Eingaben werden gespeichert

[pcartist]

Hallo,

schon zum zweiten mal wird mir die Seite kurzzeitig zerschossen, weil Redakteure ungültige Eingaben im Backend speichern können.
Im aktuellen Fall wurde bei den unregelmäßigen Wiederholungen Monat und Jahr mit 2 Punkten getrennt.

Beim Versuch den Eintrag zu speichern wird zwar ein Fehler angezeigt, gespeichert wird aber dennoch. Klick man also auf "Speichern & schließen" bemerkt man den Fehler erst, wenn das Frontend zerschossen ist.

Beim letzten mal wurde eine regelmäßige Wiederholung ohne Angabe eines Intervalls gespeichert, was dann zu einer schönen Endlosschleife bis zu Timeout geführt hat.

[kmielke]

Hi,

welche Version setzt du ein von
- der Erweiterung kmielke/calendar_extended
- der Erweiterung menatwork/contao-multicolumnwizard

Gruß Kester

[pcartist]

Hallo Kester,

hier die Versionen:

menatwork/contao-multicolumnwizard 3.3.10
kmielke/calendar_extended 3.5.16.1

Contao ist die 3.5.24


Danke vorab für Deine Mühen!

[kmielke]

Ist in der aktuellen dev Version gefixt.

Gruß Kester

[pcartist]

Hallo Kester,

leider konnte ich bei den unregelmäßigen Wiederholungen auch nach dem Update ein ungültiges Datum 01..01.2018 speichern. Zwar bekommt man im BE den Hinweis auf einen ungültigen Eintrag, wenn man nur auf "Speichern" klickt, der Wert wird aber dennoch in der DB abgelegt, was im FE zu einer Fehlermeldung führt.

Fatal error: Uncaught exception OutOfBoundsException with message Invalid date "01..01.2018 00:00" thrown in system/modules/core/library/Contao/Date.php on line 438
#0 system/modules/core/library/Contao/Date.php(79): Contao\Date->dateToUnix()
#1 system/modules/calendar_extended/classes/EventsExt.php(392): Contao\Date->__construct('01..01.2018 00:...', 'd.m.Y H:i')
#2 system/modules/calendar_extended/modules/ModuleEventlist.php(201): Contao\EventsExt->getAllEventsExt(Array, 1491151020, 1506962219, Array)
#3 system/modules/core/modules/Module.php(287): Contao\ModuleEventlist->compile()
#4 system/modules/calendar_extended/modules/ModuleEventlist.php(112): Contao\Module->generate()
#5 system/modules/core/library/Contao/Controller.php(321): Contao\ModuleEventlist->generate()
#6 system/modules/core/library/Contao/InsertTags.php(447): Contao\Controller::getFrontendModule('19')
#7 system/modules/core/library/Contao/Controller.php(692): Contao\InsertTags->replace('...', false)
#8 system/modules/core/classes/FrontendTemplate.php(100): Contao\Controller::replaceInsertTags('...', false)
#9 system/modules/core/pages/PageRegular.php(190): Contao\FrontendTemplate->output(true)
#10 system/modules/core/controllers/FrontendIndex.php(285): Contao\PageRegular->generate(Object(Contao\PageModel), true)
#11 index.php(20): Contao\FrontendIndex->run()
#12 {main}

Bildschirmfoto 2017-04-02 um 18.38.08.png

[kmielke]

Was passiert bei Speichern und Schließen?

Bei mir war es nicht mehr möglich das Formular zu verlassen. Man muss also was richtiges eingeben.

Gruß Kester

[pcartist]

Auch dann wird der Wert gespeichert. Wenn man das Event dann erneut öffnet, ist die Anzeige wie im Screenshot.
Sehr seltsam...

[Spooky]

@kmielke das Formular kannst du immer verlassen - die Fehlermeldung alleine hindert dich ja daran nicht.

[kmielke]

@kmielke das Formular kannst du immer verlassen - die Fehlermeldung alleine hindert dich ja daran nicht.

und was kann man machen? ich habe ein save_callback(), die eine Exception schmeisst, wenn was nicht stimmt. Was geht denn sonst noch?

Gruß Kester

[PaddySD]

Du müsstest in dem Fall über den onsubmit_callback (global) gehen. Der save_callback hat den Nachteil, genau was hier passiert, dass die Daten trotzdem schon gespeichert sind. Wenn Du auf die Daten vor dem speichern drauf willst, dann nur über den onsubmit_callback.

Ist zwar kein Field-Callback, aber sollte gehen, um damit zu arbeiten....

[kmielke]

Hallo,

der Bug ist gefixt und in der aktuellen DEV Version im Git.

Gruß Kester