Ticket #18160: 18160.patch

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

@@ -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;
@@ -819 +816 @@
      */
     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 (Layer layer : layerManager.getLayers()) {
+            if (layer.getChangesetSourceTag() != null && !layer.getChangesetSourceTag().trim().isEmpty()) {
+                layerInfo.add(layer.getChangesetSourceTag());
             }
         }
-        for (final ImageryLayer i : layerManager.getLayersOfType(ImageryLayer.class)) {
-            if (i.isVisible()) {
-                layerInfo.add(i.getInfo().getSourceName());
-            }
-        }
         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() {
+        return getInfo().getSourceName();
+    }
 }

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() {
+        return getName();
+    }
 }