Bugfix - bei Hochkomma in Optionswertname

[web28]

User snowseeker hat wohl einen Bug gefunden:

Wenn man bei den Optionswerten etwas nach diesem Muster '09 oder abc' eingibt, kommt beim Warenkorb diese Fehlermeldung:

Code: [Auswählen]

1064 - You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '09' AND pa.products_id='73'' at line 11

SELECT pa.attributes_model FROM products_attributes pa INNER JOIN products_options po ON po.products_options_id = pa.options_id INNER JOIN products_options_values pov ON pa.options_values_id = pov.products_options_values_id WHERE po.language_id = '1' AND po.products_options_name = 'downloads' AND pov.language_id = '1' AND pov.products_options_values_name = 'Pages '09' AND pa.products_id='73'

[XT SQL Error]
Hinzufügen von addslashes bei xtc_get_attributes_model.inc.php sollte das Problem fixen
(für $options_name und $attribute_name)

Hier die gefixte trunc 382 Datei (ohne Gewähr)



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

[Tomcraft]

Gefixt in r385!

Danke web28!

Grüße

Torsten

[mei chan]

Hallo,

muss das alte Thema leider ausgraben.

Bekomme mit dem addslashes Bugfix ab php 5.3.6 folgende Warning sofern Artikel mit Attributen im Warenkorb sind:

Code: PHP  [Auswählen]

addslashes() expects parameter 1 to be string, array given in /inc/xtc_get_attributes_model.inc.php on line 30

Kann man  addslashes() bedenkenlos wieder entfernen, sofern der Shop nicht vom Ursprünglichen [XT SQL Error] betroffen ist oder gibt es dann an anderer Stelle Probleme?

Danke und Gruße

[web28]

$options_name darf kein Array sein, der Fehler ist woanders zu suchen. Der SP1b Demoshop läuft mit PHP 5.3.13 und da funktioniert alles.

[mei chan]

Hallo,
der Array kommt bei mir wohl aus einem Modul für die Anpassung der Attribute im Warenkorb.
Möchte da nun nicht unnötig das Modul "verbiegen" sofern addslashes() nur ein bugfix für den einen [XT SQL Error] in Verbindung mit Optionswerten nach dem Muster '09 oder abc' ist.

Daher meine Frage, ob man die beiden addslashes() auch bedenkenlos wieder rausnehmen kann?

Danke und Gruß