Opened 11 years ago

Last modified 11 years ago

#10851 closed defect

Validation tests for foot/footway/sidewalk — at Version 9

Reported by: naoliv Owned by: team
Priority: normal Milestone: 14.12
Component: Core validator Version:
Keywords: mapcss Cc: Klumbumbus

Description (last modified by Klumbumbus)

First a problem: validating a way with highway=footway + foot=no gives us a footway used with foot= (note that there isn't a value after the =)

Now the enhancements: highway=residential + sidewalk=both + foot=no doesn't issue a warning (it's as contradictory as using highway=footway + foot=no)

Also, highway=residential + footway=both doesn't warn that footway is deprecated (and that sidewalk should be used instead) done.

Change History (9)

comment:1 by Klumbumbus, 11 years ago

Cc: Klumbumbus added

comment:2 by Klumbumbus, 11 years ago

Milestone: 14.12
Summary: Validation tests for foot/footwat/sidewalkValidation tests for foot/footway/sidewalk

with highway=foot you mean highway=footway?

in reply to:  2 comment:3 by naoliv, 11 years ago

Replying to Klumbumbus:

with highway=foot you mean highway=footway?

Yes, sorry :-)
Also, sidewalk=both (on second paragraph) can be both|left|right|yes and footway=both (third paragraph) can be both|left|right

comment:4 by naoliv, 11 years ago

Description: modified (diff)

comment:5 by Klumbumbus, 11 years ago

Description: modified (diff)

typos

in reply to:  description comment:6 by Klumbumbus, 11 years ago

Replying to naoliv:

First a problem: validating a way with highway=footway + foot=no gives us a footway used with foot= (note that there isn't a value after the =)

In this case we could fix this by simply changing !? to =no in trunk/data/validator/highway.mapcss#L67, but I made a ticket for this, see #10859

in reply to:  description comment:7 by Klumbumbus, 11 years ago

Replying to naoliv:

Now the enhancements: highway=residential + sidewalk=both + foot=no doesn't issue a warning

It seems some people tag this way when the sidewalks are mapped as separate ways, however I'm not sure if this is good tagging.
Example: way/220407308.
In this case should the sidewalk key set to no or be removed?

Overpass turbo querry: http://overpass-turbo.eu/s/6yN

comment:8 by Klumbumbus, 11 years ago

In 7814/josm:

see #10851 - add validator rules for footway=*

comment:9 by Klumbumbus, 11 years ago

Description: modified (diff)
Note: See TracTickets for help on using tickets.