Anführungszeichen in title- und alt-Tags maskieren

[Sailor]

Cool, gestern hatte ich genau den selben Gedanken und wollte gerade das posten

Ich habs jetzt erstmal bei uns gefixt, damit die HTML-Fehler weniger und wir wieder besser gelistet werden-

[Hobbybastler]

Code: PHP  [Auswählen]

{$PRODUCTS_NAME|escape:"htmlall"}

löst es nicht?

Hallo

Das löst das Problem beim ALT Atribut

Code: PHP  [Auswählen]

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

und das beim Title Atribut

Code: PHP  [Auswählen]

<a title="{$PRODUCTS_NAME|escape:"htmlall"}" href="

Dateien die geändert werden müssen:
box_whatsnew.html
product_listing_v1.html
product_info_tabs_v1.html
product_info_v1.html
product_info_x_accordion_v1.html
new_products_default.html
new_products.html
new_products_overview.html
specials.html

ich finde es immer besser im Template das selber zu lösen, da bei einem Update immer die .php Dateien angepasst werden müssen.....

Alternative müsste man halt immer mehr Änderungen in den Core einfliesen lassen.

gruß
Fritz

[Bonsai]

Das sollte meiner Meinung nach nicht von jedem einzeln gefixt werden, sondern es sollte generell funktionieren!

Wenn man z.B. Bücher oder Filme verkauft, hat man des öfteren im Titel blödsinnige Zeichen.

Also z.B. so etwas:
" ' &

Der Titel verliert ohne diese Zeichen eventuell enorm an Aussagekraft ....
Beispiel:
Im Namen Gottes <- klingt nach religiösem Buch
Im Namen "Gottes" <- klingt nach religionskritischem Buch

Also sämtliche Aussagen "solche Zeichen nicht zu verwenden" sind hiermit entkräftet.

Warum ist das im Standardtemplate nicht schon gefixt?  Der Fehler ist doch schon eine Weile bekannt.

@Hobbybastler  Gleich mal Lesezeichen setzen und auf die todo Liste packen ....

[Hobbybastler]

@Bonsai

Das ist aber nicht der Artikeltitel (Artikelname), der wird angezeigt wie er soll ("Gottes") und ist valide,
Sondern der Titel für den Link und das alt Atribut für das Artikelbild!

Und das Standardtemplate kann auch nix dafür, das müsste, wenn, im Core gefixt werden.
Ich bevorzuge hier meist die Template/Smarty Lösung, da der core unberührt bleibt, wegen Updates.

Gruß Fritz

[Bonsai]

Ich weiß schon um was es geht .... Da ich den Artikel im Shop habe
[ Für Gäste sind keine Dateianhänge sichtbar ]

Da ich sowieso ein neues eigenes Template bauen will, bleibt das erst mal so. Da fließt dann in das neue Template ein. Ich ärgere mich nur darüber, dass jeder der das xtc5 Template als Vorlage benutzt diesen Bug gleich mit umsetzt .... So etwas muss nicht sein bei bekannten Fehlern.

[Bonsai]

Info für alle die diesen Thread finden:
Keine Korrektur

Code: PHP  [Auswählen]

{$PRODUCTS_NAME|escape:"htmlall"}
alt="{$module_data.PRODUCTS_NAME|escape:'htmlall'}"
title="{$module_data.PRODUCTS_NAME|escape:'htmlall'}"
 

Liste verlängert:
Dateien die geändert werden müssen:
box_whatsnew.html
product_listing_v1.html
product_info_tabs_v1.html
product_info_v1.html
product_info_x_accordion_v1.html
new_products_default.html
new_products.html
new_products_overview.html
specials.html
cross_selling.html
reverse_cross_selling.html
upcoming_products.html

[Bonsai]

Liste nochmal verlängert:
Dateien die geändert werden müssen:
also_purchased.html
box_whatsnew.html
box_best_sellers.html
box_last_viewed.html
box_popcart.html
product_listing_v1.html
product_info_tabs_v1.html
product_info_v1.html
product_info_x_accordion_v1.html
product_reviews.html
product_reviews_info.html
product_reviews_write.html
reviews.html
new_products_default.html
new_products.html
new_products_overview.html
specials.html
cross_selling.html
reverse_cross_selling.html
upcoming_products.html
order_details.html
print_order.html
print_product_info.html

Wenn das Modul elektronische Widerrufsformular eingebaut ist noch:
widerruf_step2.html

[Bonsai]

Wichtig! In der box_best_sellers.html darf man das escape:"htmlall" NICHT beim Produktlink einbauen, weil es sonst zu Darstellungsfehlern kommt durch den textformat smarty. Allerdings kann man den alt Tatg füllen der hier fehlerhafterweise nicht gefüllt ist.      

Code: PHP  [Auswählen]

<dt><a href="{$box_data.PRODUCTS_LINK}">{if $box_data.PRODUCTS_IMAGE}<img src="{$box_data.PRODUCTS_IMAGE}" alt="{$box_data.PRODUCTS_NAME|escape:"htmlall"}" title="" />{/if}</a></dt>
      <dd>
        <h3>{$box_data.COUNT}. <a href="{$box_data.PRODUCTS_LINK}">{textformat wrap=18 wrap_cut=true}{$box_data.PRODUCTS_NAME}{/textformat}</a></h3>
 

[noRiddle (revilonetz)]

Also man kann das mit "nichts am Core ändern" auch übertreiben.
Bevor ich zig Template-Dateien anfasse mache ich einfach das:
In /includes/classes/product.php in der Funktion buildDataArray() das suchen

Code: PHP  [Auswählen]

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

und daraus das machen

Code: PHP  [Auswählen]

'PRODUCTS_NAME' => htmlentities($array['products_name'], ENT_QUOTES, $_SESSION['language_charset']),

(somit auch PHP 5.5 aufwärts kompatibel)

Daß das noch immer nicht gefixt ist... tststs 

Gruß,
noRiddle

[Bonsai]

[noRiddle (revilonetz)]

Man muß das dann auch in /includes/modules/product_info.php machen (für die product_info-Templates):

Code: PHP  [Auswählen]

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

da dort der products_name aus dem Array $products->data geholt wird und die Funktion buildDataArray() an der Stelle nicht benutzt wird.

Noch sauberer wäre vielleicht sich einen extra alt-tag zu definieren.

Gruß,
noRiddle