Anführungszeichen in title- und alt-Tags maskieren

[speedy]

Hallo,

modified eCommerce Shopsoftware hat noch ein Problem mit Anführungszeichen " " in z.B. Produktnamen.
Das Problem ist hier beschrieben: http://www.traum-projekt.com/forum/89-content-management-systeme-and-shop/117996-anfuehrungszeichen-in-alt-tags.html

Alt- und Title-Tags werden dadurch verunstaltet und sorgen auch für nicht-validen Code.

Ein Beispiel aus dem Thread:

Code: PHP  [Auswählen]

<a href="http://www.DiesIstEinBeispiel.de/shop/product_info.php?info=p1499_Postkarte--Weihnachtsmann-.html"><img src="images/product_images/thumbnail_images/1499_0.jpg" alt="Postkarte " weihnachtsmann="" class="thumb_img"></a>

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

[DokuMan]

Code: PHP  [Auswählen]

{$PRODUCTS_NAME|escape:"htmlall"}

löst es nicht?

[speedy]

Hallo DokuMan,

ich weiß nicht. Ich konnte

Code: PHP  [Auswählen]

htmlall

nirgends in modified eCommerce Shopsoftware 1.05 finden, zumindest nicht in Zusammenhang mit Title, oder Alt-Tags.
Nur bei drei xtbooster Dateien und zwei Smarty-Dateien kommt "htmlall" vor, tlw. aber nur als Kommentar.

Auch im Trunk-Demoshop gibt es das Problem.

[DokuMan]

Es ist nicht im Trunk.
Es sollte ggf. so mit der product_info_v1.html funktionieren:
statt

Code: PHP  [Auswählen]

alt="{$PRODUCTS_NAME}

also

Code: PHP  [Auswählen]

alt="{$PRODUCTS_NAME|escape:"htmlall"}

das Gleiche sollte beim title-tag auftreten.

[speedy]

Achso, ist nur die Frage, ob man das im Template lösen sollte.
Im Template gibt es $PRODUCTS_NAME nur in print_product_info, product_info, product_reviews
Der Rest wäre dann z.B. $module_data.PRODUCTS_NAME
Oder korrigiert man es nicht besser im Core wie bei Shopstat bei der Erzeugung der Tags.

[web28]

Bei den ALT und TITLE Tags ist das nur im Template lösbar, dazu muss man aber erst alle Stellen suchen und finden wo der Produktname in TITLE oder ALT Tag geschrieben wird.

Für die Metatags müsste die metatags.php bearbeitet werden.

Alles in allen bleibt als Fazit:

Doppelte Anführungszeichen im Produkt/Kategorienamen NICHT verwenden, ansonsten müssen die Anführungszeichen mühsam rausgefiltert werden.
Damit werden sie also sowieso nicht angezeigt!

Gruss Web28

[speedy]

Das kann man leider nicht vermeiden, würde unmöglich aussehen.

[speedy]

Hat keiner eine Idee ?
Kann man die Anführungszeichen nicht systemweit gegen den HTML-Code ersetzen oder es maskieren ?
&quot ;
http://www.digitalmediaminute.com/reference/entity/index.php

[Hetfield]

In der includes/classes/product.php nach dieser Zeile suchen:

Code: PHP  [Auswählen]

return array ('PRODUCTS_NAME' => $array['products_name'],

und davor diese Zeile einfügen:

Code: PHP  [Auswählen]

$array['products_name'] = htmlentities($array['products_name'],  ENT_QUOTES);

Ungetestet, sollte aber funktionieren!  

MfG Hetfield

[speedy]

Hallo Hetfield,

vielen Dank, hat aber leider zumindest in der product_info keine Auswirkung gezeigt.
Der Validator meckert noch genau so viel Fehler an

[Hetfield]

in der product_info.php wird {$PRODUCTS_NAME} nochmal gebildet. Also dort nochmal nach dieser Zeile suchen:

Code: PHP  [Auswählen]

$info_smarty->assign('PRODUCTS_NAME', $product->data['products_name']);

und gegen diese ersetzen:

Code: PHP  [Auswählen]

$info_smarty->assign('PRODUCTS_NAME', htmlentities($product->data['products_name'],  ENT_QUOTES));

MfG Hetfield

[speedy]

Perfekt, Hetfield, das war's - vielen Dank. Funktioniert perfekt bei Title und Alt-Tags.
Könnt Ihr gerne in den Trunk übernehmen, dann würdet Ihr verhindern, dass die Links zerschossen werden und auch der Validator meckert

Für andere es ist die /includes/modules/product_info.php gemeint

[Tomcraft]

Ich setze es mal auf die Roadmap.

Grüße

Torsten

[DokuMan]

Bug gefixt -> r1517
und Roadmap 1.0x aktualisiert.

[voodoopupp]

ja, ich weiß: das Thema ist bereits mehr als alt. Aber kann einer vielleicht ne Info dazu geben, warum der Fix r1517 durch den r1525 wieder revidiert wurde? Bzw. wie werden denn nun die Anführungszeichen W3C konform ausgelesen?

Denn bei unserer Seite wird das leider irgendwie immer moniert....

Grüße
Dominik