Multiple Ship-To Addresses [Support Thread]

More information on the matter. Thoughts about the process.

First, apologies. Not sure I mentioned mod Order Delivery Date is loaded.

I've found that the option to multi-ship does not appear for PayPal as well as Square. Only money order allows the option to appear.

Regardless of payment method, the shopping cart is reacting as it should. Tells the customer that single goes to primary and multiples may be sent to multiple addresses.

With money order as the example, if only one item in the cart, it correctly does not show the multi-ship option. If more than one item in the cart, it does show the multi-ship option.

PayPal from the shopping cart page does come back for shipping but does not offer multi-ship for multiple items.

Square stays on site of course, but also does not offer multi-ship items.

Extrapolation leads to thought that whatever triggers the multi-ship option is not coming from PayPal or Square. OR... Order Delivery Date is munging something up.

It's the time of year that this site gets orders sent to multiple addresses. One car salesman sends to all who purchased this year.

If the PayPal method being used is PayPal Express Checkout (i.e. via the shortcut button), note that the payment method prohibits changing the shipping address; if the method is, instead, PayPal Payments Pro, the checkout_confirmation page is actually displayed via AJAX on the checkout_payment page.

In either case, that could be part of the issue. I don't remember if Square declares itself as a "collects payments on site" type of payment method, but if it does then that's the same base issue as PayPal Payments Pro.

@dbltoe, the issue is/was that the non-moneyorder payment methods weren't updated to include a multiple_shipping_addresses method. Looking through the readme, I see that I haven't included any instructions for that required update.

I'll get that readme updated for a follow-on release.

Thanks so much for your help. It is a little bit difficult to read the non-existing part of a readme file.:D Guess it's like my putts in golf. Better than 99% of the putts that don't make it to the hole fail to go in.

Seriously, :hug:for all you do for Zen Cart and the forum. Customer got $1500 order with 30 addresses shortly after your fix this morning.

No problems, @dbltoe. That's great news about the site!

Good Morning

allpecans is still going strong with the multiship. Site has been upgraded to 1.5.6c and we've made some changes aesthetically that I'll probably address on github. Really all minor but, the move to PHP 7.2 has created a problem with includes/modules/pages/account_history/header_php_account_history_multiship.php on line 10.

Code:

---

<?php// ---------------------------------------------------------------------------
// Part of the Multiple Shipping Addresses plugin for Zen Cart v1.5.5 and later
//
// Copyright (C) 2014-2017, Vinos de Frutas Tropicales (lat9)
//
// @license http://www.zen-cart.com/license/2_0.txt GNU Public License V2.0
// ---------------------------------------------------------------------------
//
if (count($accountHistory) > 0) {
    foreach ($accountHistory as &$currentOrder) {
        $check = $db->Execute(
            "SELECT o.order_total FROM " . TABLE_ORDERS_MULTISHIP . " om
                INNER JOIN " . TABLE_ORDERS . " o
                    ON o.orders_id = om.orders_id
              WHERE om.orders_id = " . $currentOrder['orders_id'] . "
              LIMIT 1"
        );
        if (!$check->EOF) {
            $currentOrder['order_name'] = MULTISHIP_MULTIPLE_ADDRESSES;
            $currentOrder['order_total'] = $check->fields['order_total'];
        }
    }
}

---

I've spent the morning trying to learn about 7.2's change for count and recursive versus plain count or https://www.php.net/manual/en/function.count.php and none of my trials seem to work. I figure you'll do your usual and get the right combo as soon as you see it.

THANX for your help and work with Zen Cart.

Quote:

---

Originally Posted by dbltoe

Good Morning

allpecans is still going strong with the multiship. Site has been upgraded to 1.5.6c and we've made some changes aesthetically that I'll probably address on github. Really all minor but, the move to PHP 7.2 has created a problem with includes/modules/pages/account_history/header_php_account_history_multiship.php on line 10.

Code:

---

<?php
// ---------------------------------------------------------------------------
// Part of the Multiple Shipping Addresses plugin for Zen Cart v1.5.5 and later
//
// Copyright (C) 2014-2017, Vinos de Frutas Tropicales (lat9)
//
// @license http://www.zen-cart.com/license/2_0.txt GNU Public License V2.0
// ---------------------------------------------------------------------------
//
if (count($accountHistory) > 0) {
    foreach ($accountHistory as &$currentOrder) {
        $check = $db->Execute(
            "SELECT o.order_total FROM " . TABLE_ORDERS_MULTISHIP . " om
                INNER JOIN " . TABLE_ORDERS . " o
                    ON o.orders_id = om.orders_id
              WHERE om.orders_id = " . $currentOrder['orders_id'] . "
              LIMIT 1"
        );
        if (!$check->EOF) {
            $currentOrder['order_name'] = MULTISHIP_MULTIPLE_ADDRESSES;
            $currentOrder['order_total'] = $check->fields['order_total'];
        }
    }
}

---

I've spent the morning trying to learn about 7.2's change for count and recursive versus plain count or https://www.php.net/manual/en/function.count.php and none of my trials seem to work. I figure you'll do your usual and get the right combo as soon as you see it.

THANX for your help and work with Zen Cart.

---

Unfortunately, the $accountHistory array is created by the as-shipped header_php.php only if a customer has placed orders. Line 10 needs to be updated as described below:

Code:

---

<?php
// ---------------------------------------------------------------------------
// Part of the Multiple Shipping Addresses plugin for Zen Cart v1.5.5 and later
//
// Copyright (C) 2014-2017, Vinos de Frutas Tropicales (lat9)
//
// @license http://www.zen-cart.com/license/2_0.txt GNU Public License V2.0
// ---------------------------------------------------------------------------
//
if (!empty($accountHistory) && is_array($accountHistory)) {
    foreach ($accountHistory as &$currentOrder) {
        $check = $db->Execute(
            "SELECT o.order_total FROM " . TABLE_ORDERS_MULTISHIP . " om
                INNER JOIN " . TABLE_ORDERS . " o
                    ON o.orders_id = om.orders_id
              WHERE om.orders_id = " . $currentOrder['orders_id'] . "
              LIMIT 1"
        );
        if (!$check->EOF) {
            $currentOrder['order_name'] = MULTISHIP_MULTIPLE_ADDRESSES;
            $currentOrder['order_total'] = $check->fields['order_total'];
        }
    }
}

---

:hug:

Was looking at the empty but couldn't see the comparison between the example and the code.

Off to find out why Square is bombing today. Certainly NOT OPC.

THANX

Another odd item has reared its head on the pecan site. Still not using OPC

It's taken me a week to duplicate what is happening and it is multiship reporting to PayPal.

Three things have to happen:

1. All items are taxable in Texas based on shipping
2. Four different addresses -- one not taxable - three taxable
3. must be in order non-tax, tax, tax, tax

In Step 3 of 3, a review will show
Attachment 19304

All amounts in this image are correct.

However, when presented to PayPal, the system tells PayPal that the tax is zero.

PayPal refuses the transaction because it sees taxable addresses but no tax.

Square doesn't care and processes the correct charge.

Sorry I am unable to let you play at this time. Busy season for them.

This has only happened three times that we know of and the system has been set for notify on errors since the upgrade to 1.5.6c.

I know I'm leaving something out but, let me know if you need any info.

@dbltoe, if you could enable PayPal logging, rerun the failing case and send the associated PayPal logs to my direct email, that would give me something to work with.