Changeset 6301 in osm for applications/editors/josm/plugins/validator/src/org

Timestamp:

2008-01-09T18:24:09+01:00 (17 years ago)

Author:

gabriel

Message:

Validator plugin:

• Fix fixing duplicate way nodes errors.
• Report 'fixme' and 'todo' tags as FIXME warnings.

Location:

applications/editors/josm/plugins/validator/src/org/openstreetmap/josm/plugins/validator/tests

Files:

• DuplicatedWayNodes.java (modified) (2 diffs)
• SpellCheck.java (modified) (1 diff)

applications/editors/josm/plugins/validator/src/org/openstreetmap/josm/plugins/validator/tests/DuplicatedWayNodes.java

@@ -25 +25 @@
                         if (lastN == null) {
                                 lastN = n;
-                                break;
+                                continue;
                         }
                         if (lastN == n) {
@@ -57 +57 @@
                 }
         }
+
+        @Override public boolean isFixable(TestError testError) {
+                return testError.getTester() instanceof DuplicatedWayNodes;
+        }       
 }

applications/editors/josm/plugins/validator/src/org/openstreetmap/josm/plugins/validator/tests/SpellCheck.java

@@ -251 +251 @@
             if( checkFixmes && value != null && value.length() > 0 )
             {
-                if( value.contains("FIXME") && !withErrors.contains(p, "FIXME"))
+                if( (value.contains("FIXME") || value.contains("todo") || value.contains("fixme"))
+                                  && !withErrors.contains(p, "FIXME"))
                 {
                     errors.add( new TestError(this, Severity.OTHER, tr("FIXMES"), p, FIXME) );