From: Sarah Hoffmann <lonvia@denofr.de>
Date: Mon, 26 May 2014 20:24:54 +0000 (+0200)
Subject: Merge remote-tracking branch 'upstream/master'
X-Git-Tag: deploy~529
X-Git-Url: https://git.openstreetmap.org./nominatim.git/commitdiff_plain/f1cbca788531e9eefe5fbbadd7ac0d218f6aa076?hp=-c

Merge remote-tracking branch 'upstream/master'
---

f1cbca788531e9eefe5fbbadd7ac0d218f6aa076
diff --combined lib/lib.php
index b26038d8,1e05def3..b54023a4
--- a/lib/lib.php
+++ b/lib/lib.php
@@@ -45,7 -45,7 +45,7 @@@
  		if (!$sUserError) $sUserError = $sError;
  		error_log('ERROR: '.$sError);
  		echo $sUserError."\n";
- 		exit;
+ 		exit(-1);
  	}
  
  
@@@ -94,8 -94,7 +94,8 @@@
  
  	function bySearchRank($a, $b)
  	{
 -		if ($a['iSearchRank'] == $b['iSearchRank']) return 0;
 +		if ($a['iSearchRank'] == $b['iSearchRank'])
 +            return strlen($a['sOperator']) + strlen($a['sHouseNumber']) - strlen($b['sOperator']) - strlen($b['sHouseNumber']);
  		return ($a['iSearchRank'] < $b['iSearchRank']?-1:1);
  	}
  
@@@ -178,7 -177,7 +178,7 @@@
  	{
  		$aResult = array(array(join(' ',$aWords)));
  		$sFirstToken = '';
 -		if ($iDepth < 8) {
 +		if ($iDepth < 7) {
  			while(sizeof($aWords) > 1)
  			{
  				$sWord = array_shift($aWords);
diff --combined sql/functions.sql
index a6fa20ea,e0f6de72..488d340f
--- a/sql/functions.sql
+++ b/sql/functions.sql
@@@ -936,11 -936,6 +936,11 @@@ DECLAR
  BEGIN
    --DEBUG: RAISE WARNING '% %',NEW.osm_type,NEW.osm_id;
  
 +  -- remove operator tag for most places, messes too much with search_name indexes
 +  IF NEW.class not in ('amenity', 'shop') THEN
 +    NEW.name := delete(NEW.name, 'operator');
 +  END IF;
 +
    -- just block these
    IF NEW.class in ('landuse','natural') and NEW.name is null THEN
  --    RAISE WARNING 'empty landuse %',NEW.osm_id;
@@@ -2064,11 -2059,6 +2064,11 @@@ BEGI
      --DEBUG: RAISE WARNING '%', existingplacex;
    END IF;
  
 +  -- remove operator tag for most places, messes too much with search_name indexes
 +  IF NEW.class not in ('amenity', 'shop') THEN
 +    NEW.name := delete(NEW.name, 'operator');
 +  END IF;
 +
    -- Just block these - lots and pointless
    IF NEW.class in ('landuse','natural') and NEW.name is null THEN
      -- if the name tag was removed, older versions might still be lurking in the place table
@@@ -2259,12 -2249,6 +2259,12 @@@
  
    END IF;
  
 +  -- refuse to update multiplpoygons with too many objects, too much of a performance hit
 +  IF ST_NumGeometries(NEW.geometry) > 2000 THEN
 +    RAISE WARNING 'Dropping update of % % because of geometry complexity.', NEW.osm_type, NEW.osm_id;
 +    RETURN NULL;
 +  END IF;
 +
    IF coalesce(existing.name::text, '') != coalesce(NEW.name::text, '')
       OR coalesce(existing.extratags::text, '') != coalesce(NEW.extratags::text, '')
       OR coalesce(existing.housenumber, '') != coalesce(NEW.housenumber, '')
@@@ -2503,7 -2487,7 +2503,7 @@@ BEGI
        CASE WHEN class = 'place' and type = 'postcode' THEN hstore('name', postcode) ELSE name END as name,
        class, type, admin_level, true as fromarea, true as isaddress,
        CASE WHEN rank_address = 0 THEN 100 WHEN rank_address = 11 THEN 5 ELSE rank_address END as rank_address,
-       0 as distance, calculated_country_code
+       0 as distance, calculated_country_code, postcode
        from placex
        where place_id = for_place_id 
    LOOP
@@@ -2514,6 -2498,9 +2514,9 @@@
      IF searchpostcode IS NOT NULL and location.type = 'postcode' THEN
        location.isaddress := FALSE;
      END IF;
+     IF searchpostcode IS NULL and location.postcode IS NOT NULL THEN
+       searchpostcode := location.postcode;
+     END IF;
      IF location.rank_address = 4 AND location.isaddress THEN
        hadcountry := true;
      END IF;