Merge branch 'PSR2-fixes-for-wikidata' of https://github.com/mtmail/Nominatim into mtmail-misc-psr-fixes

half-reverting 51155b63, We dont need two of three break statements after all

PSR2 fixes for /wikidata and /tests-php directories

PHP linter looks for phpcs.xml by default. Also removed a couple of exceptions that didnt cause warnings

fix indentation and misc errors according to PSR2 coding style guide

PHP linter looks for phpcs.xml by default. Also removed a couple of exceptions that didnt cause warnings

case statement need : instead of ;. Added more breaks

UI: viewbox parameter was set in wrong parameter order

Merge pull request #524 from mtmail/PSR2-function-declarations

fix function declaration errors according to PSR2 coding style guide

fix function declaration errors according to PSR2 coding style guide

code style definitions: we no longer use tabs

add PHP code-sniffer to installation instructions (marked optional)

Merge pull request #522 from mtmail/PSR2-arrays

fix array-related errors according to PSR2 coding style guide

make apt-get noninteractive. Also fixes the grub error state

grub complains about disc unique identifier having changed

fix array-related errors according to PSR2 coding style guide

bracket spacing for if/else/for/foreach/while/switch according to PSR2 standard

Merge branch 'delete-us-postcode-without-name' of https://github.com/mtmail/Nominatim

adapt coding style for new spaces-only policy

tabs-to-spaces

one US postcode has no name. Unable to find secondary source what it would have been

Merge pull request #498 from mtmail/move-mouse-position-info-on-map

HTML map: new button -show map bounds-

Merge pull request #508 from mtmail/addtype-in-installation.md

typo in Installation.md

HTML map: new button -show map bounds- in top right corner

Update Installation.md

trivial typo: AddTpe => AddType

Merge remote-tracking branch 'upstream/master' into move-mouse-position-info-on-map

Merge pull request #504 from lonvia/parameter-parser-class

factor out parameter parsing into its own class

Merge pull request #506 from mtmail/primary-vagrant-vm

specify ubuntu as primary Vagrant VM

specify ubuntu as primary Vagrant VM

factor out parameter parsing into its own class

Allows to use the parsing functions in website/search.php and
utils/query.php as well.

update tests for newer planet

docs: give recomendations about flatnode file size

closes #501

mark checkpoint_segments parameter as deprecated

Postgresql 9.5 has checkpoint_segments removed and fails to start
if the options is given. Adds a comment to that effect.

Fixes #494.

Merge pull request #499 from mtmail/travis-ci

Travis ci setup - tests pass

Merge remote-tracking branch 'upstream/master' into travis-ci

travis-ci: enable river tests again, numpy package was missing

travis-ci: enable river tests again, numpy package was missing

update paths in vagrant docs

Merge pull request #497 from lonvia/forbid-execution-from-sourcedir

fail on execution of util scripts from source directory

Merge pull request #476 from mtmail/travis-ci

travis-ci setup

HTML map: new button -show map bounds-

fail on execution of util scripts from source directory

Utils scripts must be run from the build directory to make sure
we get the right paths. Rename the settings file in source and
replace the original one with an error, so that scripts
fail with an understandable error message when run from the
source directory.

travis: exclude poldi, waterway tests

travis: exclude poldi, waterway tests

travis: exclude poldi, waterway tests

Merge remote-tracking branch 'upstream/master' into travis-ci

Merge pull request #490 from lonvia/streamline-class-interfaces

Code cleanup for geocode classes

remove unused functions in Geocode

cleanup interface of PlaceLookup

Move lookup parameters (place id, type, frac etc.) from members
to parameters for lookup().

remove unnecessary local variable

refactor ReverseGeocode

- remove unused functions and members
- move coordinates from being members to being parameters of lookup()

fix typos and grammar in installation instructions

Merge pull request #487 from lonvia/unify-reverse

Remove duplicate reverse geocoding code

remove duplicate reverse geocoding code

Geocode now uses ReverseGeocode directly. Note that it currently
cannot support any interpolations, neither OSM nor Tiger. But
neither did the previous version.

adapt Tiger reverse test to new output format

remove IP blocking and rate-limiting code

Closes #472.

Merge pull request #486 from IrlJidel/patch-1

pgsqlRunPartitionScript needs global $aCMDResult

pgsqlRunPartitionScript needs global $aCMDResult

function pgsqlRunPartitionScript  was missing global $aCMDResult

setup was failing with

...
PHP Notice:  Undefined variable: aCMDResult in /home/roles/nominatim/app/Nominatim/build/utils/setup.php on line 768
NOTICE:  type "nearplace" does not exist, skipping
..
ERROR:  relation "location_area_large_0" already exists
ERROR: pgsql returned with error code (3)
pgsql returned with error code (3)
Command exited with non-zero status 255

travis-ci setup, some tests fails

allow coordinates in query only with name or classtype

Coordinates with country or housenumber only do not make much
sense. Currently these combination result in queries that
require a sequential scan of search_name and normally time
out before yielding any results.

fixes #462

fix links to documentation in README

more instructions for Tiger imports

reverse do not downgrade tiger and interpolation results

Also fix the type that is returned for Tiger data.

Fixes #470

Merge pull request #469 from lonvia/refactor-php

Some refactoring of website php

add instructions to enable use of Tiger data

fix more syntax errors in error functions

allow parameters to be empty

Consider empty parameters as unset and use the default. Simplifies
use in forms.

fix syntax errors and update tests

Some of the tests with bad parameters now return a HTTP 400.

use chksql everywhere in lib/

Replaces manual PEAR error checks and adds the chksql wrapper
where the error checks were still missing.

introduce chksql function for website code and move other fail functions

The default error screen now reports the full SQL even when not
in debug mode as this message is often more helpful than our
custom error message.

factor out link formatting function and osm type translation

update links to OSM objects

/browse/ is no longer necesary in the URL

remove now unused bAsPoints in reverse API

introduce accessor function for URL parameter

These functions take care of type conversion and check that
the parameters contain legal values. The API now returns a
Bad Request error if the format is wrong.

polygons.php: print total number of broken polygons

remove dead code

Merge pull request #219 from mizabrik/master

Fix case sensitivity of cyrillic ghe

introduce chksql function for phpscripts

Checks SQL query results for errors and bails out if it finds one.
Avoids some heavy code duplication all over the place.

remove --create-website function

Symlinking does not go well with the separate build directory.
The recommended method is now to serve the website directly from
build/website via an alias. 'website installation' may be
reintroduced later via cmake.

Merge pull request #464 from lonvia/vagrant-with-docs

Move installation documentation from wiki into /docs directory

remove old vagrant scripts

fix formatting of docs

add compiled vagrant docs in docs/ directory

add vagrant script for ubuntu 16 and polish everything

fix cut&paste error in php tests

move installation instructions from wiki page here

fix apache permission problems in CentOS vagrant script

split instruction into software installation and import

Basically leaves the part that needs to be done by the automatic
vagrant script in the bash script and moves the remaining part
into an OS-independent md file.

Also fixes some of the instructions for CentOS. Now almost runs
through (some minor issue with permission remains).

initial version of documented CentOS7 vagrant script

provide srcdir for postgres module in VPATH and CPPFLAGS

The pgxs makefile in Postgres <= 9.2 cannot yet handle VPATH
correctly. Provide header paths explicitly and overwrite
VPATH after sourcing the pgxs file.

Fixes #454.

use place_id instead of parent_place_id for excluded place_ids in interpolation lines query

remove reference assignment where unnecessary

Avoids a number of warnings on PHP7.

Merge branch 'interpolations'

fill place and osmline independently for interpolations on update

interpolation lines may be missing in osmline when the interpolation
is broken, so we cannot conclude that a way is not in place, just
because there are no entries in location_property_osmline.

reinstate tests for geometrytype

fix variable name in search xml output

cmake: use explicit VPATH when calling pgxs makefile

Some postgres installations report problems with the makefile
magic that should look for the source where the Makefile is
located. So hand in the path explicitly via VPATH instead.

Fixes #453.

cmake: force-set options and explicitly check for pgxs makefile

fixes #443

reintroduce addr:place for interpolation

It is perfectly valid that interpolated addresses refer to
something else than a street.

Also gets rid of the maximum interpolation size. As we don't
expand, arbitrary sizes are fine.

tests: permit duplicate entries in interpolation table

fixes self-intersecting interpolation test

tests for interpolation to building type change