If requesting a shiping quote for any finish address apears do be not recieved from the web service.
The same addresses work on the fedex website quotation tool.

CommentFileSizeAuthor
#4 finland.patch3.95 KBTR
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

TR’s picture

Priority: Critical » Normal
Status: Active » Postponed (maintainer needs more info)

Please turn on the debug output and paste the request/response here so I can try to reproduce this. Don't forget to remove your account number and password.

Code-WorX’s picture

There seems to be no debug output.
What i discover by trying to get a debug output is that there is no selection for country finland for state/province.

For sweden e.g. there will be a state/province selection necessary.

Maybe the Finland bug is because of the missing state selection?

When i compare the necessary fields to the fedex.com website shipping quotes, there won't be any field for state/province to enter at all.

Why is Drupal requesting a state/province by some countries but not for finland even if it seems to bot required be the fedex webservice?

TR’s picture

Title: Shipping quotes for Finland will not requested » Finland CIF doesn't contain ISO 3166-2 zones
Project: FedEx Shipping » Ubercart
Version: 7.x-2.0 » 7.x-3.x-dev
Component: Miscellaneous » Code
Assigned: Unassigned » TR
Status: Postponed (maintainer needs more info) » Active

It looks like the country definition file for Finland is wrong.

finland_246_1.cif was written 5 years ago for the original version of Ubercart, and has not been changed since because no-one mentioned any problems with it. The original country definition files didn't conform to ISO 3166, and in the case of Finland the zones were omitted. (All of the newer .cif files, which are the majority, were written from the start to ISO 3166 standards.)

I will create a patch later today to address this issue with the .cif file.

TR’s picture

Status: Active » Needs review
FileSize
3.95 KB

Try this and let me know if it's correct.

longwave’s picture

Patch works, and the zones are now the same as the ISO 3166-2:FI entry in Wikipedia.

However, "Ahvenanmaan maakunta" apparently translates to "Åland Islands" - which we already have as a separate country?

TR’s picture

Status: Needs review » Fixed

Committed #4. I'll back port it to 6.x-2.x later today.

A lot of countries have that sort of thing. The US has Puerto Rico, for example, which is both a separate country and a zone within the main US file. Whether to use the separate CIF file or not is really something each store has to manage, because it depends on other things. In the US case, shipping by USPS to Puerto Rico is considered a domestic service, so in order to use USPS shipping the customer *must* choose destination = US, state = PR. It will not work at all if the customer chooses country = Puerto Rico. But for FedEx shipping, Puerto Rico is considered an international destination, so the opposite is true.

I don't see any one way to satisfy every requirement, which is why I think it's best to just stick to the ISO 3166 definitions of countries and zones.

Code-WorX’s picture

It works for us too ... will it be included into the uc_store module by the maintainers?

longwave’s picture

When TR said "committed", that means this will be included in the next release of Ubercart, or you can get it now by downloading the 7.x-3.x-dev release.

Code-WorX’s picture

Great .. Thanks so much .. i'm quite new to Drupal, but doing webcoding since 15 years ... everyday i learn something new ;)

TR’s picture

Version: 7.x-3.x-dev » 6.x-2.x-dev
Status: Fixed » Patch (to be ported)

Forgot to do the back port - setting status so I don't forget again.

longwave’s picture

Status: Patch (to be ported) » Fixed

Ported and committed to 6.x.

Status: Fixed » Closed (fixed)

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