Ticket #22879: josmrelcut.diff

src/org/openstreetmap/josm/gui/dialogs/relation/GenericRelationEditor.java

@@ -308 +308 @@
         key = Shortcut.getCopyKeyStroke();
         if (key != null) {
             // handle uncommon situation, that user has no keystroke assigned to copy
-            registerCopyPasteAction(new CopyMembersAction(actionAccess),
+            registerCopyPasteAction(new CopyMembersAction(actionAccess, true),
                     "COPY_MEMBERS", key, getRootPane(), memberTable, selectionTable);
         }
+        key = Shortcut.getCutKeyStroke();
+        if (key != null) {
+                        // handle uncommon situation, that user has no keystroke assigned to cut
+                        registerCopyPasteAction(new CopyMembersAction(actionAccess, false),
+                                        "CUT_MEMBERS", key, getRootPane(), memberTable, selectionTable);
+                }
         tagEditorPanel.setNextFocusComponent(memberTable);
         selectionTable.setFocusable(false);
         memberTableModel.setSelectedMembers(selectedMembers);

src/org/openstreetmap/josm/gui/dialogs/relation/actions/CopyMembersAction.java

@@ -14 +14 @@
  */
 public class CopyMembersAction extends AddFromSelectionAction {
     private static final long serialVersionUID = 1L;
+    private boolean keepCopiedMembers = true;
 
     /**
      * Constructs a new {@code CopyMembersAction}.
      * @param editorAccess An interface to access the relation editor contents.
+     * @param keepCopiedMembers if true, copied members are kept in the table; otherwise they are removed (cut)
      */
-    public CopyMembersAction(IRelationEditorActionAccess editorAccess) {
+    public CopyMembersAction(IRelationEditorActionAccess editorAccess, boolean keepCopiedMembers) {
         super(editorAccess);
+        this.keepCopiedMembers = keepCopiedMembers;
     }
 
     @Override
@@ -29 +32 @@
 
         if (!members.isEmpty()) {
             ClipboardUtils.copy(new RelationMemberTransferable(members));
+            if (!this.keepCopiedMembers) {
+                getMemberTableModel().remove(getMemberTableModel().getSelectedIndices());
+            }
         }
     }
 

test/unit/org/openstreetmap/josm/gui/dialogs/relation/actions/RelationEditorActionsTest.java

@@ -48 +48 @@
         new OKAction(relationEditorAccess).actionPerformed(null);
         new CancelAction(relationEditorAccess).actionPerformed(null);
 
-        new CopyMembersAction(relationEditorAccess).actionPerformed(null);
+        new CopyMembersAction(relationEditorAccess, true).actionPerformed(null);
+        new CopyMembersAction(relationEditorAccess, false).actionPerformed(null);
         new PasteMembersAction(relationEditorAccess).actionPerformed(null);
 
         new SelectAction(relationEditorAccess).actionPerformed(null);