Changeset 14685 in josm for trunk/src/org/openstreetmap/josm/gui/dialogs/RelationListDialog.java

Timestamp:

2019-01-12T21:19:22+01:00 (7 years ago)

Author:

simon04

Message:

fix #14462 - Harmonize relation actions provided in popup menus

File:

• trunk/src/org/openstreetmap/josm/gui/dialogs/RelationListDialog.java (modified) (11 diffs)

trunk/src/org/openstreetmap/josm/gui/dialogs/RelationListDialog.java

@@ -34 +34 @@
 
 import org.openstreetmap.josm.actions.ExpertToggleAction;
-import org.openstreetmap.josm.actions.IPrimitiveAction;
 import org.openstreetmap.josm.actions.relation.AddSelectionToRelations;
 import org.openstreetmap.josm.actions.relation.DeleteRelationsAction;
-import org.openstreetmap.josm.actions.relation.DownloadMembersAction;
-import org.openstreetmap.josm.actions.relation.DownloadSelectedIncompleteMembersAction;
 import org.openstreetmap.josm.actions.relation.DuplicateRelationAction;
 import org.openstreetmap.josm.actions.relation.EditRelationAction;
@@ -44 +41 @@
 import org.openstreetmap.josm.actions.relation.ExportRelationToGpxAction.Mode;
 import org.openstreetmap.josm.actions.relation.RecentRelationsAction;
-import org.openstreetmap.josm.actions.relation.SelectMembersAction;
+import org.openstreetmap.josm.actions.relation.SelectInRelationListAction;
 import org.openstreetmap.josm.actions.relation.SelectRelationAction;
 import org.openstreetmap.josm.data.osm.DataSet;
@@ -73 +70 @@
 import org.openstreetmap.josm.gui.SideButton;
 import org.openstreetmap.josm.gui.dialogs.relation.RelationEditor;
+import org.openstreetmap.josm.gui.dialogs.relation.RelationPopupMenus;
 import org.openstreetmap.josm.gui.layer.LayerManager.LayerAddEvent;
 import org.openstreetmap.josm.gui.layer.LayerManager.LayerChangeListener;
@@ -99 +97 @@
  */
 public class RelationListDialog extends ToggleDialog
-        implements DataSetListener, NavigatableComponent.ZoomChangeListener, ExpertToggleAction.ExpertModeChangeListener {
+        implements DataSetListener, NavigatableComponent.ZoomChangeListener {
     /** The display list. */
     private final JList<IRelation<?>> displaylist;
@@ -120 +118 @@
     /** the duplicate action */
     private final DuplicateRelationAction duplicateAction = new DuplicateRelationAction();
-    private final DownloadMembersAction downloadMembersAction = new DownloadMembersAction();
-    private final DownloadSelectedIncompleteMembersAction downloadSelectedIncompleteMembersAction =
-            new DownloadSelectedIncompleteMembersAction();
-    private final SelectMembersAction selectMembersAction = new SelectMembersAction(false);
-    private final SelectMembersAction addMembersToSelectionAction = new SelectMembersAction(true);
+    /** the select relation action */
     private final SelectRelationAction selectRelationAction = new SelectRelationAction(false);
-    private final SelectRelationAction addRelationToSelectionAction = new SelectRelationAction(true);
     /** add all selected primitives to the given relations */
     private final AddSelectionToRelations addSelectionToRelations = new AddSelectionToRelations();
-    private transient JMenuItem addSelectionToRelationMenuItem;
 
     /** export relation to GPX track action */
@@ -245 +237 @@
         SelectionEventManager.getInstance().addSelectionListener(addSelectionToRelations);
         dataChanged(null);
-        ExpertToggleAction.addExpertModeChangeListener(this);
-        expertChanged(ExpertToggleAction.isExpert());
     }
 
@@ -256 +246 @@
         DatasetEventManager.getInstance().removeDatasetListener(this);
         SelectionEventManager.getInstance().removeSelectionListener(addSelectionToRelations);
-        ExpertToggleAction.removeExpertModeChangeListener(this);
     }
 
@@ -632 +621 @@
         List<JMenuItem> checkDisabled = new ArrayList<>();
 
-        // -- select action
-        popupMenuHandler.addAction(selectRelationAction);
-        popupMenuHandler.addAction(addRelationToSelectionAction);
-
-        // -- select members action
-        popupMenuHandler.addAction(selectMembersAction);
-        popupMenuHandler.addAction(addMembersToSelectionAction);
-
-        // -- download members action
-        popupMenuHandler.addSeparator();
-        popupMenuHandler.addAction(downloadMembersAction);
-        popupMenuHandler.addAction(downloadSelectedIncompleteMembersAction);
+        RelationPopupMenus.setupHandler(popupMenuHandler, SelectInRelationListAction.class);
 
         // -- export relation to gpx action
@@ -658 +636 @@
         popupMenuHandler.addAction(deleteRelationsAction).setVisible(false);
 
-        addSelectionToRelationMenuItem = popupMenuHandler.addAction(addSelectionToRelations);
+        ExpertToggleAction.addVisibilitySwitcher(popupMenuHandler.addAction(addSelectionToRelations));
 
         popupMenuHandler.addListener(new PopupMenuListener() {
@@ -664 +642 @@
             public void popupMenuWillBecomeVisible(PopupMenuEvent e) {
                 for (JMenuItem mi: checkDisabled) {
-                    mi.setVisible(((IPrimitiveAction) mi.getAction()).isEnabled());
-
-                    Component sep = popupMenu.getComponent(
-                            Math.max(0, popupMenu.getComponentIndex(mi)-1));
+                    mi.setVisible(mi.getAction().isEnabled());
+                    Component sep = popupMenu.getComponent(Math.max(0, popupMenu.getComponentIndex(mi) - 1));
                     if (!(sep instanceof JMenuItem)) {
                         sep.setVisible(mi.isVisible());
@@ -770 +746 @@
         }
     }
-
-    @Override
-    public void expertChanged(boolean isExpert) {
-        addSelectionToRelationMenuItem.setVisible(isExpert);
-    }
 }