Looking at the new code
if (empty($cart_virtuemart_product_id)) $cart_virtuemart_product_id = vRequest::getString('cart_virtuemart_product_id');
if ($quantity === null) $quantity = vRequest::getInt('quantity');
Should it not use the vmrequest function ??
if (empty($cart_virtuemart_product_id)) $cart_virtuemart_product_id = vmRequest::getString('cart_virtuemart_product_id');
if ($quantity === null) $quantity = vmRequest::getInt('quantity');
vmRequest got renamed to vRequest. Thats all.
Milbo thanks for the clarification, I can see that it has been renamed in other files as well.