Opened 16 months ago

Closed 16 months ago

Last modified 15 months ago

#23604 closed defect (fixed)

[Patch] false warning: combination of sport and building:part — at Version 3

Reported by: Fred73000 Owned by: GerdP
Priority: normal Milestone: 24.03
Component: Core validator Version:
Keywords: combination sport building:part Cc:

Description (last modified by skyper)

Note : I don't use JOSM so I cannot create a report for this ticket.

The problem in josm is here :
https://josm.openstreetmap.de/browser/josm/trunk/resources/data/validator/combinations.mapcss#L545

The problem is : [!"building:part"] is missing.

To illustrate, see this building https://www.openstreetmap.org/way/71485515

This sports_hall has 2 building parts = one big area where people can play basketball, volleyball, handball. And a smaller room dedicated to boxing. The tag "sport" is on each "building:part" but osmose (and probably josm) shows a warning "sport without physical feature" while a physical feature is present.

Change History (4)

by GerdP, 16 months ago

Attachment: 23604.patch added

comment:1 by GerdP, 16 months ago

Milestone: 24.03
Owner: changed from team to GerdP
Status: newassigned
Summary: false warning in osmose because of a problem in JOSM[Patch] false warning in osmose because of a problem in JOSM

Thanks for the suggested fix. If nobody complains I'll commit the patch tomorrow.

comment:2 by GerdP, 16 months ago

Resolution: fixed
Status: assignedclosed

In 19035/josm:

fix #23604: false warning in osmose because of a problem in JOSM

  • add tag key building:part as a physical feature for sport=*

comment:3 by skyper, 16 months ago

Component: CoreCore validator
Description: modified (diff)
Keywords: combination sport building:part added
Summary: [Patch] false warning in osmose because of a problem in JOSM[Patch] false warning: combination of sport and building:part

I guess it does not matter for the assertNoMatch but building:part on a node is wrong, in general, and way would be a better choice.

Note: See TracTickets for help on using tickets.