No Shipping To Po Box?

[zenned]

hOW do I set zen to not allow people to ship to PO Boxes??
thanks.

[DrByte]

I did the following on one site:

includes/modules/shipping/ups.php
line 118 starts with:

  function quote($method = '') {
    global $_POST, $order, $shipping_weight, $shipping_num_boxes;

    if (zen_not_null($this->icon)) $this->quotes['icon'] = zen_image($this->icon, $this->title); 

I added a couple lines after that to check for PO boxes, like this:

  function quote($method = '') {
    global $_POST, $order, $shipping_weight, $shipping_num_boxes;

    if (zen_not_null($this->icon)) $this->quotes['icon'] = zen_image($this->icon, $this->title);

    // disable this module if address contains "PO Box"
    $addr_check = str_replace(array('.',' ', '/'),'',strtoupper($order->delivery['street_address']));
    if (strstr($addr_check,'POBOX') || strstr($addr_check,'POSTOFFICEBOX')) {
      $this->quotes = array('module' => $this->title,
                            'error' => 'UPS cannot ship to a PO Box address.');
      return $this->quotes;
    } 

[zenned]

Dr Byte: I am using the Flat Shipping, is this the same code?

[DrByte]

most likely would work the same.... don't add the code I repeated in the second box, just the extra code added

[jeff_fwd]

I did the following on one site:

includes/modules/shipping/ups.php
line 118 starts with:

  function quote($method = '') {
    global $_POST, $order, $shipping_weight, $shipping_num_boxes;

    if (zen_not_null($this->icon)) $this->quotes['icon'] = zen_image($this->icon, $this->title); 

what version of the module are you quoting above? in the latest version, that piece of code reads

     if ( (zen_not_null($method)) && (isset($this->types[$method])) ) { 

so are you embedding the new code within the body of that IF statement, or before it now?

[DrByte]

oops ... skip that line altogether ... I had relocated it from later in the file for other purposes

Skip this in my examples above:

 if (zen_not_null($this->icon)) $this->quotes['icon'] = zen_image($this->icon, $this->title); 

- thx for the catch Jeff

[jeff_fwd]

no problem - just wanted to make sure I wasnt missing something

so now, with the addition of your code snippet, it will look like this (correct?):

    if (zen_not_null($this->icon)) $this->quotes['icon'] = zen_image($this->icon, $this->title);

// disable this module if address contains "PO Box" 
    $addr_check = str_replace(array('.',' ', '/'),'',strtoupper($order->delivery['street_address'])); 
    if (strstr($addr_check,'POBOX') || strstr($addr_check,'POSTOFFICEBOX')) { 
      $this->quotes = array('module' => $this->title, 
                            'error' => 'UPS cannot ship to a PO Box address.'); 
      return $this->quotes; 
    }  
    
    return $this->quotes;
  } 

[gjh42]

You also need to check for "BOX" at the beginning of the string, because not everyone specifies "PO" in their address.

[DrByte]

 if (strstr($addr_check,'POBOX') || strstr($addr_check,'POSTOFFICEBOX') || strstr($addr_check,'BOX')) { 

[Ajeh]

BOX alone may not be ideal ...

The problem being folks using Mailboxes etc. my say Box 12 where they won't say PO Box 12 ...

And Mailboxes etc. is a real location so UPS does deliver ...