Opened 9 years ago
Closed 7 years ago
#10976 closed enhancement (fixed)
Fix subtags together with footway=both/left/right
Reported by: | skyper | Owned by: | team |
---|---|---|---|
Priority: | normal | Milestone: | 16.12 |
Component: | Core validator | Version: | |
Keywords: | fix footway sidewalk | Cc: |
Description
There are some subtags of footway=both/left/right
like footway:surface
or footway:width
plus width:footway
, width:footway
and similar.
Please, with changing footway
to sidewalk
also change all other keys, too.
Attachments (1)
Change History (9)
comment:1 by , 9 years ago
comment:2 by , 9 years ago
Replying to Klumbumbus:
Can you please be a bit more precise?
Sorry, yes
In validator you get a warning about footway=both/left/right
and an option to automatically fix it changing the key to sidewalk
but there could be subtags for the footway
which should be changed to like surface:footway=*
which need to be adjusted, too.
Have a look at attached example file
comment:3 by , 7 years ago
Still get these warnings and always have to be really careful to check all tags before auto-fixing.
follow-up: 6 comment:5 by , 7 years ago
Auto fixing the other tags is a bit more complex and not possible in mapcss I think. I don't think someone has time for a Java test atm. So I disabled the autofix for now if another key which contains footway: is present.
comment:6 by , 7 years ago
Replying to Klumbumbus:
Auto fixing the other tags is a bit more complex and not possible in mapcss I think. I don't think someone has time for a Java test atm. So I disabled the autofix for now if another key which contains footway: is present.
Thanks a lot.
comment:7 by , 7 years ago
@Klumbumbus:
I can live well with the current behaviour and the numbers of hits will hopefully drop. Please, go ahead and mark this as fixed.
Can you please be a bit more precise?