Changeset 6886 in josm for trunk/src/org/openstreetmap/josm/data

Timestamp:

2014-02-26T23:33:44+01:00 (11 years ago)

Author:

Don-vip

Message:

fix #6373 - proper conflict resolution after failed attempt to delete an object referenced by an unknown one

File:

• trunk/src/org/openstreetmap/josm/data/osm/DataSet.java (modified) (7 diffs)

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

@@ -10 +10 @@
 import java.util.Collections;
 import java.util.HashMap;
+import java.util.HashSet;
 import java.util.Iterator;
 import java.util.LinkedHashSet;
@@ -814 +815 @@
 
     /**
-     * removes all references from ways in this dataset to a particular node
+     * Removes all references from ways in this dataset to a particular node.
      *
      * @param node the node
-     */
-    public void unlinkNodeFromWays(Node node) {
+     * @return The set of ways that have been modified
+     */
+    public Set<Way> unlinkNodeFromWays(Node node) {
+        Set<Way> result = new HashSet<Way>();
         beginUpdate();
         try {
@@ -829 +832 @@
                         way.setNodes(wayNodes);
                     }
+                    result.add(way);
                 }
             }
@@ -834 +838 @@
             endUpdate();
         }
+        return result;
     }
 
@@ -840 +845 @@
      *
      * @param primitive the primitive
-     */
-    public void unlinkPrimitiveFromRelations(OsmPrimitive primitive) {
+     * @return The set of relations that have been modified
+     */
+    public Set<Relation> unlinkPrimitiveFromRelations(OsmPrimitive primitive) {
+        Set<Relation> result = new HashSet<Relation>();
         beginUpdate();
         try {
@@ -859 +866 @@
                 if (removed) {
                     relation.setMembers(members);
+                    result.add(relation);
                 }
             }
@@ -864 +872 @@
             endUpdate();
         }
-    }
-
-    /**
-     * removes all references from other primitives to the
-     * referenced primitive
+        return result;
+    }
+
+    /**
+     * Removes all references from other primitives to the referenced primitive.
      *
      * @param referencedPrimitive the referenced primitive
-     */
-    public void unlinkReferencesToPrimitive(OsmPrimitive referencedPrimitive) {
+     * @return The set of primitives that have been modified
+     */
+    public Set<OsmPrimitive> unlinkReferencesToPrimitive(OsmPrimitive referencedPrimitive) {
+        Set<OsmPrimitive> result = new HashSet<OsmPrimitive>();
         beginUpdate();
         try {
             if (referencedPrimitive instanceof Node) {
-                unlinkNodeFromWays((Node)referencedPrimitive);
-                unlinkPrimitiveFromRelations(referencedPrimitive);
-            } else {
-                unlinkPrimitiveFromRelations(referencedPrimitive);
-            }
+                result.addAll(unlinkNodeFromWays((Node)referencedPrimitive));
+            }
+            result.addAll(unlinkPrimitiveFromRelations(referencedPrimitive));
         } finally {
             endUpdate();
         }
+        return result;
     }