make sure no invalid geometries end up in placex

Merge branch 'tetris' of https://github.com/roques/Nominatim into roques-tetris

use SMALLINT for ranks and partition; reorder columns for alignment

fix typo

get pg error on correct connection

send address interpolation import to correct postgres conn

also use address column for word count computation

explicitly name columns at INSERT

deduplicate words: do not look for duplicates in empty partitions

update: getPostgresVersion requires a database connection

with PHP7 the result of new() can no longer be assigned by reference

correct warm.php --search-only help text

Merge pull request #700 from lonvia/structured-address-info

Move address information into hstore column

update osm2pgsql (structured addresses and new libosmium)

use centroid for near feature search on rank 30

Rank 30 has some very large geometries (peninsulas, time zones,
etc.) for which a near feature search for the full geometry is
too expensive, so do the search on the centroid only.

fix operator precedence for hstore

remove street and addr_place columns in placex

The information is onyl needed for parenting,
so no need to store it permanently.

Remove street and addr_place from location_property_osmline

Derived columns are not needed because parent information is
always computed from scratch. So the columns are just duplicate
information.

Also get address information on nodes from address columns. The
other columns are not necessarily reliable when the node have not
been indexed yet.

adapt more BDD tests to hstore

fix DB bd tests for address hstore

adapt osm2pgsql tests for new address column

introduce address column for place tables

The column is a hstore containing the full OSM tag with key and
value.

abort osmline update correctly when way nodes are missing

Fixes #695.

Merge pull request #690 from ImreSamu/fix_install_sql_warnings

fix table/sequence ... does not exist

fix sql warnings
* ERROR:  sequence "seq_postcodes" does not exist
* ERROR:  table "import_polygon_error" does not exist
* ERROR:  table "import_polygon_delete" does not exist
* ERROR:  sequence "file" does not exist

Merge pull request #683 from lonvia/more-url-function

Factor out moreurl computation

move moreURL computation into Geocode and include all params

Fixes #678.

remove unused code

Merge pull request #682 from mtmail/update-docs

update docs/*.md based on vagrant/*.sh

update docs/*.md based on vagrant/*.sh

Merge pull request #680 from mtmail/git-submodule-hint

Update CMakeLists.txt

Update CMakeLists.txt

Ran into the issue this morning.
`git submodules` => `git submodule`

Merge pull request #676 from mtmail/html-map-position-box-disappeared

On /search.php the map-position box disappeared behind the map tiles

Merge pull request #674 from mtmail/leaflet-image-path-changed

leaflet 1.0 changed the path of the marker icon

On /search.php the map-position box disappeared behind the map tiles

leaflet 1.0 changed the path of the marker icon

Merge pull request #671 from mtmail/html-view-use-geojson-geometries

Html view use geojson geometries

update leaflet CSS file as well. Fixes delayed layer animation

Merge pull request #669 from lonvia/address-part-on-intersect

Add all intersecting boundaries to address list for roads

for search,details in format=HTML return geometry in geojson, not wkt. No longer need leaflet-omnivore plugin

update leaflet.js 0.7.7 => 1.0.3

remove obsolete openlayers theme files

for roads add all intersecting boundaries to address list

When roads cross boundaries, both administrative entities should
be added to the address list, so that both entities can be used
for searching. Also allows in a second step to better sort out
addresses of POIs on such roads.

Fixes #121.

Merge pull request #667 from lonvia/refactor-nearpoint

Factor out near point into class

Merge pull request #668 from lonvia/warn-on-missing-osm2pgsql

cmake: add a more explicit hint when osm2pgsql is missing

cmake: add a more explicit hint when osm2pgsql is missing

Fixes #642.

fix style

Merge pull request #665 from openstreetmap/lonvia-patch-1

add unzip tool to requirements for Tiger imports

add tests for nearpoint sql functions

use NearPoint class in Search array

adapt unit tests for new NearPoint type

make NearPoint its own class

add unzip tool to requirements for Tiger imports

Fixes #663

Merge pull request #661 from mtmail/travis-ci-badge

add travis-ci badge ("build passing") to README file

Merge pull request #660 from mtmail/new-github-URL

github.com/twain47 => github.com/openstreetmap

add travis-ci badge ("build passing") to README file

github.com/twain47 => github.com/openstreetmap

Merge pull request #659 from lonvia/full-indexing-in-tests

Fix travis tests and update CONTRIBUTING.md

adapt CONTRIBUTING.md to new test locations and tools

test: rerun indexing after updates until really all is done

There are caess where indexing needs to be done a second round
because related objects need to be updated.

Merge pull request #656 from EdwardBetts/master

correct spelling mistakes

correct spelling mistakes

include housenumbers and postcodes when rechecking for exact matches

Merge pull request #649 from ldgeo/master

UI: Extra HTML headers for compatibility on MSIE>8

extra headers for compatibility on IE7

make sure the flatnode file is disabled while testing

add behave settings

removes @Fail tests

fix geometry creation from scene

add more interpolation update scenarios (currently failing)

test: enable grid support for DB tests

move interpolaton computation into indexing step

The interpolaton computation needs information from the osm2pgsql
slim tables which may not be available when the data is inserted.

Insertion now only adds a line with basic address information to
location_property_osmline. The line is then split during the
indexing, leading to more lines (which are complete in that case)
being inserted.

Fixes #598.

add node grids for tests and test for interpolation update

Merge pull request #644 from ldgeo/master

fix compilation error (cannot find -lpq)

fix compilation error (cannot find -lpq)

Set osmosis if available

add Kosovo to list of country names

Kosovo will remain in the same partition as Serbia to not disrupt
existing databases.

don't delete names on unknown country boundaries

If a country boundary has a country_code that is unknown to
Nominatim, it would delete all names because the coalascing
with country_name would not yield any result.

Merge pull request #632 from melvyn-sopacua/libxml2-fixes

Fix warnings:

Fix warnings:

- be consistent with (const char *) casts when assigning
- use xmlStrlen instead of strlen when dealing with xmlChar *

Merge pull request #630 from lonvia/cascading-make

use properly cascading make for external make file

use properly cascading make for external make file

If make is called directly then the -j option is not
properly propagated. Instead make gives a warning
'No jobserver avilable'.

Merge pull request #629 from melvyn-sopacua/release-2.6.0a1

Build fixes for BSD-like systems

Use CMAKE_MAKE_PROGRAM instead of hardcoded make

FreeBSD uses `gmake` instead of `make` if GNU make is required to build
a port.

BSD compat: use sys/endian.h if available

<byteswap.h> is a linuxism. On BSD-like systems this is <sys/endian.h>

Merge pull request #626 from rory/patch-1

Syntax improvements

Syntax improvements

use backticks for more fixed-width type things

reverse: choose interpolation only when further than point

The ST_DWithin check also includes objects that are the same distance.

Another attempt at #545.

reverse: disable interpolation when going up the address hierarchy

Fixes #621.

Merge pull request #620 from sebkur/improve-readme-formatting

Improve markdown of Installation section

Improve markdown of Installation section

Due to indentation issues (code within numbered list needs at least 8
spaces instead of 4) the code would not appear as code but as plain text
without a fixed size font and also without newlines appearing in the
HTML.

Merge pull request #613 from mtmail/travis-behave-and-split-tests

Replace lettuce with behave in tests. Use python3 Split import and (p…

Replace lettuce with behave in tests. Use python3 Split import and (python,php) tests into two travis executions

Merge pull request #610 from lonvia/set-lanugage-of-indexer

force language of pgsql to C

Merge pull request #605 from lonvia/fix-fraction-on-interpolations

Better search radius for interpolations when reverse geocoding

force language of pgsql to C

Fixes #558.

fix viewbox tests

Hamburg may not be complete in the test set

reverse: reduce search radius for interpolations

Interpolations should have the same maximum search radius as
address points. Change search strategy and only search for
interpolations either before decreasing the maxRank to road
or when a POI was found and we need to check that there isn't
an interpolation that is closer.

add test for moreURL issue and debugging of test query contents

Merge branch 'moreurl-with-countrycodes' of https://github.com/mtmail/Nominatim into mtmail-moreurl-with-countrycodes

Merge pull request #600 from lonvia/behave-tests-rebased

Rework tests