Changeset 297 in josm for src/org/openstreetmap/josm/data/osm/DataSet.java

Timestamp:

2007-07-27T10:50:54+02:00 (19 years ago)

Author:

imi

Message:

• fixed bug that selection listener were informed twice on selection changes

File:

• src/org/openstreetmap/josm/data/osm/DataSet.java (modified) (3 diffs)

src/org/openstreetmap/josm/data/osm/DataSet.java

@@ -82 +82 @@
         /**
          * Remove the selection of the whole dataset.
+         * @deprecated Use setSelected() instead.
          */
+        @Deprecated
         public void clearSelection() {
                 clearSelection(nodes);
@@ -103 +105 @@
 
         public void setSelected(Collection<? extends OsmPrimitive> selection) {
-                clearSelection();
+                clearSelection(nodes);
+                clearSelection(segments);
+                clearSelection(ways);
                 for (OsmPrimitive osm : selection)
                         osm.selected = true;
@@ -110 +114 @@
 
         public void setSelected(OsmPrimitive... osm) {
-                clearSelection();
-                if (osm.length == 0 || (osm.length == 1 && osm[0] == null))
+                if (osm.length == 1 && osm[0] == null) {
+                        setSelected();
                         return;
+                }
+                clearSelection(nodes);
+                clearSelection(segments);
+                clearSelection(ways);
                 for (OsmPrimitive o : osm)
-                        o.selected = true;
+                        if (o != null)
+                                o.selected = true;
                 fireSelectionChanged(Arrays.asList(osm));
         }