]> git.openstreetmap.org Git - nominatim.git/log
nominatim.git
5 years agofix indent in docs
Sarah Hoffmann [Mon, 29 Apr 2019 21:13:37 +0000 (23:13 +0200)]
fix indent in docs

5 years agoMerge branch 'markdown-syntax-fix-gb-postcodes' of https://github.com/mtmail/Nominati...
Sarah Hoffmann [Mon, 29 Apr 2019 21:12:37 +0000 (23:12 +0200)]
Merge branch 'markdown-syntax-fix-gb-postcodes' of https://github.com/mtmail/Nominatim into mtmail-markdown-syntax-fix-gb-postcodes

5 years agoReadme for GB postcodes had markdown syntax error
marc tobias [Sun, 28 Apr 2019 21:18:36 +0000 (23:18 +0200)]
Readme for GB postcodes had markdown syntax error

5 years agoIgnore no-fatal errors during dump file restore
Sarah Hoffmann [Sun, 28 Apr 2019 20:44:42 +0000 (22:44 +0200)]
Ignore no-fatal errors during dump file restore

The owner should never be restored, the table should be owned
by the caller instead. Non-existing indexes and similar only
started to throw a warning with Postgresql 9.4 and later, so
ignore them explicitly there.

5 years agoMerge pull request #1321 from mtmail/interpolating-0-housenumbers
Sarah Hoffmann [Fri, 19 Apr 2019 16:29:43 +0000 (18:29 +0200)]
Merge pull request #1321 from mtmail/interpolating-0-housenumbers

Support housenumber=0 in interpolations

5 years agoMerge pull request #1359 from mtmail/fix-export-script
Sarah Hoffmann [Wed, 17 Apr 2019 21:04:51 +0000 (23:04 +0200)]
Merge pull request #1359 from mtmail/fix-export-script

utils/export.php broke after switch to PDO DB abstraction

5 years agoutils/export.php broke after switch to PDO DB abstraction
marc tobias [Wed, 17 Apr 2019 16:51:22 +0000 (18:51 +0200)]
utils/export.php broke after switch to PDO DB abstraction

5 years agoMerge pull request #1358 from mtmail/travis-php-7dot1
Sarah Hoffmann [Wed, 17 Apr 2019 20:05:28 +0000 (22:05 +0200)]
Merge pull request #1358 from mtmail/travis-php-7dot1

travis-CI: use PHP 7.1

5 years agotravis-CI: use PHP 7.1
marc tobias [Wed, 17 Apr 2019 12:03:35 +0000 (14:03 +0200)]
travis-CI: use PHP 7.1

5 years agoMerge remote-tracking branch 'upstream/master'
Sarah Hoffmann [Tue, 16 Apr 2019 21:15:17 +0000 (23:15 +0200)]
Merge remote-tracking branch 'upstream/master'

5 years agoexclude all objects without address rank from reverse
Sarah Hoffmann [Tue, 16 Apr 2019 21:13:27 +0000 (23:13 +0200)]
exclude all objects without address rank from reverse

This was forgotten when looking for a housenumber for
a street point.

Fixes #1319.

5 years agoadd reverse zoom level that includes minor streets
Sarah Hoffmann [Mon, 15 Apr 2019 20:43:07 +0000 (22:43 +0200)]
add reverse zoom level that includes minor streets

Zoom 17 now also resolves service roads and similar.

Fixes #1350.

5 years agoMerge remote-tracking branch 'upstream/master'
Sarah Hoffmann [Sun, 14 Apr 2019 10:07:06 +0000 (12:07 +0200)]
Merge remote-tracking branch 'upstream/master'

5 years agoisaddress field may be missing in details view
Sarah Hoffmann [Sun, 14 Apr 2019 10:03:37 +0000 (12:03 +0200)]
isaddress field may be missing in details view

5 years agoMerge branch 'observe-bounded-viewbox-in-postcode-search' of https://github.com/mtmai...
Sarah Hoffmann [Sun, 14 Apr 2019 09:29:28 +0000 (11:29 +0200)]
Merge branch 'observe-bounded-viewbox-in-postcode-search' of https://github.com/mtmail/Nominatim into mtmail-observe-bounded-viewbox-in-postcode-search

5 years agoMerge pull request #1336 from mtmail/faq-entry-about-buffer-not-owned
Sarah Hoffmann [Sun, 14 Apr 2019 09:27:36 +0000 (11:27 +0200)]
Merge pull request #1336 from mtmail/faq-entry-about-buffer-not-owned

New FAQ entry about -buffer is not owned by resource owner-

5 years agoMerge pull request #1348 from mtmail/checkmodulepresence-to-raise-exception
Sarah Hoffmann [Sun, 14 Apr 2019 09:25:52 +0000 (11:25 +0200)]
Merge pull request #1348 from mtmail/checkmodulepresence-to-raise-exception

checkModulePresence now raises exception instead of its callers

5 years agoMerge pull request #1347 from mtmail/pdo-returns-proper-boolean
Sarah Hoffmann [Sun, 14 Apr 2019 09:24:23 +0000 (11:24 +0200)]
Merge pull request #1347 from mtmail/pdo-returns-proper-boolean

PDO library returns proper boolean. We dont need string comparison

5 years agoEnsure that postcode relations are used in addresses
Sarah Hoffmann [Sun, 14 Apr 2019 09:20:03 +0000 (11:20 +0200)]
Ensure that postcode relations are used in addresses

Postcode nodes are normally thrown away as they only play
a role for computing artifical postcodes. However, if we
have a postcode area this still should take part of the
address.

Fixes #1330.

5 years agocheckModulePresence now raises exception instead of its callers
marc tobias [Tue, 2 Apr 2019 16:37:11 +0000 (18:37 +0200)]
checkModulePresence now raises exception instead of its callers

5 years agoPDO library returns proper boolean. We dont need string comparison
marc tobias [Tue, 2 Apr 2019 14:52:37 +0000 (16:52 +0200)]
PDO library returns proper boolean. We dont need string comparison

5 years agoSupport housenumber=0 in interpolations
marc tobias [Sun, 3 Mar 2019 19:44:22 +0000 (20:44 +0100)]
Support housenumber=0 in interpolations

5 years agoOn postcode searches observe given bounded viewbox
marc tobias [Tue, 2 Apr 2019 12:49:31 +0000 (14:49 +0200)]
On postcode searches observe given bounded viewbox

5 years agoadd test for issue #1343
Sarah Hoffmann [Tue, 26 Mar 2019 20:49:44 +0000 (21:49 +0100)]
add test for issue #1343

Keyword details for countries (which don't have address details).

5 years agoif nameaddress_vector was {} the database queries failed
marc tobias [Tue, 26 Mar 2019 16:59:27 +0000 (17:59 +0100)]
if nameaddress_vector was {} the database queries failed

5 years agoNew FAQ entry about -buffer is not owned by resource owner-
marc tobias [Tue, 19 Mar 2019 00:52:35 +0000 (01:52 +0100)]
New FAQ entry about -buffer is not owned by resource owner-

5 years agoMerge pull request #1334 from mtmail/fix-PDOException-call
Sarah Hoffmann [Mon, 18 Mar 2019 20:26:23 +0000 (21:26 +0100)]
Merge pull request #1334 from mtmail/fix-PDOException-call

PDOException call in catch was causing exception itself

5 years agoPDOException call in catch was causing exception itself
marc tobias [Sun, 17 Mar 2019 01:47:28 +0000 (02:47 +0100)]
PDOException call in catch was causing exception itself

5 years agoMerge pull request #1333 from Arun179/patch-1
Sarah Hoffmann [Thu, 14 Mar 2019 18:06:07 +0000 (19:06 +0100)]
Merge pull request #1333 from Arun179/patch-1

Rectified a small spelling mistake

5 years agoRectified a small spelling mistake
Arun Kumar [Thu, 14 Mar 2019 17:23:09 +0000 (22:53 +0530)]
Rectified a small spelling mistake

changed "mailinglist" to "mailing list"

5 years agoMerge pull request #1328 from mtmail/php-pdo-with-prepare
Sarah Hoffmann [Wed, 13 Mar 2019 10:10:17 +0000 (11:10 +0100)]
Merge pull request #1328 from mtmail/php-pdo-with-prepare

Nominatim::DB support input variables, custom error messages

5 years agoNominatim::DB support input variables, custom error messages
marc tobias [Sun, 10 Mar 2019 14:42:58 +0000 (15:42 +0100)]
Nominatim::DB support input variables, custom error messages

5 years agoremove outdated last_update script
Sarah Hoffmann [Sat, 9 Mar 2019 20:41:46 +0000 (21:41 +0100)]
remove outdated last_update script

No longer used by server monitoring.

5 years agoMerge remote-tracking branch 'upstream/master'
Sarah Hoffmann [Sat, 9 Mar 2019 10:51:24 +0000 (11:51 +0100)]
Merge remote-tracking branch 'upstream/master'

5 years agoMerge pull request #1318 from mtmail/php-pdo
Sarah Hoffmann [Sat, 9 Mar 2019 10:27:51 +0000 (11:27 +0100)]
Merge pull request #1318 from mtmail/php-pdo

replace database abstraction DB with PDO

5 years agoreplace database abstraction DB with PDO
marc tobias [Sun, 24 Feb 2019 15:14:36 +0000 (16:14 +0100)]
replace database abstraction DB with PDO

5 years agoadd logEnd to reverse.php, just like search.php
marc tobias [Tue, 26 Feb 2019 14:03:40 +0000 (15:03 +0100)]
add logEnd to reverse.php, just like search.php

5 years agoMerge remote-tracking branch 'upstream/master'
Sarah Hoffmann [Sun, 24 Feb 2019 16:51:32 +0000 (17:51 +0100)]
Merge remote-tracking branch 'upstream/master'

5 years agoMerge pull request #1315 from mtmail/ui-initialize-switch-to-reverse-link
Sarah Hoffmann [Sun, 24 Feb 2019 16:51:09 +0000 (17:51 +0100)]
Merge pull request #1315 from mtmail/ui-initialize-switch-to-reverse-link

UI: update the switch-to-reverse link after each map click

5 years agoMerge pull request #1314 from mtmail/query-php-5dot4
Sarah Hoffmann [Sun, 24 Feb 2019 16:50:09 +0000 (17:50 +0100)]
Merge pull request #1314 from mtmail/query-php-5dot4

query.php - we no longer support PHP < 5.4

5 years agoUI: update the switch-to-reverse link after each map click
marc tobias [Sun, 24 Feb 2019 15:34:21 +0000 (16:34 +0100)]
UI: update the switch-to-reverse link after each map click

5 years agoquery.php - we no longer support PHP < 5.4
marc tobias [Sun, 24 Feb 2019 15:22:55 +0000 (16:22 +0100)]
query.php - we no longer support PHP < 5.4

5 years agoMerge remote-tracking branch 'upstream/master'
Sarah Hoffmann [Sun, 24 Feb 2019 13:40:17 +0000 (14:40 +0100)]
Merge remote-tracking branch 'upstream/master'

5 years agodowngrade housenumbers without numbers
Sarah Hoffmann [Sun, 24 Feb 2019 13:39:14 +0000 (14:39 +0100)]
downgrade housenumbers without numbers

Fixes #1312.

5 years agoMerge remote-tracking branch 'upstream/master'
Sarah Hoffmann [Sun, 24 Feb 2019 10:29:26 +0000 (11:29 +0100)]
Merge remote-tracking branch 'upstream/master'

5 years agoChange accepted features for reverse geocoding at rank 30
Sarah Hoffmann [Sun, 24 Feb 2019 10:00:33 +0000 (11:00 +0100)]
Change accepted features for reverse geocoding at rank 30

Always exclude line features (removes railways, tunnels,
piers, historical streets etc.) and boundaries (removes
electoral, historical boundaries etc.)

Fixes #1313.

5 years agoreuse DB connections by default
Sarah Hoffmann [Thu, 21 Feb 2019 22:40:40 +0000 (23:40 +0100)]
reuse DB connections by default

5 years agoMerge remote-tracking branch 'upstream/master'
Sarah Hoffmann [Thu, 21 Feb 2019 22:39:48 +0000 (23:39 +0100)]
Merge remote-tracking branch 'upstream/master'

5 years agoadd osm_id index for osmline table
Sarah Hoffmann [Thu, 21 Feb 2019 22:26:31 +0000 (23:26 +0100)]
add osm_id index for osmline table

Needed when deleting address interpolation.

5 years agoadd hint that setup.php must be run from build directory
Sarah Hoffmann [Wed, 13 Feb 2019 20:58:59 +0000 (21:58 +0100)]
add hint that setup.php must be run from build directory

Fixes #1307.

5 years agoMerge pull request #1295 from mtmail/move-searchrank-labels-to-php
Sarah Hoffmann [Sun, 10 Feb 2019 16:22:49 +0000 (17:22 +0100)]
Merge pull request #1295 from mtmail/move-searchrank-labels-to-php

Remove get_addressrank_label. Move get_searchrank_label to PHP

5 years agoMerge pull request #1294 from mtmail/behave-support-for-DB-PORT
Sarah Hoffmann [Sun, 10 Feb 2019 16:20:57 +0000 (17:20 +0100)]
Merge pull request #1294 from mtmail/behave-support-for-DB-PORT

BDD: support for DB_PORT environment variable

5 years agoMerge pull request #1305 from mtmail/travis-ci-ubuntu-16
Sarah Hoffmann [Sun, 10 Feb 2019 16:20:06 +0000 (17:20 +0100)]
Merge pull request #1305 from mtmail/travis-ci-ubuntu-16

Travis CI Ubuntu 14 => Ubuntu 16

5 years agoBDD: support for DB_PORT environment variable
marc tobias [Mon, 28 Jan 2019 18:33:22 +0000 (19:33 +0100)]
BDD: support for DB_PORT environment variable

5 years agoTravis CI Ubuntu 14 => Ubuntu 16
marc tobias [Sat, 9 Feb 2019 19:22:34 +0000 (20:22 +0100)]
Travis CI Ubuntu 14 => Ubuntu 16

5 years agoRemove get_addressrank_label. Move get_searchrank_label to PHP
marc tobias [Mon, 28 Jan 2019 18:38:05 +0000 (19:38 +0100)]
Remove get_addressrank_label. Move get_searchrank_label to PHP

5 years agoMerge pull request #1282 from lonvia/remove-self-from-place-addressline
Sarah Hoffmann [Sat, 9 Feb 2019 15:00:46 +0000 (16:00 +0100)]
Merge pull request #1282 from lonvia/remove-self-from-place-addressline

Remove self from place addressline

5 years agodocs: adapt sizes to smaller place_address table
Sarah Hoffmann [Tue, 22 Jan 2019 21:42:18 +0000 (22:42 +0100)]
docs: adapt sizes to smaller place_address table

5 years agoremove self-reference from place-addressline
Sarah Hoffmann [Sun, 13 Jan 2019 20:07:01 +0000 (21:07 +0100)]
remove self-reference from place-addressline

5 years agoMerge pull request #1304 from mtmail/travis-use-composer-instead-of-pear-install
Sarah Hoffmann [Sat, 9 Feb 2019 14:24:04 +0000 (15:24 +0100)]
Merge pull request #1304 from mtmail/travis-use-composer-instead-of-pear-install

pear.php.net offline. Use composer instead of pear install

5 years agoMerge pull request #1303 from mtmail/remove-deprecated-phpunit-config-key
Sarah Hoffmann [Sat, 9 Feb 2019 08:54:34 +0000 (09:54 +0100)]
Merge pull request #1303 from mtmail/remove-deprecated-phpunit-config-key

remove phpunit config key deprecated since version 3.5

5 years agopear.php.net offline. Use composer instead of pear install
marc tobias [Sat, 9 Feb 2019 00:08:47 +0000 (01:08 +0100)]
pear.php.net offline. Use composer instead of pear install

5 years agoremove phpunit config key deprecated since version 3.5
marc tobias [Fri, 8 Feb 2019 23:34:04 +0000 (00:34 +0100)]
remove phpunit config key deprecated since version 3.5

5 years agoMerge pull request #1296 from mtmail/readme-markdown-syntax-error
Sarah Hoffmann [Mon, 28 Jan 2019 20:19:46 +0000 (21:19 +0100)]
Merge pull request #1296 from mtmail/readme-markdown-syntax-error

README: tiny markdown syntax error

5 years agoREADME: tiny markdown syntax error
marc tobias [Mon, 28 Jan 2019 18:42:40 +0000 (19:42 +0100)]
README: tiny markdown syntax error

5 years agoDo not allow --no-index together with --import-osmosis-all
Sarah Hoffmann [Sat, 26 Jan 2019 14:25:00 +0000 (15:25 +0100)]
Do not allow --no-index together with --import-osmosis-all

Fixes #1283.

5 years agoset import_status when indexing only
Sarah Hoffmann [Sat, 26 Jan 2019 14:01:39 +0000 (15:01 +0100)]
set import_status when indexing only

Makes sure that things work as expected when running
`--import-osmosis --no-index` and `--index` separately.

Fixes #1284.

5 years agoMerge remote-tracking branch 'upstream/master'
Sarah Hoffmann [Sat, 26 Jan 2019 13:04:35 +0000 (14:04 +0100)]
Merge remote-tracking branch 'upstream/master'

5 years agoignore admin boundary ways for countries and states
Sarah Hoffmann [Sat, 26 Jan 2019 12:34:28 +0000 (13:34 +0100)]
ignore admin boundary ways for countries and states

Countries and states are mapped world-wide as relations by now.

Fixes #543 and #1291.

5 years agoMerge remote-tracking branch 'upstream/master'
Sarah Hoffmann [Thu, 10 Jan 2019 19:37:10 +0000 (20:37 +0100)]
Merge remote-tracking branch 'upstream/master'

5 years agoMerge pull request #1277 from lonvia/osm2pgsql-import-from-json
Sarah Hoffmann [Thu, 10 Jan 2019 19:36:20 +0000 (20:36 +0100)]
Merge pull request #1277 from lonvia/osm2pgsql-import-from-json

Osm2pgsql import from json style file

5 years agopostcodes also need fallback
Sarah Hoffmann [Tue, 8 Jan 2019 22:46:18 +0000 (23:46 +0100)]
postcodes also need fallback

5 years agoadd final missing import numbers
Sarah Hoffmann [Tue, 8 Jan 2019 21:20:42 +0000 (22:20 +0100)]
add final missing import numbers

5 years agoupdate osm2pgsql (custom style)
Sarah Hoffmann [Sun, 6 Jan 2019 18:20:32 +0000 (19:20 +0100)]
update osm2pgsql (custom style)

5 years agomore style docs
Sarah Hoffmann [Thu, 3 Jan 2019 20:08:38 +0000 (21:08 +0100)]
more style docs

5 years agoadd tests for import of interpolations
Sarah Hoffmann [Thu, 3 Jan 2019 20:08:16 +0000 (21:08 +0100)]
add tests for import of interpolations

5 years agoadd postcodes and interpolations to osm2pgsql style
Sarah Hoffmann [Wed, 2 Jan 2019 13:50:55 +0000 (14:50 +0100)]
add postcodes and interpolations to osm2pgsql style

5 years agodocument import style variants
Sarah Hoffmann [Tue, 4 Dec 2018 21:22:19 +0000 (22:22 +0100)]
document import style variants

5 years agofix typo
Sarah Hoffmann [Sun, 2 Dec 2018 22:09:11 +0000 (23:09 +0100)]
fix typo

5 years agofixup admin import style and add two new ones
Sarah Hoffmann [Sun, 2 Dec 2018 22:05:45 +0000 (23:05 +0100)]
fixup admin import style and add two new ones

Remove unnamed landuses and postcode points from
importing. The latter will cause all objects with
address tags to be imported after all. Not expected
in the admin import style.

5 years agoadd documentation for new import style
Sarah Hoffmann [Sun, 2 Dec 2018 16:38:25 +0000 (17:38 +0100)]
add documentation for new import style

5 years agoSwitch to configurable style for osm2pgsql
Sarah Hoffmann [Mon, 26 Nov 2018 23:17:00 +0000 (00:17 +0100)]
Switch to configurable style for osm2pgsql

Includes the full style, which is the same as now (minus
sidwalk exclusion) and a minimal style for boundaries only.

5 years agoadapt warm.php to new Result objects
Sarah Hoffmann [Tue, 8 Jan 2019 21:45:21 +0000 (22:45 +0100)]
adapt warm.php to new Result objects

Fixes #1276.

5 years agokeep country_osm_grid table when dropping update tables
Sarah Hoffmann [Tue, 8 Jan 2019 21:44:33 +0000 (22:44 +0100)]
keep country_osm_grid table when dropping update tables

5 years agoemphasize the need for valid HTTP referrer/user agent even more
Sarah Hoffmann [Sat, 5 Jan 2019 21:20:38 +0000 (22:20 +0100)]
emphasize the need for valid HTTP referrer/user agent even more

5 years agoMerge remote-tracking branch 'upstream/master'
Sarah Hoffmann [Fri, 4 Jan 2019 22:29:03 +0000 (23:29 +0100)]
Merge remote-tracking branch 'upstream/master'

5 years agoRemove postcodes also from word table when they no longer exist
Sarah Hoffmann [Fri, 4 Jan 2019 22:11:47 +0000 (23:11 +0100)]
Remove postcodes also from word table when they no longer exist

Also adds tests for postcode updates.

Fixes #1273.

5 years agoDo not log file sizes in the index step on updates
Sarah Hoffmann [Fri, 4 Jan 2019 20:51:18 +0000 (21:51 +0100)]
Do not log file sizes in the index step on updates

Fixes #1274.

5 years agoMerge remote-tracking branch 'upstream/master'
Sarah Hoffmann [Thu, 3 Jan 2019 20:51:59 +0000 (21:51 +0100)]
Merge remote-tracking branch 'upstream/master'

5 years agocorrectly discard partially matching duplicates
Sarah Hoffmann [Thu, 3 Jan 2019 20:49:50 +0000 (21:49 +0100)]
correctly discard partially matching duplicates

The same result may be found with different result ranks
in the same search loop when housenumber or postcode are
part of the name or address. In this case we need to keep
the result with the lower result rank.

Fixes #1264.

6 years agoMerge pull request #1263 from mtmail/add-new-postcodes-to-searchnames
Sarah Hoffmann [Tue, 11 Dec 2018 20:39:28 +0000 (21:39 +0100)]
Merge pull request #1263 from mtmail/add-new-postcodes-to-searchnames

when updating GB postcodes, also run SQL getorcreate_postcode_id

6 years agodocumentation: when updating GB postcodes, also run SQL getorcreate_postcode_id ...
marc tobias [Tue, 11 Dec 2018 20:34:56 +0000 (21:34 +0100)]
documentation: when updating GB postcodes, also run SQL getorcreate_postcode_id [SKIP CI]

6 years agoMerge remote-tracking branch 'upstream/master'
Sarah Hoffmann [Thu, 6 Dec 2018 21:03:17 +0000 (22:03 +0100)]
Merge remote-tracking branch 'upstream/master'

6 years agotraverse address list backwards when computing admin levels
Sarah Hoffmann [Thu, 6 Dec 2018 20:18:47 +0000 (21:18 +0100)]
traverse address list backwards when computing admin levels

By starting with the lowest address level, when collecting
administrative boundaries by level, there is a better chance
to actually get the boundary when the capital of the
administrative boundary is tagged with the level of the
boundary as well.

This is just a heuristics adaption to make the best out of
the imprecise admin_level definition for place nodes.

Fixes #1261.

6 years agoMerge remote-tracking branch 'upstream/master'
Sarah Hoffmann [Wed, 5 Dec 2018 20:40:52 +0000 (21:40 +0100)]
Merge remote-tracking branch 'upstream/master'

6 years agoDon't escape slashes in json output
Sarah Hoffmann [Tue, 4 Dec 2018 21:28:29 +0000 (22:28 +0100)]
Don't escape slashes in json output

Fixes #1256.

6 years agoMerge pull request #1252 from mtmail/update-and-document-gb-postcode-data2
Sarah Hoffmann [Tue, 4 Dec 2018 21:20:55 +0000 (22:20 +0100)]
Merge pull request #1252 from mtmail/update-and-document-gb-postcode-data2

GB postcode: new conversion script, documentation

6 years agoMerge pull request #1255 from mtmail/faq-about-permanent-ids
Sarah Hoffmann [Tue, 4 Dec 2018 21:03:41 +0000 (22:03 +0100)]
Merge pull request #1255 from mtmail/faq-about-permanent-ids

new FAQ entry about place_id values

6 years agoMerge pull request #1258 from lonvia/cleanup-utils
Sarah Hoffmann [Tue, 4 Dec 2018 21:00:31 +0000 (22:00 +0100)]
Merge pull request #1258 from lonvia/cleanup-utils

Restructure script and website installation

6 years agoExplain place_id, i.e. shouldn't be use as permanent id
marc tobias [Wed, 28 Nov 2018 18:36:56 +0000 (19:36 +0100)]
Explain place_id, i.e. shouldn't be use as permanent id