Changeset 14960 in josm for trunk/src/org/openstreetmap/josm/data/osm/Node.java

Timestamp:

2019-04-04T11:34:48+02:00 (5 years ago)

Author:

GerdP

Message:

see #13538:

• Add new method isOutSideWorld() to class Node and use the method where LatLon.isOutSideWorld() was used before
• Add a check in MoveNodeAction to make sure that rounding errors near 180 longitude don't move the node from east to west or vice versa

(I've removed the changes to reactions implemented in 13538-v2.patch, they would be too confusing.)

TODO: Plugins should use the new method in Node

File:

• trunk/src/org/openstreetmap/josm/data/osm/Node.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/data/osm/Node.java

@@ -403 +403 @@
         return referrers(Way.class).collect(Collectors.toList());
     }
+
+    /**
+     * Determines if this node is outside of the world. See also #13538.
+     * @return <code>true</code>, if the coordinate is outside the world, compared by using lat/lon and east/north
+     * @since 14960
+     */
+    public boolean isOutSideWorld() {
+        LatLon ll = getCoor();
+        if (ll != null) {
+            if (ll.isOutSideWorld())
+                return true;
+            if (!new Node(ll).getEastNorth().equalsEpsilon(getEastNorth(), 1.0)) {
+                // we get here if a node was moved or created left from -180 or right from +180
+                return true;
+            }
+        }
+        return false;
+    }
 }