MSRP contrib

[MortalWombat]

Retail Price: £449.99
You save: £100.00 (27%)
Our Price: £329.99



can you spot the error above??


post this a while ago and still no fix... any help??

Spotted this problem throughout the thread... This was something I also was having difficulty figuring out. This is the fix I can up with:

Around line #294 you'll find this snippet of code:

if ($display_normal_price == 0) {
      // don't show the $0.00
	  if(($display_msrp_price!='')&&($display_msrp_price!='0')){
	  $save=$display_msrp_price-$display_normal_price;
      	$final_display_price = '<span class="smalltext1"><s>'.TEXT_PRODUCT_MSRP.$currencies->display_price($display_msrp_price, zen_get_tax_rate($product_check->fields['products_tax_class_id'])).'</s></span><br><span class="save">'.TEXT_PRODUCT_SAVE .$currencies->display_price($save, '').'&nbsp;&nbsp;('. number_format((($display_msrp_price -  $display_normal_price) / $display_msrp_price), 2, '.', '') * 100 . '%)'.'</span><br><span class="norm">'.TEXT_PRODUCT_OUR.$show_special_price . $show_sale_price . $show_sale_discount.'</span>' ;
	  }else{
	  	$final_display_price = $show_special_price . $show_sale_price . $show_sale_discount;
	  }
    } else {
	 if(($display_msrp_price!='')&&($display_msrp_price!='0')){
	 $save=$display_msrp_price-$display_normal_price;
      	$final_display_price = '<span class="smalltext1"><s>'.TEXT_PRODUCT_MSRP.$currencies->display_price($display_msrp_price, zen_get_tax_rate($product_check->fields['products_tax_class_id'])).'</s></span><br><span class="save">'.TEXT_PRODUCT_SAVE .$currencies->display_price($save, '').'&nbsp;&nbsp;('. number_format((($display_msrp_price -  $display_normal_price) / $display_msrp_price), 2, '.', '') * 100 . '%)'.'</span><br><span class="norm">'.TEXT_PRODUCT_OUR.$show_normal_price . $show_special_price . $show_sale_price . $show_sale_discount.'</span>' ;
	  }else{
	  	$final_display_price = $show_normal_price . $show_special_price . $show_sale_price . $show_sale_discount;
	  }
    }

Alter the 2 lines in blue (note the section in red, which adds the tax to the saving):

if ($display_normal_price == 0) {
      // don't show the $0.00
	  if(($display_msrp_price!='')&&($display_msrp_price!='0')){
	  $save=$display_msrp_price-$display_normal_price;
      	$final_display_price = '<span class="smalltext1"><s>'.TEXT_PRODUCT_MSRP.$currencies->display_price($display_msrp_price, zen_get_tax_rate($product_check->fields['products_tax_class_id'])).'</s></span><br><span class="save">'.TEXT_PRODUCT_SAVE .$currencies->display_price($save, zen_get_tax_rate($product_check->fields['products_tax_class_id'])).'&nbsp;&nbsp;('. number_format((($display_msrp_price -  $display_normal_price) / $display_msrp_price), 2, '.', '') * 100 . '%)'.'</span><br><span class="norm">'.TEXT_PRODUCT_OUR.$show_special_price . $show_sale_price . $show_sale_discount.'</span>' ;
	  }else{
	  	$final_display_price = $show_special_price . $show_sale_price . $show_sale_discount;
	  }
    } else {
	 if(($display_msrp_price!='')&&($display_msrp_price!='0')){
	 $save=$display_msrp_price-$display_normal_price;
      	$final_display_price = '<span class="smalltext1"><s>'.TEXT_PRODUCT_MSRP.$currencies->display_price($display_msrp_price, zen_get_tax_rate($product_check->fields['products_tax_class_id'])).'</s></span><br><span class="save">'.TEXT_PRODUCT_SAVE .$currencies->display_price($save, zen_get_tax_rate($product_check->fields['products_tax_class_id'])).'&nbsp;&nbsp;('. number_format((($display_msrp_price -  $display_normal_price) / $display_msrp_price), 2, '.', '') * 100 . '%)'.'</span><br><span class="norm">'.TEXT_PRODUCT_OUR.$show_normal_price . $show_special_price . $show_sale_price . $show_sale_discount.'</span>' ;
	  }else{
	  	$final_display_price = $show_normal_price . $show_special_price . $show_sale_price . $show_sale_discount;
	  }
    }

Hoping this assists those who encountered this problem :)

- Tija