Changeset 8542 in josm for trunk/src/org/openstreetmap/josm/gui

Timestamp:

2015-06-30T13:57:53+02:00 (9 years ago)

Author:

wiktorn

Message:

Add warnings about supported projections for WMS layers

Location:

trunk/src/org/openstreetmap/josm/gui/layer

Files:

• AbstractTileSourceLayer.java (modified) (2 diffs)
• Layer.java (modified) (1 diff)
• WMSLayer.java (modified) (5 diffs)

trunk/src/org/openstreetmap/josm/gui/layer/AbstractTileSourceLayer.java

@@ -139 +139 @@
     public AbstractTileSourceLayer(ImageryInfo info) {
         super(info);
-
-        if (!isProjectionSupported(Main.getProjection())) {
-            JOptionPane.showMessageDialog(Main.parent,
-                    tr("This layer do not support the projection {0}.\n{1}\n"
-                            + "Change the projection or remove the layer.",
-                            Main.getProjection().toCode(), nameSupportedProjections()),
-                            tr("Warning"),
-                            JOptionPane.WARNING_MESSAGE);
-        }
         setBackgroundLayer(true);
         this.setVisible(true);
-
-        initTileSource(getTileSource(info));
         MapView.addZoomChangeListener(this);
     }
@@ -462 +451 @@
     @Override
     public void hookUpMapView() {
+        initTileSource(getTileSource(info));
+        projectionChanged(null, Main.getProjection()); // check if projection is supported
+
         // keep them final here, so we avoid namespace clutter in the class
         final JPopupMenu tileOptionMenu = new JPopupMenu();

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

@@ -447 +447 @@
         if (!isProjectionSupported(newValue)) {
             JOptionPane.showMessageDialog(Main.parent,
-                    tr("The layer {0} does not support the new projection {1}.\n{2}\n"
+                    tr("The layer {0} does not support the new projection {1}.\n"
+                            + "Supported projections are: {2}\n"
                             + "Change the projection again or remove the layer.",
                             getName(), newValue.toCode(), nameSupportedProjections()),

trunk/src/org/openstreetmap/josm/gui/layer/WMSLayer.java

@@ -17 +17 @@
 import org.openstreetmap.gui.jmapviewer.interfaces.TileLoaderListener;
 import org.openstreetmap.gui.jmapviewer.interfaces.TileSource;
-import org.openstreetmap.josm.Main;
 import org.openstreetmap.josm.data.imagery.CachedTileLoaderFactory;
 import org.openstreetmap.josm.data.imagery.ImageryInfo;
@@ -28 +27 @@
 import org.openstreetmap.josm.data.preferences.IntegerProperty;
 import org.openstreetmap.josm.data.projection.Projection;
-import org.openstreetmap.josm.data.projection.ProjectionChangeListener;
-import org.openstreetmap.josm.gui.MapView;
-import org.openstreetmap.josm.gui.MapView.LayerChangeListener;
 
 /**
@@ -42 +38 @@
     /** should WMS layer autozoom in default mode */
     public static final BooleanProperty PROP_DEFAULT_AUTOZOOM = new BooleanProperty("imagery.wms.default_autozoom", true);
+    private List<String> supportedProjections;
 
     /**
@@ -47 +44 @@
      * @param info ImageryInfo description of the layer
      */
+
+
     public WMSLayer(ImageryInfo info) {
         super(info);
-    }
-
-    @Override
-    public void hookUpMapView() {
-        super.hookUpMapView();
-        final ProjectionChangeListener listener = new ProjectionChangeListener() {
-            @Override
-            public void projectionChanged(Projection oldValue, Projection newValue) {
-                if (!oldValue.equals(newValue) && tileSource instanceof TemplatedWMSTileSource) {
-                    ((TemplatedWMSTileSource) tileSource).initProjection(newValue);
-                }
-
-            }
-        };
-        Main.addProjectionChangeListener(listener);
-
-        MapView.addLayerChangeListener(new LayerChangeListener() {
-            @Override
-            public void activeLayerChange(Layer oldLayer, Layer newLayer) {
-                // empty
-            }
-
-            @Override
-            public void layerAdded(Layer newLayer) {
-                // empty
-            }
-
-            @Override
-            public void layerRemoved(Layer oldLayer) {
-                if (oldLayer == WMSLayer.this) {
-                    Main.removeProjectionChangeListener(listener);
-                }
-            }
-        });
+        this.supportedProjections = info.getServerProjections();
     }
 
@@ -155 +121 @@
         return null;
     }
+
+    @Override
+    public boolean isProjectionSupported(Projection proj) {
+        return supportedProjections == null || supportedProjections.isEmpty() || supportedProjections.contains(proj.toCode());
+    }
+
+    @Override
+    public String nameSupportedProjections() {
+        StringBuffer ret = new StringBuffer();
+        for (String e: supportedProjections) {
+            ret.append(e).append(", ");
+        }
+        String appendix = "";
+        if (supportedProjections.contains("EPSG:4326")) {
+            appendix = ". " + tr("JOSM will use EPSG:4326 to query the server, but results may vary "
+                    + "depending on the WMS server");
+        }
+        return ret.substring(0, ret.length()-2);
+    }
+
+    @Override
+    public void projectionChanged(Projection oldValue, Projection newValue) {
+        super.projectionChanged(oldValue, newValue);
+
+        if (!newValue.equals(oldValue) && tileSource instanceof TemplatedWMSTileSource) {
+            ((TemplatedWMSTileSource) tileSource).initProjection(newValue);
+        }
+
+    }
+
 }