Modify

Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#15746 closed defect (fixed)

Validator wrong message, "area is unnecessary for leisure"

Reported by: dieterdreist Owned by: team
Priority: normal Milestone: 18.01
Component: Core validator Version:
Keywords: area leisure track Cc: Klumbumbus

Description

How would JOSM know about all leisure tags? Area=yes/no is unnecessary for leisure=pitch/swimming_pool/beach_resort and a lot of others, but it is necessary for leisure=<user defined> and leisure=track.

Attachments (0)

Change History (5)

comment:1 by Don-vip, 7 years ago

Checks is defined as follows:

area:closed[amenity ][area?][!highway],
area:closed[building][area?],
area:closed[landuse ][area?][!highway],
area:closed[leisure ][area?][!highway],
area:closed[natural ][area?],
area:closed[shop    ][area?] {
  throwWarning: tr("{0} is unnecessary for {1}", "{2.key}", "{1.key}");
  group: tr("unnecessary tag");
  fixRemove: "{2.key}";
}

comment:2 by Don-vip, 7 years ago

Keywords: area leisure added

comment:3 by Don-vip, 7 years ago

Cc: Klumbumbus added
Keywords: track added

We should probably disable the test for leisure=track

comment:4 by Klumbumbus, 7 years ago

Resolution: fixed
Status: newclosed

In 13302/josm:

fix #15746 - don't warn about area=* on leisure=track and leisure=slipway

comment:5 by Klumbumbus, 7 years ago

Milestone: 18.01
Version: latest

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.