Coupon discount and free shipping

[zanardi]

VirtueMart 2.0.10 on Joomla 2.5

I would like a confirmation or clarifications about free shipping and coupon discounts.

I have a shipping method which gets free shipping over € 150. So if my total amount is, e.g., € 180, i get free shipping.
Now I use a coupon, e.g. a 50% coupon, that sets the total to € 90.
I still get free shipping, even if I am going to pay less than € 150.

Is that intended behaviour? Is it configurable?

Thank you for any suggestion.

[zanardi]

This (possible) issue is still there in VirtueMart 2.0.14. Free shipping threshold is checked against total price BEFORE coupon discount is calculated.
I see that in

  /plugins/vmshipment/weight_countries/weight_countries.php

there's a method

  plgVmShipmentWeight_countries::getCosts

which does the check:

  if ($method->free_shipment && $cart_prices['salesPrice'] >= $method->free_shipment) {

I changed that to

  if ($method->free_shipment && ( $cart->pricesUnformatted['billTotal'] >= $method->free_shipment ) ) {

to get the desired behaviour for free shipping.

Thank you for any opinion on this matter.

[bytelord]

Hello,

Please try with 2.0.16d. The coupon calculation have been changed, please test it.

[zanardi]

@bytelord:
no difference in 2.0.16d, the issue is still there.

Please note that I am not talking about a *coupon* issue, but about a *free shipping* issue. The coupon is only a condition that triggers the possibly unintended behaviour.

If you look at the code of the  plgVmShipmentWeight_countries::getCosts you will see that it still checks free shipping threshold against the "salesPrice" instead of the "billTotal" so of course the coupon discount is not taken into account.

In my previous post i suggest a possible solution (which I am actually using on a production site).

[veeco]

@zanardi, by common sense, i think you got the point right.. 

[kermitfrog]

This (possible) issue is still there in VirtueMart 2.0.14. Free shipping threshold is checked against total price BEFORE coupon discount is calculated.
I see that in

  /plugins/vmshipment/weight_countries/weight_countries.php

there's a method

  plgVmShipmentWeight_countries::getCosts

which does the check:

  if ($method->free_shipment && $cart_prices['salesPrice'] >= $method->free_shipment) {

I changed that to

  if ($method->free_shipment && ( $cart->pricesUnformatted['billTotal'] >= $method->free_shipment ) ) {

to get the desired behaviour for free shipping.

Thank you for any opinion on this matter.

Thanks zanardi, exactly what I was looking for - I Have the same problem.
Price: 18,70 Euro
Coupon: 20% off
-> 14,96 Euro
(Free Shipping when more than 15 Euro)
14,96 -> Free Shipping 

But on my Joomla 2.5.9 and Virtuemart 2.0.18a 
$cart->pricesUnformatted['billTotal']    is not working. When I use this, the "if" never becomes true.

I'm not that good in php, but i tried  return $cart->pricesUnformatted['billTotal']; to check the value... the card showed nothing.
return $cart_prices['salesPrice']; correctly returns the amount (but without discount).

Adding $cart->pricesUnformatted['salesPriceCoupon']   also doesn't work for me (empty, too).

I also tried to solve this problem by adding another shipment method with the minimum amount 15 und shipping costs 0 ...
But the minumum / maximum amount for a shipping method is also not effected by the coupon amount.

Any ideas?
Thank a lot!

[kermitfrog]

Maybe I found why this is working in 2.0.14 and no longer in 2.0.18a...
cart.php  line 225  in the function setCartIntoSession()

in Version 2.0.18 pricesUnformatted is commented:
//$sessionCart->pricesUnformatted    = $this->pricesUnformatted;

in Version 2.0.14, this is without // comments...

I guess, it must have a reason why this is done. But this makes your trick impossible -.-

[ArtRight]

Thank you zanardi!  Your solution worked for me also, VM 2.0.2.

[zanardi]

Maybe I found why this is working in 2.0.14 and no longer in 2.0.18a...
cart.php  line 225  in the function setCartIntoSession()

in Version 2.0.18 pricesUnformatted is commented:
//$sessionCart->pricesUnformatted    = $this->pricesUnformatted;

Thank you for pointing that out.
I tried different methods but I can't find a different solution to get the final price from the available objects.
So the original issue remains and the proposed fix has been disabled. Great.

[kermitfrog]

This error also effects the payment method if you are using a min or max amount.

I tried to reactivate some lines in the code to make this solution work again. But the problem is, that the shopping card has to be manually reloaded after using a discount coupon.
Only after a manual reload (F5) the settings are correct. Seems that the Shipping and Payment methods are calculated before the total amount is calculated.

Well, I'm no good programmer - so I give up ;o)

[zanardi]

Wrote some code to work around the issue.

/plugins/vmshipment/weight_countries/weight_countries.php, change this code

Code Select Expand


/**
* @param VirtueMartCart $cart
* @param                $method
* @param                $cart_prices
* @return int
*/
function getCosts (VirtueMartCart $cart, $method, $cart_prices) {

  if ($method->free_shipment && $cart_prices['salesPrice'] >= $method->free_shipment) {
    return 0;
  } else {
    return $method->cost + $method->package_fee;
  }
}


to this

Code Select Expand



/**
* @param VirtueMartCart $cart
* @param                $method
* @param                $cart_prices
* @return int
*/
function getCosts (VirtueMartCart $cart, $method, $cart_prices) {

  if ($cart->couponCode) {
    $cart_prices['salesPrice'] = $this->applyCoupon($cart->couponCode, $cart_prices['salesPrice'] );
  }
  if ($method->free_shipment && $cart_prices['salesPrice'] >= $method->free_shipment) {
    return 0;
  } else {
    return $method->cost + $method->package_fee;
  }
}

/**
* Custom function to get total price with coupon applied
*
* @param string $couponcode
* @param float $salesprice
*/
private function applyCoupon($couponcode, $salesprice)
{
  require_once JPATH_VM_SITE.'/helpers/coupon.php';
  if (!($data = CouponHelper::getCouponDetails($couponcode))) {
    return $salesprice;
  }
  if ($data->percent_or_total == 'total') {
    return $salesprice - $data->coupon_value;
  }
  else {
    return $salesprice - ($salesprice * $data->coupon_value / 100);
  }
} 


Basically, I have added a custom check for the presence of a coupon code, and then a custom method to get the coupon-discounted total.
Of course the "correct" way is not to hack the core plugin, but to create a custom plugin instead... but that is left as an exercise to the reader. This works and today it's already way too late.