Changeset 1351 in josm for trunk/src/org/openstreetmap

Timestamp:

2009-01-31T14:43:27+01:00 (16 years ago)

Author:

stoecker

Message:

close #1669. Patch by xeen

Location:

trunk/src/org/openstreetmap/josm/actions

Files:

• CopyAction.java (modified) (3 diffs)
• DuplicateAction.java (modified) (2 diffs)
• PasteAction.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/actions/CopyAction.java

@@ -44 +44 @@
 
     public void actionPerformed(ActionEvent e) {
-        Collection<OsmPrimitive> sel = Main.ds.getSelected();
-        if (sel.isEmpty()) {
-            JOptionPane.showMessageDialog(Main.parent,
-                    tr("Please select something to copy."));
-            return;
+        if(noSelection()) return;
+        
+        Main.pasteBuffer = copyData();
+        Main.main.menu.paste.setEnabled(true); /* now we have a paste buffer we can make paste available */
+
+        for(JosmAction a : listeners) {
+            a.pasteBufferChanged(Main.pasteBuffer);
         }
-
+    }
+    
+    public static DataSet copyData() {
         /* New pasteBuffer - will be assigned to the global one at the end */
         final DataSet pasteBuffer = new DataSet();
         final HashMap<OsmPrimitive,OsmPrimitive> map = new HashMap<OsmPrimitive,OsmPrimitive>();
         /* temporarily maps old nodes to new so we can do a true deep copy */
-
+        
+        if(noSelection()) return pasteBuffer;
+        
         /* scan the selected objects, mapping them to copies; when copying a way or relation,
          * the copy references the copies of their child objects */
@@ -105 +111 @@
             }
         }.visitAll();
-
-        Main.pasteBuffer = pasteBuffer;
-        Main.main.menu.paste.setEnabled(true); /* now we have a paste buffer we can make paste available */
-
-        for(JosmAction a : listeners) {
-            a.pasteBufferChanged(Main.pasteBuffer);
-        }
+        
+        return pasteBuffer;
     }
 
@@ -117 +118 @@
         setEnabled(! newSelection.isEmpty());
     }
+    
+    private static boolean noSelection() {
+        Collection<OsmPrimitive> sel = Main.ds.getSelected();
+        if (sel.isEmpty()) {
+            JOptionPane.showMessageDialog(Main.parent,
+                    tr("Please select something to copy."));
+            return true;
+        }
+        return false;
+    }
 }

trunk/src/org/openstreetmap/josm/actions/DuplicateAction.java

@@ -10 +10 @@
 
 import org.openstreetmap.josm.Main;
+import org.openstreetmap.josm.actions.CopyAction;
+import org.openstreetmap.josm.actions.PasteAction;
 import org.openstreetmap.josm.data.SelectionChangedListener;
 import org.openstreetmap.josm.data.osm.DataSet;
@@ -22 +24 @@
             Shortcut.registerShortcut("system:duplicate", tr("Edit: {0}", tr("Duplicate")), KeyEvent.VK_D, Shortcut.GROUP_MENU), true);
         setEnabled(false);
-            DataSet.selListeners.add(this);
+        DataSet.selListeners.add(this);
     }
 
     public void actionPerformed(ActionEvent e) {
-        Main.main.menu.copy.actionPerformed(e);
-        Main.main.menu.paste.actionPerformed(e);
+        PasteAction.pasteData(CopyAction.copyData(), e);
     }
 

trunk/src/org/openstreetmap/josm/actions/PasteAction.java

@@ -31 +31 @@
         super(tr("Paste"), "paste", tr("Paste contents of paste buffer."),
             Shortcut.registerShortcut("system:paste", tr("Edit: {0}", tr("Paste")), KeyEvent.VK_V, Shortcut.GROUP_MENU), true);
-            setEnabled(false);
+        setEnabled(false);
     }
 
     public void actionPerformed(ActionEvent e) {
-        DataSet pasteBuffer = Main.pasteBuffer;
-
+        pasteData(Main.pasteBuffer, e);
+    }
+    
+    public static void pasteData(DataSet pasteBuffer, ActionEvent e) {
         /* Find the middle of the pasteBuffer area */
         double maxEast = -1E100, minEast = 1E100, maxNorth = -1E100, minNorth = 1E100;