BUG: Fehlende Statusmeldung -> Gutschein-Code eingelöst

[Barcelo]

Moin Moin,

im Warenkorb hat man ja die Möglichkeit einen Gutschein-Code einzulösen.

Ich bekommen jedoch keine Statusmeldung angezeigt wenn ich einen Gutschein-Code eingelöst habe.

In der shopping_cart.html ist das wohl dieser Part:

Code: PHP  [Auswählen]

{if $info_message_1!=''}<div class="errormessage">{$info_message_1}{$min_order}{$info_message_2}{$order_amount}</div>{/if}

In der URL steht nach dem Absenden folgendes:
shopping_cart.php?info=1info_message=Ihr+Gutschein+wurde+erfolgreich+auf+Ihr+Konto+verbucht.+Gutscheinwert%3A+10%2C00+EUR

Der Gutschein wird verbucht, aber wie gesagt ohne Status-Info.

Kennt jemand dieses Szenario???

Gruß Barcelo

EDIT:
Hab' Eben mal zum Spaß die URL wie folgt abgeändert:

shopping_cart.php?info=1&info_message=Ihr+Gutschein+wurde+erfolgreich+auf+Ihr+Konto+verbucht.+Gutscheinwert%3A+650%2C00+EUR

-> Also ein "&" zwischen "info=1" und "info_message" eingefügt und siehe da: Meldung wird im Template ausgespuckt.

Nun also die Frage: wo könnte/muss das "&"-Zeichen fehlen...?

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

[Barcelo]

Habe den Fehler eben gefunden - ist ein BUG in der xtc_collect_posts.inc.php !

Habe eben mit der xtc_collect_posts.inc.php aus dem Original-Paket modified-shop_1.06-r4642_SP2_install.zip verglichen.

Zeile 80:

Code: PHP  [Auswählen]

          xtc_redirect(xtc_href_link(FILENAME_SHOPPING_CART, 'info=1info_message=' . urlencode(REDEEMED_AMOUNT. $xtPrice->xtcFormat($gv_amount,true,0,true)), 'NONSSL')); // web28 - 2011-04-13  New  class distinction  error / info // web28 - 2011-04-14 - change SSL -> NONSSL

muss lauten:

Code: PHP  [Auswählen]

          xtc_redirect(xtc_href_link(FILENAME_SHOPPING_CART, 'info=1&info_message=' . urlencode(REDEEMED_AMOUNT. $xtPrice->xtcFormat($gv_amount,true,0,true)), 'NONSSL')); // web28 - 2011-04-13  New  class distinction  error / info // web28 - 2011-04-14 - change SSL -> NONSSL

Kann das dann bitte der Mod dieses Forums an die Dev's weitergeben?

Gruß Barcelo

[noRiddle (revilonetz)]

Benutze bitte dazu den "Moderator informieren"-Button, dafür ist der u.a. da.

In der kommenden Version werden Meldungen nicht mehr über GET-Parameter in der URL im Template ausgegeben werden, zumindest habe ich läuten hören, daß man das abschaffen möchte (wie z.B. auch die Error-Meldung auf der checkout_payment.php bei nicht ausgewählter Zahlungsart).

Gruß,
noRiddle

[Tomcraft]

Danke für den Hinweis!

Der Fix wurde soeben in r6871 im SVN korrigiert und wird mit dem SP3 ausgeliefert werden.

Grüße

Torsten

[Caro]

Hallo,
bei Download von Zip-Archiv erhalte ich folgende Meldung.

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

[Barcelo]

@noRiddle
Danke für die Infos.
Den Link hatte ich glatt übersehen & auch nicht daran gedacht

@Tomcraft
Gerne doch 

[Tomcraft]

Hallo,
bei Download von Zip-Archiv erhalte ich folgende Meldung.
[...]

Unser SVN ist nicht mehr öffentlich zugänglich. Die notwendigen Änderungen findest du aber in Beitrag #2.

Grüße

Torsten

[Caro]

Danke für die Antwort Tomcraft
ich dachte das wäre mehr zu ändern.

Danke auch an Barcelo.

wenn ich Fragen darf

und wird mit dem SP3 ausgeliefert werden.

wann ist der Auslieferungstermin ?

[SonGoku]

@ Caro
nimm das als Leitfaden

Keine Antwort ist auch eine Antwort und die wäre in dem Fall womöglich: kein Interesse! Aber hey, lass den Kopf nicht hängen! Je mehr Beiträge du schreibst, desto höher ist die Wahrscheinlichkeit, dass du auch eine Antwort bekommst. Hau‘ in die Tasten und probier‘ verschiedene Schreibstile aus: Sei nett, freundlich, ironisch, romantisch, draufgängerisch. Aber bleib‘ immer du selbst. Und wenn auch dann keine Antwort kommt – schreib noch mehr Beiträge! Die bekannten Mottos „Quantität vor Qualität“, „Masse statt Klasse“ oder „Manchmal ist mehr mehr“ zahlen sich bestimmt irgendwann aus.

[Caro]

ich merke schon da ist was drann

Keine Antwort ist auch eine Antwort und die wäre in dem Fall womöglich: kein Interesse!

Naja Zahlende Kunden sind mir auch Lieber als Schmatrotzer.

so ist das Leben halt.

aber Danke für die Antworten die eventl. kommen.

[Barcelo]

Nachtrag:

Zeile 135

Code: PHP  [Auswählen]

        xtc_redirect(xtc_href_link(FILENAME_SHOPPING_CART, 'info_message=' . urlencode(REDEEMED_COUPON), 'NONSSL')); // web28 - 2011-04-14 - change SSL -> NONSSL

Müsste dann auch wie folgt sein, wenn ich mich nicht irre. 

Code: PHP  [Auswählen]

        xtc_redirect(xtc_href_link(FILENAME_SHOPPING_CART, 'info=1&info_message=' . urlencode(REDEEMED_COUPON), 'NONSSL')); // web28 - 2011-04-14 - change SSL -> NONSSL

Gruß Barcelo

[Tomcraft]

Nicht notwendig, da der Parameter "info" in Shopversion 1.06 eh nicht genutzt wird. War mal angedacht, aber wird wohl anders gelöst werden. Wichtig ist wirklich nur der Fix, den du in Beitrag 2 genannt hattest, da ansonsten die "info_message" gar nicht angezeigt wurde.

Grüße

Torsten