Changeset 5719 in josm

Timestamp:

2013-02-15T21:58:25+01:00 (11 years ago)

Author:

Don-vip

Message:

see #8431 - Align in circle: Do not allow "regular polygon" mode for way nodes being referred by another primitives. The initial problem (way being reverted) remains.

File:

• trunk/src/org/openstreetmap/josm/actions/AlignInCircleAction.java (modified) (4 diffs)

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

@@ -10 +10 @@
 import java.math.MathContext;
 import java.util.Collection;
+import java.util.HashSet;
 import java.util.LinkedList;
 import java.util.List;
+import java.util.Set;
 
 import javax.swing.JOptionPane;
@@ -116 +118 @@
             // distance between two nodes.
             if (nodes.size() > 0) {
-                if (nodes.size() == 1 && way.containsNode(nodes.get(0))) {
+                if (nodes.size() == 1 && way.containsNode(nodes.get(0)) && allowRegularPolygon(way.getNodes())) {
                     regular = true;
-                } else {
-
+                } else if (nodes.size() >= 2) {
                     center = nodes.get(way.containsNode(nodes.get(0)) ? 1 : 0).getEastNorth();
                     if (nodes.size() == 2) {
@@ -125 +126 @@
                     }
                 }
-                nodes = new LinkedList<Node>();
+                nodes.clear();
             }
 
@@ -239 +240 @@
         setEnabled(selection != null && !selection.isEmpty());
     }
+    
+    /**
+     * Determines if a regular polygon is allowed to be created with the given nodes collection.
+     * @param nodes The nodes collection to check.
+     * @return true if all nodes in the given collection are referred by the same object, and no other one (see #8431)
+     */
+    protected boolean allowRegularPolygon(Collection<Node> nodes) {
+        Set<OsmPrimitive> allReferrers = new HashSet<OsmPrimitive>();
+        for (Node n : nodes) {
+            List<OsmPrimitive> referrers = n.getReferrers();
+            if (referrers.size() > 1 || (allReferrers.addAll(referrers) && allReferrers.size() > 1)) {
+                return false;
+            }
+        }
+        return true;
+    }
 }