Changeset 4408 in josm for trunk/src/org/openstreetmap/josm/actions/HistoryInfoAction.java

Timestamp:

2011-09-08T00:08:02+02:00 (13 years ago)

Author:

simon04

Message:

fix #6773 - shortcuts for History and Advanced info dialog

File:

• trunk/src/org/openstreetmap/josm/actions/HistoryInfoAction.java (modified) (1 diff)

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

@@ -2 +2 @@
 package org.openstreetmap.josm.actions;
 
-import static org.openstreetmap.josm.tools.I18n.tr;
-
+import java.awt.event.ActionEvent;
 import java.awt.event.KeyEvent;
+import java.util.Collection;
 
 import org.openstreetmap.josm.Main;
+import org.openstreetmap.josm.data.osm.DataSet;
 import org.openstreetmap.josm.data.osm.OsmPrimitive;
-import org.openstreetmap.josm.data.osm.OsmPrimitiveType;
+import org.openstreetmap.josm.gui.history.HistoryBrowserDialogManager;
 import org.openstreetmap.josm.tools.Shortcut;
+
+import static org.openstreetmap.josm.tools.I18n.tr;
 import static org.openstreetmap.josm.gui.help.HelpUtil.ht;
 
-public class HistoryInfoAction extends AbstractInfoAction {
+public class HistoryInfoAction extends JosmAction {
 
-    public HistoryInfoAction() {
-        super(tr("Object history"), "about",
-                tr("Display history information about OSM ways, nodes, or relations."),
-                Shortcut.registerShortcut("core:history",
-                        tr("Object history"), KeyEvent.VK_H, Shortcut.GROUP_HOTKEY), false);
-        putValue("help", ht("/Action/ObjectHistory"));
-        putValue("toolbar", "action/historyinfo");
-        Main.toolbar.register(this);
-    }
+        public HistoryInfoAction() {
+                super(tr("History"), "about",
+                                tr("Display history information about OSM ways, nodes, or relations."),
+                                Shortcut.registerShortcut("core:historyinfo",
+                                tr("History"), KeyEvent.VK_H, Shortcut.GROUP_HOTKEY), false);
+                putValue("help", ht("/Action/ObjectHistory"));
+                putValue("toolbar", "action/historyinfo");
+                Main.toolbar.register(this);
+        }
 
-    @Override
-    protected  String createInfoUrl(Object infoObject) {
-        OsmPrimitive primitive = (OsmPrimitive)infoObject;
-        return getBaseBrowseUrl() + "/" + OsmPrimitiveType.from(primitive).getAPIName() + "/" + primitive.getId() + "/history";
-    }
+        @Override
+        public void actionPerformed(ActionEvent ae) {
+                DataSet set = getCurrentDataSet();
+                if (set != null) {
+                        HistoryBrowserDialogManager.getInstance().showHistory(set.getSelected());
+                }
+        }
+
+        @Override
+        public void updateEnabledState() {
+                if (getCurrentDataSet() == null) {
+                        setEnabled(false);
+                } else {
+                        updateEnabledState(getCurrentDataSet().getSelected());
+                }
+        }
+
+        @Override
+        protected void updateEnabledState(Collection<? extends OsmPrimitive> selection) {
+                setEnabled(!selection.isEmpty());
+        }
 }