+ * This should either never be called or only called in the constructor of this action. + *
+ * All registered adapters should be removed in {@link #destroy()}
+ */
+ protected void installAdapters() {
+ // make this action listen to layer change and selection change events
+ if (listenToLayerChange()) {
+ layerChangeAdapter = new LayerChangeAdapter();
+ activeLayerChangeAdapter = new ActiveLayerChangeAdapter();
+ getLayerManager().addLayerChangeListener(layerChangeAdapter);
+ getLayerManager().addActiveLayerChangeListener(activeLayerChangeAdapter);
+ }
+ if (listenToSelectionChange()) {
+ selectionChangeAdapter = new SelectionChangeAdapter();
+ DataSet.addSelectionListener(selectionChangeAdapter);
+ }
+ initEnabledState();
+ }
+
+ /**
+ * Overwrite this if {@link #updateEnabledState()} should be called when the active / availabe layers change. Default is true.
+ * @return true if a {@link LayerChangeListener} and a {@link ActiveLayerChangeListener} should be registered.
+ * @since 10352
+ */
+ protected boolean listenToLayerChange() {
+ return true;
+ }
+
+ /**
+ * Overwrite this if {@link #updateEnabledState()} should be called when the selection changed. Default is true.
+ * @return true if a {@link SelectionChangedListener} should be registered.
+ * @since 10352
+ */
+ protected boolean listenToSelectionChange() {
+ return true;
+ }
+
@Override
public void destroy() {
@@ -184,7 +219,10 @@
}
if (layerChangeAdapter != null) {
- Main.getLayerManager().removeActiveLayerChangeListener(layerChangeAdapter);
- }
- DataSet.removeSelectionListener(selectionChangeAdapter);
+ getLayerManager().removeLayerChangeListener(layerChangeAdapter);
+ getLayerManager().removeActiveLayerChangeListener(activeLayerChangeAdapter);
+ }
+ if (selectionChangeAdapter != null) {
+ DataSet.removeSelectionListener(selectionChangeAdapter);
+ }
}
@@ -198,4 +236,18 @@
/**
+ * Returns the shortcut for this action.
+ * @return the shortcut for this action, or "No shortcut" if none is defined
+ */
+ public Shortcut getShortcut() {
+ if (sc == null) {
+ sc = Shortcut.registerShortcut("core:none", tr("No Shortcut"), KeyEvent.CHAR_UNDEFINED, Shortcut.NONE);
+ // as this shortcut is shared by all action that don't want to have a shortcut,
+ // we shouldn't allow the user to change it...
+ // this is handled by special name "core:none"
+ }
+ return sc;
+ }
+
+ /**
* Sets the tooltip text of this action.
* @param tooltip The text to display in tooltip. Can be {@code null}
@@ -208,10 +260,24 @@
/**
+ * Gets the layer manager used for this action. Defaults to the main layer manager but you can overwrite this.
+ *
+ * The layer manager must be available when {@link #installAdapters()} is called and must not change. + * + * @return The layer manager. + * @since 10352 + */ + public MainLayerManager getLayerManager() { + return Main.getLayerManager(); + } + + /** * Replies the current edit layer * * @return the current edit layer. null, if no edit layer exists - */ + * @deprecated Use {@link #getLayerManager()}.getEditLayer() instead. To be removed in end of 2016. + */ + @Deprecated public static OsmDataLayer getEditLayer() { - return Main.main != null ? Main.main.getEditLayer() : null; + return Main.getLayerManager().getEditLayer(); } @@ -220,16 +286,9 @@ * * @return the current dataset. null, if no current dataset exists - */ + * @deprecated Use {@link #getLayerManager()}.getEditDataSet() instead. To be removed in end of 2016. + */ + @Deprecated public static DataSet getCurrentDataSet() { - return Main.main != null ? Main.main.getCurrentDataSet() : null; - } - - protected void installAdapters() { - // make this action listen to layer change and selection change events - layerChangeAdapter = new LayerChangeAdapter(); - selectionChangeAdapter = new SelectionChangeAdapter(); - Main.getLayerManager().addActiveLayerChangeListener(layerChangeAdapter); - DataSet.addSelectionListener(selectionChangeAdapter); - initEnabledState(); + return Main.getLayerManager().getEditDataSet(); } @@ -273,4 +332,5 @@ * @see #updateEnabledState(Collection) * @see #initEnabledState() + * @see #listenToLayerChange() */ protected void updateEnabledState() { @@ -286,4 +346,5 @@ * @see #updateEnabledState() * @see #initEnabledState() + * @see #listenToSelectionChange() */ protected void updateEnabledState(Collection selection) { @@ -293,5 +354,30 @@ * Adapter for layer change events. Runs updateEnabledState() whenever the active layer changed. */ - protected class LayerChangeAdapter implements ActiveLayerChangeListener { + protected class LayerChangeAdapter implements LayerChangeListener { + @Override + public void layerAdded(LayerAddEvent e) { + updateEnabledState(); + } + + @Override + public void layerRemoving(LayerRemoveEvent e) { + updateEnabledState(); + } + + @Override + public void layerOrderChanged(LayerOrderChangeEvent e) { + updateEnabledState(); + } + + @Override + public String toString() { + return "LayerChangeAdapter [" + JosmAction.this.toString() + ']'; + } + } + + /** + * Adapter for layer change events. Runs updateEnabledState() whenever the active layer changed. + */ + protected class ActiveLayerChangeAdapter implements ActiveLayerChangeListener { @Override public void activeOrEditLayerChanged(ActiveLayerChangeEvent e) { @@ -301,5 +387,5 @@ @Override public String toString() { - return "LayerChangeAdapter [" + JosmAction.this.toString() + ']'; + return "ActiveLayerChangeAdapter [" + JosmAction.this.toString() + ']'; } }