Merge pull request #1836 from lonvia/rework-large-location-II

Change processing of place nodes in addresses

docs: minor typo and grammar fixes

Merge branch 'split-off-test-tool-installation-instructions' of https://github.com/mtmail/Nominatim into mtmail-split-off-test-tool-installation-instructions

Merge pull request #1841 from mtmail/faq-entry-about-rebuilding-nominatim-so

FAQ addition when to rebuild nominatim.so

Merge pull request #1834 from mtmail/faq-invalid-page-in-block

FAQ entry for PostgreSQL -invalid page in block-

FAQ addition when to rebuild nominatim.so

Put install instructions of test tools into separate docs/ markdown file

FAQ entry for PostgreSQL -invalid page in block-

Merge pull request #1838 from lonvia/make-serve

add 'make serve' command

add 'make serve' command

Starts up PHP's built-in webserver in the website/ directory.
Useful for testing and development.

See #1831.

adapt tests to new search ranks

change place node expansion for large area table

So far we've used a buffer around a place node to define its
potential address reach. This had two problems: the buffer was
so large that addresses often contain false positives and the
buffer is really distorted when getting closer to the poles.

Change the buffer here to draw a bounndig box at a certain
distance in meter. This means that we always use the same
box everywhere on the planet and can make the extent much
smaller. Using a box has the advantage that it is much faster
to figure out if a point is within the box.

Merge pull request #1830 from lonvia/docs-for-nominatim-ui

Add usage docs for nominatim-ui

add usage docs for nominatim-ui

Includes migration guides for Apache and nginx.

Merge pull request #1827 from mtmail/centos8-postgresql12-without-proj

Vagrant centos8: proj52 not needed, use postgresql 12/postgis 3.0

remove version warning from ubuntu 20 installation

cmake: only require php for import and api

Vagrant centos8: proj52 not needed, use postgresql 12/postgis 3.0

prepare 3.5.0 release

Merge pull request #1822 from lonvia/document-address-labels

document which labels may appear in the address info

document which labels may appear in the address info

The list is manually generated and only valid for the default
configuration as used on openstreetmap.org.

Fixes #1808.

parents for large POIs must be address features

There are a couple of places with a search rank < 25 which are
not addressable like waterways and islands. We don't want them
to function as parents for POI-level objects. So use the
address rank for finding parents, not the search rank.

See #1815.

Merge pull request #1813 from lonvia/revert-concurrent-indexing

revert building indexes concurrently

Merge pull request #1814 from lonvia/disable-jit

Disable JIT and parallel workers when indexing

remove warnings about postgres 12 and postgis 3

Disable JIT and parallel workers when indexing

Locally disable jit and parallel workers in the connection that
do indexing. The query planner tends to be overenthusiatic about
using JIT. But with the rather less complex queries we have, the
overhead tends to be larger than the performance gain.

Fixes #1677.

revert building indexes concurrently

This does not really solve the issue with blocking autovacuum
requests and can lead to incomplete indexes and bogus
out-of-disk messages.

Fixes #1549.

Merge pull request #1809 from lonvia/fix-display-names

Fix some glitches in choice of address tags

Merge pull request #1804 from lonvia/ranking-improvement-germany

Localized ranking adaptions for Germany, Sweden and Norway

prefer linked place type over own place type

For state cities, tagging might prefer the place=state on
the admin boundary. The linked place is a more reliable indicator.

each address line must fill at most one geocodejson field

This fixes an issue where a postcode with rank_address 5
would also appear in the state field.

Fix script names in README (#1805)

adapt tests to new ranks

Adapt boundary labels for Sweden and Norway

This also gives us the correct labels for address output in
json and xml.

fix admin levels for Norway and Sweden

Admin levels 3 and 4 are used for region and county respectively,
so downgrade the ranking.

adapt municipality and region for Germany

Merge pull request #1801 from lonvia/rework-classtypes

Rework ClassTypes helper functions

fix style

adapt tests to modified address types

encapsulate icon URL in a function

also adapt uses of ClassTypes in website/

adapt php tests

Also fixes some errors found by the tests.

reorganise class/type information

Add a separate function for each property which saves necessary
information independently. Simplify computation of labels and
simple labels to not explicitly save the labels.

first draft

nominatim: always use deadlock-protected wait

Fixes #1785.

Merge pull request #1793 from lonvia/remove-struct-params-in-gui

search UI: hide unused query parameters

Update travis to bionic=Ubuntu18 (#1800)

Merge pull request #1797 from mtmail/jquery-3-5-1

update jquery dependency 3.5.0 => 3.5.1

update jquery dependency 3.5.0 => 3.5.1

improve syntax highlighting for apache and nginx examples

switch to php-fpm 7.3

Also fixes indent.

Merge branch 'patch-1' of https://github.com/ganeshkrishnan1/Nominatim into ganeshkrishnan1-patch-1

search UI: hide unused query parameters

Only send query parameters relevant for the current query
type (simple/structured), hide the other input fields.

This is quite a bit of CSS state changing, so move the intial
setup of the input field states into Javascript.

show simple query field when no parameters are given

Fix PHP errors in structured HTML output

Correctly handle missing parameters.

Merge pull request #1792 from lonvia/remove-from-location-area

remove linked places also from the location_area_large tables

remove linked places also from the location_area_large tables

We don't want linked places to show up in addresses either,
so remove them from the address lookup table.

Add simple/structured query selector to HTML search page (#1722)

Merge pull request #1786 from lonvia/remove-ubuntu-1604

remove Ubuntu 16 installation instructions

Merge pull request #1782 from Simon-Will/1781-make-tests-work-with-phpunit-8

Make tests work with phpunit 8

remove Ubuntu 16 installation instructions

Also fixes up CentOS 8 links in documentation.

Use Ubuntu’s packaged composer, not the custom installation

Merge pull request #1780 from Simon-Will/1768-vagrant-installation-for-ubuntu-20

Add vagrant machine for Ubuntu 20.04

Integrate Ubuntu 20 instructions into documentation

Document minimum phpunit version

Install phpunit 8 on Ubuntu 18 with composer

Merge pull request #1784 from krahulreddy/patch-1

Removed redundant question

Removed redundant question

Use assertEqualsWithDelta for float comparisons

PHPUnit 7.3 introduced the functions assertEqualsWithDelta for comparing
floats with a delta. The old four-argument version of assertEquals is
deprecated in PHPUnit 8 and removed in PHPUnit 9.

This commit means that the tests will fail with PHPUnit < 7.3 because
assertEqualsWithDelta is not defined there.

Declare return type of testcase setUp method

PHPUnit 7 changed the signature of the TestCase methods to include the
return type.

Add vagrant machine for Ubuntu 20.04

The instructions in
[`VAGRANT.md`](https://github.com/osm-search/Nominatim/blob/42c80893cb1038e21c05c2d0d30e3a7a20f0fa26/VAGRANT.md)
still work as before. The names of the Vagrant machines are updated so
that Ubuntu 18.04 (previously called `ubuntu`) is now called `ubuntu18`
and Ubuntu 20.04 is now called `ubuntu20`.

The version changes from Ubuntu 18.04 to Ubuntu 20.04 are:

  - Python: 3.6 to 3.8
  - Postgres: 10 to 12
  - PHP: 7.2 to 7.4

In the `apt-get`, I changed `--force` to `--allow-downgrades --allow-remove-essential --allow-change-held-packages`, because the former is deprecated. Cf. the [manpage of apt-get](http://manpages.ubuntu.com/manpages/focal/man8/apt-get.8.html)

The php module `codesniffer` was previously installed via Composer, but it is available via the Ubuntu repository, so I installed it via `apt-get` now.

docs: add link to new status page

documenation for /status endpoint

properly escape class parameter

The class parameter was used as is, allowing for potential
SQL injection via the API.

Thanks to @bladeswords for finding this.

Merge pull request #1770 from lonvia/eyusupov-separate-compilation

Separate compilation

Merge pull request #1769 from lonvia/display-name-order

 Ensure that result object name is always first in display_name

cmake: reintroduce check script

fix docs typos

cmake: restructure splitting between modules

Make a clear distinction between parts used for the importer
and parts used for the API.

Merge branch 'separate-compilation' of https://github.com/eyusupov/Nominatim into eyusupov-separate-compilation

Merge pull request #1754 from mtmail/nominatim-db-tests-against-postgres

Nominatim::DB tests against separate postgresql database

lonvia PR feedback

simplify display_name computation

ensure that result object name is always first in display_name

The display name might be mixed up if the result object has a lower
rank_address than its address members.

Merge pull request #1752 from mtmail/new-oo-shell-class

new PHP Nominatim\Shell class to wrap shell escaping

Merge pull request #1758 from krahulreddy/advanced-installations

Advanced installations

Merge pull request #1764 from mtmail/docs-countrycodes-based-on-adminlevel-2

API docs: countrycode assignment happens using admin_level=2 tags

highway:construction should appear as 'road' in the address list

Fixes #1763.

API documentation: clarification countrycode assignment happens using admin_level=2 tags

nominatim.py: fix wrong use of assert

Fixes #1762.

Fix documentation links (#1760)

Update installation documentation link in VAGRANT.md, update.php

Adjustments made to documentation

Added Advanced Installations documentation

Added scripts for multiple country setup and updates

Merge pull request #1756 from lonvia/downgrade-waterways

downgrade waterways

downgrade waterways

A lot of streams in OSM are of minor importance, they certainly
should show up lower in the list of results than villages. Those
rivers/streams that are well known have a wikipedia page and get
a higher importance from that.

The disadvantage with downgrading is that the address gets even
more useless but that's something that needs to be solved outside
the rank search.

Merge pull request #1753 from mtmail/fix-travis-ci-badge

fix Travis-CI badge in README output

Nominatim::DB tests against separate postgresql database

fix Travis-CI badge in README output

fix logging of lookup calls

Log start was called but the actual writing was missing.