Village recognized as suburb: Grzybowo near Kołobrzeg

[dbapl]

In serach results it is returned with classification suburb which is wrong, it is independent village:
http://nominatim.openstreetmap.org/search.php?q=grzybowo%2C+ko%C5%82obrzeg&viewbox=15.43%2C54.18%2C15.55%2C54.14&polygon=1

[lonvia]

Nominatim takes the classification from the admin_level=10 tag which in most countries means suburb. Poland seems to be an exception. We'd need country-specific mappings to get around that issue.

[dbapl]

Where I can find it in source code?
I would like to fix that for Poland, but have no idea where to start.
Please point me in general direction.

[lonvia]

That's rather tricky. The classification term is created by each website individually. Modifying the code in Nominatim, won't give you a different term on osm.org for example. Until that changes I doubt it is really worth special casing things like this.

[dbapl]

In functions.sql code there is:

      NEW.rank_search := NEW.admin_level * 2;
      IF NEW.type = 'administrative' THEN
        NEW.rank_address := NEW.rank_search;
      ELSE
        NEW.rank_address := 0;
      END IF;

So rank_address = admin_level * 2
For village in Poland it will be rank_address = 10*2 = 20
But for village standard rank_address = 16.
rank_address = 20 is mapped to suburb, which gives us wrong results.

      ELSEIF NEW.type in ('village','hamlet','municipality','district','unincorporated_area','borough') THEN
        NEW.rank_search := 19;
        NEW.rank_address := 16;

      ELSEIF NEW.type in ('suburb','croft','subdivision','isolated_dwelling') THEN
        NEW.rank_search := 20;
        NEW.rank_address := NEW.rank_search;

I will just change admin_level * 2 to mapping used in Poland.
So when we compare them in relations it will match.
I will use this modification on our servers, just give me a clue if I am on right track.

[lonvia]

That'll work as well. You can use NEW.calculated_country_code to figure out if an object is in Poland.

[dbapl]

Thank you.

[lonvia]

This is basically an issue of #213.