UPS shipping module

Anonymous · 11-19-2002, 01:37 AM

During testing (shipping to myself) I get the following options:
Ground Residential
Next Day Air Early AM
Next Day Air
Next Day Air Saver
2nd Day Air
3 Day Select
3 Day Select
Ground Commercial

is there a way to have it only show:
2nd Day Air
Ground

and to places out of the country just show:
2nd Day Air

or something along those lines?

**modom** · 11-23-2002, 07:43 AM

Hi,

You need to change this in the admin area.

**someguy** · 11-23-2002, 09:47 AM

where can that be changed in the admin area? im looking in modules/shipping and only have these options

Enable UPS Shipping
1

UPS Pickup Method
CC

UPS Packaging?
CP

Residential Delivery?
RES

**modom** · 11-23-2002, 10:45 AM

Hi,

I'm sorry but I misunderstood your question before.

Maybe you could comment out some of these in one of the files.

I will look around and see if I can find out how to do this or if it is possible.

**modom** · 11-23-2002, 11:14 AM

Hi,

You can get rid of the names in catalog/includes/modules/shipping/ups.php by commenting out this code:

//'1DM' => 'Next Day Air Early AM',
//'1DML' => 'Next Day Air Early AM Letter',
//'1DA' => 'Next Day Air',
//'1DAL' => 'Next Day Air Letter',
//'1DAPI' => 'Next Day Air Intra (Puerto Rico)',
//'1DP' => 'Next Day Air Saver',
//'1DPL' => 'Next Day Air Saver Letter',
'2DM' => '2nd Day Air AM',
'2DML' => '2nd Day Air AM Letter',
'2DA' => '2nd Day Air',
'2DAL' => '2nd Day Air Letter',
//'3DS' => '3 Day Select',
'GNDCOM' => 'Ground Commercial',
'GNDRES' => 'Ground Residential');
//'STD' => 'Canada Standard',
//'XPR' => 'Worldwide Express',
//'XPRL' => 'worldwide Express Letter',
//'XDM' => 'Worldwide Express Plus',
//'XDML' => 'Worldwide Express Plus Letter',
//'XPD' => 'Worldwide Expedited');

Notice that I put the ending ); tag after Ground Residential listing.

I haven't figured out how to get rid of the amounts or the radio buttons for the commented out listings.

Anonymous · 11-23-2002, 12:57 PM

if you figure out how to eliminate those from showing up even tho they are commented out please let me know how to do it!!!

thanks!

looker@fwdepot.com

**stan** · 11-26-2002, 02:26 PM

You just need to wrap the code in the quote() method to only add the method if it is supported. It's around line 76 of catalog/includes/modules/shipping/ups.php.

Change:

list($type, $cost) = each($upsQuote[$i]);
$methods[] = array('id' => $type,
'title' => $this->types[$type],
'cost' => (SHIPPING_HANDLING + $cost) * $shipping_num_boxes);

To:
list($type, $cost) = each($upsQuote[$i]);
if( isset($this->types[$type]) )
{
$methods[] = array('id' => $type,
'title' => $this->types[$type],
'cost' => (SHIPPING_HANDLING + $cost) * $shipping_num_boxes);
}

I don't see why this check shouldn't be made part of the normal distribution imho...

hth,
-- stan c",)

**stan** · 11-26-2002, 02:29 PM

Hmm... as a side note, I really like the latest changes in the shipping stuff. It looks much nicer! Thanks, Harald!

-- stan c",)

**modom** · 11-27-2002, 07:04 AM

Hi,

There was a parse error.

The code should be changed to:

list($type, $cost) = each($upsQuote[$i]);
if( isset($this->types[$type]) )
$methods[] = array('id' => $type,
'title' => $this->types[$type],
'cost' => (SHIPPING_HANDLING + $cost) * $shipping_num_boxes);
}

It works great!

**stan** · 11-27-2002, 10:51 AM

i had myself confused for awhile there! i thought i was missing the opening curly brace!

But I checked and figured out the real misunderstanding: I cut & pasted just the inside of the for-loop, which just so happens to end right after my original snippet. I'm guessing that people are mixing up my ending curly with the ending curly for the for-loop.

Here's an expanded snippet that hopefully clears it up. The other way works too, but is trickier cuz the if will only evaluate the next statement -- I prefer explicit blocks, but hey, it's just a style thang!

Expanded snippet to follow:

Code:

        for &#40;$i=0; $i&lt;sizeof&#40;$upsQuote&#41;; $i++&#41; &#123;
          list&#40;$type, $cost&#41; = each&#40;$upsQuote&#91;$i&#93;&#41;;
          if&#40; isset&#40;$this->types&#91;$type&#93;&#41; &#41;
          &#123;
            $methods&#91;&#93; = array&#40;'id' => $type,
                               'title' => $this->types&#91;$type&#93;,
                               'cost' => &#40;SHIPPING_HANDLING + $cost&#41; * $shipping_num_boxes&#41;;
          &#125;
        &#125;

mystery solved! whew!

-- stan c",)

p.s. humph! i noticed something else sucky... i haven't been marking my code snippets with the proper tags, so all my spaces/tabs were just being munched because it was just normal html. doh!

oth, it's almost thanksgiving! yay! i'm counting down the hours.