source: josm/trunk/resources/data/validator/highway.mapcss@ 18548

Last change on this file since 18548 was 18523, checked in by taylor.smock, 2 years ago

Fix #21235: Stop whitelisting footway=*, and add footway=separate to blacklist candidates for sidewalk=* (patch by JeroenHoek)

Patch notes by JeroenHoek (modified):

  • Add a check for footway=separate.
  • Remove special treatment for footway=none, none is added to the general footway=* check.
  • The whitelist for footway=* is removed.
  • Property svn:eol-style set to native
File size: 11.4 KB
Line 
1way[highway=~/^(motorway|trunk|primary|secondary|tertiary)$/] {
2 set major_road;
3}
4way[highway=~/^.*_link$/] {
5 set link_road;
6}
7way[highway=~/^(unclassified|residential|living_street|service)$/] {
8 set minor_road;
9}
10
11/* r3669, #9384, #20117 */
12way[highway][name =~ /(?i).* (Ave|Blvd|Bnd|Br|Brg|Cct|Cir|Cl|Cr|Crct|Cres|Crt|Ct|Cv|Dr|Drv|Esp|Espl|Hwy|Ln|Mw|Mwy|Pky|Pkwy|Pl|Rd|Qy|Qys|Sq|St|Str|Ter|Tce|Tr|Trl|Vw|Wy|Xing)[.]?$/] {
13 throwWarning: tr("abbreviated street name");
14 assertMatch: "way highway=unclassified name=\"Foo Ave\"";
15 assertMatch: "way highway=unclassified name=\"Foo Ave.\"";
16 assertMatch: "way highway=unclassified name=\"Bou Blvd.\"";
17 assertMatch: "way highway=unclassified name=\"Bou blvd.\"";
18}
19
20node[highway =~ /motorway|trunk|primary|secondary|tertiary|unclassified|residential|service|living_street|pedestrian|track|path|footway|cycleway|busway|bus_guideway|bridleway/][highway!=motorway_junction][highway!=services] {
21 throwWarning: tr("wrong highway tag on a node");
22 assertMatch: "node highway=primary";
23 assertMatch: "node highway=primary_link";
24 assertNoMatch: "node highway=traffic_calming";
25 assertNoMatch: "node highway=bus_stop";
26 assertNoMatch: "node highway=crossing";
27 assertNoMatch: "node highway=emergency_access_point";
28 assertNoMatch: "node highway=give_way";
29 assertNoMatch: "node highway=mini_roundabout";
30 assertNoMatch: "node highway=motorway_junction";
31 assertNoMatch: "node highway=passing_place";
32 assertNoMatch: "node highway=rest_area";
33 assertNoMatch: "node highway=speed_camera";
34 assertNoMatch: "node highway=street_lamp";
35 assertNoMatch: "node highway=services";
36 assertNoMatch: "node highway=stop";
37 assertNoMatch: "node highway=traffic_signals";
38 assertNoMatch: "node highway=turning_circle";
39}
40
41way[highway=crossing],
42way[railway=crossing],
43way[railway=level_crossing] {
44 throwWarning: tr("wrong crossing tag on a way");
45 assertMatch: "way highway=crossing";
46 assertNoMatch: "node highway=crossing";
47}
48
49/* #18060 */
50node[footway=crossing] {
51 throwWarning: tr("{0} on a node", "{0.tag}");
52 suggestAlternative: "highway=crossing";
53 suggestAlternative: "railway=crossing";
54}
55node[cycleway=crossing] {
56 throwWarning: tr("{0} on a node", "{0.tag}");
57 suggestAlternative: "highway=crossing + bicycle=yes";
58 suggestAlternative: "railway=crossing + bicycle=yes";
59}
60
61way[highway=unclassified][!name][noname!=yes] {
62 throwOther: tr("Unnamed unclassified highway");
63 assertMatch: "way highway=unclassified";
64 assertNoMatch: "way highway=unclassified noname=yes";
65 assertNoMatch: "way highway=unclassified name=Foo";
66}
67
68/* #17970 */
69relation[type=route][route=road][ref] > way.major_road {
70 set road_route_has_ref;
71}
72way.major_road!.road_route_has_ref[!ref][!destination:ref][noref!=yes][junction!=roundabout] {
73 throwOther: tr("highway without a reference");
74 group: tr("missing tag");
75 assertMatch: "way highway=primary";
76 assertNoMatch: "way highway=primary ref=123";
77 assertNoMatch: "way highway=primary destination:ref=123";
78 assertNoMatch: "way highway=primary noref=yes";
79}
80
81way[highway=yes],
82way[highway=road] {
83 throwWarning: tr("Unspecific highway type");
84 assertMatch: "way highway=road";
85 assertNoMatch: "way highway=residential";
86}
87
88way[highway=footway][maxspeed],
89way[highway=steps][maxspeed],
90way[highway=cycleway][bicycle=no],
91way[highway=footway][foot=no],
92way[highway=cycleway][cycleway=lane] {
93 throwWarning: tr("{0} together with {1}", "{0.tag}", "{1.tag}");
94 assertNoMatch: "way highway=cycleway bicycle=yes";
95 assertNoMatch: "way highway=cycleway";
96 assertNoMatch: "way highway=footway foot=yes";
97 assertNoMatch: "way highway=footway";
98 assertMatch: "way highway=cycleway cycleway=lane";
99 assertNoMatch: "way highway=cycleway";
100 assertNoMatch: "way highway=residential cycleway=lane";
101 assertMatch: "way highway=footway maxspeed=20";
102 assertNoMatch: "way highway=residential maxspeed=20";
103 assertNoMatch: "way highway=footway";
104}
105
106/* footway, see #10851, #10976, #15439, #19389, #21235 */
107way[footway=left][/^footway:/],
108way[footway=right][/^footway:/],
109way[footway=both][/^footway:/],
110way[footway=separate][/^footway:/],
111way[footway=no][/^footway:/] {
112 throwWarning: tr("{0} is deprecated, use {1} instead. Also check similar tags like {2}", "{0.tag}", "sidewalk", "{1.key}");
113 group: tr("deprecated tagging");
114 set not_fixable_footway;
115 assertMatch: "way footway=both footway:surface=asphalt";
116 assertMatch: "way footway=separate footway:surface=asphalt";
117}
118way[footway=none][/^footway:/] {
119 throwWarning: tr("{0} is deprecated, use {1} instead. Also check similar tags like {2}", "{0.tag}", "sidewalk=no", "{1.key}");
120 group: tr("deprecated tagging");
121 set not_fixable_footway;
122 assertMatch: "way footway=none footway:surface=asphalt";
123 assertNoMatch: "way footway=no footway:surface=asphalt";
124}
125
126way[footway=left]!.not_fixable_footway,
127way[footway=right]!.not_fixable_footway,
128way[footway=both]!.not_fixable_footway,
129way[footway=separate]!.not_fixable_footway,
130way[footway=no]!.not_fixable_footway {
131 throwWarning: tr("{0} is deprecated", "{0.tag}");
132 suggestAlternative: "sidewalk";
133 group: tr("deprecated tagging");
134 fixChangeKey: "footway => sidewalk";
135 set fixable_footway;
136}
137
138way[footway=none]!.not_fixable_footway {
139 throwWarning: tr("{0} is deprecated", "{0.tag}");
140 suggestAlternative: "sidewalk=no";
141 group: tr("deprecated tagging");
142 fixRemove: "footway";
143 fixAdd: "sidewalk=no";
144 set fixable_footway;
145}
146
147/* #11270, #9297, #15439, #16424, #17022 */
148way[railway =~ /^(miniature|narrow_gauge|preserved|rail)$/][!highway] > node { set .is_in_railway }
149way[highway =~ /^(bridleway|cycleway|footway|path|steps)$/] > node { set .is_in_minor_road }
150way[highway =~ /^(busway|bus_guideway|living_street|motorway|motorway_link|pedestrian|primary|primary_link|raceway|residential|road|secondary|secondary_link|service|tertiary|tertiary_link|track|trunk|trunk_link|unclassified)$/] > node { set .is_in_major_road }
151node[railway!=crossing][crossing!=no].is_in_railway.is_in_minor_road!.is_in_major_road {
152 throwWarning: tr("pedestrian railway crossing without {0}", "{0.tag}");
153 group: tr("missing tag");
154 fixAdd: "railway=crossing";
155}
156node[railway!=level_crossing].is_in_railway.is_in_major_road!.is_in_minor_road {
157 throwWarning: tr("railway crossing without {0}", "{0.tag}");
158 group: tr("missing tag");
159 fixAdd: "railway=level_crossing";
160}
161
162node[highway=crossing][barrier=kerb].is_in_major_road {
163 throwWarning: tr("{0} together with {1}", "{0.tag}", "{1.tag}");
164 suggestAlternative: "kerb=*";
165 group: tr("suspicious tag combination");
166}
167
168/* #20987 */
169way[turn ][turn !~ /^(none|((sharp_|slight_|merge_to_|slide_)?(left|right)|reverse|through)(;((sharp_|slight_|merge_to_|slide_)?(left|right)|reverse|through))*)$/],
170way[turn:forward ][turn:forward !~ /^(none|((sharp_|slight_|merge_to_|slide_)?(left|right)|reverse|through)(;((sharp_|slight_|merge_to_|slide_)?(left|right)|reverse|through))*)$/],
171way[turn:backward ][turn:backward !~ /^(none|((sharp_|slight_|merge_to_|slide_)?(left|right)|reverse|through)(;((sharp_|slight_|merge_to_|slide_)?(left|right)|reverse|through))*)$/],
172way[turn:both_ways ][turn:both_ways !~ /^(none|((sharp_|slight_|merge_to_|slide_)?(left|right)|reverse|through)(;((sharp_|slight_|merge_to_|slide_)?(left|right)|reverse|through))*)$/],
173way[turn:both_ways:forward ][turn:both_ways:forward !~ /^(none|((sharp_|slight_|merge_to_|slide_)?(left|right)|reverse|through)(;((sharp_|slight_|merge_to_|slide_)?(left|right)|reverse|through))*)$/],
174way[turn:both_ways:backward ][turn:both_ways:backward !~ /^(none|((sharp_|slight_|merge_to_|slide_)?(left|right)|reverse|through)(;((sharp_|slight_|merge_to_|slide_)?(left|right)|reverse|through))*)$/],
175way[turn:lanes ][turn:lanes !~ /^(|none|((sharp_|slight_|merge_to_|slide_)?(left|right)|reverse|through)(;((sharp_|slight_|merge_to_|slide_)?(left|right)|reverse|through))*)(\|(|none|((sharp_|slight_|merge_to_|slide_)?(left|right)|reverse|through)(;((sharp_|slight_|merge_to_|slide_)?(left|right)|reverse|through))*))*$/],
176way[turn:lanes:forward ][turn:lanes:forward !~ /^(|none|((sharp_|slight_|merge_to_|slide_)?(left|right)|reverse|through)(;((sharp_|slight_|merge_to_|slide_)?(left|right)|reverse|through))*)(\|(|none|((sharp_|slight_|merge_to_|slide_)?(left|right)|reverse|through)(;((sharp_|slight_|merge_to_|slide_)?(left|right)|reverse|through))*))*$/],
177way[turn:lanes:backward ][turn:lanes:backward !~ /^(|none|((sharp_|slight_|merge_to_|slide_)?(left|right)|reverse|through)(;((sharp_|slight_|merge_to_|slide_)?(left|right)|reverse|through))*)(\|(|none|((sharp_|slight_|merge_to_|slide_)?(left|right)|reverse|through)(;((sharp_|slight_|merge_to_|slide_)?(left|right)|reverse|through))*))*$/],
178way[turn:lanes:both_ways ][turn:lanes:both_ways !~ /^(|none|((sharp_|slight_|merge_to_|slide_)?(left|right)|reverse|through)(;((sharp_|slight_|merge_to_|slide_)?(left|right)|reverse|through))*)(\|(|none|((sharp_|slight_|merge_to_|slide_)?(left|right)|reverse|through)(;((sharp_|slight_|merge_to_|slide_)?(left|right)|reverse|through))*))*$/],
179way[turn:lanes:both_ways:forward ][turn:lanes:both_ways:forward !~ /^(|none|((sharp_|slight_|merge_to_|slide_)?(left|right)|reverse|through)(;((sharp_|slight_|merge_to_|slide_)?(left|right)|reverse|through))*)(\|(|none|((sharp_|slight_|merge_to_|slide_)?(left|right)|reverse|through)(;((sharp_|slight_|merge_to_|slide_)?(left|right)|reverse|through))*))*$/],
180way[turn:lanes:both_ways:backward][turn:lanes:both_ways:backward !~ /^(|none|((sharp_|slight_|merge_to_|slide_)?(left|right)|reverse|through)(;((sharp_|slight_|merge_to_|slide_)?(left|right)|reverse|through))*)(\|(|none|((sharp_|slight_|merge_to_|slide_)?(left|right)|reverse|through)(;((sharp_|slight_|merge_to_|slide_)?(left|right)|reverse|through))*))*$/] {
181 throwWarning: tr("unusual value of {0}", "{0.key}");
182 assertMatch: "way turn=straight"; /* through would be correct */
183 assertMatch: "way turn=slight_reverse"; /* wrong value */
184 assertMatch: "way turn=through|right"; /* :lanes missing in key */
185 assertNoMatch: "way turn=through;right";
186 assertMatch: "way turn:lanes:forward=straight|right"; /* through would be correct */
187 assertMatch: "way turn:lanes:forward=slight_reverse|right"; /* wrong value */
188 assertMatch: "way turn:lanes=left;none|right"; /* "none" needs to be a solo value */
189 assertNoMatch: "way turn:lanes:forward=sharp_left;left||left;through;slight_right|slight_right;right|";
190}
191
192/* #20442 traffic signals or similar on roundabouts */
193node[highway=traffic_signals ][ traffic_signals=signal] < way[highway][junction=roundabout],
194node[highway=~/^(traffic_signals|stop|give_way)$/][!traffic_signals ] < way[highway][junction=roundabout] {
195 throwWarning: tr("{0} without right of way", "{1.tag}");
196 suggestAlternative: "junction=circular";
197}
198
199/* #18217 */
200way[highway =~ /^((motorway|trunk|primary|secondary|tertiary)(_link)?|residential|unclassified)$/][area=yes],
201relation[highway =~ /^((motorway|trunk|primary|secondary|tertiary)(_link)?|residential|unclassified)$/][type=multipolygon] {
202 throwError: tr("Area with {0} above {1} is invalid", "highway=*", "highway=service");
203 group: tr("suspicious tag combination");
204 suggestAlternative: "area:highway=*";
205 assertMatch: "way highway=trunk area=yes";
206 assertMatch: "relation highway=trunk type=multipolygon";
207 assertNoMatch: "way highway=service area=yes";
208 assertNoMatch: "way highway=trunk";
209}
Note: See TracBrowser for help on using the repository browser.