Changeset 18444 in josm


Ignore:
Timestamp:
2022-05-10T18:54:08+02:00 (2 years ago)
Author:
taylor.smock
Message:

Fix #21775: Remove deprecated code in ImageData

The code in question has been deprecated since r18246 (2021-10-04).

Location:
trunk
Files:
2 edited

Legend:

Unmodified
Added
Removed

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

    r18421 r18444  
    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
    … …  
    165154        }
    166155        return null;
    167     }
    168 
    169     /**
    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);
    176156    }
    177157
    … …  
    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
    … …  
    235204    public boolean hasPreviousImage() {
    236205        return (selectedImagesIndex.size() == 1 && selectedImagesIndex.get(0) - 1 > -1);
    237     }
    238 
    239     /**
    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));
    249206    }
    250207

  • trunk/test/unit/org/openstreetmap/josm/data/ImageDataTest.java

    r18049 r18444  
    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());
    … …  
    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));
    … …  
    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));
    … …  
    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
    … …  
    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();
    138         assertEquals(1, data.getSelectedImages().size());
    139         assertEquals(list.get(0), data.getSelectedImages().get(0));
    140         data.selectPreviousImage();
    141         assertEquals(list.get(0), data.getSelectedImages().get(0));
     137        data.setSelectedImage(data.getPreviousImage());
     138        assertEquals(1, data.getSelectedImages().size());
     139        assertEquals(list.get(0), data.getSelectedImages().get(0));
     140        data.setSelectedImage(data.getPreviousImage());
     141        assertTrue(data.getSelectedImages().isEmpty());
    142142    }
    143143
    … …  
    177177        }};
    178178        data.addImageDataUpdateListener(listener);
    179         data.selectFirstImage();
    180         data.selectFirstImage();
     179        data.setSelectedImage(data.getFirstImage());
     180        data.setSelectedImage(data.getFirstImage());
    181181    }
    182182
    … …  
    187187
    188188        ImageData data = new ImageData(list);
    189         data.selectFirstImage();
     189        data.setSelectedImage(data.getFirstImage());
    190190        data.addImageToSelection(list.get(1));
    191191        data.removeSelectedImages();
    … …  
    199199        list.add(new ImageEntry());
    200200        list.add(new ImageEntry());
    201 
    202         ImageData data = new ImageData(list);
    203         data.selectLastImage();
     201        // The images cannot be equal -- otherwise, some code gives us the wrong result.
     202        list.get(1).setPos(LatLon.NORTH_POLE);
     203
     204        ImageData data = new ImageData(list);
     205        data.setSelectedImage(data.getLastImage());
    204206        data.addImageToSelection(list.get(1));
    205207        data.removeSelectedImages();
    … …  
    214216
    215217        ImageData data = new ImageData(list);
    216         data.selectFirstImage();
     218        data.setSelectedImage(data.getFirstImage());
    217219        data.removeSelectedImages();
    218220        assertEquals(1, data.getImages().size());
    … …  
    227229
    228230        ImageData data = new ImageData(list);
    229         data.selectLastImage();
     231        data.setSelectedImage(data.getLastImage());
    230232        data.removeSelectedImages();
    231233        assertEquals(1, data.getImages().size());
    … …  
    250252        }};
    251253        data.addImageDataUpdateListener(listener);
    252         data.selectFirstImage();
     254        data.setSelectedImage(data.getFirstImage());
    253255        data.removeSelectedImages();
    254256    }
    … …  
    270272        }};
    271273        data.addImageDataUpdateListener(listener);
    272         data.selectFirstImage();
     274        data.setSelectedImage(data.getFirstImage());
    273275        data.removeSelectedImages();
    274276    }
    … …  
    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    }
    … …  
    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));
    … …  
    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    }
    … …  
    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));
Note: See TracChangeset for help on using the changeset viewer.