Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.But they're now fixed ;)
Problem was that the imported file had countries with no geonameid. The new postal_load_countries checks for that.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | postal-1027646.patch | 3.02 KB | barraponto |
| postal-fix-install-errors.patch | 3.02 KB | barraponto |
Comments
Comment #1
barrapontowrote a buggy patch, now it works properly.
Comment #2
wylbur commentedApplied the patch and tested an import of sates/provinces. Using Drupal 6.20 and Postal 6.x-1.0.
Import of US states added all US states to postal database as entries.
Import of Denmark states added 5 states, Capital Region, Central Jutland, North Jutland, South Denmark, Zealand.
Import of Antartica failed:
warning: Invalid argument supplied for foreach() in C:\sites\lupino\drupal\sites\lupino.us\modules\postal\includes\postal.admin.inc on line 136.
Import of Austrian states added 9 entries
Import of Australia generated error:
warning: Invalid argument supplied for foreach() in C:\sites\lupino\drupal\sites\lupino.us\modules\postal\includes\postal.admin.inc on line 136.
Comment #3
barrapontoThe issues on install happened while the countries were imported. These errors are now fixed. Importing states and provinces will need further debugging, and I need to take a look at it to make Brazilian states work.
Comment #4
bochenn commentedHi @barraponto, I don't know how to use the patch file, can you publish an updated version of the module with the issues fixed?
Thanks
Comment #5
barraponto@bochenn: http://drupal.org/patch/apply explains the whole patching process.
i have posted a patched version of the module here https://github.com/barraponto/postal (but it is still bugged)