Falsch aufgerufene Smarty-Variable in xtc5/module/gift_cart.html

[Modulfux]

In der "/templates/xtc5/module/gift_cart.html" findet sich folgende Zeile:

Code: PHP  [Auswählen]

<p>{#text_send_to_friend#} <a href="{$GV_SEND_TO_FRIEND_LINK}">{#box_send_to_friend#}</a></p>

Die Smarty-Variable {$GV_SEND_TO_FRIEND_LINK} wird aber in der "/templates/xtc5/source/boxes/shopping_cart.php" folgendermaßen zugewiesen:

Code: PHP  [Auswählen]

$box_smarty->assign('GV_SEND_TO_FRIEND_LINK', '<a href="' . xtc_href_link(FILENAME_GV_SEND) . '">');

Somit würde im generierten Quellcode der "gift_cart.html" folgendes Konstrukt stehen:

Code: PHP  [Auswählen]

<a href="<a href="{$GV_SEND_TO_FRIEND_LINK}">{$GV_SEND_TO_FRIEND_LINK}">

Korrektur in der "gift_cart.html" wäre diese:

Code: PHP  [Auswählen]

<p>{#text_send_to_friend#} {$GV_SEND_TO_FRIEND_LINK}{#box_send_to_friend#}</a></p>

Das lustige daran ist, diesen Fehler gab es schon im xt:Commerce selber.

Gruß
Ronny

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

[DokuMan]

Hi Ronny,

danke für den Hinweis - hab ich im Trunk mit r2193 übernommen.
Ja, wer weiß, wie viele xt:Commerce-"Leichen" wir noch im Keller haben.

[h-h-h]

Na, da warst Du schneller, denn ich wollte es so lösen:

Code: PHP  [Auswählen]

$box_smarty->assign('GV_SEND_TO_FRIEND_LINK', xtc_href_link(FILENAME_GV_SEND));

Damit hätten wir eine sauberere Trennung zwischen HTML und PHP.

Gruß

h-h-h

[Modulfux]

Klar h-h-h,

die Lösung ist die eleganteste, dann sieht die Zerhackstückelung im HTML-Teil der "gift_cart" auch nicht so dämlich aus.

Gruß
Ronny

[DokuMan]

@h-h-h: Das wäre natürlich noch besser.
Magst du das entsprechend in den Tunk einpflegen?

[Tomcraft]

Jupp, wäre cool, wenn h-h-h das noch in den Trunk schieben würde.

Danke für's Melden, Ronny!

Grüße

Torsten

[h-h-h]

Habe gerade festgestellt, dass es sich hier um keinen Fehler handelt.
Ronny hatte sich nur mit der Datei-Zugehörigkeit versehen:

gift_cart.php

Code: PHP  [Auswählen]

    $gift_smarty->assign('GV_SEND_TO_FRIEND_LINK', xtc_href_link(FILENAME_GV_SEND));

gift_cart.html

Code: XML  [Auswählen]

    <p>{#text_send_to_friend#} <a href="{$GV_SEND_TO_FRIEND_LINK}">{#box_send_to_friend#}</a></p>

Absolut korrekt und wird in der "shopping_cart.php" (shop-root) eingebunden. Also revoke r2193 in r2194

shopping_cart.php (boxes)

Code: XML  [Auswählen]

$box_smarty->assign('GV_SEND_TO_FRIEND_LINK', '<a href="'.xtc_href_link(FILENAME_GV_SEND).'">');

Kann in der "box_cart.html" bei Bedarf im Template verwendet werden.^^


Was anderes hirnverbranntes habe ich dann gefunden:

gift_cart.php

Code: XML  [Auswählen]

  //BOC - web28 | GTB - 2010-11-05 - LOST cID / SSL FIX | correct href and added title tag
  //$gift_smarty->assign('COUPON_HELP_LINK', '<a target="_blank" class="thickbox" href="'.xtc_href_link(FILENAME_POPUP_COUPON_HELP.'?KeepThis=true&TB_iframe=true&height=400&width=600').'">Information</a>');
    $gift_smarty->assign('COUPON_HELP_LINK', '<a target="_blank" class="thickbox" title="Information" href="'.xtc_href_link(FILENAME_POPUP_COUPON_HELP, 'cID='.$_SESSION['cc_id'].'&KeepThis=true&TB_iframe=true&height=400&width=600', $request_type).'">Information</a>');
  //EOC - web28 | GTB - 2010-11-05 - LOST cID / SSL FIX | correct href and added title tag

Die Korrektur ist gut, aber ...

gift_cart.html

Code: XML  [Auswählen]

    <p>{#cart_coupon#} {$COUPON_HELP_LINK} {#cart_coupon_info#}</a></p>

Jetzt sagt mir mal wie man das abwärtskompatibel lösen kann?

Gruß

h-h-h

[DokuMan]

Abwärtskompatibel muss es nicht sein: wir ergänzen den Schritt einfach im Wiki, was beim Template zu ändern ist, wenn die modified eCommerce Shopsoftware Version aktualisiert wird.

[h-h-h]

Ok, hab den Fehler mal in r2198 korrigiert.

Gruß

h-h-h

[Tomcraft]

Prima, Danke!

Grüße

Torsten