Changeset 15352 in josm for trunk/src/org

Timestamp:

2019-09-15T23:54:42+02:00 (5 years ago)

Author:

Don-vip

Message:

fix #18130 - Select next image after remove (patch by francois2)

File:

• trunk/src/org/openstreetmap/josm/data/ImageData.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/data/ImageData.java

@@ -247 +247 @@
      */
     public void removeSelectedImage() {
-        List<ImageEntry> selected = getSelectedImages();
-        if (selected.size() > 1) {
+        List<ImageEntry> selectedImages = getSelectedImages();
+        if (selectedImages.size() > 1) {
             throw new IllegalStateException(tr("Multiple images have been selected"));
         }
-        if (selected.isEmpty()) {
+        removeImages(selectedImages);
+    }
+
+    /**
+     * Remove the current selected image from the list
+     * @since 15348
+     */
+    public void removeSelectedImages() {
+        List<ImageEntry> selectedImages = getSelectedImages();
+        removeImages(selectedImages);
+    }
+
+    private void removeImages(List<ImageEntry> selectedImages) {
+        if (selectedImages.isEmpty()) {
             return;
         }
-        data.remove(getSelectedImages().get(0));
+        for (ImageEntry img: getSelectedImages()) {
+            data.remove(img);
+        }
         if (selectedImagesIndex.get(0) == data.size()) {
             setSelectedImageIndex(data.size() - 1);
@@ -263 +278 @@
 
     /**
-     * Remove the current selected image from the list
-     * @since 15348
-     */
-    public void removeSelectedImages() {
-        List<ImageEntry> selected = getSelectedImages();
-        if (selected.isEmpty()) {
-            return;
-        }
-        for (ImageEntry img: getSelectedImages()) {
-            data.remove(img);
-        }
-        setSelectedImageIndex(-1, true);
-    }
-
-    /**
      * Determines if the image is selected
      * @param image the {@link ImageEntry} image