UPSXML - once crashed under a bad address no good addresses work

[jpietrowiak]

Strange problem I ran into. Using UPS XML module. I have an an address called in from a customer that does not work. Comes up with blank screen. I tried logging in separate account and tried anther address from austrailia in a different city to see if it worked and it did. Then I tried changing just the address to some not even close address and left zip city and such alone. Then got the blank screen. Then changed it back and tried checkout. Still blank screen. Then I went from firefox to IE and logged in. then tried with same shipping address that I fixed and it is fine. So I changed the address once more in IE and again blank screen. CHanged back and same thing. Changing back to a good address does not work. So is there something built into the browsers that is holding it up from refreshing. I tried refreshing every page before going to checkout and it did no good.

[jpietrowiak]

Anyone want to try this address in their store to see if it works?

18 Ardena Court
East Bentleigh, 3165
VIC, Australia

I also tried the below thinking maybe this would be the correct city
18 Ardena Court
Melbourne, 3165
VIC, Australia

[jpietrowiak]

Update:
The address provided was infact wrong. HOWEVER......why does the checkout shipping page not respond and still give blank screen once the correct address is input? If I turned off the module then refresh the checkout page it works, then turn it back on and it then works with a good address?

Also is there a way to not get a blank page or when it does to tell the customer that you entered in your address wrong? Otherwise there is a chance to loose customers.

[Ajeh]

Assuming you are using v1.3.9, check and see if you have any debug logs in the /cache directory ...

[jpietrowiak]

Yes 1.3.9
From debug log
[29-Sep-2011 06:33:26] PHP Fatal error: Cannot use string offset as an array in /home/jpietrowiak/cnc-specialty-store.com/includes/modules/shipping/upsxml.php on line 265


Tracking this down I even enabled the debug for shipping module and the below is what came up. Which is just that UPS didn't recognize the address.

UPS RESPONSE: <?xml version="1.0"?><TimeInTransitResponse><Response><TransactionReference><CustomerC ontext>Time in Transit</CustomerContext><XpciVersion>1.0001</XpciVersion></TransactionReference><ResponseStatusCode>0</ResponseStatusCode><ResponseStatusDescription>Failure</ResponseStatusDescription><Error><ErrorSeverity>Hard</ErrorSeverity><ErrorCode>270032</ErrorCode><ErrorDescription>Invalid Destination Postal Code and City</ErrorDescription></Error></Response></TimeInTransitResponse>

Thanks

[Ajeh]

This is rather odd ... try changing line 261 to:

                if (isset($this->servicesTimeintransit[$type]) && sizeof($this->servicesTimeintransit[$type]) > 1) {

[jpietrowiak]

I think you might have just bought yourself a cup of coffee. Yes this worked. No blank page. I ran a couple tests changed the city to something that made no sense and it did not error. Also did the same with the zipcode and address and no error. Shipping rates still came up fine. So I assume it goes by the city victoria and austrailia only at that point. I am guessing this is ok, but I wonder if the shipping rates will be accurate if the address is completly wrong. Will have to test on USA states once to see if there is a difference.

Will let you know what I find.

[jpietrowiak]

Ajeh
You are the man! It looks like all the shipping values are ok with that change. If I change the zip the first 5 numbers and it does not match the city then ups errors but shows postal service shipping options, so that works. If I add any 4 numbers after the first 5 it still displays shipping quotes. So I think this is a good fix.

Thank you for this fix!!

Now my question would be what is the process to get this fixed or updated and tested to patch the UPSXML mod. Because I think this is important for most people who use this mod.

Thanks again

[Ajeh]

Need someone to fix the bugs and add it to the contributions ...

Another common bug is changing this line:

        $this->host = ((MODULE_SHIPPING_UPSXML_RATES_TEST_MODE == 'Test') ? 'www.ups.com' : 'www.ups.com');

so both use the ups.com site ...

[jpietrowiak]

I would do it if I new how to code. I'm technically inclined just code stupid . I have no idea what the little code change you recommended did, but it worked.

As far as the other code. I have mine set at this "'wwwcie.ups.com' : 'onlinetools.ups.com" which I found awhile back that someone had recommended the change since I had issues with something or UPS changed there servers or something. Can't remember why exactly?

Not sure how the cup of coffee works but I sent something for your help.