Ticket #13538: 13538-deprecate.patch

src/org/openstreetmap/josm/data/coor/LatLon.java

@@ -17 +17 @@
 import java.util.Objects;
 
 import org.openstreetmap.josm.data.Bounds;
+import org.openstreetmap.josm.data.osm.Node;
 import org.openstreetmap.josm.data.projection.ProjectionRegistry;
 import org.openstreetmap.josm.tools.Logging;
 import org.openstreetmap.josm.tools.Utils;
@@ -192 +193 @@
     /**
      * Determines if this lat/lon is outside of the world
      * @return <code>true</code>, if the coordinate is outside the world, compared by using lat/lon.
+     * @deprecated use {@link Node#isOutSideWorld} instead, see also #13538.
      */
+    @Deprecated
     public boolean isOutSideWorld() {
         Bounds b = ProjectionRegistry.getProjection().getWorldBoundsLatLon();
         return lat() < b.getMinLat() || lat() > b.getMaxLat() ||

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

@@ -10 +10 @@
 import java.util.function.Predicate;
 import java.util.stream.Collectors;
 
+import org.openstreetmap.josm.data.Bounds;
 import org.openstreetmap.josm.data.coor.EastNorth;
 import org.openstreetmap.josm.data.coor.LatLon;
 import org.openstreetmap.josm.data.osm.visitor.OsmPrimitiveVisitor;
@@ -411 +412 @@
     public boolean isOutSideWorld() {
         LatLon ll = getCoor();
         if (ll != null) {
-            if (ll.isOutSideWorld())
+            Bounds b = ProjectionRegistry.getProjection().getWorldBoundsLatLon();
+            if (lat() < b.getMinLat() || lat() > b.getMaxLat() || lon() < b.getMinLon() || lon() > b.getMaxLon()) {
                 return true;
+            }
             if (!ProjectionRegistry.getProjection().latlon2eastNorth(ll).equalsEpsilon(getEastNorth(), 1.0)) {
                 // we get here if a node was moved or created left from -180 or right from +180
                 return true;