Changeset 13043 in josm

Timestamp:

2017-10-27T00:59:01+02:00 (7 years ago)

Author:

Don-vip

Message:

see #15474 - fix warnings

File:

• trunk/src/org/openstreetmap/josm/gui/history/OpenChangesetPopupMenu.java (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/gui/history/OpenChangesetPopupMenu.java

@@ -56 +56 @@
             new ChangesetViewerEntry(tr("Open {0}", "OSMCha (OSM Changeset Analyzer)"), "https://osmcha.mapbox.com/changesets/{0}"),
             new ChangesetViewerEntry(tr("Open {0}", "OSM History Viewer"), "http://osmhv.openstreetmap.de/changeset.jsp?id={0}"),
-            new ChangesetViewerEntry(tr("Open {0}", "WhoDidIt (OSM Changeset Analyzer)"), "http://simon04.dev.openstreetmap.org/whodidit/index.html?changeset={0}&show=1")
+            new ChangesetViewerEntry(tr("Open {0}", "WhoDidIt (OSM Changeset Analyzer)"),
+                    "http://simon04.dev.openstreetmap.org/whodidit/index.html?changeset={0}&show=1")
     );
 
@@ -63 +64 @@
      */
     public static class ChangesetViewerEntry {
+        /** Name to be displayed in popup menu */
         @StructEntry
         public String name;
+        /** Templated service url. <code>{0}</code> will be replaced by changeset id */
         @StructEntry
         public String url;
@@ -80 +83 @@
 
         Action toAction(final long changesetId) {
-            final String url = MessageFormat.format(this.url, Long.toString(changesetId));
-            return new OpenBrowserAction(name, url);
+            return new OpenBrowserAction(name, MessageFormat.format(this.url, Long.toString(changesetId)));
         }
     }