BUG: Fehlerhafte ALT- und TITLE-Tags

[innuXTC]

Hallo Zusammen,

beim 'basteln' ist mir ein kleiner, süßer "Käfer" über den Weg gelaufen

Diesen "Bug" konnte ich soeben im aktuellem Demo-Shop nachstellen.

Vergibt man Artikelnamen mit Anführungszeichen, z.B.: Holzkiste "Mozart", dann wird folgender Quelltext für die Bilder/Fotos generiert:

Code: PHP  [Auswählen]

<a class="thickbox" rel="1" title="Holzkiste " Mozart""="" href="images/product_images/popup_images/1_1.jpg">
<img src="images/product_images/info_images/1_1.jpg" alt="Holzkiste " Mozart""="" align="left"></a>

Der ALT- und der TITLE-TAG wird beimm ersten Anführungszeichen beendet und man erhält - in diesem Fall - einen MOZART-TAG, wenn auch einen fehlerhaften...

Vg, innuXTC

P.S.: Möglicherweise führen die Anführungszeichen im Artikel-Namen auch an anderen Stellen zu Problemen..?!

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

[fishnet]

Wie so oft:
- gelöst
- auf die Roadmap gesetzt
-vergessen

Anführungszeichen in title- und alt-Tags maskieren

[innuXTC]

Wie so oft

Du meinst "So wie immer!" *lach*

Ich 'programmiere' z.Zt. eine komplett neue "Bilder-Anzeige" für die Produkt-Info. Da habe ich das Problem bereits gelöst:

Sonderzeichen usw. werden dabei herausgefiltert und/oder umgeschrieben. Der Hauptgrund für diesen Schritt war jedoch, dass bei mir die Bilder direkt entsprechend umbenannt und gespeichert werden in z.B. "Kamera_MarkeX_ModellX_Foto_1.jpg".

Bilder wie "Käse-Straße Gemälde.jpg" würde in diesem Fall zu "Kaese_Strasse_Gemaelde_Foto_1.jpg" umbenannt.

Wenn ich fertig bin und interesse besteht, stelle ich das Script für "Testzwecke & Weiterentwicklung" gerne zur Verfügung.

Meine Version ist bislang nicht zu 100% XTC-Komform, jedoch zu 100% kompatibel.... hoffe ich

Vlg, innuXTC

[noRiddle (revilonetz)]

Am einfachsten wäre das ganze zu lösen indem man in der /includes/classes/product.php analog zu

Code: PHP  [Auswählen]

'PRODUCTS_IMAGE_TITLE' => str_replace('"','',$array['products_name']),

ein PRODUCTS_IMAGE_ALT ins Array mit aufnimmt und auch dort mittels str_replace() die Ersetzung vornimmt, in der /includes/modules/product_info.php dann ebenfalls einen alt-Inhalt definiert und dann im Template eben an den entsprechenden Stellen das neue Smarty benutzt.
Man könnte es auch anstelle von str_replace() mit htmlentities() machen (dabei ab PHP 5.4  Kodierungsangabe nicht vergessen ($_SESSION['language_charset'])).

Da evtl. auch meta-tags betroffen sein können am Besten mit der zweitgenannten Variante machen und dafür lediglich PRODUCTS_NAME mit htmlentities() versehen.

Gruß,
noRiddle