Modify

Opened 13 years ago

Closed 13 years ago

Last modified 13 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 by simon04, 13 years ago

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 by richlv, 13 years ago

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 by simon04, 13 years ago

Component: CoreInternal preset
Owner: changed from team to ce

comment:4 by simon04, 13 years ago

Resolution: fixed
Status: newclosed

In [4443/josm]:

fix #6803 - remove wire_fence and wood_fence presets

in reply to:  1 comment:5 by simon04, 13 years ago

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. 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.