Modify

Opened 6 years ago

Closed 6 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 Changed 6 years ago by Don-vip

Milestone: 14.02

comment:2 Changed 6 years ago by naoliv

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 Changed 6 years ago by mkoniecz

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

comment:4 Changed 6 years ago by mkoniecz

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

comment:5 Changed 6 years ago by simon04

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.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.