Add parent_osm_id for mapcss

[Ialokim <ialokim@…>]

Currently, it is only possible to access the parent_tag but it would be very helpful to be able to get the parent's osm_id. It should be very easy to implement following https://josm.openstreetmap.de/browser/josm/trunk/src/org/openstreetmap/josm/gui/mappaint/mapcss/ExpressionFactory.java#L457 and https://josm.openstreetmap.de/browser/josm/trunk/src/org/openstreetmap/josm/gui/mappaint/mapcss/ExpressionFactory.java#L780.

[Klumbumbus]

What if the object has several parent objects?

[bastiK]

In 13094/josm:

fixed #15525 - Add parent_osm_id for mapcss

[bastiK]

Replying to Klumbumbus:

What if the object has several parent objects?

It is now restricted to the parent object matched by child selector, i.e. in

relation[type=route][route=foot] > way {...}

it selects the first parent relation with given tags.

[Don-vip]

This piece of code is not correct:

        public static long parent_osm_id(final Environment env) { // NO_UCD (unused code)
            return env.parent == null ? null : env.parent.getUniqueId();
        }

if env.parent is null, the auto-boxing of (Long) null to long will throw an NPE.

[bastiK]

In 13105/josm:

see #15525 - fix NPE

[bastiK]

Replying to Don-vip:

This piece of code is not correct:

        public static long parent_osm_id(final Environment env) { // NO_UCD (unused code)
            return env.parent == null ? null : env.parent.getUniqueId();
        }

if env.parent is null, the auto-boxing of (Long) null to long will throw an NPE.

Okay, this happens when you commit code without testing. :)