Hello,

I don't understand why location module have the country and regions in files (location/supported/). That's for performance reason? But if you have all or a lot fo country activate, performance still ok with all this include?

DB isn't a better choice for import/export, extensibility and communication between other module....

And why don't eliminate completely the country files? For formating and country special case, http://www.ubercart.org have a nice little pattern system.

Comments

gagarine’s picture

gagarine commented

An example of country information (or import?) file for ubercart: http://www.ubercart.org/files/italy_380_1.cif

nicholasthompson’s picture

nicholasthompson
Primary language English
commented

I've also wondered this.

When looking up states (for example), would it not be more efficient to fire off 1 DB Query rather than including an array of hundreds of items?

bdragon’s picture

bdragon commented

I agree, it's inefficient.

It will be changing soon. At the very least, all the "common" stuff like region names can be moved into a table.

gagarine’s picture

gagarine commented

Nice!

Kerel’s picture

Kerel commented

That would give a lot of advantages indeed!
May be also look at the possibility to include cities in the database to extend the scope of this module.

yesct’s picture

yesct commented
Issue tags: +location performance improvements

tagging.

clashar’s picture

clashar commented

subscribe

gagarine’s picture

gagarine commented
Category: support » feature
suffering drupal’s picture

suffering drupal commented
Issue tags: -location performance improvements +location performance improvements regions

The typical state/province field is rather limited and offers only mandatory options.
Is there a way to manually add an additional "Region" field (to location, not CCK) somehow? To enable the use of for example Yellowstone Park, Patagonia, the Alps, Côte d'Azur, Veluwe, Toscana, Costa del Sol and all those other marvelous regions that are not politically defined, but generally more interesting.

gagarine’s picture

gagarine commented

They are also a http://drupal.org/project/countries field now

suffering drupal’s picture

suffering drupal commented

No regions however, right?

suffering drupal’s picture

suffering drupal commented

I get it, it has to be handmade, yes?

legolasbo’s picture

legolasbo
he/him
Primary language Dutch
Location Middelburg
commented
Issue summary: View changes
Status: Active » Closed (outdated)

Closing old D6 issues as D6 is end of life