Merge remote-tracking branch 'upstream/master'

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 remote-tracking branch 'upstream/master'

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 remote-tracking branch 'upstream/master'

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 remote-tracking branch 'upstream/master'

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

Merge remote-tracking branch 'upstream/master'

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

add lookup call to munin output

Merge remote-tracking branch 'upstream/master'

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 remote-tracking branch 'upstream/master'

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

Merge remote-tracking branch 'upstream/master'

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 remote-tracking branch 'upstream/master'

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