source: josm/trunk/data/validator/geometry.mapcss@ 7793

Last change on this file since 7793 was 7793, checked in by Klumbumbus, 10 years ago

fix #10825 - add validator warning for tags on isolated nodes, which should be part of a way

File size: 4.5 KB
Line 
1/* {0} on a node, should be a way */
2node[oneway],
3node[bridge?],
4node[sidewalk],
5node[waterway=river],
6node[waterway=riverbank],
7node[waterway=canal],
8node[waterway=wadi],
9node[waterway=stream],
10node[waterway=ditch],
11node[natural=coastline],
12node[natural=ridge],
13node[natural=tree_row] {
14 throwWarning: tr("{0} on a node. Should be used on a way.", "{0.key}");
15 assertMatch: "node oneway=-1";
16 assertNoMatch: "way oneway=-1";
17 assertMatch: "node bridge=yes";
18 assertNoMatch: "node bridge=13";
19}
20
21/* {0} on a node, should be an area; see ticket #10679 */
22node[landuse],
23node[natural=scree],
24node[natural=scrub],
25node[natural=fell],
26node[natural=heath],
27node[natural=wood],
28node[natural=grassland],
29node[natural=wetland],
30node[natural=water],
31node[natural=mud],
32node[natural=beach],
33node[natural=sand],
34node[natural=wood],
35node[natural=bare_rock],
36node[waterway=riverbank],
37node[source:outline] {
38 throwWarning: tr("{0} on a node. Should be drawn as an area.", "{0.tag}");
39}
40
41/* {0} on a node, should be a relation; see #10252, #10769 */
42node[route],
43node[restriction] {
44 throwWarning: tr("{0}=* on a node. Should be used in a relation", "{0.key}");
45}
46
47/* {0} on a way, should be a relation; see #10252 */
48way[route=bus] {
49 throwWarning: tr("{0} on a way. Should be used in a relation", "{0.tag}");
50}
51
52/* see ticket:#10125 */
53node[source:geometry] {
54 throwWarning: tr("{0} on a node", "{0.key}");
55 fixChangeKey: "source:geometry => source:position";
56}
57
58/* Building inside building (spatial test) */
59*[building][building!~/no|entrance/][any(tag("layer"),"0") = any(parent_tag("layer"),"0")]
60*[building][building!~/no|entrance/] {
61 throwWarning: tr("Building inside building");
62}
63
64/* Overlapping areas (spatial test) */
65area[natural =~ /^(water|wetland|coastline)$/], area[landuse=reservoir] {
66 set water_area;
67}
68
69/* area:closed:areaStyle.water_area ⧉ area:closed:areaStyle.water_area -- does not work for now -- see ticket#10215 */
70area:closed:areaStyle[natural =~ /^(water|wetland|coastline)$/] area:closed:areaStyle.water_area,
71area:closed:areaStyle[landuse=reservoir] area:closed:areaStyle.water_area {
72 throwWarning: tr("Overlapping Water Areas");
73}
74
75area:closed:areaStyle area:closed:areaStyle {
76 throwOther: tr("Overlapping Areas");
77}
78
79/* see ticket #9311 */
80node[amenity=parking]["capacity:disabled" !~ /^(yes|[0-9]+)$/] *[amenity=parking] {
81 throwWarning: tr("{0} inside {1}", "amenity=parking", "amenity=parking");
82}
83
84/* see ticket #9556 */
85area:closed:areaStyle[tag("natural") = parent_tag("natural")] area:closed:areaStyle[natural] {
86 throwWarning: tr("Overlapping Identical Natural Areas");
87}
88
89area:closed:areaStyle[tag("landuse") = parent_tag("landuse")] area:closed:areaStyle[landuse] {
90 throwWarning: tr("Overlapping Identical Landuses");
91}
92
93/* see ticket:#9522 */
94node[tag("amenity") = parent_tag("amenity")] *[amenity][amenity != parking] {
95 throwWarning: tr("{0} inside {1}", concat("amenity=", tag("amenity")), concat("amenity=", tag("amenity")));
96}
97node[tag("leisure") = parent_tag("leisure")] *[leisure] {
98 throwWarning: tr("{0} inside {1}", concat("leisure=", tag("leisure")), concat("leisure=", tag("leisure")));
99}
100node[tag("tourism") = parent_tag("tourism")] *[tourism] {
101 throwWarning: tr("{0} inside {1}", concat("tourism=", tag("tourism")), concat("tourism=", tag("tourism")));
102}
103node[tag("shop") = parent_tag("shop")] *[shop] {
104 throwWarning: tr("{0} inside {1}", concat("shop=", tag("shop")), concat("shop=", tag("shop")));
105}
106node[tag("power") = parent_tag("power")] *[power] {
107 throwWarning: tr("{0} inside {1}", concat("power=", tag("power")), concat("power=", tag("power")));
108}
109
110/* isolated nodes which should be part of a way, see ticket:#10825 */
111node:unconnected[barrier=entrance],
112node:unconnected[entrance],
113node:unconnected[traffic_calming],
114node:unconnected[highway=passing_place],
115node:unconnected[highway=mini_roundabout],
116node:unconnected[highway=motorway_junction],
117node:unconnected[highway=turning_loop],
118node:unconnected[highway=turning_circle],
119node:unconnected[highway=traffic_signals],
120node:unconnected[highway=crossing],
121node:unconnected[crossing],
122node:unconnected[railway=crossing],
123node:unconnected[railway=level_crossing],
124node:unconnected[railway=buffer_stop],
125node:unconnected[noexit],
126node:unconnected[waterway=dam],
127node:unconnected[waterway=weir],
128node:unconnected[waterway=waterfall],
129node:unconnected[amenity=ferry_terminal],
130node:unconnected[mountain_pass=yes] {
131 throwWarning: tr("{0} must be connected to a way", "{1.tag}");
132}
Note: See TracBrowser for help on using the repository browser.