fix variable name typos

note phpcs command in contribution readme

details: check for unknown object and icons

fix syntax error for route boxes

avoid passing temporary array by reference

PHP7 complains: Only variables should be passed by reference.

better error message for missing osm2pgsql

replace deprecated postgis functions

ST_Line_Locate_Point and ST_Distance_Spheroid have changed
spelling in newer version and throw a deprecation warning.

check for correct array for country codes

fixes trac ticket 5427

Change on get_addressdata check for matching the condition in placex_update

On placex_update we stop the indexing to the first parent if the rank_search is above 27. We should do the same check in get_adressdata, because place with a rank_address != 30 and a search_rank > 27 will have only 1 parent.
https://github.com/twain47/Nominatim/issues/534

docs: fix link

fill interpolation array for reverse lookups via search endpoint

Fixes #540

fix namespace use

Merge pull request #538 from lonvia/update-default-languages

Update default languages and move country search term creation to setup.php

move country search term creation into setup script

Search results can become odd without the country search
terms, so make their inclusion a mandatory part of the
setup.

Also adds a new configuration variable to restrict the
languages taken into account by Nominatim.

update per-country language defaults

Added missing defaults for all mono-lingual countries as per
http://wiki.openstreetmap.org/wiki/Nominatim/Country_Codes

Merge pull request #533 from mtmail/PSR2-php-tests

fix PSR2 warnings in php tests

Merge pull request #532 from mtmail/PSR2-namespaces

add namespaces, method visibility according to PSR2 standard

fix PSR2 warnings in php tests

add namespaces, method visibility according to PSR2 standard

vagrant: use old configs when updating

Otherwise we may overwrite vagrant-specific settings like
nopassword for sudo.

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

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 /settings

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.