From: Sarah Hoffmann <lonvia@denofr.de>
Date: Wed, 30 Oct 2013 20:52:06 +0000 (+0100)
Subject: Merge remote-tracking branch 'upstream/master'
X-Git-Tag: deploy~565
X-Git-Url: https://git.openstreetmap.org./nominatim.git/commitdiff_plain/1ccf4fb92fe71e9e2bc0baabfb2e92cd79166dae?hp=-c

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

1ccf4fb92fe71e9e2bc0baabfb2e92cd79166dae
diff --combined sql/functions.sql
index 2407db5b,4670a71f..4c040fdc
--- a/sql/functions.sql
+++ b/sql/functions.sql
@@@ -347,7 -347,7 +347,7 @@@ BEGI
  
      w := getorcreate_word_id(s);
  
-     IF NOT (ARRAY[w] <@ result) THEN
+     IF w IS NOT NULL AND NOT (ARRAY[w] <@ result) THEN
        result := result || w;
      END IF;
  
@@@ -415,7 -415,7 +415,7 @@@ BEGI
  
    w := getorcreate_word_id(s);
  
-   IF NOT (ARRAY[w] <@ result) THEN
+   IF w IS NOT NULL AND NOT (ARRAY[w] <@ result) THEN
      result := result || w;
    END IF;
  
@@@ -1357,26 -1357,13 +1357,26 @@@ BEGI
      NEW.centroid := null;
  
      -- reclaculate country and partition
 -    IF NEW.rank_search >= 4 THEN
 -      --NEW.calculated_country_code := lower(get_country_code(NEW.geometry, NEW.country_code));
 -      NEW.calculated_country_code := lower(get_country_code(place_centroid));
 +    IF NEW.rank_search = 4 THEN
 +      -- for countries, believe the mapped country code,
 +      -- so that we remain in the right partition if the boundaries
 +      -- suddenly expand.
 +      NEW.partition := get_partition(place_centroid, lower(NEW.country_code));
 +      IF NEW.partition = 0 THEN
 +        NEW.calculated_country_code := lower(get_country_code(place_centroid));
 +        NEW.partition := get_partition(place_centroid, NEW.calculated_country_code);
 +      ELSE
 +        NEW.calculated_country_code := lower(NEW.country_code);
 +      END IF;
      ELSE
 -      NEW.calculated_country_code := NULL;
 +      IF NEW.rank_search > 4 THEN
 +        --NEW.calculated_country_code := lower(get_country_code(NEW.geometry, NEW.country_code));
 +        NEW.calculated_country_code := lower(get_country_code(place_centroid));
 +      ELSE
 +        NEW.calculated_country_code := NULL;
 +      END IF;
 +      NEW.partition := get_partition(place_centroid, NEW.calculated_country_code);
      END IF;
 -    NEW.partition := get_partition(place_centroid, NEW.calculated_country_code);
      NEW.geometry_sector := geometry_sector(NEW.partition, place_centroid);
  
      -- Adding ourselves to the list simplifies address calculations later