Several cif files use the uc_set_address_format() function, but pass the first parameter as an octal value. For instance, Belgium is passed as 056 instead of 56. 056 turns into 46 in the database, so the address formats are not available for some countries.
And regarding Belgium as well, all the zones are named in Dutch, even the French ones. I believe having the zones in their local language will be much clearer.
I attached a patch.

CommentFileSizeAuthor
#3 address_format.patch1.06 KBalpritt
belgium_56_1.cif_.patch1.01 KBYannickD

Comments

YannickD’s picture

YannickD commented

Forgot to mention:
The countries affected by the "octal bug" are Argentina, Australia, Austria, Belgium, Brazil, Korea and Saint Lucia.

alpritt’s picture

alpritt commented
Version: 5.x-1.4 » 6.x-2.x-dev
Priority: Minor » Normal
Status: Active » Needs work

This affects 6.x-2.x-dev too. It is customary in Drupal core to fix the latest version first and then backport, but perhaps that is not how things are done in Ubercart; I'm not sure of the policy.

Patch only covers one of the affected countries.

Renaming the zones is probably a good idea but should probably be a separate issue.

This will likely conflict with #292577: Update .cif files to use the serial zone_id column.

alpritt’s picture

alpritt commented
Status: Needs work » Needs review
StatusFileSize
new address_format.patch1.06 KB

Belgium seems to be fixed already, so that just leaves these two.

Island Usurper’s picture

Island Usurper commented
Status: Needs review » Fixed

Cool beans.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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