Merge pull request #2068 from lonvia/fix-reverse-only

Do not create POI search terms in reverse-only mode

diff --git a/.github/workflows/ci-tests.yml b/.github/workflows/ci-tests.yml
index 79830673b25d68a89a001d7a42cfc37502cb0076..f1da461686f148db0a438c2f0432c6dfac962198 100644 (file)
--- a/.github/workflows/ci-tests.yml
+++ b/.github/workflows/ci-tests.yml
@@ -97,3 +97,9 @@ jobs:
                    php ./utils/update.php --init-updates
                    php ./utils/update.php --import-osmosis
               working-directory: build
+
+            - name: Run reverse-only import
+              run : |
+                  dropdb nominatim
+                  php ./utils/setup.php --osm-file ../monaco-latest.osm.pbf --reverse-only --all
+              working-directory: build

diff --git a/sql/functions/placex_triggers.sql b/sql/functions/placex_triggers.sql
index 7d6352b4eda6ea2bdd7fe0dfc6475ba710065f42..6fd9e2258a4f67c91e372a00fd98509f8dfe6343 100644 (file)
--- a/sql/functions/placex_triggers.sql
+++ b/sql/functions/placex_triggers.sql
@@ -814,8 +814,8 @@ BEGIN
 
       END IF;
 
-      IF array_length(name_vector, 1) is not NULL
-         OR inherited_address is not NULL OR NEW.address is not NULL
+      IF not %REVERSE-ONLY% AND (array_length(name_vector, 1) is not NULL
+         OR inherited_address is not NULL OR NEW.address is not NULL)
       THEN
         SELECT * INTO name_vector, nameaddress_vector
           FROM create_poi_search_terms(NEW.place_id,
@@ -824,7 +824,7 @@ BEGIN
                                        NEW.country_code, NEW.housenumber,
                                        name_vector, NEW.centroid);
 
-        IF not %REVERSE-ONLY% AND array_length(name_vector, 1) is not NULL THEN
+        IF array_length(name_vector, 1) is not NULL THEN
           INSERT INTO search_name (place_id, search_rank, address_rank,
                                    importance, country_code, name_vector,
                                    nameaddress_vector, centroid)