From: Sarah Hoffmann <lonvia@denofr.de>
Date: Wed, 7 Jun 2017 21:26:24 +0000 (+0200)
Subject: Merge remote-tracking branch 'upstream/master'
X-Git-Tag: deploy~382
X-Git-Url: https://git.openstreetmap.org./nominatim.git/commitdiff_plain/ebbd03efe08c33fe81fdcf17e498d1d14546a314?ds=inline;hp=-c

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

ebbd03efe08c33fe81fdcf17e498d1d14546a314
diff --combined sql/functions.sql
index d3207843,77af9263..d55b0d65
--- a/sql/functions.sql
+++ b/sql/functions.sql
@@@ -1492,7 -1492,8 +1492,8 @@@ BEGI
        FOR relMember IN select get_osm_rel_members(relation_members,ARRAY['label']) as member LOOP
  
          FOR linkedPlacex IN select * from placex where osm_type = upper(substring(relMember.member,1,1))::char(1) 
-           and osm_id = substring(relMember.member,2,10000)::bigint order by rank_search desc limit 1 LOOP
+           and osm_id = substring(relMember.member,2,10000)::bigint
+           and class = 'place' order by rank_search desc limit 1 LOOP
  
            -- If we don't already have one use this as the centre point of the geometry
            IF NEW.centroid IS NULL THEN
@@@ -1523,7 -1524,8 +1524,8 @@@
          FOR relMember IN select get_osm_rel_members(relation_members,ARRAY['admin_center','admin_centre']) as member LOOP
  
            FOR linkedPlacex IN select * from placex where osm_type = upper(substring(relMember.member,1,1))::char(1) 
-             and osm_id = substring(relMember.member,2,10000)::bigint order by rank_search desc limit 1 LOOP
+             and osm_id = substring(relMember.member,2,10000)::bigint
+             and class = 'place' order by rank_search desc limit 1 LOOP
  
              -- For an admin centre we also want a name match - still not perfect, for example 'new york, new york'
              -- But that can be fixed by explicitly setting the label in the data
@@@ -2401,7 -2403,7 +2403,7 @@@ BEGI
        CASE WHEN class = 'place' and type = 'postcode' THEN hstore('name', postcode) ELSE name END as name,
        CASE WHEN extratags ? 'place' THEN 'place' ELSE class END as class,
        CASE WHEN extratags ? 'place' THEN extratags->'place' ELSE type END as type,
 -      admin_level, fromarea, isaddress,
 +      admin_level, fromarea, isaddress and linked_place_id is NULL as isaddress,
        CASE WHEN address_place_id = for_place_id AND rank_address = 0 THEN 100 WHEN rank_address = 11 THEN 5 ELSE rank_address END as rank_address,
        distance,country_code,postcode
        from place_addressline join placex on (address_place_id = placex.place_id) 
@@@ -2418,7 -2420,7 +2420,7 @@@
      IF searchpostcode IS NOT NULL and location.type = 'postcode' THEN
        location.isaddress := FALSE;
      END IF;
-     IF searchpostcode IS NULL and location.isaddress and location.type != 'postcode' and location.postcode IS NOT NULL THEN
+     IF searchpostcode IS NULL and location.isaddress and location.type != 'postcode' and location.postcode IS NOT NULL and location.postcode not similar to '%(,|;)%' THEN
        searchpostcode := location.postcode;
      END IF;
      IF location.rank_address = 4 AND location.isaddress THEN