Ticket #18160: 18160.2.patch

src/org/openstreetmap/josm/gui/MapView.java

@@ -28 +28 @@
 import java.util.LinkedHashSet;
 import java.util.List;
 import java.util.Set;
-import java.util.TreeSet;
 import java.util.concurrent.CopyOnWriteArrayList;
 import java.util.concurrent.atomic.AtomicBoolean;
+import java.util.stream.Collectors;
 
 import javax.swing.AbstractButton;
 import javax.swing.JComponent;
@@ -50 +50 @@
 import org.openstreetmap.josm.gui.MapViewState.MapViewRectangle;
 import org.openstreetmap.josm.gui.autofilter.AutoFilterManager;
 import org.openstreetmap.josm.gui.datatransfer.OsmTransferHandler;
-import org.openstreetmap.josm.gui.layer.GpxLayer;
-import org.openstreetmap.josm.gui.layer.ImageryLayer;
 import org.openstreetmap.josm.gui.layer.Layer;
 import org.openstreetmap.josm.gui.layer.LayerManager;
 import org.openstreetmap.josm.gui.layer.LayerManager.LayerAddEvent;
@@ -66 +64 @@
 import org.openstreetmap.josm.gui.layer.MapViewPaintable.PaintableInvalidationEvent;
 import org.openstreetmap.josm.gui.layer.MapViewPaintable.PaintableInvalidationListener;
 import org.openstreetmap.josm.gui.layer.OsmDataLayer;
-import org.openstreetmap.josm.gui.layer.geoimage.GeoImageLayer;
 import org.openstreetmap.josm.gui.layer.markerlayer.PlayHeadMarker;
 import org.openstreetmap.josm.gui.mappaint.MapPaintStyles;
 import org.openstreetmap.josm.gui.mappaint.MapPaintStyles.MapPaintSylesUpdateListener;
@@ -818 +815 @@
      * @return A String of sources separated by ';'
      */
     public String getLayerInformationForSourceTag() {
-        final Set<String> layerInfo = new TreeSet<>();
-        if (!layerManager.getLayersOfType(GpxLayer.class).isEmpty()) {
-            // no i18n for international values
-            layerInfo.add("survey");
-        }
-        for (final GeoImageLayer i : layerManager.getLayersOfType(GeoImageLayer.class)) {
-            if (i.isVisible()) {
-                layerInfo.add(i.getName());
-            }
-        }
-        for (final ImageryLayer i : layerManager.getLayersOfType(ImageryLayer.class)) {
-            if (i.isVisible()) {
-                layerInfo.add(i.getInfo().getSourceName());
-            }
-        }
+        final Set<String> layerInfo = layerManager.getLayers().stream()
+                .filter(layer -> layer.getChangesetSourceTag() != null && !layer.getChangesetSourceTag().trim().isEmpty())
+                .map(layer -> layer.getChangesetSourceTag().trim()).distinct().collect(Collectors.toSet());
         return Utils.join("; ", layerInfo);
     }
 

src/org/openstreetmap/josm/gui/layer/GpxLayer.java

@@ -473 +473 @@
     public void expertChanged(boolean isExpert) {
         this.isExpertMode = isExpert;
     }
+
+    @Override
+    public String getChangesetSourceTag() {
+        return "survey";
+    }
 }

src/org/openstreetmap/josm/gui/layer/ImageryLayer.java

@@ -354 +354 @@
     public String toString() {
         return getClass().getSimpleName() + " [info=" + info + ']';
     }
+
+    @Override
+    public String getChangesetSourceTag() {
+        if (isVisible())
+            return getInfo().getSourceName();
+        return null;
+    }
 }

src/org/openstreetmap/josm/gui/layer/Layer.java

@@ -674 +674 @@
         return v.getBounds();
     }
 
+    /**
+     * Get the source for the layer
+     * @return The string for the changeset source tag or {@code null}
+     * @since xxx
+     */
+    public String getChangesetSourceTag() {
+        return null;
+    }
+
     @Override
     public String toString() {
         return getClass().getSimpleName() + " [name=" + name + ", associatedFile=" + associatedFile + ']';

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

@@ -1010 +1010 @@
     public void imageDataUpdated(ImageData data) {
         updateBufferAndRepaint();
     }
+
+    @Override
+    public String getChangesetSourceTag() {
+        if (isVisible())
+            return getName();
+        return null;
+    }
 }