Ticket #143 (closed defect: fixed)

Opened 6 years ago

Last modified 6 years ago

Location module not showing various locations

Reported by: ed Owned by: ed
Priority: minor Milestone: Phase 3
Component: Drupal modules & settings Keywords:
Cc: Estimated Number of Hours: 0.0
Add Hours to Ticket: 0 Billable?: yes
Total Hours: 0.1

Description

The location module is an issue. Users in different places aren't able to find suitable regions to set for their initiatives, so can't set up their profiles. I've looked for a list of the regions the location module uses but found nothing so can't help them other than choose random regions in their countries, which they're not happy with.

Setting to high for reply from Jim so I can bodge it while we work out how the location module works and do something longer term

  1. Denmark: Fejo - can't find correct region
  2. France: Le Coteau de Plombières-Les-Bains en Transition - can't find correct region
  3. Hong Kong: doesn't work at all

A: is there a list of the regions that the location module uses that we can refer to to bodge it?

B: is there a location moduel information page and ticket we can work with (if suitable?)

Change History

comment:1 Changed 6 years ago by ed

See also Capilla Del Monte, in Argentina, which has full location data but is not showing up on maps...
http://www.transitionnetwork.org/initiatives/capilla-del-monte-transition

comment:2 Changed 6 years ago by ed

another in france:
State/province: - Region Provence Alpes Cote d'Azur

comment:3 Changed 6 years ago by ed

Also now Portugal (Minho) not findable

Can we simply make the region *NOT* mandatory? Or is region mandatory for the map? Can they just add their zip or city?

comment:4 Changed 6 years ago by jim

It'd be wise to update the Locations database when I next do an update to the site - there's a high chance the newer version has some corrected/extra area details.

We *should* be able to make region non mandatory... Be nice to have the data we need though.

comment:5 Changed 6 years ago by ed

OK - is there a list of the regions anywhere? according to the users, there is no logic to the choice of regions... at least offering them a list would be a start...

comment:6 Changed 6 years ago by ed

  • Milestone set to Phase 3

comment:7 Changed 6 years ago by ed

so important my pants are on fire

comment:8 Changed 6 years ago by ed

might be worth looking at locations database which might be out of date - not necc the module -

comment:9 Changed 6 years ago by jim

  • Owner changed from jim to ed
  • Priority changed from critical to minor
  • Status changed from new to assigned

OK. I was wrong about the database - that's just for postcode lookups.

Regions are held in the the location module's 'supported' folder as arrays in include files - in other words, easily editable.

BUT I'm afraid, having looked through a few examples above show consistent USER ERROR! They just don't know the correct part of the world (officially speaking) that they reside in:

So Ed, this is a case of users not knowing the correct province/state that they need, simple as that. What do you want to do about it?

My 2p: I recommend handling/educating the occasional person with an issue rather than dumping the province choice as it's central to the site to have accurate geolocation stuff occurring. And, since addresses are looked up via Google, having the wrong one isn't going to help anyone.

comment:10 Changed 6 years ago by jim

FYI, location module country region files (by 2 letter country code) are here:
https://tech.transitionnetwork.org/trac/browser/www/trunk/sites/all/modules/location/supported

comment:11 Changed 6 years ago by ed

will look into this in more detail on monday..

comment:12 Changed 6 years ago by ed

handy list - that was my next request - will send over to those in question and see what they say...

comment:13 Changed 6 years ago by ed

feedback from fejo:
"'ST' => "Storstrom" is the best (though I think you have an old list of 'amts' (Counties)). We used to be in Storstrom, and everyone has an idea of where that is.
We are actually in Region Zealand, which includes both West Zealand and Storstrom."

comment:14 Changed 6 years ago by ed

  • Owner changed from ed to jim

so we can add to the table, yes? please see french response:

France folks response: "I corrected a few ones and added a lot more. The list must be about complete now."

"Bas-Rhin - Alsace
Haut-Rhin - Alsace
Pyrenees-Atlantiques - Aquitaine
Charente - Poitou-Charentes
Charente-Maritime - Poitou-Charentes
Deux-Sevres - Poitou-Charentes
Vienne - Poitou-Charente
Allier - Auvergne
Cantal - Auvergne
Haute-Loire - Auvergne
Puy-de-Dome - Auvergne
Cote-d'Or - Bourgogne
Nievre - Bourgogne
Saone-et-Loire - Bourgogne
Yonne - Bourgogne
Cotes-d'Armor - Bretagne
Finistere - Bretagne
Ille-et-Vilaine - Bretagne
Morbihan - Bretagne
Cher - Centre
Eure-et-Loir - Centre
Indre - Centre
Indre-et-Loire - Centre
Loir-et-Cher - Centre
Loiret - Centre
Ardennes - Champagne-Ardenne
Aube - Champagne-Ardenne
Marne - Champagne-Ardenne
Haute-Marne - Champagne-Ardenne
Corse-du-Sud - Corse
Haute-Corse - Corse
Doubs - Franche-Comte
Jura - Franche-Comte
Haute-Saone - Franche-Comte
Territoire de Belfort - Franche-Comte
Paris - Ile-de-France
Seine-et-Marne - Ile-de-France
Yvelines - Ile-de-France
Essonne - Ile-de-France
Hauts-de-Seine - Ile-de-France
Seine-Saint-Denis - Ile-de-France
Val-de-Marne - Ile-de-France
Val-d'Oise - Ile-de-France
Aude - Languedoc-Roussillon
Gard - Languedoc-Roussillon
Hérault - Languedoc-Roussillon
Lozère -Languedoc Roussillon
Pyrenees Orientales - Languedoc-Roussillon
Corrèze - Limousin
Creuse - Limousin
Haute-Vienne - Limousin
Meurthe-et-Moselle - Lorraine
Meuse - Lorraine
Moselle - Lorraine
Vosges - Lorraine
Ariege -Midi-Pyrenees
Aveyron -Midi-Pyrenees
Gers - Midi-Pyrenees
Haute-Garonne -Midi-Pyrenees
Lot - Midi-Pyrenees
Lot-et-Garonne-Midi-Pyrenees
Tarn - Midi-Pyrenees
Tarn-et-Garonne - Midi-Pyrennees
Calvados - Basse Normandie
Manche - Basse Normandie
Orne - Basse Normandie
Eure - Haute Normandie
Seine-Maritime - Haute Normandie
Nord - Nord-Pas-de-Calais
Pas-de-Calais - Nord-Pas-de-Calis
Loire-Atlantique - Pays de Loire
Maine-et-Loire - Pays de Loire
Vendée - Pays de Loire
Mayenne - Pays de Loire
Sarthe - Pays de Loire
Aisne - Picardie
Oise - Picardie
Somme - Picardie
Alpes-de-Haute-Provence - Provence-Alpes-Cote d'Azur
Hautes-Alpes - Provence-Alpes-Cote d'Azur
Alpes-Maritimes - Provence-Alpes-Cote d'Azur
Bouches-du-Rhone - Provence-Alpes-Cote d'Azur
Var - Provence-Alpes-Cote d'Azur
Vaucluse - Provence-Alpes-Cote d'Azur
Ain - Rhone-Alpes
Ardèche - Rhone-Alpes
Drome - Rhone-Alpes
Isere - Rhone-Alpes
Loire - Rhone-Alpes
Rhone - Rhone-Alpes
Savoie - Rhone-Alpes
Haure-Savoie - Rhone-Alpes
Guadeloupe - DOM
Martinique - DOM
Guyane - DOM
La Reunion - DOM
Mayotte - DOM
Polynesie francaise - TOM
Nouvelle-Caledonie - TOM
Saint-Pierre-et-Miquelon - TOM

comment:15 Changed 6 years ago by jim

  • Status changed from assigned to closed
  • Resolution set to worksforme

Mais cette liste n'est pas correct! Regardez ici: http://en.wikipedia.org/wiki/Regions_of_France

I STRONGLY recommend against basically forking our version of the Location module just because certain users are unable to accept the official administrative regions of their countries.

Ou, en autre termes, c'est pas un bon idee mon frere.

Brutally updating status...

comment:17 Changed 6 years ago by ed

  • Status changed from closed to reopened
  • Resolution worksforme deleted

OK bringing it back in, the drupal list of supported regions does not match the wiki link saying there are 26... this needs discussion not closing.

comment:18 Changed 6 years ago by ed

Agreed - Jim to add patch as per node:
http://drupal.org/node/635958#comment-3078802

Ed to blog status and engage the french

comment:19 Changed 6 years ago by jim

  • Add Hours to Ticket changed from 0.0 to 0.1
  • Total Hours changed from 0.0 to 0.1

Patch applied and in SVN pending push to DEV then LIVE.

comment:20 Changed 6 years ago by jim

  • Owner changed from jim to ed
  • Status changed from reopened to assigned

LIVE now patched - over to Ed.

comment:21 Changed 6 years ago by ed

  • Status changed from assigned to closed
  • Resolution set to fixed
Note: See TracTickets for help on using tickets.