Index: trunk/src/org/openstreetmap/josm/gui/bbox/SlippyMapBBoxChooser.java =================================================================== --- trunk/src/org/openstreetmap/josm/gui/bbox/SlippyMapBBoxChooser.java (revision 14298) +++ trunk/src/org/openstreetmap/josm/gui/bbox/SlippyMapBBoxChooser.java (revision 14300) @@ -16,10 +16,10 @@ import java.util.Collections; import java.util.HashMap; -import java.util.HashSet; +import java.util.LinkedHashMap; import java.util.List; import java.util.Map; -import java.util.Set; import java.util.concurrent.CopyOnWriteArrayList; import java.util.concurrent.TimeUnit; +import java.util.stream.Collectors; import javax.swing.ButtonModel; @@ -53,4 +53,5 @@ import org.openstreetmap.josm.gui.MainApplication; import org.openstreetmap.josm.gui.layer.AbstractCachedTileSourceLayer; +import org.openstreetmap.josm.gui.layer.ImageryLayer; import org.openstreetmap.josm.gui.layer.MainLayerManager; import org.openstreetmap.josm.gui.layer.TMSLayer; @@ -61,6 +62,6 @@ * This panel displays a map and lets the user chose a {@link BBox}. */ -public class SlippyMapBBoxChooser extends JMapViewer implements BBoxChooser, ChangeListener, MainLayerManager.ActiveLayerChangeListener { - +public class SlippyMapBBoxChooser extends JMapViewer implements BBoxChooser, ChangeListener, + MainLayerManager.ActiveLayerChangeListener, MainLayerManager.LayerChangeListener { /** * A list of tile sources that can be used for displaying the map. @@ -76,12 +77,13 @@ /** - * TMS TileSource provider for the slippymap chooser - */ - public static class TMSTileSourceProvider implements TileSourceProvider { - private static final Set existingSlippyMapUrls = new HashSet<>(); - static { - // Urls that already exist in the slippymap chooser and shouldn't be copied from TMS layer list - existingSlippyMapUrls.add("https://{switch:a,b,c}.tile.openstreetmap.org/{zoom}/{x}/{y}.png"); // Mapnik - } + * TileSource provider for the slippymap chooser. + * @since 14300 + */ + public abstract static class AbstractImageryInfoBasedTileSourceProvider implements TileSourceProvider { + /** + * Returns the list of imagery infos backing tile sources. + * @return the list of imagery infos backing tile sources + */ + public abstract List getImageryInfos(); @Override @@ -89,8 +91,5 @@ if (!TMSLayer.PROP_ADD_TO_SLIPPYMAP_CHOOSER.get()) return Collections.emptyList(); List sources = new ArrayList<>(); - for (ImageryInfo info : ImageryLayerInfo.instance.getLayers()) { - if (existingSlippyMapUrls.contains(info.getUrl())) { - continue; - } + for (ImageryInfo info : this.getImageryInfos()) { try { TileSource source = TMSLayer.getTileSourceStatic(info); @@ -112,4 +111,30 @@ /** + * TileSource provider for the slippymap chooser - providing sources from imagery sources menu + * @since 14300 + */ + public static class TMSTileSourceProvider extends AbstractImageryInfoBasedTileSourceProvider { + @Override + public List getImageryInfos() { + return ImageryLayerInfo.instance.getLayers(); + } + } + + /** + * TileSource provider for the slippymap chooser - providing sources from current layers + * @since 14300 + */ + public static class CurrentLayersTileSourceProvider extends AbstractImageryInfoBasedTileSourceProvider { + @Override + public List getImageryInfos() { + return MainApplication.getLayerManager().getLayers().stream().filter( + layer -> layer instanceof ImageryLayer + ).map( + layer -> ((ImageryLayer) layer).getInfo() + ).collect(Collectors.toList()); + } + } + + /** * Plugins that wish to add custom tile sources to slippy map choose should call this method * @param tileSourceProvider new tile source provider @@ -123,4 +148,5 @@ addTileSourceProvider(() -> Arrays.asList(new OsmTileSource.Mapnik())); addTileSourceProvider(new TMSTileSourceProvider()); + addTileSourceProvider(new CurrentLayersTileSourceProvider()); } @@ -178,5 +204,5 @@ setMaxTilesInMemory(Config.getPref().getInt("slippy_map_chooser.max_tiles", 1000)); - List tileSources = getAllTileSources(); + List tileSources = new ArrayList<>(getAllTileSources().values()); this.showDownloadAreaButtonModel = new JToggleButton.ToggleButtonModel(); @@ -211,10 +237,14 @@ } - private static List getAllTileSources() { - List tileSources = new ArrayList<>(); - for (TileSourceProvider provider: providers) { - tileSources.addAll(provider.getTileSources()); - } - return tileSources; + private static LinkedHashMap getAllTileSources() { + // using a LinkedHashMap of to retain ordering but provide deduplication + return providers.stream().flatMap( + provider -> provider.getTileSources().stream() + ).collect(Collectors.toMap( + TileSource::getId, + ts -> ts, + (oldTs, newTs) -> oldTs, + LinkedHashMap::new + )); } @@ -359,7 +389,10 @@ this.setTileSource(tileSource); PROP_MAPSTYLE.put(tileSource.getName()); // TODO Is name really unique? - if (this.iSourceButton.getCurrentSource() != tileSource) { // prevent infinite recursion - this.iSourceButton.setCurrentMap(tileSource); - } + + // we need to refresh the tile sources in case the deselected source should no longer be present + // (and only remained there because its removal was deferred while the source was still the + // selected one). this should also have the effect of propagating the new selection to the + // iSourceButton & menu: it attempts to re-select the current source when rebuilding its menu. + this.refreshTileSources(); } @@ -417,5 +450,28 @@ */ public final void refreshTileSources() { - iSourceButton.setSources(getAllTileSources()); - } + final LinkedHashMap newTileSources = getAllTileSources(); + final TileSource currentTileSource = this.getTileController().getTileSource(); + + // re-add the currently active TileSource to prevent inconsistent display of menu + newTileSources.putIfAbsent(currentTileSource.getId(), currentTileSource); + + this.iSourceButton.setSources(new ArrayList<>(newTileSources.values())); + } + + @Override + public void layerAdded(MainLayerManager.LayerAddEvent e) { + if (e.getAddedLayer() instanceof ImageryLayer) { + this.refreshTileSources(); + } + } + + @Override + public void layerRemoving(MainLayerManager.LayerRemoveEvent e) { + if (e.getRemovedLayer() instanceof ImageryLayer) { + this.refreshTileSources(); + } + } + + @Override + public void layerOrderChanged(MainLayerManager.LayerOrderChangeEvent e) {} } Index: trunk/src/org/openstreetmap/josm/gui/bbox/SourceButton.java =================================================================== --- trunk/src/org/openstreetmap/josm/gui/bbox/SourceButton.java (revision 14298) +++ trunk/src/org/openstreetmap/josm/gui/bbox/SourceButton.java (revision 14300) @@ -82,5 +82,5 @@ // attempt to initialize button group matching current state of slippyMapBBoxChooser - buttonModel.setSelected(this.slippyMapBBoxChooser.getTileController().getTileSource() == ts); + buttonModel.setSelected(this.slippyMapBBoxChooser.getTileController().getTileSource().getId().equals(ts.getId())); } Index: trunk/src/org/openstreetmap/josm/gui/dialogs/MinimapDialog.java =================================================================== --- trunk/src/org/openstreetmap/josm/gui/dialogs/MinimapDialog.java (revision 14298) +++ trunk/src/org/openstreetmap/josm/gui/dialogs/MinimapDialog.java (revision 14300) @@ -38,4 +38,5 @@ slippyMap.setSizeButtonVisible(false); slippyMap.addPropertyChangeListener(BBoxChooser.BBOX_PROP, this); + MainApplication.getLayerManager().addLayerChangeListener(slippyMap); }