]> git.openstreetmap.org Git - nominatim.git/commitdiff
Updated reverse geocoding to improve handling of zoom=0
authorBrian Quinion <openstreetmap@brian.quinion.co.uk>
Thu, 24 May 2012 15:54:51 +0000 (16:54 +0100)
committerSarah Hoffmann <lonvia@denofr.de>
Thu, 24 May 2012 19:29:23 +0000 (21:29 +0200)
Conflicts:

website/reverse.php

website/reverse.php

index ed7090e264ba68e871a482534a6bc4b4f89ddc7b..253b84a59357e7368d3d1087573050ea3e0a49cf 100755 (executable)
                        4 => 4,
                        5 => 8, // State
                        6 => 10, // Region
-                       7 => 10, 
+                       7 => 10,
                        8 => 12, // County
-                       9 => 12,  
+                       9 => 12,
                        10 => 17, // City
-                       11 => 17, 
+                       11 => 17,
                        12 => 18, // Town / Village
-                       13 => 18, 
+                       13 => 18,
                        14 => 22, // Suburb
                        15 => 22,
                        16 => 26, // Street, TODO: major street?
-                       17 => 26, 
+                       17 => 26,
                        18 => 30, // or >, Building
                        19 => 30, // or >, Building
                        );
                        if ($fSearchDiam > 0.008 && $iMaxRank > 22) $iMaxRank = 22;
                        if ($fSearchDiam > 0.001 && $iMaxRank > 26) $iMaxRank = 26;
 
-                       $sSQL = 'select place_id,parent_place_id from placex';
+                       $sSQL = 'select place_id,parent_place_id,rank_search from placex';
                        $sSQL .= ' WHERE ST_DWithin('.$sPointSQL.', geometry, '.$fSearchDiam.')';
                        $sSQL .= ' and rank_search != 28 and rank_search >= '.$iMaxRank;
                        $sSQL .= ' and (name is not null or housenumber is not null)';
 //var_dump($sSQL);
                        $aPlace = $oDB->getRow($sSQL);
                        $iPlaceID = $aPlace['place_id'];
+                       $iParentPlaceID = $aPlace['parent_place_id'];
                        if (PEAR::IsError($iPlaceID))
                        {
                                failInternalError("Could not determine closest place.", $sSQL, $iPlaceID); 
                }
 
                // The point we found might be too small - use the address to find what it is a child of
-               if ($iPlaceID)
+               if ($iPlaceID && $iMaxRank < 28)
                {
-                       $sSQL = "select address_place_id from place_addressline where cached_rank_address <= $iMaxRank and place_id = $iPlaceID order by cached_rank_address desc,isaddress desc,distance desc limit 1";
-//var_dump($sSQL);
+                       if ($aPlace['rank_search'] > 28 && $iParentPlaceID) {
+                               $iPlaceID = $iParentPlaceID;
+                       }
+                       $sSQL = "select address_place_id from place_addressline where place_id = $iPlaceID order by abs(cached_rank_address - $iMaxRank) asc,cached_rank_address desc,isaddress desc,distance desc limit 1";
                        $iPlaceID = $oDB->getOne($sSQL);
                        if (PEAR::IsError($iPlaceID))
                        {
                                failInternalError("Could not get parent for place.", $sSQL, $iPlaceID); 
                        }
-
-                       if ($iPlaceID && $aPlace['place_id'] && $iMaxRank < 28)
-                       {
-                               $sSQL = "select address_place_id from place_addressline where cached_rank_address <= $iMaxRank and place_id = ".$aPlace['place_id']." order by cached_rank_address desc,isaddress desc,distance desc";
-//var_dump($sSQL);
-                               $iPlaceID = $oDB->getOne($sSQL);
-                               if (PEAR::IsError($iPlaceID))
-                               {
-                                       failInternalError("Could not get larger parent for place.", $sSQL, $iPlaceID); 
-                               }
-                       }
                        if (!$iPlaceID)
                        {
                                $iPlaceID = $aPlace['place_id'];