contao/import — Issue #3568: Verbesserung der Änderungen aus r1085
Author: importer
Created: 2011-11-29 21:48:36+00:00
State: open
Labels: Feature, framework

Bei den Änderungen in c00cf7bd04fdb01cc6b8ae076cc75225 an der System.php sind mir ein paar Dinge aufgefallen, die man Verbessern könnte (patch siehe Anhang).

Änderungen die ich gemacht habe:
* Die Nachrichtengruppen wurden bei resetMessage auf einen leeren String gesetzt. Geprüft wird jedoch immer, ob es ein array ist oder nicht, was nach einem reset damit natürlich immer schief geht -> resetMessage setzt auf array() statt auf ''
* addXXXXMessage war immer gleich aufgebaut, nur das der Key ein anderer war -> doppelten Code ausgelagert
* Nachrichtengruppen als Klassenvariable festgelegt, so dass erbende Klassen einfach neue anlegen können (sofern benötigt), ohne das die System.php angefasst werden muss oder unnötig viel abgeleitet werden muss
* Wenn

![]($_POST gilt, wurden alle Nachrichten gelöscht - aber verteilt. Dies habe ich jetzt zentral über resetMessages gelöst (tritt der Fall )$_POST eigentlich jemals auf? Denn die Variable existiert ja eigentlich immer [Kommentar wäre an der Stelle evtl. gut, wann das auftritt])

Was man noch hätte machen können: Die Session-Nachrichtengruppen einmalig anlegen, wodurch man sichergehen könnte, dass sie garantiert gesetzt sind und vom Typ array sind, wodurch man einige Überprüfungen wegfallen lassen könnte. Ich weiß aber gerade nicht, wie oft Nachrichten darüber angezeigt werden, also ob es eher ein Performancevor- oder nachteil wäre.

[Download the attachments](http://www.contao.org/tl_files/tickets/3568.zip "Download the attachments of this ticket")

--- _Originally created by SunBlack on October 21st, 2011, at 04:26pm (ID 3568)_