Modify

Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#6803 closed enhancement (fixed)

remove wire_fence and wood_fence presets

Reported by: richlv Owned by: ce
Priority: normal Milestone:
Component: Internal preset Version: tested
Keywords: Cc:

Description

#6220 added wire_fence and wood_fence.
these are considered to be very bad choices by some, as they endlessly proliferate keys/tags.

a much better approach would be barrier=fence coupled with fence=wood, fence=wire etc

otherwise there is no limit to what could be stuffed in the barrier tag - barrier=wood_fence_with_3_cm_openings...

this makes renderer and other software authors' life much harder, as they can't know in general that barrier=fence is a fence - they have to consider about gazillion of other possibilities

Attachments (0)

Change History (5)

comment:1 Changed 8 years ago by simon04

Before changing the preset, we should take into consideration the actual number of tag usages:

Probably we should discuss how to deal with deprecated but common tags in general.

comment:2 Changed 8 years ago by richlv

i'd guess those existing values come from previous presets ;)

general handling principles would be great, but might be better solved in a separate ticket

comment:3 Changed 8 years ago by simon04

Component: CoreInternal preset
Owner: changed from team to ce

comment:4 Changed 8 years ago by simon04

Resolution: fixed
Status: newclosed

In [4443/josm]:

fix #6803 - remove wire_fence and wood_fence presets

comment:5 in reply to:  1 Changed 8 years ago by simon04

Replying to simon04:

Probably we should discuss how to deal with deprecated but common tags in general.

A validation test for deprecated tags was added in r4442.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain ce.
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.