Modify

Opened 8 years ago

Last modified 6 years ago

#1375 new Bug/Fehler

class messageStack doppelt

Reported by: webald <elertsm@…> 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 Ronald Parcinski, 8 years ago

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.

comment:2 by webald <elertsm@…>, 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.

Version 0, edited 8 years ago by webald <elertsm@…> (next)

comment:3 by Torsten Riemer, 8 years ago

Milestone: modified-shop-2.0.4.0

comment:4 by Gerhard Waldemair, 6 years ago

Milestone: modified-shop-2.1.0.0

Modify Ticket

Action
as new The owner will remain somebody.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.