Fatal error in includes/modules/product_attributes.php on Line: 130

[info@…]

Hi,

Ich hatte gerade einen Shop lahmgelegt, da die Addition in dieser Zeile aufgrund der Datentypen nicht funktioniert hat:

$full = $products_price + $attr_price;

Das Problem war, das ein Drittanbieter-Modul unter templates/source/boxes... die Variable $products_price definierte. Allerdings mit Formatierungen, sodass es sich bei $products_price um ein Array handelte, welches nicht addiert werden kann.

Problematisch ist jedoch, dass in Zeile 116 der product_attributes.php mit
"if (!isset($products_price)) {"
geprüft wird, ob überhaupt schon eine Preisangabe existiert und diese dann verwendet wird, anstelle des originalen Produktpreises.
Damit kann der Produktpreis komplett (beabsichtigt/oder nicht) manipuliert werden, indem einfach ein vorheriges Modul die Variable $products_price einfügt.

Ich glaube, auf die Prüfung von Zeile 116 sollte verzichtet werden und der Preis sollte immer neu geholt werden.

Viele Grüße
Andreas

[Gerhard Waldemair]

In 11511:

fix #1508