Changeset 5828 in josm

Timestamp:

2013-04-06T13:39:18+02:00 (12 years ago)

Author:

Don-vip

Message:

fix #8580 - validator offers to fix empty new relations (by deleting them)

File:

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

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

@@ -13 +13 @@
 import java.util.Set;
 
+import org.openstreetmap.josm.command.Command;
+import org.openstreetmap.josm.command.DeleteCommand;
 import org.openstreetmap.josm.data.osm.Node;
 import org.openstreetmap.josm.data.osm.OsmPrimitive;
@@ -211 +213 @@
         }
     }
+
+    /* (non-Javadoc)
+     * @see org.openstreetmap.josm.data.validation.Test#fixError(org.openstreetmap.josm.data.validation.TestError)
+     */
+    @Override
+    public Command fixError(TestError testError) {
+        if (isFixable(testError)) {
+            return new DeleteCommand(testError.getPrimitives());
+        }
+        return null;
+    }
+
+    /* (non-Javadoc)
+     * @see org.openstreetmap.josm.data.validation.Test#isFixable(org.openstreetmap.josm.data.validation.TestError)
+     */
+    @Override
+    public boolean isFixable(TestError testError) {
+        return testError.getCode() == RELATION_EMPTY && testError.getPrimitives().iterator().next().isNew();
+    }
 }