DEL button deletes Maps from Projects

Jody Garnett added a comment - 30/Nov/11 6:40 AM - edited Okay I am gradually finding out the limits on this problem: - the delete key does the correct thing from the layers view (DEL removes a layer) - it does NOT do the correct thing from the projects view (i.e. Project Explorer) In comparing the two implementations; the LayersView takes a few extra steps with the creation of its own Delete action (rather than recycling the one from ToolbarManager). I am going to make them work the same and then test. Update: Okay that did not work out (i.e. no effect other than consistent code). ------- Looking at UDIGGenericAction (the super class of Delete) now. The base UDIGGenericAction class it checks the current workbench selection; and dispatches accordingly. 1) The selection (for a Map) comes from ReplaceableSelectionProvider. 2) For the Zoom and Pan tools that class delegates to MapEditorSelectionProvider - which has the Map as the selected item So for the zoom and pan tools considers the map to be the current selection (incase people want to look at properties or something). So the delete key is correctly deleting the current selection ... what can we do about that.

Jody Garnett added a comment - 30/Nov/11 8:43 AM - edited Okay I have made delete action have a creation flag to indicate if it can delete project elements or not. The ToolManager copy of Delete cannot do this; but the one created int the Projects Explorer can. I also cleaned up a lot of the code to be explicit with its return type of Pair<boolean,Int> and fixed a few NPE and was explicit about checking Window.CANCEL. Will submit a pull request now: - https://github.com/uDig/udig-platform/pull/80

Jody Garnett added a comment - 30/Nov/11 9:20 AM So my pull request is fine; in that it fixes the symptom. The real problem here is the tool selection is being given to the ToolProxy; but the ReplaceableSelectionProvider used by the Map is not correctly delegating to the ToolProxy.

Jody Garnett added a comment - 30/Nov/11 9:20 AM So my pull request is fine; in that it fixes the symptom. The real problem here is the tool selection is being given to the ToolProxy; but the ReplaceableSelectionProvider used by the Map is not correctly delegating to the ToolProxy.

Jody Garnett added a comment - 30/Nov/11 10:31 AM Okay I have introduced additional methods into MapEditorPart interface; and cleaned up the code that handled selection to work against this interface.