Changeset 18649 in josm for trunk/src/org/openstreetmap

Timestamp:

2023-02-06T19:39:32+01:00 (15 months ago)

Author:

taylor.smock

Message:

Fix #22712: ignore list doesn't work

This occurred due to OsmValidator#save modifying the list, and expecting
all error codes to be positive.

TestErrorTest now checks one of the tests known to have a negative hashcode.

This also adds some additional JUnit 5 annotations. The additional
annotations are:

• @LayerManager (cleans up layers after each test run)
  • This is automatically registered for all tests
• @Projection (sets up the ProjectionRegistry for each test)

File:

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

trunk/src/org/openstreetmap/josm/data/validation/OsmValidator.java

@@ -467 +467 @@
                 if (tr("Ignore list").equals(description))
                     description = "";
-                if (!key.matches("^[0-9]+(_.*|$)")) {
+                if (!key.matches("^-?[0-9]+(_.*|$)")) {
                     description = key;
                     key = "";
@@ -480 +480 @@
                     // single element
                     entry = key + ":" + item;
-                } else if (item.matches("^[0-9]+(_.*|)$")) {
+                } else if (item.matches("^-?[0-9]+(_.*|)$")) {
                     // no element ids
                     entry = item;