Opened 8 years ago
Closed 6 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 Changed 8 years ago by
comment:2 Changed 8 years ago by
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 Changed 6 years ago by
Still get these warnings and always have to be really careful to check all tags before auto-fixing.
comment:5 follow-up: 6 Changed 6 years ago by
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 Changed 6 years ago by
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 Changed 6 years ago by
@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?