Hi,

I think there's a problem here : prices are VAT exclusive in Views blocks.
As I have read in the uc_vat_price_handler_alter function, these cases should be covered by the switch... Maybe a wrong context name?

For Upsell, doesn't work either, but works well in dev version.

cfab

Comments

tahiticlic’s picture

tahiticlic commented

I answer myself as I found the problem cause : there's no "type" attribute in the $values array produced by views... Instead, there's a "node_type" attribute, so the switch becomes for views case :

 case 'views-price-handler':     // Prices displayed by Views module
      $node = $context['subject']['node'];
      $node->type=$node->node_type;
      break;
longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Active » Fixed

Thanks for spotting this, fixed in CVS. I modified your patch to load the full node data, as the type itself isn't enough - we also need the "shippable" setting and the product data in the case that the node is a product kit.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.