UPS XML: Support Thread

Update: Cindy was right - I had other issues that prevented the values returned from ups.php vs upsxml.php from being identical.
Thanks Cindy!

swguy:

Update: Cindy was right - I had other issues that prevented the values returned from ups.php vs upsxml.php from being identical.
Thanks Cindy!
No problem, @swguy ... what were those other issues?

• I had some custom logic in UPSXML which was not in UPS.
• UPSXML has its own origin zip code but UPS uses the one from Admin > Configuration > Shipping, which was incorrectly set.

Is this plugin compitible with PHP8?

ntestinfo11:

Is this plugin compitible with PHP8?
I haven't specifically tested with PHP8, but there are no deprecation issues present.

lat9:

I haven't specifically tested with PHP8, but there are no deprecation issues present.
... but someone else has, and there's a fatal error due to the fact that the *XMLParser *class is now in the PHP base. See https://github.com/lat9/upsxml/issues/27; v1.7.10 forthcoming.

I've just submitted v1.7.10 of the UPSXML shipping method for the Zen Cart moderators' review and will post back here when it's available for download.

This release corrects a PHP 8.0+ Fatal error; XMLParse class now included in PHP base; see GitHub issue #27.

lat9:

I've just submitted v1.7.10 of the UPSXML shipping method for the Zen Cart moderators' review and will post back here when it's available for download.

This release corrects a PHP 8.0+ Fatal error; XMLParse class now included in PHP base; see GitHub issue #27.
Now available for download: https://www.zen-cart.com/downloads.php?do=file&id=126

I've just submitted v1.7.11 of UPSXML for the Zen Cart moderators' review; I'll post back here when it's available for download.

This version contains changes associated with GitHub issue #28, due to Zen Cart 1.5.8's requirement for the second argument to zen_get_zone_code to be an integer.

lat9:

I've just submitted v1.7.11 of UPSXML for the Zen Cart moderators' review; I'll post back here when it's available for download.

This version contains changes associated with GitHub issue #28, due to Zen Cart 1.5.8's requirement for the second argument to zen_get_zone_code to be an integer.
Now available for download: https://www.zen-cart.com/downloads.php?do=file&id=126

Has anyone figured out the changes required to make UPSXML respect the free shipping setting (product_is_always_free_shipping) in the products table?

My solution was just to recompute $shipping_num_boxes.
Just before the loop that calls _addItem,

            // Handle free shipping
            $free_weight = $_SESSION['cart']->free_shipping_weight();
            if ($free_weight > 0) { 
               $non_free_weight = $_SESSION['cart']->show_weight() - $free_weight; 
               $zc_boxes = zen_round(($non_free_weight/SHIPPING_MAX_WEIGHT), 2);
               $shipping_num_boxes = ceil($zc_boxes);
            }

Not sure if this is perfect, still testing.

swguy:

My solution was just to recompute $shipping_num_boxes.
Just before the loop that calls _addItem,

        // Handle free shipping
        $free_weight = $_SESSION['cart']->free_shipping_weight();
        if ($free_weight > 0) { 
           $non_free_weight = $_SESSION['cart']->show_weight() - $free_weight; 
           $zc_boxes = zen_round(($non_free_weight/SHIPPING_MAX_WEIGHT), 2);
           $shipping_num_boxes = ceil($zc_boxes);
        }

> Not sure if this is perfect, still testing.
The underlying issue is that the shipping module isn't using ***zen_get_shipping_enabled*** to see if the cart's all free shipping.

GitHub tracking issue: <https://github.com/lat9/upsxml/issues/29>

My concern is not so much that the cart is all free shipping but rather that some items are and others aren't.
It makes sense for UPS to appear as an options even if a cart is free shipping (for example, UPS might be offered as expedited shipping).
I just think that (at least for some customers) respecting the per product free shipping flag would be preferred.

swguy:

My concern is not so much that the cart is all free shipping but rather that some items are and others aren't.
It makes sense for UPS to appear as an options even if a cart is free shipping (for example, UPS might be offered as expedited shipping).
I just think that (at least for some customers) respecting the per product free shipping flag would be preferred.
Since that's processing that should affect ***all ***shipping-modules, I'll suggest that the change should be in /includes/classes/shipping.php, where the $shipping_weight is calculated. It's there that the cart's weight should be reduced by the free_shipping_weight.

lat9:

Since that's processing that should affect ***all ***shipping-modules, I'll suggest that the change should be in /includes/classes/shipping.php, where the $shipping_weight is calculated. It's there that the cart's weight should be reduced by the free_shipping_weight.
... or possibly in the shopping_cart.php class' **show_weight **method? From the in-core usage, it looks like the 'assumption' made by callers of that method is that the weight returned is the weight to be used for the shipping calculations.

I'm not sure ... I'm not even sure upsxml should be changed for most people (based on the paucity of complaints).

Shipping by Item sort of works this way now (exclude free shipping items in the count that is the multiplier in the cost calculation).

Table Shipping works this way for price and item calculation, but not for weight calculation.
Zone Shipping is the same way as Table Shipping.
Weird.

We have upgraded to Zencart 1.5.7d and the UPSXML module v1.7.11 yet we are still getting charge much higher rates than our physical store module (not the shopping cart) that uses the same UPS Rates Access Key. We had hoped the upgrade would solve this issue - but it persists. Any advice would be greatly appreciated.

SPython:

We have upgraded to Zencart 1.5.7d and the UPSXML module v1.7.11 yet we are still getting charge much higher rates than our physical store module (not the shopping cart) that uses the same UPS Rates Access Key. We had hoped the upgrade would solve this issue - but it persists. Any advice would be greatly appreciated.
Have you set your "UPS Rates: Shipper Number" so that you're receiving your store's negotiated rates?

SPython:

We have upgraded to Zencart 1.5.7d and the UPSXML module v1.7.11 yet we are still getting charged much higher rates than our physical store module (not the shopping cart) that uses the same UPS Rates Access Key. We had hoped the upgrade would solve this issue - but it persists. Any advice would be greatly appreciated.

Here is an example: sending a 7lb package from New York to California the UPS charge quoted on our UPS desktop application is $19.18 and the same package has an estimated UPS charge of $28.90 via the UPSXML module for Zencart