Changeset 11129 in josm for trunk/src/org/openstreetmap/josm/data/validation/tests/Coastlines.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/Coastlines.java (modified) (2 diffs)

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

@@ -147 +147 @@
             // simple closed ways are reported by WronglyOrderedWays
             if (visited.size() > 1 && nodes.get(0) == nodes.get(nodes.size()-1) && Geometry.isClockwise(nodes)) {
-                errors.add(new TestError(this, Severity.WARNING, tr("Reversed coastline: land not on left side"),
-                        WRONG_ORDER_COASTLINE, visited));
+                errors.add(TestError.builder(this, Severity.WARNING, WRONG_ORDER_COASTLINE)
+                        .message(tr("Reversed coastline: land not on left side"))
+                        .primitives(visited)
+                        .build());
             }
         }
@@ -226 +228 @@
         }
         if (errCode != REVERSED_COASTLINE)
-            errors.add(new TestError(this, Severity.ERROR, msg, errCode, primitives, Collections.singletonList(n)));
+            errors.add(TestError.builder(this, Severity.ERROR, errCode)
+                    .message(msg)
+                    .primitives(primitives)
+                    .highlight(n)
+                    .build());
         else
-            errors.add(new TestError(this, Severity.ERROR, msg, errCode, primitives));
+            errors.add(TestError.builder(this, Severity.ERROR, errCode)
+                    .message(msg)
+                    .primitives(primitives)
+                    .build());
     }