[Done 155b] Shipping tax not added in order confirmation - possible bug??

[rued]

however when a customer pays 'on account' using the invoice mod from the add ons the shipping is once again calculated without tax on the shipping. I can rectify the issue by removing the zone from the shipping module again.

This one: https://www.zen-cart.com/downloads.php?do=file&id=131 ?

What happens if you change:

function invoice() {

into:

function __construct() {

I don't see what else should be changed in this for 1.5.5, as it doesn't have any zone handling?!

[frank18]

Just completed another upgrade from 1.5.4 to 1.5.5a for a client.

He had that store going for many years without tax issues - until up and including ZC 1.5.4.

Now, after upgrading to 1.5.5a, the tax was added correctly during the checkout process but was NOT added at the checkout_success page, the order confirmation email and the invoice.

Applied Dr Byte's fix (post #59) and we are back to normal, the shipping tax now shows correctly on the checkout_success page, the order confirmation email and the invoice.

So, that adds another (upgraded) store showing the same symptoms as I reported in post #1

[frank18]

So, from the reports of others, this symptom only seems to appear in upgraded stores. If one of the upgraders made a mistake in the upgrade then that would be understandable. But, it seems to happen in many upgraded stores as reported here in this thread and it is unlikely that the many different upgraders are making exactly the same mistake.....

On the other hand, in Wilts's and DrByte's vanilla 1.5.5a installs (ie trying to reproduce the error in a fresh installation) the symptom does not appear.

So, why the difference in symptoms between an upgraded store and a fresh installation?

[mc12345678]

Suggested "test" would be to install a fresh earlier version of ZC for a version that didn't report the issue, setup with conditions that appear to be causing the issues and then attempt to upgrade and see if the problem persists.

If the problem exists with that upgrade, compare database dumps of both a fresh install of 1.5.5a and that upgrade. If not, then unfortunately attempt the same database compare between a troubled upgraded cart and fresh ZC 1.5.5a...

[rued]

Suggested "test" would be...

+1 to it all ... :)

[frank18]

Suggested "test" would be to install a fresh earlier version of ZC for a version that didn't report the issue, setup with conditions that appear to be causing the issues and then attempt to upgrade and see if the problem persists.

If the problem exists with that upgrade, compare database dumps of both a fresh install of 1.5.5a and that upgrade. If not, then unfortunately attempt the same database compare between a troubled upgraded cart and fresh ZC 1.5.5a...

Yep, agreed! The starter could be a 1.5.1 or a 1.5.3 version and then upgrade progressively from version to version.

[frank18]

Did some digging.....

In includes/classes/shipping.php

ZC versions 1.5.1 and 1.5.4 we find around line 144

$this->notify('NOTIFY_SHIPPING_MODULE_GET_ALL_QUOTES', $quotes_array);

and in the same (corresponding) file in ZC 1.5.5a around line 176

$this->notify('NOTIFY_SHIPPING_MODULE_GET_ALL_QUOTES', $quotes_array, $quotes_array);

Is the duplication of , $quotes_array in the 1.5.5a file intended?

[DrByte]

Did some digging.....

In includes/classes/shipping.php

ZC versions 1.5.1 and 1.5.4 we find around line 144

$this->notify('NOTIFY_SHIPPING_MODULE_GET_ALL_QUOTES', $quotes_array);

and in the same (corresponding) file in ZC 1.5.5a around line 176

$this->notify('NOTIFY_SHIPPING_MODULE_GET_ALL_QUOTES', $quotes_array, $quotes_array);

Is the duplication of , $quotes_array in the 1.5.5a file intended?

Yes, intended. And, unless you're actually using something that gets triggered by that notifier hook, it wouldn't be affecting your shipping. let alone taxes

[lat9]

Yes, intended. And, unless you're actually using something that gets triggered by that notifier hook, it wouldn't be affecting your shipping. let alone taxes

To clarify, the duplication enables backwards compatibility (keeping the $quotes_array as the first, read-only, parameter) but also adds the capability for an observer to modify those values by supplying the second (updateable) parameter containing the $quotes_array.

[bottyz]

This one: https://www.zen-cart.com/downloads.php?do=file&id=131 ?

What happens if you change:

function invoice() {

into:

function __construct() {

I don't see what else should be changed in this for 1.5.5, as it doesn't have any zone handling?!

Hi Rued,

Sorry for the late reply (I was on holiday in Turkey), and thanks for your help. I found the issue and it wasn't anything to do with the invoice mod. It actually turned out to be a problem with the wp_checkout_proccess.php file with the worldpay module.

The above file is mostly a duplicate of the first part of the checkout_process.php file, which Dr Byte has suggested needs the shipping / order lines swapping round. I've swapped these round in the worldpay file and all seems well.

Thanks again.