Changeset 1712 in josm

Timestamp:

Jun 28, 2009 3:28:36 PM (4 years ago)

Author:

stoecker

Message:

added security check for unwanted moves

File:

• trunk/src/org/openstreetmap/josm/actions/mapmode/SelectAction.java (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/actions/mapmode/SelectAction.java

@@ -17 +17 @@
 import java.util.LinkedList;
 import java.util.List;
+import java.util.TreeSet;
 
 import javax.swing.JOptionPane;
@@ -36 +37 @@
 import org.openstreetmap.josm.data.osm.visitor.AllNodesVisitor;
 import org.openstreetmap.josm.data.osm.visitor.SimplePaintVisitor;
+import org.openstreetmap.josm.gui.ExtendedDialog;
 import org.openstreetmap.josm.gui.MapFrame;
 import org.openstreetmap.josm.gui.MapView;
@@ -417 +419 @@
             } else {
                 Collection<OsmPrimitive> selection = Main.ds.getSelected();
+                Collection<OsmPrimitive> s = new TreeSet<OsmPrimitive>();
+                int max = Main.pref.getInteger("warn.move.maxelements", 20);
+                for (OsmPrimitive osm : selection)
+                {
+                    if(osm instanceof Node)
+                        s.add(osm);
+                    else if(osm instanceof Way)
+                    {
+                        s.add(osm);
+                        s.addAll(((Way)osm).nodes);
+                    }
+                    if(s.size() > max)
+                    {
+                        if(1 != new ExtendedDialog(Main.parent, tr("Move elements"),
+                        tr("You did move more than {0} elements. "
+                        + "Moving a large number of elements is often an error.\n"
+                        + "Really move them?", max),
+                        new String[] {tr("Move them"), tr("Undo move")},
+                        new String[] {"reorder.png", "cancel.png"}).getValue())
+                        {
+                            Main.main.undoRedo.undo();
+                        }
+                        break;
+                    }
+                }
                 if (ctrl) {
                     Collection<Node> affectedNodes = AllNodesVisitor.getAllNodes(selection);