Changeset 7444 in josm


Ignore:
Timestamp:
2014-08-27T00:53:36+02:00 (5 years ago)
Author:
Don-vip
Message:

fix #10427 - Problems deleting a photo layer

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/gui/layer/geoimage/ImageViewerDialog.java

    r7426 r7444  
    2323
    2424import org.openstreetmap.josm.Main;
     25import org.openstreetmap.josm.gui.MapView;
     26import org.openstreetmap.josm.gui.MapView.LayerChangeListener;
    2527import org.openstreetmap.josm.gui.dialogs.DialogsPanel.Action;
    2628import org.openstreetmap.josm.gui.dialogs.ToggleDialog;
     29import org.openstreetmap.josm.gui.layer.Layer;
    2730import org.openstreetmap.josm.tools.ImageProvider;
    2831import org.openstreetmap.josm.tools.Shortcut;
    2932import org.openstreetmap.josm.tools.date.DateUtils;
    3033
    31 public final class ImageViewerDialog extends ToggleDialog {
     34public final class ImageViewerDialog extends ToggleDialog implements LayerChangeListener {
    3235
    3336    private static final String COMMAND_ZOOM = "zoom";
     
    8790        btnPrevious.getInputMap(JComponent.WHEN_IN_FOCUSED_WINDOW).put(scPrev.getKeyStroke(), APREVIOUS);
    8891        btnPrevious.getActionMap().put(APREVIOUS, prevAction);
     92        btnPrevious.setEnabled(false);
    8993
    9094        final String DELETE_TEXT = tr("Remove photo from layer");
     
    117121        btnNext.getInputMap(JComponent.WHEN_IN_FOCUSED_WINDOW).put(scNext.getKeyStroke(), ANEXT);
    118122        btnNext.getActionMap().put(ANEXT, nextAction);
     123        btnNext.setEnabled(false);
    119124
    120125        Main.registerActionShortcut(
     
    167172
    168173        add(content, BorderLayout.CENTER);
     174
     175        MapView.addLayerChangeListener(this);
     176    }
     177
     178    @Override
     179    public void destroy() {
     180        MapView.removeLayerChangeListener(this);
     181        super.destroy();
    169182    }
    170183
     
    218231    public static void showImage(GeoImageLayer layer, ImageEntry entry) {
    219232        getInstance().displayImage(layer, entry);
    220         layer.checkPreviousNextButtons();
    221     }
    222     public static void setPreviousEnabled(Boolean value) {
     233        if (layer != null) {
     234            layer.checkPreviousNextButtons();
     235        } else {
     236            setPreviousEnabled(false);
     237            setNextEnabled(false);
     238        }
     239    }
     240
     241    /**
     242     * Enables (or disables) the "Previous" button.
     243     * @param value {@code true} to enable the button, {@code false} otherwise
     244     */
     245    public static void setPreviousEnabled(boolean value) {
    223246        getInstance().btnPrevious.setEnabled(value);
    224247    }
    225     public static void setNextEnabled(Boolean value) {
     248
     249    /**
     250     * Enables (or disables) the "Next" button.
     251     * @param value {@code true} to enable the button, {@code false} otherwise
     252     */
     253    public static void setNextEnabled(boolean value) {
    226254        getInstance().btnNext.setEnabled(value);
    227255    }
     
    285313            }
    286314        }
    287 
    288315    }
    289316
     
    334361        return getInstance().currentLayer;
    335362    }
     363
     364    @Override
     365    public void activeLayerChange(Layer oldLayer, Layer newLayer) {
     366        // Ignored
     367    }
     368
     369    @Override
     370    public void layerAdded(Layer newLayer) {
     371        // Ignored
     372    }
     373
     374    @Override
     375    public void layerRemoved(Layer oldLayer) {
     376        // Clear current image and layer if current layer is deleted
     377        if (currentLayer != null && currentLayer.equals(oldLayer)) {
     378            showImage(null, null);
     379        }
     380    }
    336381}
Note: See TracChangeset for help on using the changeset viewer.