[Patch] footway=* instead of sidewalk=* on roads

[Famlam]

​The key footway is quite often incorrectly used when (in most cases, most likely) sidewalk=* is meant.

Perhaps it is an idea to make the validator complain about "disallowed" highway values combined with footway=*? (Formally it's allowed on highway=footway/path/construction only, although it seems quite a common tag to use on highway=cycleway too)
​overpass-turbo for the area around The Netherlands

If not, then it might be worth adding a check for footway=* without highway=* tag (or prefixed highway keys)? ​Overpass-turbo

/* 19603 */
way[footway][highway][footway=sidewalk][highway!=footway][highway!=path][highway!=construction][highway!=cycleway] {
  throwWarning: tr("{0} together with {1}", "{0.tag}", "{1.tag}");
  group: tr("suspicious tag combination");
  suggestAlternative: "sidewalk=left";
  suggestAlternative: "sidewalk=right";
  suggestAlternative: "sidewalk=both";
}
/* 19603 */
way[footway][highway][footway!=sidewalk][highway!=footway][highway!=path][highway!=construction][highway!=cycleway] {
  throwWarning: tr("{0} together with {1}", "{0.tag}", "{1.tag}");
  group: tr("suspicious tag combination");
}

/* 19603 */
way[footway][!highway][!/:highway$/],
way[cycleway][!highway][!/:highway$/] {
  throwWarning: tr("{0} without {1}", "{0.key}", "{1.key}");
  set MissingKeyWarning;
  group: tr("missing tag");
}

[skyper]

I use path=sidewalk. One reason that cycleway=sidewalk is not used is the validator warning about it.

Nowadays, with *=use_sidepath the proper tag for all three might be side_path=yes instead of *=sidewalk but that needs discussion.

[anonymous]

@skyper, I didn't fully understand your reply (side_path?), but to clarify: this patch doesn't modify tags on i.e. highway=path/cycleway/footway.

It just adds a warning on i.e. highway=residential + footway=sidewalk.
Here the correct way to map it would be sidewalk=*

[skyper]

Replying to anonymous:

@skyper, I didn't fully understand your reply (side_path?)

Sorry, should have quoted:

Replying to Famlam:

Perhaps it is an idea to make the validator complain about "disallowed" highway values combined with footway=*? (Formally it's allowed on highway=footway/path/construction only, although it seems quite a common tag to use on highway=cycleway too)

What I think is that instead of footway=sidewalk or cycleway=sidewalk a new tag sidepath=yes would fit to highway=cycleway/footway/path and bicycle/foot=use_sidepath. We would not need footway=sidewalk on cycleways or three different tags cycleway/footway/path=sidewalk.

but to clarify: this patch doesn't modify tags on i.e. highway=path/cycleway/footway.
It just adds a warning on i.e. highway=residential + footway=sidewalk.
Here the correct way to map it would be sidewalk=*

Yes, sure, did not want to criticize your suggestions.

[Klumbumbus]

It was already on my todo list.

[Klumbumbus]

In 17051/josm:

fix #19603 - Warn about suspicious tag combinations with footway=* (based on patch by Famlam)