Opened 11 years ago
Closed 9 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 , 11 years ago
comment:2 by , 11 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 , 9 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 , 9 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 , 9 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 , 9 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?