Merge pull request #483 from lonvia/fix-county-ranks

fix address ranks for counties
komoot · Jul 29, 2020 · feffe0a · feffe0a
2 parents d0a940f + dec4454
commit feffe0a
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 16 deletions.
diff --git a/src/main/java/de/komoot/photon/PhotonDoc.java b/src/main/java/de/komoot/photon/PhotonDoc.java
@@ -153,6 +153,7 @@ public void completeFromAddress() {
         district = extractAddress(district, "suburb");
         locality = extractAddress(locality, "neighbourhood");
         county = extractAddress(county, "county");
+        state = extractAddress(state, "state");
 
         String addressPostCode = address.get("postcode");
         if (addressPostCode != null && !addressPostCode.equals(postcode)) {

diff --git a/src/main/java/de/komoot/photon/nominatim/model/AddressRow.java b/src/main/java/de/komoot/photon/nominatim/model/AddressRow.java
@@ -21,6 +21,9 @@ public class AddressRow {
 
     static private final String[] USEFUL_CONTEXT_KEYS = new String[]{"boundary", "landuse", "place"}; // must be in alphabetic order to speed up lookup
 
+    /**
+     * @return whether nominatim thinks this place is a street
+     */
     public boolean isStreet() {
         return 26 <= rankAddress && rankAddress < 28;
     }
@@ -38,17 +41,29 @@ public boolean isLocality() {
     public boolean isDistrict() {
         return 17 <= rankAddress && rankAddress < 22;
     }
+
+    /**
+     * @return whether nominatim thinks this place is a town or city
+     */
     public boolean isCity() {
-        return 13 <= rankAddress && rankAddress <= 16;
+        return 13 <= rankAddress && rankAddress < 17;
     }
 
     /**
      * @return whether nominatim thinks this place is a county
      */
-    public boolean isCounty() {
-        return 5 <= rankAddress && rankAddress <= 9;
+    public boolean isCounty() { return 10 <= rankAddress && rankAddress < 13; }
+
+    /**
+     * @return whether nominatim thinks this place is a state
+     */
+    public boolean isState() { return 5 <= rankAddress && rankAddress < 10; }
+
+    public boolean isCountry() {
+        return (rankAddress == 4 && "boundary".equals(osmKey) && "administrative".equals(osmValue));
     }
 
+
     public boolean isPostcode() {
         if ("place".equals(osmKey) && "postcode".equals(osmValue)) {
             return true;
@@ -82,19 +97,6 @@ public boolean isUsefulForContext() {
         return false;
     }
 
-    public boolean isCountry() {
-        if (rankAddress == 4 && "boundary".equals(osmKey) && "administrative".equals(osmValue)) {
-            return true;
-        }
-
-
-        return false;
-    }
-
-    public boolean isState() {
-        return (5 <= rankAddress && rankAddress <= 9);
-    }
-
     @Override
     public String toString() {
         return MoreObjects.toStringHelper(this)