Ticket #1591 (closed developer task: fixed)

Opened 4 months ago

Last modified 4 months ago

Replace usage of cities, regions and countries with geonames_cache

Reported by: planetcruiser Owned by: planetcruiser
Priority: blocker Milestone: 0.5.7 - bugfixing
Component: BW Geo Keywords:
Cc:

Description (last modified by planetcruiser) (diff)

Issue:

  • Currently we have two ways of retrieving city, region and country information for members (city/region/country tables and geonames_* tables)
  • This should be reduced to one way in order to have less source for error

Solution:

  • Only use geonames_cache table to lookup location data
  • Remove cities, regions and countries tables

Notes:

  • Currently the cities table is corrupt/truncated on the live database, queries to it are unreliable - this increases urgency!

Change History

comment:1 Changed 4 months ago by planetcruiser

  • Owner set to planetcruiser
  • Status changed from new to accepted

will start with this now

comment:2 Changed 4 months ago by planetcruiser

last profile visitors on personal start page fixed via  https://gitorious.org/bewelcome/rox/commit/437884034d9f7a2f92527d03659e9e57c852167f

comment:3 Changed 4 months ago by planetcruiser

/places overview counters fixed via  https://gitorious.org/bewelcome/rox/commit/ffb1ae38e47b415e6b8600b7632b2abc7d809535

comment:4 Changed 4 months ago by planetcruiser

member counters for regions of a country fixed via  https://gitorious.org/bewelcome/rox/commit/cef3cf8e2244ba364b260ea7468663ba1221471e

comment:9 Changed 4 months ago by planetcruiser

  • Description modified (diff)
  • Summary changed from Replace usage of cities and countries with geonames_cache to Replace usage of cities, regions and countries with geonames_cache

comment:10 follow-up: ↓ 12 Changed 4 months ago by globetrotter_tt

There are still problems with some country codes on alpha. See  http://alpha.bewelcome.org/places/AX or  http://alpha.bewelcome.org/places/GU for example

Also "Kosovo" is not shown in the list, but two members:

have already signed up there long time ago, but were never set to active because of this.

comment:11 Changed 4 months ago by planetcruiser

replacing all remaining usage of country/region/city tables (not considering htdocs/bw/) via:

comment:12 in reply to: ↑ 10 ; follow-up: ↓ 14 Changed 4 months ago by planetcruiser

Replying to globetrotter_tt:

There are still problems with some country codes on alpha. See  http://alpha.bewelcome.org/places/AX or  http://alpha.bewelcome.org/places/GU for example

Also "Kosovo" is not shown in the list, but two members:

have already signed up there long time ago, but were never set to active because of this.

could you please create a new ticket for this?

comment:13 follow-up: ↓ 15 Changed 4 months ago by planetcruiser

  • follow_up changed from none to test on alpha
  • Status changed from accepted to closed
  • Resolution set to fixed

phew, this was a big one. i will compile a list of effected features later.

please test ;)

comment:14 in reply to: ↑ 12 Changed 4 months ago by globetrotter_tt

Replying to planetcruiser:

Replying to globetrotter_tt:

There are still problems with some country codes on alpha. See  http://alpha.bewelcome.org/places/AX or  http://alpha.bewelcome.org/places/GU for example

Also "Kosovo" is not shown in the list, but two members:

have already signed up there long time ago, but were never set to active because of this.

could you please create a new ticket for this?

new ticket: #1601

comment:15 in reply to: ↑ 13 Changed 4 months ago by planetcruiser

Replying to planetcruiser:

i will compile a list of effected features later.

see http://trac.bewelcome.org/wiki/Changelog_0.5.7

Note: See TracTickets for help on using tickets.