Changeset 14943 in josm

Timestamp:

2019-03-30T22:13:30+01:00 (6 years ago)

Author:

Don-vip

Message:

fix #17542 - Check segregated keys before autofixing foot/cycleway access (patch by mkoniecz)

Location:

trunk/data/validator

Files:

• combinations.mapcss (modified) (1 diff)
• unnecessary.mapcss (modified) (1 diff)

trunk/data/validator/combinations.mapcss

@@ -243 +243 @@
 }
 
-/* {0.tag} together with {1.tag}, see #16861, #16147, #10186 */
+/* {0.tag} together with {1.tag}, see #17542, #16861, #16147, #10186 */
 way[segregated][bicycle=no],
 way[segregated][foot=no],
 way[man_made=pipeline][tunnel=flooded],
-way[waterway=canal][tunnel=yes],
-way[highway=footway][bicycle=designated] {
-  throwWarning: tr("{0} together with {1}", "{0.tag}", "{1.tag}");
-  group: tr("suspicious tag combination");
+way[waterway=canal][tunnel=yes] {
+  throwWarning: tr("{0} together with {1}", "{0.tag}", "{1.tag}");
+  group: tr("suspicious tag combination");
+}
+
+way[highway=footway][bicycle=designated],
+way[highway=cycleway][foot=designated] {
+  throwWarning: tr("{0} together with {1}", "{0.tag}", "{1.tag}");
+  group: tr("suspicious tag combination");
+  suggestAlternative: "highway=path";
 }
 

trunk/data/validator/unnecessary.mapcss

@@ -39 +39 @@
 /* #9257, #17482 */
 way[foot =~ /^(yes|designated)$/][highway =~ /^(footway|pedestrian)$/][!access],
-way[bicycle =~ /^(yes|designated)$/][highway=cycleway][!vehicle][!access],
+way[bicycle =~ /^(yes|designated)$/][highway=cycleway][!vehicle][!access][foot!=designated],
 *[building:ruian:type=2][building=farm_auxiliary],
 *[building:ruian:type=7][building=house],