From: Sarah Hoffmann <lonvia@denofr.de>
Date: Thu, 5 Jul 2018 17:29:53 +0000 (+0200)
Subject: Merge remote-tracking branch 'upstream/master'
X-Git-Tag: deploy~317
X-Git-Url: https://git.openstreetmap.org/nominatim.git/commitdiff_plain/203b5f7de12ca09d275103eb2eb70fe5a4c20466?hp=0bbc25c9c5c3ab644d6771ccbd87f26c3d7552bf

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

diff --git a/lib/ReverseGeocode.php b/lib/ReverseGeocode.php
index 69fef37e..f854d098 100644
--- a/lib/ReverseGeocode.php
+++ b/lib/ReverseGeocode.php
@@ -72,12 +72,12 @@ class ReverseGeocode
             'Could not determine closest housenumber on an osm interpolation line.'
         );
     }
-    
+
     protected function polygonFunctions($sPointSQL, $iMaxRank)
     {
         // starts the nopolygonFound function if no polygon is found with the lookupPolygon function
         $oResult = null;
-        
+
         $aPlace = $this->lookupPolygon($sPointSQL, $iMaxRank);
         if ($aPlace) {
             $oResult = new Result($aPlace['place_id']);
@@ -91,22 +91,22 @@ class ReverseGeocode
         }
         return $oResult;
     }
-    
+
     protected function noPolygonFound($sPointSQL, $iMaxRank)
     {
         // searches for polygon in table country_osm_grid which contains the searchpoint
         // and searches for the nearest place node to the searchpoint in this polygon
-        $sSQL = 'SELECT * FROM country_osm_grid';
-        $sSQL .= ' WHERE ST_CONTAINS (geometry, '.$sPointSQL.' )';
-        
+        $sSQL = 'SELECT country_code FROM country_osm_grid';
+        $sSQL .= ' WHERE ST_CONTAINS (geometry, '.$sPointSQL.') limit 1';
+
         $aPoly = chksql(
             $this->oDB->getRow($sSQL),
             'Could not determine polygon containing the point.'
         );
         if ($aPoly) {
             $sCountryCode = $aPoly['country_code'];
-            
-            $sSQL = 'SELECT *, ST_distance('.$sPointSQL.', geometry) as distance';
+
+            $sSQL = 'SELECT place_id, ST_distance('.$sPointSQL.', geometry) as distance';
             $sSQL .= ' FROM placex';
             $sSQL .= ' WHERE osm_type = \'N\'';
             $sSQL .= ' AND country_code = \''.$sCountryCode.'\'';
@@ -118,7 +118,7 @@ class ReverseGeocode
             $sSQL .= ' AND ST_DWithin('.$sPointSQL.', geometry, 1.0)';
             $sSQL .= ' ORDER BY distance ASC, rank_address DESC';
             $sSQL .= ' LIMIT 1';
-            
+
             if (CONST_Debug) var_dump($sSQL);
             $aPlacNode = chksql(
                 $this->oDB->getRow($sSQL),
@@ -129,19 +129,19 @@ class ReverseGeocode
             }
         }
     }
-    
+
     protected function lookupPolygon($sPointSQL, $iMaxRank)
     {
         // searches for polygon where the searchpoint is within
         // if a polygon is found, placenodes with a higher rank are searched inside the polygon
-    
+
         // polygon search begins at suburb-level
         if ($iMaxRank > 25) $iMaxRank = 25;
         // no polygon search over country-level
         if ($iMaxRank < 4) $iMaxRank = 4;
         // search for polygon
-        $sSQL = 'SELECT * FROM';
-        $sSQL .= '(select place_id,parent_place_id,rank_address, rank_search, country_code, geometry';
+        $sSQL = 'SELECT place_id, parent_place_id, rank_address, rank_search FROM';
+        $sSQL .= '(select place_id, parent_place_id, rank_address, rank_search, country_code, geometry';
         $sSQL .= ' FROM placex';
         $sSQL .= ' WHERE ST_GeometryType(geometry) in (\'ST_Polygon\', \'ST_MultiPolygon\')';
         $sSQL .= ' AND rank_address Between 4 AND ' .$iMaxRank;
@@ -163,7 +163,7 @@ class ReverseGeocode
             $iRankAddress = $aPoly['rank_address'];
             $iRankSearch = $aPoly['rank_search'];
             $iPlaceID = $aPoly['place_id'];
-            
+
             if ($iRankAddress != $iMaxRank) {
             //search diameter for the place node search
                 if ($iMaxRank <= 4) {
@@ -181,8 +181,8 @@ class ReverseGeocode
                 } elseif ($iMaxRank <= 25) {
                     $fSearchDiam = 0.1;
                 }
-                
-                $sSQL = 'SELECT *';
+
+                $sSQL = 'SELECT place_id';
                 $sSQL .= ' FROM (';
                 $sSQL .= ' SELECT place_id, rank_address,country_code, geometry,';
                 $sSQL .= ' ST_distance('.$sPointSQL.', geometry) as distance';
@@ -209,7 +209,7 @@ class ReverseGeocode
                 $sSQL .= ' WHERE ST_CONTAINS((SELECT geometry FROM placex WHERE place_id = '.$iPlaceID.'), geometry )';
                 $sSQL .= ' ORDER BY distance ASC, rank_address DESC';
                 $sSQL .= ' LIMIT 1';
-                
+
                 if (CONST_Debug) var_dump($sSQL);
                 $aPlacNode = chksql(
                     $this->oDB->getRow($sSQL),
@@ -222,7 +222,7 @@ class ReverseGeocode
         }
         return $aPoly;
     }
-    
+
 
     public function lookup($fLat, $fLon, $bDoInterpolation = true)
     {
@@ -246,7 +246,7 @@ class ReverseGeocode
         $aPlace = null;
         $fMaxAreaDistance = 1;
         $bIsTigerStreet = false;
-       
+
         // for POI or street level
         if ($iMaxRank >= 26) {
             $sSQL = 'select place_id,parent_place_id,rank_address,country_code,';
@@ -275,14 +275,14 @@ class ReverseGeocode
                 $this->oDB->getRow($sSQL),
                 'Could not determine closest place.'
             );
-            
+
             if ($aPlace) {
-                    
+
                 $iDistance = $aPlace['distance'];
                 $iPlaceID = $aPlace['place_id'];
                 $oResult = new Result($iPlaceID);
                 $iParentPlaceID = $aPlace['parent_place_id'];
-                
+
                 if ($bDoInterpolation && $iMaxRank >= 30) {
                     if ($aPlace['rank_address'] <=27) {
                         $iDistance = 0.001;
@@ -294,7 +294,7 @@ class ReverseGeocode
                         $oResult->iHouseNumber = closestHouseNumber($aHouse);
                     }
                 }
-                
+
                 // if street and maxrank > streetlevel
                 if (($aPlace['rank_address'] <=27)&& $iMaxRank > 27) {
                     // find the closest object (up to a certain radius) of which the street is a parent of
@@ -320,7 +320,7 @@ class ReverseGeocode
                         $iPlaceID = $aStreet['place_id'];
                         $oResult = new Result($iPlaceID);
                         $iParentPlaceID = $aStreet['parent_place_id'];
-                            
+
                         if ($bDoInterpolation && $iMaxRank >= 30) {
                             $aHouse = $this->lookupInterpolation($sPointSQL, $iDistance, $iParentPlaceID);
 
@@ -331,7 +331,7 @@ class ReverseGeocode
                         }
                     }
                 }
-                
+
                   // In the US we can check TIGER data for nearest housenumber
                 if (CONST_Use_US_Tiger_Data && $aPlace['country_code'] == 'us' && $this->iMaxRank >= 28) {
                     $fSearchDiam = $aPlace['rank_address'] > 28 ? $aPlace['distance'] : 0.001;