Ticket #21775: 21775.patch

File 21775.patch, 8.9 KB (added by taylor.smock, 3 years ago)
  • src/org/openstreetmap/josm/data/ImageData.java

     
    144144    }
    145145
    146146    /**
    147      * Select the first image of the sequence
    148      * @deprecated Use {@link #getFirstImage()} in conjunction with {@link #setSelectedImage}
    149      */
    150     @Deprecated
    151     public void selectFirstImage() {
    152         if (!data.isEmpty()) {
    153             setSelectedImageIndex(0);
    154         }
    155     }
    156 
    157     /**
    158147     * Get the last image in the layer
    159148     * @return The last image
    160149     * @since 18246
     
    167156    }
    168157
    169158    /**
    170      * Select the last image of the sequence
    171      * @deprecated Use {@link #getLastImage()} with {@link #setSelectedImage}
    172      */
    173     @Deprecated
    174     public void selectLastImage() {
    175         setSelectedImageIndex(data.size() - 1);
    176     }
    177 
    178     /**
    179159     * Check if there is a next image in the sequence
    180160     * @return {@code true} is there is a next image, {@code false} otherwise
    181161     */
     
    206186    }
    207187
    208188    /**
    209      * Select the next image of the sequence
    210      * @deprecated Use {@link #getNextImage()} in conjunction with {@link #setSelectedImage}
    211      */
    212     @Deprecated
    213     public void selectNextImage() {
    214         if (hasNextImage()) {
    215             setSelectedImageIndex(selectedImagesIndex.get(0) + 1);
    216         }
    217     }
    218 
    219     /**
    220189     * Get the image previous to the current image
    221190     * @return The previous image
    222191     * @since 18246
     
    237206    }
    238207
    239208    /**
    240      * Select the previous image of the sequence
    241      * @deprecated Use {@link #getPreviousImage()} with {@link #setSelectedImage}
    242      */
    243     @Deprecated
    244     public void selectPreviousImage() {
    245         if (data.isEmpty()) {
    246             return;
    247         }
    248         setSelectedImageIndex(Integer.max(0, selectedImagesIndex.get(0) - 1));
    249     }
    250 
    251     /**
    252209     * Select as the selected the given image
    253210     * @param image the selected image
    254211     */
  • test/unit/org/openstreetmap/josm/data/ImageDataTest.java

     
    4444        ImageData data = new ImageData();
    4545        assertEquals(0, data.getImages().size());
    4646        assertNull(data.getSelectedImage());
    47         data.selectFirstImage();
     47        data.setSelectedImage(data.getFirstImage());
    4848        assertNull(data.getSelectedImage());
    49         data.selectLastImage();
     49        data.setSelectedImage(data.getLastImage());
    5050        assertNull(data.getSelectedImage());
    51         data.selectFirstImage();
     51        data.setSelectedImage(data.getFirstImage());
    5252        assertNull(data.getSelectedImage());
    53         data.selectPreviousImage();
     53        data.setSelectedImage(data.getPreviousImage());
    5454        assertNull(data.getSelectedImage());
    5555        assertFalse(data.hasNextImage());
    5656        assertFalse(data.hasPreviousImage());
     
    9595        List<ImageEntry> list = getOneImage();
    9696
    9797        ImageData data = new ImageData(list);
    98         data.selectFirstImage();
     98        data.setSelectedImage(data.getFirstImage());
    9999        assertEquals(1, data.getSelectedImages().size());
    100100        assertEquals(list.get(0), data.getSelectedImages().get(0));
    101101    }
     
    106106        list.add(new ImageEntry());
    107107
    108108        ImageData data = new ImageData(list);
    109         data.selectLastImage();
     109        data.setSelectedImage(data.getLastImage());
    110110        assertEquals(1, data.getSelectedImages().size());
    111111        assertEquals(list.get(1), data.getSelectedImages().get(0));
    112112    }
     
    117117
    118118        ImageData data = new ImageData(list);
    119119        assertTrue(data.hasNextImage());
    120         data.selectNextImage();
     120        data.setSelectedImage(data.getNextImage());
    121121        assertEquals(1, data.getSelectedImages().size());
    122122        assertEquals(list.get(0), data.getSelectedImages().get(0));
    123123        assertFalse(data.hasNextImage());
    124         data.selectNextImage();
    125         assertEquals(list.get(0), data.getSelectedImages().get(0));
     124        data.setSelectedImage(data.getNextImage());
     125        assertTrue(data.getSelectedImages().isEmpty());
    126126    }
    127127
    128128    @Test
     
    132132
    133133        ImageData data = new ImageData(list);
    134134        assertFalse(data.hasPreviousImage());
    135         data.selectLastImage();
     135        data.setSelectedImage(data.getLastImage());
    136136        assertTrue(data.hasPreviousImage());
    137         data.selectPreviousImage();
     137        data.setSelectedImage(data.getPreviousImage());
    138138        assertEquals(1, data.getSelectedImages().size());
    139139        assertEquals(list.get(0), data.getSelectedImages().get(0));
    140         data.selectPreviousImage();
    141         assertEquals(list.get(0), data.getSelectedImages().get(0));
     140        data.setSelectedImage(data.getPreviousImage());
     141        assertTrue(data.getSelectedImages().isEmpty());
    142142    }
    143143
    144144    @Test
     
    176176            listener.selectedImageChanged(data); times = 1;
    177177        }};
    178178        data.addImageDataUpdateListener(listener);
    179         data.selectFirstImage();
    180         data.selectFirstImage();
     179        data.setSelectedImage(data.getFirstImage());
     180        data.setSelectedImage(data.getFirstImage());
    181181    }
    182182
    183183    @Test
     
    186186        list.add(new ImageEntry());
    187187
    188188        ImageData data = new ImageData(list);
    189         data.selectFirstImage();
     189        data.setSelectedImage(data.getFirstImage());
    190190        data.addImageToSelection(list.get(1));
    191191        data.removeSelectedImages();
    192192        assertEquals(0, data.getImages().size());
     
    198198        List<ImageEntry> list = getOneImage();
    199199        list.add(new ImageEntry());
    200200        list.add(new ImageEntry());
     201        // The images cannot be equal -- otherwise, some code gives us the wrong result.
     202        list.get(1).setPos(LatLon.NORTH_POLE);
    201203
    202204        ImageData data = new ImageData(list);
    203         data.selectLastImage();
     205        data.setSelectedImage(data.getLastImage());
    204206        data.addImageToSelection(list.get(1));
    205207        data.removeSelectedImages();
    206208        assertEquals(1, data.getImages().size());
     
    213215        list.add(new ImageEntry());
    214216
    215217        ImageData data = new ImageData(list);
    216         data.selectFirstImage();
     218        data.setSelectedImage(data.getFirstImage());
    217219        data.removeSelectedImages();
    218220        assertEquals(1, data.getImages().size());
    219221        assertEquals(1, data.getSelectedImages().size());
     
    226228        list.add(new ImageEntry());
    227229
    228230        ImageData data = new ImageData(list);
    229         data.selectLastImage();
     231        data.setSelectedImage(data.getLastImage());
    230232        data.removeSelectedImages();
    231233        assertEquals(1, data.getImages().size());
    232234        assertEquals(1, data.getSelectedImages().size());
     
    249251            listener.selectedImageChanged(data); times = 2;
    250252        }};
    251253        data.addImageDataUpdateListener(listener);
    252         data.selectFirstImage();
     254        data.setSelectedImage(data.getFirstImage());
    253255        data.removeSelectedImages();
    254256    }
    255257
     
    269271            listener.selectedImageChanged(data); times = 2;
    270272        }};
    271273        data.addImageDataUpdateListener(listener);
    272         data.selectFirstImage();
     274        data.setSelectedImage(data.getFirstImage());
    273275        data.removeSelectedImages();
    274276    }
    275277
     
    354356        list.add(new ImageEntry());
    355357
    356358        ImageData data = new ImageData(list);
    357         data.selectLastImage();
     359        data.setSelectedImage(data.getLastImage());
    358360        data.removeImageToSelection(list.get(1));
    359361        assertEquals(0, data.getSelectedImages().size());
    360         data.selectFirstImage();
     362        data.setSelectedImage(data.getFirstImage());
    361363        assertEquals(1, data.getSelectedImages().size());
    362364    }
    363365
     
    368370
    369371        ImageData data = new ImageData(list);
    370372        assertFalse(data.isImageSelected(list.get(0)));
    371         data.selectFirstImage();
     373        data.setSelectedImage(data.getFirstImage());
    372374        assertTrue(data.isImageSelected(list.get(0)));
    373375        data.addImageToSelection(list.get(1));
    374376        assertTrue(data.isImageSelected(list.get(0)));
     
    393395        data.clearSelectedImage();
    394396        assertEquals(0, data.getSelectedImages().size());
    395397        data.addImageToSelection(list.get(1));
    396         data.selectFirstImage();
     398        data.setSelectedImage(data.getFirstImage());
    397399        assertEquals(1, data.getSelectedImages().size());
    398400    }
    399401
     
    413415            listener.selectedImageChanged(data); times = 3;
    414416        }};
    415417        data.addImageDataUpdateListener(listener);
    416         data.selectFirstImage();
     418        data.setSelectedImage(data.getFirstImage());
    417419        data.addImageToSelection(list.get(1));
    418420        data.removeImageToSelection(list.get(0));
    419421    }