Changeset 8658 in josm for trunk/src/org


Ignore:
Timestamp:
2015-08-11T23:34:03+02:00 (5 years ago)
Author:
Don-vip
Message:

fix #11756 - NPEs

Location:
trunk/src/org/openstreetmap/josm/gui/layer/geoimage
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/gui/layer/geoimage/CorrelateGpxWithImages.java

    r8653 r8658  
    10961096     */
    10971097    private List<ImageEntry> getSortedImgList(boolean exif, boolean tagged) {
     1098        if (yLayer.data == null) {
     1099            return Collections.emptyList();
     1100        }
    10981101        List<ImageEntry> dateImgLst = new ArrayList<>(yLayer.data.size());
    10991102        for (ImageEntry e : yLayer.data) {
  • trunk/src/org/openstreetmap/josm/gui/layer/geoimage/GeoImageLayer.java

    r8540 r8658  
    252252                Main.main.addLayer(layer);
    253253
    254                 if (!canceled && !layer.data.isEmpty()) {
     254                if (!canceled && layer.data != null && !layer.data.isEmpty()) {
    255255                    boolean noGeotagFound = true;
    256256                    for (ImageEntry e : layer.data) {
     
    317317    public GeoImageLayer(final List<ImageEntry> data, GpxLayer gpxLayer, final String name, boolean useThumbs) {
    318318        super(name != null ? name : tr("Geotagged Images"));
    319         Collections.sort(data);
     319        if (data != null) {
     320            Collections.sort(data);
     321        }
    320322        this.data = data;
    321323        this.gpxLayer = gpxLayer;
     
    755757
    756758    public void checkPreviousNextButtons() {
    757         ImageViewerDialog.setNextEnabled(currentPhoto < data.size() - 1);
     759        ImageViewerDialog.setNextEnabled(data != null && currentPhoto < data.size() - 1);
    758760        ImageViewerDialog.setPreviousEnabled(currentPhoto > 0);
    759761    }
     
    10871089     */
    10881090    public List<ImageEntry> getImages() {
     1091        if (data == null) {
     1092            return Collections.emptyList();
     1093        }
    10891094        List<ImageEntry> copy = new ArrayList<>(data.size());
    10901095        for (ImageEntry ie : data) {
  • trunk/src/org/openstreetmap/josm/gui/layer/geoimage/ShowThumbnailAction.java

    r8378 r8658  
    5151     */
    5252    private static boolean enabled(GeoImageLayer layer) {
    53         return !layer.data.isEmpty();
     53        return layer.data != null && !layer.data.isEmpty();
    5454    }
    5555
Note: See TracChangeset for help on using the changeset viewer.