Opened 8 years ago
Last modified 6 years ago
#1375 new Bug/Fehler
class messageStack doppelt
| Reported by: | Owned by: | somebody | |
|---|---|---|---|
| Priority: | normal | Milestone: | modified-shop-2.1.0.0 |
| Component: | Shop | Version: | 2.0.3.0 |
| Keywords: | Cc: | ||
| Blocked By: | Blocking: |
Description
die Klasse messageStack gibt es 2x; einmal in includes/classes und in admin/includes/classes.
Leider sind die aber nicht identisch und haben stellenweise unterschiedliche Funktions-Parameter, z. B.;
function add($class, $message, $type = 'error') {
...
und
function add($message, $type = 'error') {
...
Ich habe den Sinn nicht geprüft, finde aber, dass Klassenbezeichnungen im Shop nicht doppelt vorkommen sollten. Im Endeeffekt hängt der erfolgreiche Aufruf einer Klasse/Funktion sonst davon ab, ob ein Codeteil im Admin oder im Shopbereich ausgeführt wird.
Attachments (0)
Change History (4)
comment:1 by , 8 years ago
comment:2 by , 8 years ago
Ergänzung:
die messages in MessageStack sollten ohne HTML-Code auskommen. Formatierung ist Sache des Frontend, ggf. ein weitere Klasse zur Formatierung.
Nachteil bei vorhandenen HTML-Formatierungen sind die schlechte Übertragbarkei in andere Systeme. DieNachricht selber enthält genug Infos.
comment:3 by , 8 years ago
| Milestone: | modified-shop-2.0.4.0 |
|---|
comment:4 by , 6 years ago
| Milestone: | → modified-shop-2.1.0.0 |
|---|

Es gibt durchaus einige Funktioen und Klassen die doppelt sind.
Wo es möglich war habe wir diese schon vereinheitlicht.
In der messageStack habe wir das Problem mit den Methoden Parametern, das wurde hier total verbockt, wieder eine alte xtc Erblast.