The fix is to change the country_name in the database to 'Korea, Republic of (South Korea)' instead of just 'Korea, Republic of'.

Comments

greenskin’s picture

There might be more of these types of issues for other countries. The list which I found to compare country names with is http://pe.usps.gov/text/imm/immctry.htm. Perhaps all country names need to correspond to those in this list to ensure that USPS always recognizes the country.

rszrama’s picture

Title: 'Korea, Republic of' returns as invalid country for USPS » Map country names in USPS from ISO-3166 to USPS value - ex: 'Korea, Republic of'
Version: 5.x-1.4 » 5.x-1.5
Priority: Critical » Normal

I'd rather ensure that country names correspond to the ISO-3166 standard. The name it has now is the correct name, so we probably just need to make sure the USPS module has some way to map names to what they're expecting. Would be nice if they could go with the international standard, too. : P

cha0s’s picture

Assigned: Unassigned » cha0s
Status: Active » Needs review

Generated an array from that USPS page. I attached a patch for 2.x here...

cha0s’s picture

Hrm, looks like my patch didn't get attached. Drupal update burp probably... tryin' again:

cha0s’s picture

Well, looks like it isn't working. I've put it up on my site for your viewing pleasure. http://therealcha0s.net/tmp/uc_usps.country.patch

rszrama’s picture

Version: 5.x-1.5 » 5.x-1.7

What does that func do?

cha0s’s picture

The function maps a 3 digit ISO code to a country name that USPS can understand. I got patches for 1.x and 2.x now, as well. Drupal's attach function here is still borked though, so...

http://therealcha0s.net/tmp/uc_usps.country.1.x.patch
http://therealcha0s.net/tmp/uc_usps.country.2.x.patch

Island Usurper’s picture

Status: Needs review » Needs work

So, I can't get a quote to ship something to Great Britain. That's probably the most often requested change that this issue needs to address. I think that USPS would use different names for some of the same country codes (Wales, Scotland, England) and this array wouldn't be able to handle that.

kmonty’s picture

Title: Map country names in USPS from ISO-3166 to USPS value - ex: 'Korea, Republic of' » Map country names in USPS from ISO-3166 to USPS value - ex: 'Korea, Republic of' and "Great Britain"
Version: 5.x-1.7 » 6.x-2.x-dev
Priority: Normal » Critical

Any status updates? This is a huge problem with Ubercart, in my opinion.

Island Usurper’s picture

Priority: Critical » Normal
kmonty’s picture

Priority: Normal » Critical

How is this not a critical bug? A core module of Ubercart does not perform its advertised function. According to the forums, this has been an ignored issue since late 2007. That seems like a big deal to me.

If I want to use USPS over fedex/ups for international shipping (which I do), I have to accept the fact that a number of customers will not be able to purchase products from my website.

greenskin’s picture

I agree, this issue should be set as critical as it should either not have USPS option for international or fix this issue.

GreyHawk’s picture

kmonty’s picture

#326836: Update the UK country file to include all zones Is not related, although it is an important issue. The ubercart forum issue is related.

Eric Link’s picture

Version: 6.x-2.x-dev » 6.x-2.0-rc1

This is a show stopper for my client right now; they are shipping UPS, FedEx, USPS and USPS is the preferred for international. I appreciate the need for a long term solution but right now it's difficult to figure out what the work around is? I could put code in the USPS module to check for GB and switch to UK, or change the country file. I'm going to change the country file and see if fedex/ups care. If they do, I'll change the code for the USPS module...

Eric Link’s picture

StatusFileSize
new5.86 KB

Update: for my case, the attached works as a workaround (USPS module should deal w/ this really).

I ship international via USPS only, so switching the country file to this one, w/ name changed from "United Kingdom" to "United Kingdom - (Great Britain)" allows the country name to pass USPS validation.

Island Usurper’s picture

Issue tags: +Release blocker

I think I'd rather have substitute the right country names in the messages to USPS than change the country name on all of the select elements throughout Ubercart.

Cha0s, I don't think your patches are available from your machine anymore. Can you post them here again?

rszrama’s picture

Priority: Critical » Normal
Island Usurper’s picture

Assigned: cha0s » Island Usurper
Category: task » bug
Status: Needs work » Needs review
StatusFileSize
new2.21 KB

Actually sat down and compared Ubercart's list of countries to the Postal Service's. We probably should have specified that country import files should be in English and use the t() function, but that's water under the bridge now. Even still, the list isn't as long as I thought it would be.

Island Usurper’s picture

Version: 6.x-2.0-rc1 » 6.x-2.x-dev
Status: Needs review » Fixed

I can get quotes for shipments to the UK and the Republic of Korea with this patch. I think it's interesting that you only need to know the country to get a quote.

Committed.

elijah lynn’s picture

What is the status of this functionality making it into the next RC?

We can't get shipping quotes for Turkey with RC6.

Island Usurper’s picture

It's in RC7.

Status: Fixed » Closed (fixed)
Issue tags: -Release blocker

Automatically closed -- issue fixed for 2 weeks with no activity.