Modify

Opened 10 years ago

Closed 10 years ago

#10140 closed enhancement (fixed)

Add new validation rule- consider building:levels on nodes as invalid

Reported by: Xxzme Owned by: team
Priority: minor Milestone: 14.06
Component: Core validator Version: latest
Keywords: Cc:

Description

Attachments (0)

Change History (12)

comment:1 by Don-vip, 10 years ago

Component: unspecifiedCore validator

comment:2 by bastiK, 10 years ago

There are two parts to this:

(1) building:levels without building or building:part
(2) building on a node.

(1) seems reasonable to me.

However I would reject (2): there are already 743941 nodes tagged with building=*. Even though it isn't allowed according to the wiki, there might be good reasons for this, e.g. if you want to tag the number of levels for a building, but there is no aerial imagery to trace the outline.

comment:3 by bastiK, 10 years ago

Resolution: fixed
Status: newclosed

In 7243/josm:

fixed #10140 - validation rule for building:levels

comment:4 by Don-vip, 10 years ago

Milestone: 14.06

comment:5 by anonymous, 10 years ago

Thanks! Maybe this should be expanded to warn about any 'building:*' without 'building' or 'building:part' ?

comment:6 by Xxzme, 10 years ago

Resolution: fixed
Status: closedreopened

bastiK, well you talk about different case. My case irrelevant to houses mapped as single point.

My original case:
way 1000 contains points 2000, 2001.
way 1000 have tag building=yes
points 2000 and 2001 have tag building:levels

Validator should warn if point that part of way with building=* tagged as building:levels.

comment:7 by anonymous, 10 years ago

Another case:
way 1000 contains points 2000, 2001.
way 1000 tagged as building=* and building:levels=*
points 2000 and (or?) 2001 have tag building:levels=*

comment:8 by anonymous, 10 years ago

Summary: Add new validation ruleAdd new validation rule- consider building:levels on nodes as invalid

comment:9 by bastiK, 10 years ago

@Xxzme: These cases should be covered by the new validation rule.

comment:10 by Xxzme, 10 years ago

Well, but they are not fixed by changeset 7243...

comment:11 by bastiK, 10 years ago

Please explain, what you think is missing.

comment:12 by Don-vip, 10 years ago

Resolution: fixed
Status: reopenedclosed

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.