]> git.openstreetmap.org Git - nominatim.git/commitdiff
Merge remote-tracking branch 'upstream/master'
authorSarah Hoffmann <lonvia@denofr.de>
Sun, 2 Oct 2016 12:23:11 +0000 (14:23 +0200)
committerSarah Hoffmann <lonvia@denofr.de>
Sun, 2 Oct 2016 12:23:11 +0000 (14:23 +0200)
1  2 
lib/Geocode.php
tests/features/api/search_params.feature

diff --combined lib/Geocode.php
index a3066b7c858087cfbe837ea21fbda547afc02457,ec7728d2d2cf9e5fc7c5916695cabe68bc28da3c..54e738ff5aa90e7ece10478c3f5588a7c55509b9
@@@ -24,7 -24,7 +24,7 @@@ class Geocod
  
      protected $aExcludePlaceIDs = array();
      protected $bDeDupe = true;
 -    protected $bReverseInPlan = false;
 +    protected $bReverseInPlan = true;
  
      protected $iLimit = 20;
      protected $iFinalLimit = 10;
                          // TODO: filter out the pointless search terms (2 letter name tokens and less)
                          // they might be right - but they are just too darned expensive to run
                          if (sizeof($aSearch['aName'])) $aTerms[] = "name_vector @> ARRAY[".join($aSearch['aName'], ",")."]";
 -                        if (sizeof($aSearch['aNameNonSearch'])) $aTerms[] = "array_cat(name_vector,ARRAY[]::integer[]) @> ARRAY[".join($aSearch['aNameNonSearch'], ",")."]";
 +                        //if (sizeof($aSearch['aNameNonSearch'])) $aTerms[] = "array_cat(name_vector,ARRAY[]::integer[]) @> ARRAY[".join($aSearch['aNameNonSearch'], ",")."]";
                          if (sizeof($aSearch['aAddress']) && $aSearch['aName'] != $aSearch['aAddress']) {
                              // For infrequent name terms disable index usage for address
                              if (CONST_Search_NameOnlySearchFrequencyThreshold
                                  && sizeof($aSearch['aName']) == 1
                                  && $aWordFrequencyScores[$aSearch['aName'][reset($aSearch['aName'])]] < CONST_Search_NameOnlySearchFrequencyThreshold
                              ) {
 -                                $aTerms[] = "array_cat(nameaddress_vector,ARRAY[]::integer[]) @> ARRAY[".join(array_merge($aSearch['aAddress'], $aSearch['aAddressNonSearch']), ",")."]";
 +                                //$aTerms[] = "array_cat(nameaddress_vector,ARRAY[]::integer[]) @> ARRAY[".join(array_merge($aSearch['aAddress'], $aSearch['aAddressNonSearch']), ",")."]";
 +                                $aTerms[] = "array_cat(nameaddress_vector,ARRAY[]::integer[]) @> ARRAY[".join($aSearch['aAddress'],",")."]";
                              } else {
                                  $aTerms[] = "nameaddress_vector @> ARRAY[".join($aSearch['aAddress'], ",")."]";
 -                                if (sizeof($aSearch['aAddressNonSearch'])) {
 +                                /*if (sizeof($aSearch['aAddressNonSearch'])) {
                                      $aTerms[] = "array_cat(nameaddress_vector,ARRAY[]::integer[]) @> ARRAY[".join($aSearch['aAddressNonSearch'], ",")."]";
 -                                }
 +                                }*/
                              }
                          }
                          if ($aSearch['sCountryCode']) $aTerms[] = "country_code = '".pg_escape_string($aSearch['sCountryCode'])."'";
  
              if ($aLookup['place_id']) {
                  $aSearchResults = $this->getDetails(array($aLookup['place_id'] => -1));
+                 $aResultPlaceIDs[$aLookup['place_id']] = -1;
              } else {
                  $aSearchResults = array();
              }
index 7cb597a86e1835759af15badd02c17c607cb6526,de6bd53d0fe545c2dd8f74e8677f66a52469fe49..cd0db091e44a4669ec15757c6585cafc887361d3
@@@ -35,6 -35,12 +35,12 @@@ Feature: Search querie
            | country      | Canada
            | country_code | ca
  
+     Scenario: coordinate search with addressdetails
+         When sending json search query "51.193058013916,15.5245780944824" with address
+         Then result addresses contain
+           | village    | country | country_code
+           | Kraszowice | Polska  | pl
      Scenario: Address details with unknown class types
          When sending json search query "foobar, Essen" with address
          Then results contain
@@@ -84,7 -90,7 +90,7 @@@
      Scenario: bounded search remains within viewbox, even with no results
          Given the request parameters
           | bounded | viewbox
 -         | 1       | 43.54285,-5.662003,43.5403125,-5.6563282
 +         | 1       | 43.5403125,-5.6563282,43.54285,-5.662003
           When sending json search query "restaurant"
          Then less than 1 result is returned