Changeset 11129 in josm for trunk/src/org/openstreetmap/josm/data/validation/tests/OverlappingWays.java

Timestamp:

2016-10-15T00:17:47+02:00 (8 years ago)

Author:

simon04

Message:

fix #13799 - Use builder pattern for TestError

File:

• trunk/src/org/openstreetmap/josm/data/validation/tests/OverlappingWays.java (modified) (2 diffs)

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

@@ -140 +140 @@
                     }
 
-                    preliminaryErrors.add(new TestError(this,
-                            type < OVERLAPPING_HIGHWAY_AREA ? Severity.WARNING : Severity.OTHER,
-                                    errortype, type, prims, duplicated));
+                    Severity severity = type < OVERLAPPING_HIGHWAY_AREA ? Severity.WARNING : Severity.OTHER;
+                    preliminaryErrors.add(TestError.builder(this, severity, type)
+                            .message(errortype)
+                            .primitives(prims)
+                            .highlightWaySegments(duplicated)
+                            .build());
                     seenWays.put(currentWays, duplicated);
                 } else { /* way seen, mark highlight layer only */
@@ -202 +205 @@
         final Set<WaySegment> duplicateWaySegment = checkDuplicateWaySegment(w);
         if (duplicateWaySegment != null) {
-            errors.add(new TestError(this, Severity.ERROR, tr("Way contains segment twice"),
-                    DUPLICATE_WAY_SEGMENT, Collections.singleton(w), duplicateWaySegment));
+            errors.add(TestError.builder(this, Severity.ERROR, DUPLICATE_WAY_SEGMENT)
+                    .message(tr("Way contains segment twice"))
+                    .primitives(w)
+                    .highlightWaySegments(duplicateWaySegment)
+                    .build());
             return;
         }