Ignore:
Timestamp:
2018-04-10T19:02:58+02:00 (8 months ago)
Author:
Don-vip
Message:

fix #16176 - NPE

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/actions/MergeNodesAction.java

    r13453 r13611  
    2828import org.openstreetmap.josm.data.coor.EastNorth;
    2929import org.openstreetmap.josm.data.coor.LatLon;
    30 import org.openstreetmap.josm.data.osm.DataSet;
    3130import org.openstreetmap.josm.data.osm.DefaultNameFormatter;
    3231import org.openstreetmap.josm.data.osm.Node;
    3332import org.openstreetmap.josm.data.osm.OsmPrimitive;
     33import org.openstreetmap.josm.data.osm.OsmUtils;
    3434import org.openstreetmap.josm.data.osm.TagCollection;
    3535import org.openstreetmap.josm.data.osm.Way;
     
    365365    @Override
    366366    protected void updateEnabledState(Collection<? extends OsmPrimitive> selection) {
    367         if (selection == null || selection.isEmpty()
    368                 || selection.stream().map(OsmPrimitive::getDataSet).anyMatch(DataSet::isLocked)) {
    369             setEnabled(false);
    370             return;
    371         }
    372         boolean ok = true;
    373         for (OsmPrimitive osm : selection) {
    374             if (!(osm instanceof Node)) {
    375                 ok = false;
    376                 break;
     367        boolean ok = OsmUtils.isOsmCollectionEditable(selection);
     368        if (ok) {
     369            for (OsmPrimitive osm : selection) {
     370                if (!(osm instanceof Node)) {
     371                    ok = false;
     372                    break;
     373                }
    377374            }
    378375        }
Note: See TracChangeset for help on using the changeset viewer.