Spurious 'key inside key' warning for multipolygons

[maxerickson@…]

What steps will reproduce the problem?

1. Create a multipolygon tagged leisure=track.
2. Run validation

What is the expected result?

There should be no warning.

What happens instead?

There is warning for leisure inside leisure, reporting the relation and a node from an outer way.

Please provide any additional information below. Attach a screenshot if possible.

Presumably impacts keys other than leisure.

URL:http://josm.openstreetmap.de/svn/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2017-07-30 17:07:42 +0200 (Sun, 30 Jul 2017)
Build-Date:2017-07-30 15:14:00
Revision:12542
Relative:URL: ^/trunk

Identification: JOSM/1.5 (12542 en) Linux Arch Linux
Memory Usage: 704 MB / 711 MB (242 MB allocated, but free)
Java version: 1.8.0_144-b01, Oracle Corporation, Java HotSpot(TM) Server VM
Screen: :0.0 1280x800
Maximum Screen Size: 1280x800
VM arguments: [-Djosm.restart=true]
Dataset consistency test: No problems found

Plugins:
+ Mapillary (v1.5.5)
+ OpeningHoursEditor (33185)
+ apache-commons (32994)
+ apache-http (32699)
+ buildings_tools (33004)
+ ejml (32680)
+ geotools (33380)
+ jts (32699)
+ measurement (33088)
+ opendata (33438)
+ reverter (33088)
+ tagging-preset-tester (33004)
+ todo (30200)
+ turnrestrictions (33088)
+ undelete (33480)
+ utilsplugin2 (33328)
+ wikipedia (33288)

Tagging presets:
+ ${HOME}/josm/my.presets.xml
+ ${HOME}/name-suggestion-index/name-suggestions.presets.xml

Map paint styles:
+ https://josm.openstreetmap.de/josmfile?page=Styles/TigerReviewedNo&zip=1
- https://josm.openstreetmap.de/josmfile?page=Styles/Surface&zip=1
- https://josm.openstreetmap.de/josmfile?page=Styles/Power&zip=1
- https://josm.openstreetmap.de/josmfile?page=Styles/PowerMapping&zip=1
- https://josm.openstreetmap.de/josmfile?page=Styles/AddressValidator&zip=1
+ ${HOME}/josm/Style.BuildingYes.mapcss
- https://josm.openstreetmap.de/josmfile?page=Styles/Noname&zip=1

Validator rules:
+ ${HOME}/josm/my.validator.mapcss

Last errors/warnings:
- W: Unable to convert property z-index to type class java.lang.Float: found -1
- W: Unable to convert property z-index to type class java.lang.Float: found -1
- W: Unable to convert property z-index to type class java.lang.Float: found -1
- W: Unable to convert property z-index to type class java.lang.Float: found -1
- W: Unable to convert property z-index to type class java.lang.Float: found -1
- W: Unable to use English input method
- W: Unable to use English input method
- W: Unable to use English input method
- W: Unable to use English input method
- W: Unable to use English input method

[Don-vip]

By following the bug description I got no error, can you please attach a sample .osm file? Thanks.

[maxerickson@…]

sample data

[maxerickson@…]

screenshot

[Klumbumbus]

I too get no error even with your sample file. Please check if you still get the error when you disable your custom mappaint style ${HOME}/josm/Style.BuildingYes.mapcss and your custom rule ${HOME}/josm/my.validator.mapcss.

[maxerickson@…]

Deactivated (but did not remove) the local customizations. Also removed custom presets. Restarted and still got the spurious warning.

[Klumbumbus]

What happens if you download and validate another track multipolygon, e.g. at ​https://www.openstreetmap.org/#map=19/50.70419/12.84406?

[maxerickson@…]

Same warning. My initial report was after I had seen it several times, also on other leisure multipolygons (a golf course).

I doubt it matters, but additional information not included above: My JRE is installed via AUR.

[Klumbumbus]

Do you get a similar warning for ​https://www.openstreetmap.org/relation/7005977?

[Don-vip]

I don't see how the mapCSS rule:

node[tag("leisure") = parent_tag("leisure")] ∈ *[leisure] {
  throwWarning: tr("{0} inside {1}", concat("leisure=", tag("leisure")), concat("leisure=", tag("leisure")));
  group: tr("{0} inside {1}", "leisure", "leisure");
}

can issue a warning if the node does not have any leisure tag. You're sure about that I imagine?

[Don-vip]

Replying to maxerickson@…:

My JRE is installed via AUR.

Can you please try with another JRE?

[maxerickson@…]

Switched to the Arch OpenJDK package. ​https://www.openstreetmap.org/relation/7005977 warns for building and for tourism.

Before switching I had ran with the Oracle JRE and --skip-plugins and also still had the error.

[maxerickson@…]

Using https://josm.openstreetmap.de/download/, the spurious warning does not occur until 12542, it isn't present in 12539.

(using command lines like "java -jar josm-snapshot-12539.jar --skip-plugins", having switched various things back on and back to the HotSpot VM)

[maxerickson@…]

And the error no longer occurs in 12556. ¯\_(ツ)_/¯

[Don-vip]

OK understood, I messed up in r12541 and fixed the error a few hours too late in r12544. Unfortunately you got a buggy build in between. Thanks for the report !

[Klumbumbus]

Ticket #15105 has been marked as a duplicate of this ticket.

[Klumbumbus]

Ticket #15132 has been marked as a duplicate of this ticket.

[Klumbumbus]

Ticket #15144 has been marked as a duplicate of this ticket.