Modify

Opened 10 years ago

Closed 10 years ago

#9614 closed enhancement (fixed)

Verify for unneeded area=yes [validator rule included]

Reported by: naoliv Owned by: team
Priority: normal Milestone: 14.02
Component: Core validator Version:
Keywords: Cc:

Description

Could JOSM include a test for undeeded area=yes?
For example, a closed polygon with landuse=residential doesn't need area=yes
nor leisure=park, natural=wood, amenity=parking, building=yes, etc

Attachments (0)

Change History (5)

comment:1 by Don-vip, 10 years ago

Milestone: 14.02

comment:2 by naoliv, 10 years ago

Don't know if there is a more elegant way for this, but it should be this:

area:closed[amenity ][area?],
area:closed[building][area?],
area:closed[landuse ][area?],
area:closed[leisure ][area?],
area:closed[natural ][area?],
area:closed[shop    ][area?] {
        throwWarning: tr("unneeded {0}={1} in {2}", "{2.key}", tag("area"), "{1.key}");
        fixRemove: "{2.key}";
}

comment:3 by mkoniecz, 10 years ago

Summary: Verify for unneeded area=yesVerify for unneeded area=yes [patch included]

comment:4 by mkoniecz, 10 years ago

Summary: Verify for unneeded area=yes [patch included]Verify for unneeded area=yes [validator rule included]

comment:5 by simon04, 10 years ago

Resolution: fixed
Status: newclosed

In 6778/josm:

fix #9614 - Check and remove unnecessary area=yes - patch by naoliv

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.