Changeset 6394 in josm for trunk/src/org/openstreetmap/josm/actions

Timestamp:

2013-11-19T02:29:10+01:00 (10 years ago)

Author:

Don-vip

Message:

fix #9333 - Make 'jump to position" more visible (added in view menu) + update to new osm.org url scheme + fix NPE without mapview + javadoc

File:

• trunk/src/org/openstreetmap/josm/actions/JumpToAction.java (modified) (5 diffs)

trunk/src/org/openstreetmap/josm/actions/JumpToAction.java

@@ -26 +26 @@
 import org.openstreetmap.josm.gui.widgets.JosmTextField;
 
+/**
+ * Allows to jump to a specific location.
+ * @since 2575
+ */
 public class JumpToAction extends JosmAction {
+    
     /**
      * Constructs a new {@code JumpToAction}.
@@ -35 +40 @@
     }
 
-    private JosmTextField url = new JosmTextField();
-    private JosmTextField lat = new JosmTextField();
-    private JosmTextField lon = new JosmTextField();
-    private JosmTextField zm = new JosmTextField();
+    private final JosmTextField url = new JosmTextField();
+    private final JosmTextField lat = new JosmTextField();
+    private final JosmTextField lon = new JosmTextField();
+    private final JosmTextField zm = new JosmTextField();
 
+    /**
+     * Displays the "Jump to" dialog.
+     */
     public void showJumpToDialog() {
+        if (!Main.isDisplayingMapView()) {
+            return;
+        }
         MapView mv = Main.map.mapView;
-        if(mv == null)
-            return;
-        LatLon curPos=mv.getProjection().eastNorth2latlon(mv.getCenter());
-        lat.setText(java.lang.Double.toString(curPos.lat()));
-        lon.setText(java.lang.Double.toString(curPos.lon()));
+        LatLon curPos = mv.getProjection().eastNorth2latlon(mv.getCenter());
+        lat.setText(Double.toString(curPos.lat()));
+        lon.setText(Double.toString(curPos.lon()));
 
         double dist = mv.getDist100Pixel();
         double zoomFactor = 1/dist;
 
-        zm.setText(java.lang.Long.toString(Math.round(dist*100)/100));
+        zm.setText(Long.toString(Math.round(dist*100)/100));
         updateUrl(true);
 
@@ -123 +132 @@
 
     private void parseURL() {
-        if(!url.hasFocus()) return;
-        Bounds b = OsmUrlToBounds.parse(url.getText());
+        if (!url.hasFocus()) return;
+        String urlText = url.getText();
+        Bounds b = OsmUrlToBounds.parse(urlText);
         if (b != null) {
             lat.setText(Double.toString((b.getMinLat() + b.getMaxLat())/2));
@@ -130 +140 @@
 
             int zoomLvl = 16;
-            String[] args = url.getText().substring(url.getText().indexOf('?')+1).split("&");
-            for (String arg : args) {
-                int eq = arg.indexOf('=');
-                if (eq == -1 || !arg.substring(0, eq).equalsIgnoreCase("zoom")) continue;
-
-                zoomLvl = Integer.parseInt(arg.substring(eq + 1));
-                break;
+            int hashIndex = urlText.indexOf("#map");
+            if (hashIndex >= 0) {
+                zoomLvl = Integer.parseInt(urlText.substring(hashIndex+5, urlText.indexOf('/', hashIndex)));
+            } else {
+                String[] args = urlText.substring(urlText.indexOf('?')+1).split("&");
+                for (String arg : args) {
+                    int eq = arg.indexOf('=');
+                    if (eq == -1 || !arg.substring(0, eq).equalsIgnoreCase("zoom")) continue;
+    
+                    zoomLvl = Integer.parseInt(arg.substring(eq + 1));
+                    break;
+                }
             }
 
@@ -162 +177 @@
             dlon = Math.round(dlon * decimals);
             dlon /= decimals;
-            url.setText("http://www.openstreetmap.org/?lat="+dlat+"&lon="+dlon+"&zoom="+zoomLvl);
+            url.setText("http://www.openstreetmap.org/#map="+zoomLvl+"/"+dlat+"/"+dlon);
         } catch (NumberFormatException x) {}
     }