id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc	blockedby	blocking
1508	Fatal error in includes/modules/product_attributes.php on Line: 130	info@…	somebody	"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"	Bug/Fehler	closed	niedrig	modified-shop-2.0.5.0	Shop	2.0.4.2	fixed