Disabling UPS for certain categories

[getyler2]

I need to disable UPSXML for categories 1, 38 and 39. I've read several threads on this and think this is all I need to do, but it doesn't work. Probably a syntax thing. Ideas?

Add this code to /includes/modules/shipping/upsxml.php. The commented out code is another unsuccessful try.

            if ($check_flag == false) {
                $this->enabled = false;
            }
// disable if master_categories_id 1 is in the cart
//      if ($_SESSION['cart']->in_cart_check('master_categories_id','1') > 0) {
//          $this->enabled = false;
//      }
// skip certain categories
      if (IS_ADMIN_FLAG == false && $_SESSION['cart']->in_cart_check('master_categories_id','1') > 0) {
        $this->enabled = false;
      }  
      if (IS_ADMIN_FLAG == false && $_SESSION['cart']->in_cart_check('master_categories_id','38') > 0) {
        $this->enabled = false;
      }  
      if (IS_ADMIN_FLAG == false && $_SESSION['cart']->in_cart_check('master_categories_id','39') > 0) {
        $this->enabled = false;
      }  
          }

[Ajeh]

If you add below the line:

// skip certain categories
global $cart;

[Ajeh]

Sorry goober moment ... I think you have the code in the wrong place ...

Can you move it a little higher up and below the code:

      // disable only when entire cart is free shipping
      if (zen_get_shipping_enabled($this->code)) {
        $this->enabled = ((MODULE_SHIPPING_FLAT_STATUS == 'True') ? true : false);
      }

[Ajeh]

NOTE: in UPSXML the code will probably look like:

        $this->enabled = ((MODULE_SHIPPING_UPSXML_RATES_STATUS == 'True') ? true : false);

[getyler2]

That's it! Just pasting it higher, above this line makes it work beautifully.

          if (($this->enabled == true) && ((int)MODULE_SHIPPING_UPSXML_RATES_ZONE > 0)) {
            $check_flag = false;

Many thanks Ajeh! You rock.

[Ajeh]

You had put your code in a section where the Zone had to be tested but the Zone wasn't tested as there was no Zone set ...

Having you move it, places it outside of that Zone Test and the results will disable the Zone test from even being looked at ...

Thanks for the update that this is now working for you ...

[getyler2]

Grrrrr! Works great for frozen items in the cart as now it doesn't show the UPS (priced) shipping options when Free is selectable right below it using freeshipper and 0 weight. But put a non-frozen item in the cart and now nothing shows at all, neither regular UPS options or the freeshipper option. Just empty. Going to checkout, the only option is store walkin. That ain't gonna work so I commented it out and UPS shows on frozen items which should be just freeshipper. Hmmm.

Seems like it should just display normal shipping for categories other than 1, 38 and 39, and show Shipping Included for those categories. Guess it's the combination that's screwing it up. Maybe I need a split category that allows a combination or something. Doesn't sound fun.

http://www.texastamale.com/store/