Changeset 13964 in josm for trunk/src/org/openstreetmap/josm/actions

Timestamp:

2018-06-22T17:54:19+02:00 (6 years ago)

Author:

Don-vip

Message:

fix #16415 - NPE

File:

• trunk/src/org/openstreetmap/josm/actions/mapmode/ImproveWayAccuracyAction.java (modified) (1 diff)

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

@@ -644 +644 @@
      * state if a single way or node is selected. Extracts a way by a node in
      * the second case.
-     *
      */
     private void updateStateByCurrentSelection() {
         final List<Node> nodeList = new ArrayList<>();
         final List<Way> wayList = new ArrayList<>();
-        final Collection<OsmPrimitive> sel = getLayerManager().getEditDataSet().getSelected();
-
-        // Collecting nodes and ways from the selection
-        for (OsmPrimitive p : sel) {
-            if (p instanceof Way) {
-                wayList.add((Way) p);
-            }
-            if (p instanceof Node) {
-                nodeList.add((Node) p);
-            }
-        }
-
-        if (wayList.size() == 1) {
-            // Starting improving the single selected way
-            startImproving(wayList.get(0));
-            return;
-        } else if (nodeList.size() == 1) {
-            // Starting improving the only way of the single selected node
-            List<OsmPrimitive> r = nodeList.get(0).getReferrers();
-            if (r.size() == 1 && (r.get(0) instanceof Way)) {
-                startImproving((Way) r.get(0));
+        final DataSet ds = getLayerManager().getEditDataSet();
+        if (ds != null) {
+            final Collection<OsmPrimitive> sel = ds.getSelected();
+
+            // Collecting nodes and ways from the selection
+            for (OsmPrimitive p : sel) {
+                if (p instanceof Way) {
+                    wayList.add((Way) p);
+                }
+                if (p instanceof Node) {
+                    nodeList.add((Node) p);
+                }
+            }
+
+            if (wayList.size() == 1) {
+                // Starting improving the single selected way
+                startImproving(wayList.get(0));
                 return;
+            } else if (nodeList.size() == 1) {
+                // Starting improving the only way of the single selected node
+                List<OsmPrimitive> r = nodeList.get(0).getReferrers();
+                if (r.size() == 1 && (r.get(0) instanceof Way)) {
+                    startImproving((Way) r.get(0));
+                    return;
+                }
             }
         }