Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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.
Comment | File | Size | Author |
---|---|---|---|
#4 | finland.patch | 3.95 KB | TR |
Comments
Comment #1
TR CreditAttribution: TR commentedPlease 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.
Comment #2
Code-WorX CreditAttribution: Code-WorX commentedThere 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?
Comment #3
TR CreditAttribution: TR commentedIt 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.
Comment #4
TR CreditAttribution: TR commentedTry this and let me know if it's correct.
Comment #5
longwavePatch 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?
Comment #6
TR CreditAttribution: TR commentedCommitted #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.
Comment #7
Code-WorX CreditAttribution: Code-WorX commentedIt works for us too ... will it be included into the uc_store module by the maintainers?
Comment #8
longwaveWhen 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.
Comment #9
Code-WorX CreditAttribution: Code-WorX commentedGreat .. Thanks so much .. i'm quite new to Drupal, but doing webcoding since 15 years ... everyday i learn something new ;)
Comment #10
TR CreditAttribution: TR commentedForgot to do the back port - setting status so I don't forget again.
Comment #11
longwavePorted and committed to 6.x.