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

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

update comments and formatting

fix scene making scripts and add building-parallel-road scene

Scripts updated to newest libosmium and shebang fixed.

remove duplicate index

get housenumber data for interpolations from place table

The placex table may not yet be filled when the interpolation
lines are computed.

fix syntax in load-data function

Also makes sure that load data runs with at least 2 threads: one
for loading placex and one for loading osmline.

Merge remote-tracking branch 'markus/master' into master

Change load-data in setup.php.

Fix bug in index.c and remove column admin_level from location_property_osmline.

fix small bug in index.c

Merge pull request #447 from lonvia/fix-array-expression

replace new-style [] expression with old-style array()

replace new-style [] expression with old-style array()

Also removes some unused code.

fixes #444

Merge pull request #446 from mtmail/make-outline-clickable-in-ui

UI: the result outline (geojson/polygon) now lets the map handle clicks

UI: the result outline (geojson/polygon) now lets the map handle its clicks

fix bug in Geocode.php

index on geometry of interpolation lines, and more improvements.

Merge pull request #441 from mtmail/update-php-to-check-for-correct-js-file

update.php was checking for static file which is no longer part of the project

update.php was checking for static file which is no longer part of the project

Correct merge error in Geocode.php.

Coding style adaptions and correcting small errors from merge.

Merge remote-tracking branch 'origin/master' into osmline

Conflicts:
lib/Geocode.php
lib/PlaceLookup.php
sql/functions.sql
sql/tables.sql
utils/setup.php

Remove interpolation lines from placex and save them in an extra table.

Merge pull request #436 from lonvia/remove-location-property-tables

remove unused location_property_-partion- tables

create_sql_functions() needs the commandline parameters

Fixes one failing test.

remove unused location_property_-partion- tables

apply marker filtering to functions.sql also after table creation

fixes #430

remove duplicate CORS header

fixes #427

Merge pull request #425 from lonvia/disable-aux-tables

make tables for external data (Tiger and aux) configurable

Merge pull request #424 from lonvia/cleanup-logging

clean up query logging

make tables for external data (Tiger and aux) configurable

Introduces two new settings CONST_Use_US_Tiger_Data and
CONST_Use_Aux_Location_data, which are disabled by default.
When false the corresponding tables are ignored in queries
and updates.

Aux and tiger tables are no longer created by default. This
has to be done by the corresponding import scripts. The former
aux table creation can be found in sql/aux_tables.sql for
reference.

clean up query logging

- remove query_log table, keeping only new_query_log
- drop unused import_npi_log table
- disable DB logging per default
- use file logging structure from osm.org

Merge pull request #420 from lonvia/remove-explicit-software-versions

Remove unsupported/unused features and settings

remove deleted us_state* tables from tests as well

remove state and county data for US and make postcode import optional

remove unused settings

remove 'closed for indexing' feature

Simultanous updates are fine these days.

remove explicitly set postgres/postgis version

Get the version from the database where necessary or simply
probe for existence of features. Fake hstore_to_json when
necessary.

Bumps the minimum required versions fro postgres to 9.1 and
for postgis to 2.0.

Merge pull request #416 from lonvia/dead-code-removal

Remove unused and/or unmaintained php code

test: catch script output in as debug log output

remove unused functions in php library

remove unused files

remove unused/unmaintained options in import scripts

Fix fixing of coding style

exclude Tiger stuff from required tests

adapt tests to new tiger algorithm

make sure linked places get recomputed when unlinked

When linked the place may not be in the search index,
so it must be reindexed when being unlinked. The status
change will only have an effect during the subsequent
update, so change tests to that effect.

fix syntax errors in tiger code and adapt format of dummy tables