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.

new PHP Nominatim\Shell class to wrap shell escaping

Merge pull request #1751 from lonvia/respect-admin-hierarchy

Address ranks must not invert admin_level hierarchy

address ranks must not invert admin_level hierarchy

When inheriting an address rank from a linked place we
must be careful not to destroy the hierarchy established
through boundary admin_level. Therefore, before assigning
an address rank from a linked place, find the next higher
boundary in the admin_level hierarchy, look up its address
rank and then only use the address rank from the linked
place if it is higher.

Merge branch 'update-jquery-leaflet' of https://github.com/mtmail/Nominatim into mtmail-update-jquery-leaflet

Merge pull request #1749 from lonvia/ranking-during-updates

Reset search and address ranks on update

website dependencies: jQuery v2.1 => 3.5, leaflet 1.3 => 1.6

reset address and search ranks on update

With ranks being dynamically changed through linking of places,
it is important to reset the ranks on update, so that changes
of the rank due to changes in linking are correctly taken into
account.

factor out computation of address and search rank

make admin levels 3 and 7 distinct ones in addresses

There really is no need to conflate these two levels as they
are in use in various countries.

Also adds province as a distinct place.

Fixes #1736.

Merge pull request #1745 from lonvia/shuffle-sql-functions

Some more SQL function reorganisation

Merge branch 'deletable-and-polygons-as-json' of https://github.com/mtmail/Nominatim into mtmail-deletable-and-polygons-as-json

Add JSON output for /deletable.php and /polygons.php

move ranks-related functions in separate sql file

Also adds a common function for computing the update radius
around place nodes.

move helper functions from placex_triggers into utils

Also adds documentation for these functions.

Merge pull request #1742 from mtmail/travis-ci-add-os

Travis-Ci configuration: remove -sudo-, add -os-

travis-ci configuration: remove -sudo-, add -os-

Merge pull request #1741 from filimongeorge/patch-1

Updated Import and Update .md file

Merge pull request #1740 from mtmail/setupclass-index-outputfile-not-used

SetupClass.php: remove unused variable

Merge pull request #1739 from lonvia/remove-self-from-geojson

Further tweaks to geocodejson output

Updated Import and Update .md file

SetupClass.php: remove unused variable

remove name from geocodejson when not set

adapt tests to geocodejson format adaptions

further tweaks to geocodejson address output

Removes the place itself from the address details and use
the lowest ranking element in the rank range for the output.

ignore isaddress in details output when it is not present

Merge pull request #1737 from mtmail/expose-isaddress-in-details-json

details JSON: also print isaddress addressline field

details JSON: also print isaddress addressline field

Merge pull request #1734 from krahulreddy/fixed-parselatlon

Added whitespace support for parseLatLon

avoid deletes on search_name in reverse-only mode

Added test cases for whitespaces in LatLon

Added whitespace characters support in LatLon parsing

fix phpcs issues

Merge pull request #1733 from krahulreddy/whitespaces-considered-as-single-space

Support whitespace characters(x09-x0d) as single space

Merge pull request #1732 from lonvia/improve-geocodejson-output

Improve geocodejson output

Support whitespace characters(x09-x0d) as single space

re-add district to geocodejson

update documentation for geocodejson

Address parts should be usable now.

add tests for geocodejson address fields

determine geocodejson address by rank instead of type

Using the address rank to set the address parts catches
a much wider variety of types like 'town' and 'suburb'.
With recent address ranking changes the rank ranges
are relatively reliable.

factor out geocodejson address generation

Unifies the two implementations currently used for search and address.

Merge pull request #1731 from lonvia/remove-polygon-from-docs

docs: remove example with polygon parameter

docs: remove example with polygon parameter

This parameter was undocumented, long deprecated and is gone now.

fix typo

Fixes #1730

fix handling of postcode areas in addresses

The order of preference is now:
1. a post code on the place itself
2. a post code area in the address
3. the computed postcode from the place

Fixes #1723.

adapt tests to new borough ranking

downgrade borough and remove unincorporated area

Merge pull request #1729 from lonvia/fix-details-link-for-boundaries

Fix details link for boundaries

remove polygon=1 (polypoints) feature

also emit place_type in json version of details

rather use new place_type in getAddressNames()

If for a boundary the place_type is defined, handle the address
part like a place node. This is the same behaviour as before
when class/type where patched earlier.

Fix address link for boundaries in details

Removes the special casing for boundaries with a place
type in get_addressdata(). Instead the place type is now
returned as an extra field, so that the caller has to
handle the situation.

This fixes the details link next to the address in the details
view, which previously would go to a place class instead of the
original boundary class.

Merge pull request #1720 from lonvia/better-linking-of-places

Use wikidata tags for improving linking of places with boundaries

Updated TIGER database link in documentation (#1725)

Updated TIGER database link in documentation

adapt API tests to place linkage changes

The missing district is due to a data error for wikidata tags.

use wikidata tag for linking places

Having the same wikidata is a strong indicator that the same place
is meant. There are some assignment errors where the wikidata does
not link to the object itself but to something that mentions the
place. To reduce errors there, prefer same name.

remove linking via admin_centre role

The admin_centre role is for the seat of government which is not
the same as the administrative entity. This is mostly used
correctly these days, so avoid matching by that role.

linking: better name matching for address-less places

Administrative boundaries that do not figure in the address
should still be able to take part in the name matching.
Use the rank_search for comparison in this case.