Changeset 13062 in josm

Timestamp:

2017-10-30T23:11:47+01:00 (6 years ago)

Author:

Don-vip

Message:

fix #5823 - fix calculation of "moved many elements" warning

File:

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

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

@@ -18 +18 @@
 import java.util.LinkedList;
 import java.util.Optional;
+import java.util.Set;
 
 import javax.swing.JOptionPane;
@@ -829 +830 @@
             }
         }
-        int max = Config.getPref().getInt("warn.move.maxelements", 20), limit = max;
+        Set<Node> nodes = new HashSet<>();
+        int max = Config.getPref().getInt("warn.move.maxelements", 20);
         for (OsmPrimitive osm : getLayerManager().getEditDataSet().getSelected()) {
             if (osm instanceof Way) {
-                limit -= ((Way) osm).getNodes().size();
-            }
-            if (--limit < 0) {
+                nodes.addAll(((Way) osm).getNodes());
+            } else if (osm instanceof Node) {
+                nodes.add((Node) osm);
+            }
+            if (nodes.size() > max) {
                 break;
             }
         }
-        if (limit < 0) {
+        if (nodes.size() > max) {
             final ExtendedDialog ed = new ConfirmMoveDialog();
             ed.setContent(