Forums / Bug Reports / [Not a bug] duplicate $cheapest in notify

[Not a bug] duplicate $cheapest in notify

Results 1 to 3 of 3

19 Apr 2019, 13:37

twitchtoo

Totally Zenned

Join Date:: Apr 2007
Location:: Ontario, Canada
Posts:: 1,732
Plugin Contributions:: 14

[Not a bug] duplicate $cheapest in notify

Found in 1.55f and 1.56a

includes/classes/shipping.php
line 213...
$this->notify('NOTIFY_SHIPPING_MODULE_CALCULATE_CHEAPEST', $cheapest, $cheapest, $rates);
return $cheapest;

19 Apr 2019, 15:40

lat9

Administrator

Join Date:: Sep 2009
Location:: Stuart, FL
Posts:: 14,065
Plugin Contributions:: 56

Re: [Not a bug] duplicate $cheapest in notify

I believe that the first (read-only) $cheapest is for compatibility, which the second (read-write) version allows an observer to make modifications.

19 Apr 2019, 17:38

drbyte

Sensei

Join Date:: Jan 2004
Posts:: 63,513
Plugin Contributions:: 177

Re: [Not a bug] duplicate $cheapest in notify

lat9:

I believe that the first (read-only) $cheapest is for compatibility, which the second (read-write) version allows an observer to make modifications.
Correct.