Changeset 4064 in josm

Timestamp:

2011-04-25T20:32:26+02:00 (13 years ago)

Author:

bastiK

Message:

applied #6250 (patch by akks) - move selection history to dataset

Location:

trunk/src/org/openstreetmap/josm

Files:

• actions/PurgeAction.java (modified) (1 diff)
• data/osm/DataSet.java (modified) (1 diff)
• gui/dialogs/SelectionListDialog.java (modified) (6 diffs)

trunk/src/org/openstreetmap/josm/actions/PurgeAction.java

@@ -208 +208 @@
         if (cbClearUndoRedo.isSelected()) {
             Main.main.undoRedo.clean();
-            Main.map.selectionListDialog.clearSelectionHistory();
+            getCurrentDataSet().clearSelectionHistory();
         }
     }

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

@@ -134 +134 @@
 
     /**
+     * History of selections - shared by plugins and SelectionListDialog
+     */
+    private final LinkedList<Collection<? extends OsmPrimitive>> selectionHistory = new LinkedList<Collection<? extends OsmPrimitive>>();
+
+    /**
+     * Replies the history of JOSM selections
+     *
+     * @return
+     */
+    public LinkedList<Collection<? extends OsmPrimitive>> getSelectionHistory() {
+        return selectionHistory;
+    } 
+
+    /**
+     * Clears selection history list
+     */
+    public void clearSelectionHistory() {
+            selectionHistory.clear();
+        }
+
+    /**
      * Maintain a list of used tags for autocompletion
      */

trunk/src/org/openstreetmap/josm/gui/dialogs/SelectionListDialog.java

@@ -200 +200 @@
         DatasetEventManager.getInstance().addDatasetListener(model, FireMode.IN_EDT);
         MapView.addEditLayerChangeListener(actSearch);
+        // editLayerChanged also gets the selection history of the level
+        model.editLayerChanged(null, Main.map.mapView.getEditLayer());
         if (Main.map.mapView.getEditLayer() != null) {
             model.setJOSMSelection(Main.map.mapView.getEditLayer().data.getSelected());
@@ -222 +224 @@
     }
 
-    public void clearSelectionHistory() {
-        model.clearSelectionHistory();
-    }
 
     /**
@@ -482 +481 @@
         private static final int SELECTION_HISTORY_SIZE = 10;
 
-        private final LinkedList<Collection<? extends OsmPrimitive>> history = new LinkedList<Collection<? extends OsmPrimitive>>();
+        // Variable to store history from currentDataSet()
+        private LinkedList<Collection<? extends OsmPrimitive>> history;
         private final List<OsmPrimitive> selection = new ArrayList<OsmPrimitive>();
         private DefaultListSelectionModel selectionModel;
@@ -522 +522 @@
             if (selection == null)return;
             if (selection.isEmpty())return;
+            if (history == null) return;
             if (history.isEmpty()) {
                 history.add(selection);
@@ -547 +548 @@
         public List<Collection<? extends OsmPrimitive>> getSelectionHistory() {
             return history;
-        }
-
-        public void clearSelectionHistory() {
-            history.clear();
         }
 
@@ -690 +687 @@
             if (newLayer == null) {
                 setJOSMSelection(null);
+                history = null;
             } else {
+                history = newLayer.data.getSelectionHistory();
                 setJOSMSelection(newLayer.data.getSelected());
             }