Changeset 16119 in josm for trunk/src/org/openstreetmap/josm/command/ChangeCommand.java

Timestamp:

2020-03-14T13:26:33+01:00 (4 years ago)

Author:

Don-vip

Message:

fix #18928 - fix various crashes with empty ways

File:

• trunk/src/org/openstreetmap/josm/command/ChangeCommand.java (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/command/ChangeCommand.java

@@ -32 +32 @@
      * @param osm The existing primitive to modify. It must belong to a data set
      * @param newOsm The new primitive
+     * @throws IllegalArgumentException if sanity checks fail
      */
     public ChangeCommand(OsmPrimitive osm, OsmPrimitive newOsm) {
@@ -42 +43 @@
      * @param osm The existing primitive to modify
      * @param newOsm The new primitive
+     * @throws IllegalArgumentException if sanity checks fail
      * @since 11240
      */
@@ -54 +56 @@
         CheckParameterUtil.ensureParameterNotNull(osm, "osm");
         CheckParameterUtil.ensureParameterNotNull(newOsm, "newOsm");
-        if (newOsm instanceof Way && ((Way) newOsm).getNodesCount() == 0) {
+        if (newOsm instanceof Way && ((Way) newOsm).isEmpty()) {
             // Do not allow to create empty ways (see #7465)
             throw new IllegalArgumentException(tr("New way {0} has 0 nodes", newOsm));