opencaching / opencaching-pl

The source code of Opencaching.PL (and some other domains)
https://opencaching.pl/
GNU General Public License v3.0
22 stars 33 forks source link

cache_locations missing/wrong contents #816

Open andrixnet opened 8 years ago

andrixnet commented 8 years ago

These fields should be filled out properly. It will also help with https://github.com/opencaching/opencaching-pl/issues/103

deg-pl commented 8 years ago

See my comment https://github.com/opencaching/opencaching-pl/pull/814#issuecomment-258228300

andrixnet commented 8 years ago

Let's follow the discussion here. I assume the problems might be related to :

  1. newcache
  2. editcache
  3. cache_locations script for cron

Once these are fixed, a query should be run on each node to make data consistent, since cache_locations script only acts on missing and changed caches.

Since the localized country names were moved to translation files, "countries" table should have had the corresponding columns removed. I am not sure how or if the other columns are used, since many related info is in the settings file already. (see the countries arrays). Still having and using strings from this table contradicts the whole translation infrastructure.

Since that location data is entirely based on NUTS data, I believe the country reference in the table should be the same as the NUTS data. Is the country (adm2) field really used to show cache country? I think not. Are the rest of the region strings also localized? Definitely not.

andrixnet commented 8 years ago

More so, the following only makes sense on OCPL node.

SELECT `countries`.`pl`