Ticket #6727: 0001-fix-6953-Make-sure-that-only-selectable-primitives-a.patch

src/org/openstreetmap/josm/data/validation/TestError.java

@@ -8 +8 @@
 import java.util.Collections;
 import java.util.List;
 import java.util.TreeSet;
+import java.util.ArrayList;
 
 import org.openstreetmap.josm.command.Command;
 import org.openstreetmap.josm.data.osm.Node;
@@ -121 +122 @@
     }
 
     /**
+     * Gets the list of primitives affected by this error and are selectable
+     * @return the list of selectable primitives affected by this error
+     */
+    public Collection<? extends OsmPrimitive> getSelectablePrimitives() {
+        List<OsmPrimitive> selectablePrimitives = new ArrayList<OsmPrimitive>(primitives.size());
+        for (OsmPrimitive o : primitives) {
+            if (o.isSelectable()) {
+                selectablePrimitives.add(o);
+            }
+        }
+        return selectablePrimitives;
+    }
+
+
+    /**
      * Sets the list of primitives affected by this error
      * @param primitives the list of primitives affected by this error
      */

src/org/openstreetmap/josm/gui/dialogs/ValidatorDialog.java

@@ -328 +328 @@
                 Object nodeInfo = childNode.getUserObject();
                 if (nodeInfo instanceof TestError) {
                     TestError error = (TestError) nodeInfo;
-                    sel.addAll(error.getPrimitives());
+                    sel.addAll(error.getSelectablePrimitives());
                 }
             }
         }