Opened 3 years ago

Last modified 2 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

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 Changed 3 years ago by web28

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 Changed 3 years ago by webald <elertsm@…>

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 Übertragbarkeit in andere Systeme. Die Nachricht selber enthält genug Infos.

Last edited 3 years ago by Tomcraft (previous) (diff)

comment:3 Changed 3 years ago by Tomcraft

  • Milestone modified-shop-2.0.4.0 deleted

comment:4 Changed 2 years ago by GTB

  • Milestone set to modified-shop-2.1.0.0

Add Comment

Modify Ticket

Action
as new
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.