Fix #22606: Images with no exif position or no set position were treated as if they had no position

This is incorrect behavior, since the paint code only cares about the return
value of IImageEntry#getPos, and invalid geoimages should be those that can
not be shown in the mapview.

This also fixes an issue where, when the layer has no images with positions, the
first image would be displayed, but not selected in the data.

git-svn-id: https://josm.openstreetmap.de/svn/trunk@18621 0c6e7542-c601-0410-84e7-c038aed88b3b

Author: taylor.smock

src/org/openstreetmap/josm/gui/layer/geoimage/GeoImageLayer.java

@@ -198,7 +198,12 @@ public GeoImageLayer(final List<ImageEntry> data, GpxData gpxData, final String
             }
         }
         if (getInvalidGeoImages().size() == data.size()) {
-            ImageViewerDialog.getInstance().displayImages(Collections.singletonList(this.data.getFirstImage()));
+            this.data.setSelectedImage(this.data.getFirstImage());
+            // We do have to wrap the EDT call in a worker call, since layers may be created in the EDT.
+            // And the layer must be added to the layer list in order for the dialog to work properly.
+            MainApplication.worker.execute(() -> GuiHelper.runInEDT(() -> {
+                ImageViewerDialog.getInstance().displayImages(this.getSelection());
+            }));
         }
     }
 
@@ -301,8 +306,7 @@ public List<ImageEntry> getSelection() {
 
     @Override
     public List<IImageEntry<?>> getInvalidGeoImages() {
-        return this.getImageData().getImages().stream().filter(entry -> entry.getPos() == null || entry.getExifCoor() == null
-              || !entry.getExifCoor().isValid() || !entry.getPos().isValid()).collect(toList());
+        return this.getImageData().getImages().stream().filter(entry -> entry.getPos() == null || !entry.getPos().isValid()).collect(toList());
     }
 
     @Override