ulbiopro
commented
3 years ago Maybe this can help too:
when saving billing address, the saveBillingAction is called and therefore Mage_Checkout_Model_Type_Onepage::saveBilling().
In both Mage_Checkout_Model_Type_Onepage::saveBilling() as in Mage_Checkout_Model_Type_Onepage::saveShipping() totals will be collected on the quote.
Mage_Checkout_Model_Type_Onepage
$this->getQuote()->collectTotals();Mage_Checkout_Model_Type_Onepage
$this->getQuote()->collectTotals()->save();In both cases, PackageValue in Mage_Shipping_Model_Carrier_Tablerate returns the amount including tax (50), because the address object data "base_subtotal" is 50.
This seems fine to me.
But:
On Mage_Checkout_Model_Type_Onepage::saveBilling() there are these additional lines:
Mage_Checkout_Model_Type_Onepage
if (!$this->getQuote()->isVirtual() && $this->getCheckout()->getStepData('shipping', 'complete') == true) {
//Recollect Shipping rates for shipping methods
$this->getQuote()->getShippingAddress()->setCollectShippingRates(true);
}setCollectShippingRates(true) leads to the fact, that Mage_Sales_Model_Quote_Address::requestShippingRates() is called again, when loading the shipping methods checkout template.
But this time the address object data "base_subtotal" is excl. tax (46.19).
If Magento now proceeds to collect the shipping amount it will return 3.95.
So I wonder why, we have to recollect shipping rates for shipping methods, in Mage_Checkout_Model_Type_Onepage::saveBilling()?
If we don't do it, it seems to work fine for me.
But since I don't know why Magento behaves that way, I'm afraid to change it rashly. Maybe anyone knows a little better?
Thank you for you help guys!
ADDISON74
kanevbg
Preconditions (*)
Steps to reproduce (*)
I reproduced this on fresh OpenMage LTS 20.0.6 and used the default tax-classes and zones. Nevertheless it happens with custom configuration, too.
Setup: Configure shipping method tablerates with tablerate data below (works with other countries too, but USA is less work to reproduce). Anything less than $50 should include a $3.95 shipping fee, a cart with more than $50 comes with free shipping.
Setup: Use default item, simple product with a price of 50.00 $, Tax Class = taxable goods (an existing default value)
Setup: Configuration -> Sales -> Tax -> Calculation Settings -> Catalog Prices: Including Tax
Add product to cart. In default Layout you can use the "estimate shipping and tax" box. Since we're using default values, please use state "california" because we have configured tax zones and rates for this state by default.
As expected, shipping costs are $0.
Proceed to checkout.
Checkout as guest
Fill in billing information. Make sure to use California again and select "Ship to this address"
Continue -> Shipping Information form will be skipped. 9. Shipping fee is now $3.95!
Click "Back"
Click "Continue"
Shipping fee is now $0 again.
Shipping fee is also $0, if you select "Ship to different address" in the Billing information.
Expected result (*)
I'm not sure if shipping costs are calculated on prices including or excluding taxes. Either way, the result should be the same, if you ship to the same address.
Actual result (*)
Shipping costs differ, although both times the sipping address was the same.
Notice
Maybe this information helps, finding the bug: In step 7 ("ship to this address") Mage_Shipping_Model_Carrier_Tablerate::collectRates() is called 2 times. The 1. time $request->getPackageValue() returns 50. (Price incl. tax) The 2. time $request->getPackageValue() returns 46.19. (Price excl. tax)
If you checkout with "ship to different address", Mage_Shipping_Model_Carrier_Tablerate::collectRates() is called only once. $request->getPackageValue() returns 50.