Changeset 13378 in josm

Timestamp:

2018-02-03T14:37:46+01:00 (7 years ago)

Author:

Klumbumbus

Message:

fix #15667 - adjust warnings for waterway oneway combinations

Location:

trunk/data/validator

Files:

• deprecated.mapcss (modified) (2 diffs)
• unnecessary.mapcss (modified) (1 diff)

trunk/data/validator/deprecated.mapcss

@@ -538 +538 @@
 }
 
-/* see #10346 */
-way[oneway=1] {
+/* see #10346, #15667 (waterway has its own rules in unnecessary.mapcss) */
+way[oneway=1][!waterway] {
   throwWarning: tr("{0} is deprecated", "{0.tag}");
   suggestAlternative: "oneway=yes";
@@ -546 +546 @@
 }
 
-/* see #11329 */
-way[oneway=-1] {
+/* see #11329, #15667 */
+way[oneway=-1][!waterway] {
   throwWarning: tr("{0} is not recommended. Use the Reverse Ways function from the Tools menu.", "{0.tag}");
 }

trunk/data/validator/unnecessary.mapcss

@@ -54 +54 @@
   fixRemove: "payment:cash";
 }
+
+/* see #10346, #15667 */
+way[waterway][oneway?] {
+  throwWarning: tr("{0} is unnecessary for {1}", "{1.key}", "{0.key}");
+  group: tr("unnecessary tag");
+  fixRemove: "{1.key}";
+}
+way[waterway][oneway=-1] {
+  throwWarning: tr("{0} is unnecessary for {1}. The flow direction is defined by the way direction.", "{1.key}", "{0.key}");
+  group: tr("unnecessary tag");
+}