Multiple Ship-To Addresses [Support Thread]

Regardless of hard-coding the height and width or placing them in the database as configurations, the line still retrieves only the DIR_WS_CATALOG (/) and DIR_WS_IMAGES (images). Beyond that - nothing.:frusty:```

<table border="0" width="100%" cellspacing="0" cellpadding="2"> <tr class="dataTableHeadingRow"> <!-- BOF Add Image 1 of 2 - dbltoe --> <td class="dataTableHeadingContent">Image</td> <!-- EOF Add Image 1 of 2 = dbltoe --> <td class="dataTableHeadingContent" align="center">Qty</td> <td class="dataTableHeadingContent"><?php echo TABLE_HEADING_PRODUCTS_MODEL; ?></td> <td class="dataTableHeadingContent">Option(s)</td> </tr> <?php foreach ($order->products as $currentProduct) { if ($currentProduct['orders_multiship_id'] != $multiship_id) { continue; } ?> <tr class="dataTableRow"> <!-- BOF Add Image 2 of 2 - dbltoe --> <td class="dataTableContent" valign="top" align="left"> <?php echo zen_image(DIR_WS_CATALOG . DIR_WS_IMAGES . zen_get_products_image($order->products[$i]['id']), 'Product Image', IMAGE_ON_INVOICE_IMAGE_WIDTH, IMAGE_ON_INVOICE_IMAGE_HEIGHT); ?> </td> <!-- EOF Add Image 2 of 2 - dbltoe --> <td class="dataTableContent" valign="top" align="center"><?php echo $currentProduct['qty']; ?></td> <td class="dataTableContent" valign="top"><?php echo $currentProduct['name']; ?> <?php

    if (isset($currentProduct['attributes']) && (sizeof($currentProduct['attributes']) > 0)) {
        foreach ($currentProduct['attributes'] as $currentAttribute) {

?>
<br /><span class="nobreak"><small> <i> - <?php echo $currentAttribute['option'] . ': ' . nl2br(zen_output_string_protected($currentAttribute['value'])); ?></i></small></span>

<?php } ?>

    </td>
    <td class="dataTableContent" valign="top"><?php echo $currentProduct['model']; ?></td>

<?php } ?>

  </tr>

<?php } ?>

</table>

dbltoe:

Regardless of hard-coding the height and width or placing them in the database as configurations, the line still retrieves only the DIR_WS_CATALOG (/) and DIR_WS_IMAGES (images). Beyond that - nothing.:frusty:```

<table border="0" width="100%" cellspacing="0" cellpadding="2"> <tr class="dataTableHeadingRow"> <!-- BOF Add Image 1 of 2 - dbltoe --> <td class="dataTableHeadingContent">Image</td> <!-- EOF Add Image 1 of 2 = dbltoe --> <td class="dataTableHeadingContent" align="center">Qty</td> <td class="dataTableHeadingContent"><?php echo TABLE_HEADING_PRODUCTS_MODEL; ?></td> <td class="dataTableHeadingContent">Option(s)</td> </tr> <?php foreach ($order->products as $currentProduct) { if ($currentProduct['orders_multiship_id'] != $multiship_id) { continue; } ?> <tr class="dataTableRow"> <!-- BOF Add Image 2 of 2 - dbltoe --> <td class="dataTableContent" valign="top" align="left"> <?php echo zen_image(DIR_WS_CATALOG . DIR_WS_IMAGES . zen_get_products_image([B]$order->products[$i][/B]['id']), 'Product Image', IMAGE_ON_INVOICE_IMAGE_WIDTH, IMAGE_ON_INVOICE_IMAGE_HEIGHT); ?> </td> <!-- EOF Add Image 2 of 2 - dbltoe --> <td class="dataTableContent" valign="top" align="center"><?php echo $currentProduct['qty']; ?></td> <td class="dataTableContent" valign="top"><?php echo $currentProduct['name']; ?> <?php

    if (isset($currentProduct['attributes']) && (sizeof($currentProduct['attributes']) > 0)) {
        foreach ($currentProduct['attributes'] as $currentAttribute) {

?>
<br /><span class="nobreak"><small> <i> - <?php echo $currentAttribute['option'] . ': ' . nl2br(zen_output_string_protected($currentAttribute['value'])); ?></i></small></span>

<?php } ?>

    </td>
    <td class="dataTableContent" valign="top"><?php echo $currentProduct['model']; ?></td>

<?php } ?>

  </tr>

<?php } ?>

</table>

> Any more thoughts?  Works like a champ with standard packing slip.
My thought: I didn't pay enough attention to your original post.:blush:  How about changing the highlighted section to reference the variable that's being used:

<!-- BOF Add Image 2 of 2 - dbltoe -->

    <td class="dataTableContent" valign="top" align="left">
        <?php echo zen_image(DIR_WS_CATALOG . DIR_WS_IMAGES . zen_get_products_image([B]$currentProduct[/B]['id']), 'Product Image', IMAGE_ON_INVOICE_IMAGE_WIDTH, IMAGE_ON_INVOICE_IMAGE_HEIGHT); ?>
    </td>

<!-- EOF Add Image 2 of 2 - dbltoe -->

:hug:
That did it!
Didn't occur to me that it might be a different variable than all the other packing slips.
THANX

It seems that we may be putting the cart before the horse as it were. Multi selects the shipping method then the addresses. There seems to be no way to add two items to the cart and send them to different addresses using different shipping methods.

Selecting UPS at first and then splitting the shipping with one going to a PO Box means we need to change to USPS.

Am I missing something here:lookaroun

dbltoe:

It seems that we may be putting the cart before the horse as it were. Multi selects the shipping method then the addresses. There seems to be no way to add two items to the cart and send them to different addresses using different shipping methods.

Selecting UPS at first and then splitting the shipping with one going to a PO Box means we need to change to USPS.

Am I missing something here:lookaroun
Nope, you're not missing anything. The plugin supports Multiple Ship-To Addresses, not (currently) Multiple Ship-To Methods.

I've noted the request in the plugin's GitHub repository: https://github.com/lat9/multiship/issues/31

THANX,
We would limit the shipping but USPS is so much more expensive when the item leaves Texas. Inside, we can get around dimensional weight.
After our rodeo road trip, I'll look further into the matter.

Latest of the most interesting.

allpecans.com

Site is 1.5.5f, php 5.6, Apache 3.10, MySQL 5.6.41 Initial Install not upgrade

No OPC of course.

Square, News Box Manager, Sitemap XML, Dynamic Price Updater, Magic Thumb, Monthly Sales, Display Log Files, Encrypted Master Password, Edit Orders, Captcha, USPS Shipping, Remember Me.

Don't think I left any out.

When a customer with multiple addresses goes to check out using, a credit card (Square) in Step 2, the option to multiship does NOT appear on step 3 of checkout.

If the customer selects money order in Step 2, the option to ship to multiple addresses is just fine.

Thought it might be the address format at first as one address had physical as street and PO Box as Address Line 2:. Tried different combinations and none would work with a credit card.

Then thought it might be that most of the customers were brought over from Magento, so I tried with a post-customer-move account with the same result.

Had only tested this with money/order prior to customer having a problem.

Thinking maybe a setting in includes/modules/pages/checkout_confirmation/header_php_checkout_confirmation_multiship.php.

Thoughts

@dbltoe, that's interesting behavior, which I can't explain at this time. The multi-ship addition to the standard checkout_confirmation page is a "simply" additional header file and a couple of changes to pull in content on the associated page-template.

Shoot me (via direct email) some credentials (FTP and admin) so that I can poke around a bit to see what's up.

Any more thoughts on this?
THANX

@dbltoe, not at this time; I might have more time over the weekend to investigate.

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.```

<?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.

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.

<?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: ``` <?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 [url]http://www.zen-cart.com/license/2_0.txt[/url] GNU Public License V2.0 // --------------------------------------------------------------------------- // [B]if (!empty($accountHistory) && is_array($accountHistory)) {[/B] 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 no longer available]

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.