Gutschein Meldung erscheint doppelt

[Goerni]

Hallo,

ich habe nun mal die neue Version des modified eCommerce Shopsoftware Shop aufgesetzt und der gefällt mir richtig gut. Vor allem das Backend ist nun endlich einmal nett anzusehen. Danke dafür!

Jetzt habe ich leider ein kleines Problem:

Die Meldung (Fehlermeldung) im Warenkorb, dass der Gutschein korrekt eingebucht wurde erscheint bei mir doppelt.

Siehe Screen:

[ Für Gäste sind keine Dateianhänge sichtbar ]

Ich verwende 1.03 und ein leicht angepasstes xtc5 Template.

Welche Dateien muss ich ändern damit diese errormsg nur 1x angezeigt wird?

Für Antworten bedanke ich mich recht herzlich im Voraus!



Linkback: https://www.modified-shop.org/forum/index.php?topic=3467.0

[Goerni]

Mhh, in der shopping-cart.html finde ich folgendes:

Code: PHP  [Auswählen]

{if $info_message!=''}<div class="errormessage">{$info_message}</div>{/if}
{if $error!=''}<div class="errormessage">{$error}</div>{/if}

Der Link, der die Seite aufruft sieht so aus:

www.domain.de/shopping_cart.php?info_message=Ihr+Coupon+wurde+erfolgreich+eingebuch
t+und+wird+bei+Ihrer+n%26auml%3Bchsten+Bestellung+automatisch+eingel%26ouml%3Bst.

Hier wird eine Info-Message im Link (per GET) übergeben und vermutlich wird dieselbe Variable nochmal per POST übergeben?

[Mike Randoo]

Hallo Goerni,
ich habe mir dadurch beholfen, dass ich die Zeile

Code: PHP  [Auswählen]

{if $error!=''}<div class="errormessage">{$error}</div>{/if}

aus der shopping_cart.html gestrichen habe. War die von beiden Möglichkeiten weniger schön aussehende (in meinen Augen).

Falls sich jemand auskennt und weiß, dass man ohne die "errormessage" zuviel Schaden anrichtet, lasse er es mich/uns bitte wissen.

Grüße,
Mike

[Goerni]

Mhh, ich denke nicht, dass dies der "richtige" Weg ist, da nun auch alle anderen Fehlermeldungen dann nich angezeigt werden.

Ich wüßte gerne wo genau diese "info_message=" Passage generiert wird.

@Mike: Welche Version hast Du denn?

[Goerni]

Ich habe das jetzt folgendermaßen gelöst:

Code: PHP  [Auswählen]

{if $info_message!=''}<div class="errormessage">{$info_message}</div>{/if}
{if $error!='' && $info_message==''}{$error}{/if}

So erscheint die (doppelte) errormsg nur, wenn die info_message leer ist. Ich denke damit kann ich leben.

Eine schönere Lösung wäre mir allerding lieber.

[Tomcraft]

Und was machst du, wenn {$error} und {$info_message} unterschiedlichen Inhalt haben?

Grüße

Torsten

[DokuMan]

Aber irgendwas müssen wir uns da noch einfallen lassen, eine doppelte Meldung sieht nicht schön aus.

[vr]

Im vorliegenden Fall waren Hinweis und Fehlermeldung anscheinend gleich, darauf könnte man testen. Aber ich frage mich, wieso eine Statusmeldung, die kein Fehler ist, es überhaupt in die $error-Variable schafft?

Grüße, Volker

[Hetfield]

In der header.php wird die Smarty-Variable {$error} bereits von $_GET['info_message'] so gefüllt und formatiert:

Code: PHP  [Auswählen]

if (isset($_GET['info_message']) && xtc_not_null($_GET['info_message'])) {
        $smarty->assign('error','<p class="errormessage">'.htmlspecialchars($_GET['info_message']).'</p>');
}

{$error} ist also nicht leer, da durch $_GET['info_message'] gefüllt. $_GET['info_message'] füllt aber auch die Smarty-Variable {$info_message}. Also sind weder {$info_message} noch {$error} leer, beinhalten jedoch die gleiche Meldung.

MfG Hetfield  

[vr]

Hmm, wenn wir Seiten haben, die das machen:

Code: XML  [Auswählen]

{if $info_message!=''}<div class="errormessage">{$info_message}</div>{/if}
{if $error!=''}<div class="errormessage">{$error}</div>{/if}

dürfte eine info_message eigentlich nicht noch zu einem error gemacht werden.

Code: PHP  [Auswählen]

if (isset($_GET['info_message']) && xtc_not_null($_GET['info_message'])) {
        $smarty->assign('error','<p class="errormessage">'.htmlspecialchars($_GET['info_message']).'</p>');

Vermutlich ist der Grund, dass nicht jede Seite eine info_message ausgeben kann, oder?

Grüße, Volker

[Tomcraft]

Wäre es dann nicht die sinnvollste Lösung die Stelle in der "header.php" ersatzlos zu streichen?

Code: PHP  [Auswählen]

if (isset($_GET['info_message']) && xtc_not_null($_GET['info_message'])) {
        $smarty->assign('error','<p class="errormessage">'.htmlspecialchars($_GET['info_message']).'</p>');
}

Grüße

Torsten

[Goerni]

Mhh, dann kommt aber keine Info-Message mehr durch, oder?!? Aber wenn ich Hetfields Antwort richtig interpretiere ist doch die Befüllung der $error Variable mit $_GET['info_message'] doppelt gemoppelt.

Somit tendiere ich zu Tomcrafts Vorschlag.

Müsste es denn nicht auch irgendwo eine

Code: PHP  [Auswählen]

$smarty->assign('info_message'<p class="infomessage">'.htmlspecialchars($_GET['info_message']).'</p>');}

geben?

[EDIT Tomcraft 21.01.2010: Full-Quote entfernt, siehe: FAQ: Full-Quotes und Groß-/Kleinschreibung - mit anderen Worten Netiquette!]

[geformt]

Moin,

hat schon jemand eine Lösung für dieses Problem gefunden?

[Tomcraft]

Nein, aber gut, dass du das Thema nochmal hoch geholt hast. Schon fast in Vergessenheit geraten.

Grüße

Torsten

[rattenfänger]

ich schiebe das Problem nochmal hoch. Gibt es schon einen Lösungsansatz?