Hi all,
took a quick look but couldn't find this listed as an issue though it is identified on ubercart.org: http://www.ubercart.org/forum/support/22159/ups_receiving_xml_response_q...
Essentially, according to the debug xml info, I am getting a successful return from UPS but no quote is being displayed and I am getting my standard quote error message.
I worked on the issue and turns out it was the fact that my set site currency was different from what was being returned by UPS. As a result, I wasn't getting into the if statement from line 929:
if (!isset($charge->CurrencyCode) || (string)$charge->CurrencyCode == variable_get('uc_currency_code', "USD")) {
Anyways, I tracked down the issue and fixed it but thought a patch might be in order to throw a more descriptive error when a site admin makes this type of mistake...
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | UPS-receiving-xml-response-but-no-quote-displayed-1215674.patch | 850 bytes | yaworsk |
| #1 | UPS-receiving-xml-response-but-no-quote-displayed-1215674.patch | 755 bytes | yaworsk |
Comments
Comment #1
yaworsk commentedProbably a nicer way to do this as you have to refresh the page to see the error but hopefully a step in the right direction.
Comment #2
yaworsk commentedOpps,
ignore the previous patch -- accidentally patched version 7. Also, the reference to line 929 in the issue is incorrect, it is actually 839...
D6 patch attached.
Comment #3
longwaveComment #4
tr commentedIsn't there a way to send tell UPS what currency you want to use for the rates? If so, we should implement that part of the API, rather than just throwing an error. The patch has some coding standards problems and isn't translatable.
New features should go into 7.x-3.x first.
Comment #5
longwaveAs this is such an edge case and only occurs when initially configuring UPS let's just close it out.