Modify

Opened 5 years ago

Closed 5 years ago

#17507 closed enhancement (fixed)

additional values for barrier = wall

Reported by: ma-rt-in Owned by: team
Priority: normal Milestone: 19.04
Component: Internal preset Version: latest
Keywords: barrier wall flood Cc: Klumbumbus

Description (last modified by Klumbumbus)

I ask for extension of the possible values for Tag:barrier=wall.
wall=flood_wall | de: Hochwasserschutzwand

Best regards

Attachments (0)

Change History (9)

comment:1 by Don-vip, 5 years ago

Keywords: barrier wall flood added

comment:2 by Don-vip, 5 years ago

Cc: Klumbumbus added

I agree with this:

the barrier=wall tag should be for the types of walls just like fence_type=*. For example, wall=dry_stone, wall=brick, and wall=flint should be removed from this tag as they better fit as material=* values. wall=yes and wall=no are perfectly fine as it can also describe details of building walls. Although I think we need a better wall type for regular brick or concrete barrier walls. wall=yes or wall=solid either in conjunction with a material could be a solution but I feel like wall=solid instead of wall=yes could be the best option for a generic type of barrier wall. --Mxdanger (talk) 21:59, 15 January 2018 (UTC)

The would mean for us:

  • remove wall=dry_stone from presets
  • keep wall=noise_barrier
  • add wall=castle_wall, wall=seewall, wall=flood_wall
  • deprecate wall=dry_stone, wall=brick, wall=concrete, etc. and suggest to use material instead

comment:3 by Klumbumbus, 5 years ago

Yes, the current definition in the wiki is not good "The value should indicate the principal feature of the wall, which could be the type, the purpose, or the building method,..." (mixing up different concepts).

However I'm a bit worried to go the way you suggest as e.g. wall=dry_stone has 104000 uses and is not deprecated in the wiki.

comment:4 by Don-vip, 5 years ago

After more thoughts dry_stone is not a material but a building method... This should be discussed on @tagging with a wiki proposal.

comment:5 by Klumbumbus, 5 years ago

Description: modified (diff)

in reply to:  4 ; comment:6 by Klumbumbus, 5 years ago

Replying to Don-vip:

This should be discussed on @tagging with a wiki proposal.

Yes, that would be the best. Regarding this ticket: should we add castle_wall, seawall and flood_wall for now anyway?

in reply to:  6 comment:7 by Don-vip, 5 years ago

Replying to Klumbumbus:

should we add castle_wall, seawall and flood_wall for now anyway?

I think so :)

comment:8 by Klumbumbus, 5 years ago

Milestone: 19.04

comment:9 by Klumbumbus, 5 years ago

Resolution: fixed
Status: newclosed

In 14958/josm:

fix #17507 - add wall=castle_wall, seawall and flood_wall

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.