Need to add per item fee (based on category) to the ups module

[Oldcarguy]

Hi Linda,

I'm new to PHP but I was able to modify your code somewhat to my purposes. I'm trying to create 2 additional handling categories in your UPS Module. One for shipping a 4' tube to add an additional $10 to the shipping cost, and one for shipping an 8' tube to add the $10 cost of the tube, plus the mandatory $10.50 addl. handling fee for UPS. Here's my code, but it shows 2 of every shipping option in the cart, and it only takes the 4' option. Any advice is appreciated!

// bof: customize for extra charges
      global $cart;
      $chk_molding = 0;
      $chk_molding += $_SESSION['cart']->in_cart_check('master_categories_id','141');
      $chk_molding += $_SESSION['cart']->in_cart_check('master_categories_id','136');
      $chk_molding += $_SESSION['cart']->in_cart_check('master_categories_id','137');
      $chk_molding += $_SESSION['cart']->in_cart_check('master_categories_id','139');
      $chk_molding += $_SESSION['cart']->in_cart_check('master_categories_id','138');
      $chk_molding += $_SESSION['cart']->in_cart_check('master_categories_id','140');
      $chk_molding += $_SESSION['cart']->in_cart_check('master_categories_id','142');
      $chk_molding += $_SESSION['cart']->in_cart_check('master_categories_id','171');
      $chk_molding4 = 0;
      $chk_molding4 += $_SESSION['cart']->in_cart_check('master_categories_id','164');
      $chk_molding4 += $_SESSION['cart']->in_cart_check('master_categories_id','165');
      $chk_molding4 += $_SESSION['cart']->in_cart_check('master_categories_id','166');
      $chk_molding4 += $_SESSION['cart']->in_cart_check('master_categories_id','167');
      $chk_molding4 += $_SESSION['cart']->in_cart_check('master_categories_id','169');
      $chk_molding4 += $_SESSION['cart']->in_cart_check('master_categories_id','170');
// continue adding for more categories
      $molding_handling = 0;
      if ($chk_molding > 0) {
        // add tube
        $molding_handling = 10.00;
        // add extra handling
        $molding_handling += 10.50;
      } 
$molding4_handling = 0;
      if ($chk_molding4 > 0) {
        // add tube
        $molding_handling = 10.00;
      }
// add $molding_handling to cost
    $methods[] = array('id' => $type,
                           'title' => $this->types[$type],
                           'cost' => ($cost * $shipping_num_boxes) + (MODULE_SHIPPING_UPS_HANDLING_METHOD == 'Box' ? MODULE_SHIPPING_UPS_HANDLING * $shipping_num_boxes : MODULE_SHIPPING_UPS_HANDLING) + $molding_handling );
// add $molding4_handling to cost
    $methods[] = array('id' => $type,
                           'title' => $this->types[$type],
                           'cost' => ($cost * $shipping_num_boxes) + (MODULE_SHIPPING_UPS_HANDLING_METHOD == 'Box' ? MODULE_SHIPPING_UPS_HANDLING * $shipping_num_boxes : MODULE_SHIPPING_UPS_HANDLING) + $molding4_handling );
// eof: customize for extra charges 

Thanks in advance!
Using v1.5.0 - clean install, no upgrades: http://restorationstuff.com/ecommerce
~Joe

[Ajeh]

You put the building of the cost in twice with:

// add $molding_handling to cost 
    $methods[] = array('id' => $type, 
                           'title' => $this->types[$type], 
                           'cost' => ($cost * $shipping_num_boxes) + (MODULE_SHIPPING_UPS_HANDLING_METHOD == 'Box' ? MODULE_SHIPPING_UPS_HANDLING * $shipping_num_boxes : MODULE_SHIPPING_UPS_HANDLING) + $molding_handling ); 
// add $molding4_handling to cost 
    $methods[] = array('id' => $type, 
                           'title' => $this->types[$type], 
                           'cost' => ($cost * $shipping_num_boxes) + (MODULE_SHIPPING_UPS_HANDLING_METHOD == 'Box' ? MODULE_SHIPPING_UPS_HANDLING * $shipping_num_boxes : MODULE_SHIPPING_UPS_HANDLING) + $molding4_handling ); 
// eof: customize for extra charges  

Change that to:

// add $molding_handling to cost 
// add $molding4_handling to cost 
    $methods[] = array('id' => $type, 
                           'title' => $this->types[$type], 
                           'cost' => ($cost * $shipping_num_boxes) + (MODULE_SHIPPING_UPS_HANDLING_METHOD == 'Box' ? MODULE_SHIPPING_UPS_HANDLING * $shipping_num_boxes : MODULE_SHIPPING_UPS_HANDLING) + $molding_handling + $molding4_handling ); 

[Oldcarguy]

Oh my goodness, you're a genius. Thank you so much!!! Only one little issue remains: If a customer orders both a 4' stick and an 8' stick of molding, the cart only applies the 4' tube charge. Is there an easy fix for that?

[Ajeh]

Are you charge 1 fee for all of the matches or 1 per each match?

[Oldcarguy]

Thanks for the prompt responses! Only one fee per shipment; $10 for a 4 foot tube, and $20.50 for an 8 foot tube. But if a customer needs a 4 foot and an 8 foot item we'll obviously ship them both in the 8' tube, that's where the code is only charging $10 instead of $20.50.

[Ajeh]

Try changing the code from:

// continue adding for more categories 
      $molding_handling = 0; 
      if ($chk_molding > 0) { 
        // add tube 
        $molding_handling = 10.00; 
        // add extra handling 
        $molding_handling += 10.50; 
      }  
$molding4_handling = 0; 
      if ($chk_molding4 > 0) { 
        // add tube 
        $molding_handling = 10.00; 
      } 

to be:

// continue adding for more categories 
      $molding_handling = 0; 
      if ($chk_molding > 0) { 
        // add tube 
        $molding_handling = 10.00; 
        // add extra handling 
        $molding_handling += 10.50; 
      }  
$molding4_handling = 0; 
      if ($chk_molding4 > 0) { 
        // add tube 
        $molding4_handling = 10.00; 
      } 

[Oldcarguy]

We're closer, but if now if the customer ordered both an 8 foot and a 4 foot it charges for both instead of just the required 8 foot tube. Seems like I need one of those magical 'else if' statements that I have no idea how to program LOL... You're amazing, coffee and donuts on me!

[Ajeh]

Okay let's try this:

// continue adding for more categories 
      $molding_handling = 0; 
      if ($chk_molding > 0) { 
        // add tube 
        $molding_handling = 10.00; 
        // add extra handling 
        $molding_handling += 10.50; 
      }  
$molding4_handling = 0; 
      if ($chk_molding4 > 0) { 
        // add tube 
        $molding4_handling = 10.00; 
      } 
      if ($chk_molding > 0 && $chk_molding4 > 0) { 
$molding_handling = 0;
      }

[Oldcarguy]

This modification only charges for the 4' tube if I include an 8' stick :-(

[Ajeh]

Then change this from:

      if ($chk_molding > 0 && $chk_molding4 > 0) {  $molding_handling = 0;       }

to read:

      if ($chk_molding > 0 && $chk_molding4 > 0) { 
        $molding_handling = 20.50;
        $molding4_handling= 0.00;
      }