Changeset 14501 in josm for trunk

Timestamp:

2018-12-03T12:38:12+01:00 (6 years ago)

Author:

GerdP

Message:

see #17057 : revert changes from r14494

Location:

trunk/src/org/openstreetmap/josm/data/validation/tests

Files:

• RelationChecker.java (modified) (3 diffs)
• TurnrestrictionTest.java (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/data/validation/tests/RelationChecker.java

@@ -64 +64 @@
     public static final String ROLE_VERIF_PROBLEM_MSG = tr("Role verification problem");
     private boolean ignoreMultiPolygons;
-    private boolean ignoreTurnRestrictions;
 
     /**
@@ -110 +109 @@
             if (t instanceof MultipolygonTest) {
                 ignoreMultiPolygons = true;
-            }
-            if (t instanceof TurnrestrictionTest) {
-                ignoreTurnRestrictions = true;
+                break;
             }
         }
@@ -128 +125 @@
         if (ignoreMultiPolygons && n.isMultipolygon()) {
             // see #17010: don't report same problem twice
-            return;
-        }
-        if (ignoreTurnRestrictions && n.hasTag("type","restriction")) {
-            // see #17057: TurnrestrictionTest is specialized on this
             return;
         }

trunk/src/org/openstreetmap/josm/data/validation/tests/TurnrestrictionTest.java

@@ -130 +130 @@
             }
         }
-        final String restriction = r.get("restriction");
         if (morefrom) {
-            Severity severity = "no_entry".equals(restriction) ? Severity.OTHER : Severity.ERROR;
-            errors.add(TestError.builder(this, severity, MORE_FROM)
+            errors.add(TestError.builder(this, Severity.ERROR, MORE_FROM)
                     .message(tr("More than one \"from\" way found"))
                     .primitives(r)
@@ -139 +137 @@
         }
         if (moreto) {
-            Severity severity = "no_exit".equals(restriction) ? Severity.OTHER : Severity.ERROR;
-            errors.add(TestError.builder(this, severity, MORE_TO)
+            errors.add(TestError.builder(this, Severity.ERROR, MORE_TO)
                     .message(tr("More than one \"to\" way found"))
                     .primitives(r)
@@ -173 +170 @@
         }
         if (fromWay.equals(toWay)) {
-            Severity severity = "no_u_turn".equals(restriction) ? Severity.OTHER : Severity.WARNING;
+            Severity severity = r.hasTag("restriction", "no_u_turn") ? Severity.OTHER : Severity.WARNING;
             errors.add(TestError.builder(this, severity, FROM_EQUALS_TO)
                     .message(tr("\"from\" way equals \"to\" way"))