Conditional shipping option?

[Ajeh]

What version of FedEx fedexwebservices are you running>

If you customize the fedexwebservices.php and add the code in RED:

        if (sizeof($methods) == 0) return false;
        $this->quotes['methods'] = $methods;
        if ($this->tax_class > 0) {

does it fix this issue?

[split63]

In looking at the admin panel and the shipping modules page, I noticed the error message:
ERROR: module not loaded due to missing language file:/includes/languages/english/modules/shipping/usps_xx.php

At some point I had renamed the usps.php file to usps_xx.php, as a way to make a copy. Somehow, Zen now thinks that the file is called usps_xx. I'm at a loss to understand how this has happened. I'm sure it is something I did some time ago.
Its may be unrelated to this issue, I'm not sure.

The webservices module does not appear to have a version number. But I believe it is 1.3.2

[Ajeh]

When renaming modules, you cannot leave them with the .php extension as those are auto loading directories ... instead ues something like:
usps.php_xx

now it will not auto load ...

[split63]

If I had to guess, I would say this problem was introduced when the new rate module was released for USPS

USPS Module for Zen Cart v1.3.x thru v1.6
* USPS RateV4 Intl RateV2 - January 28, 2013 Updates to: July 28, 2013 Version J

[Ajeh]

I am not sure how that change for USPS affected FedEx, but that fix for the FedEx is an educated guess as I really cannot run that code to test it ...

[Ajeh]

The webservices module does not appear to have a version number. But I believe it is 1.3.2

The current FedEx from numinix's site is Version: 1.4.5 ... you should really consider upgrading to that ...

[split63]

What version of FedEx fedexwebservices are you running>

If you customize the fedexwebservices.php and add the code in RED:

        if (sizeof($methods) == 0) return false;
        $this->quotes['methods'] = $methods;
        if ($this->tax_class > 0) {

does it fix this issue?

Yes, it did fix the issue.
I will look into to updating the webservices modules

[Ajeh]

Thanks for the update that this worked for you ...

It was a good catch to run into this in FedEx as I see FedEx needs this extra code for the:

        if (sizeof($methods) == 0) return false;
        $this->quotes['methods'] = $methods;

as well as UPS ... it only happens when the configuration has settings that result in no quotes such as configuring the shipping module to only show for National or International shipping but not both or when the selected options all result in no quote ...

USPS already has this code in it in the July 28, 2013 J5 release so that one does not run into this issue ...