Changeset 7129 in josm for trunk/src/org/openstreetmap/josm/gui/conflict/tags/CombinePrimitiveResolverDialog.java

Timestamp:

2014-05-14T23:03:49+02:00 (10 years ago)

Author:

simon04

Message:

see #10018 - Avoid broken relations after combining ways

File:

• trunk/src/org/openstreetmap/josm/gui/conflict/tags/CombinePrimitiveResolverDialog.java (modified) (4 diffs)

trunk/src/org/openstreetmap/josm/gui/conflict/tags/CombinePrimitiveResolverDialog.java

@@ -18 +18 @@
 import java.beans.PropertyChangeListener;
 import java.util.Collection;
-import java.util.HashSet;
+import java.util.Collections;
+import java.util.HashMap;
 import java.util.LinkedList;
 import java.util.List;
+import java.util.Map;
 import java.util.Set;
 
@@ -49 +51 @@
 import org.openstreetmap.josm.tools.CheckParameterUtil;
 import org.openstreetmap.josm.tools.ImageProvider;
+import org.openstreetmap.josm.tools.MultiMap;
+import org.openstreetmap.josm.tools.Predicates;
 import org.openstreetmap.josm.tools.Utils;
 import org.openstreetmap.josm.tools.Utils.Function;
@@ -297 +301 @@
 
     protected void prepareDefaultRelationDecisions() {
-        RelationMemberConflictResolverModel model = getRelationMemberConflictResolverModel();
-        Set<Relation> relations = new HashSet<>();
+        final RelationMemberConflictResolverModel model = getRelationMemberConflictResolverModel();
+        final Map<Relation, Integer> numberOfKeepResolutions = new HashMap<>();
+        final MultiMap<OsmPrimitive, Relation> resolvedRelationsPerPrimitive = new MultiMap<>();
+
         for (int i = 0; i < model.getNumDecisions(); i++) {
-            RelationMemberConflictDecision decision = model.getDecision(i);
-            if (!relations.contains(decision.getRelation())) {
+            final RelationMemberConflictDecision decision = model.getDecision(i);
+            final Relation r = decision.getRelation();
+            final OsmPrimitive p = decision.getOriginalPrimitive();
+            if (!numberOfKeepResolutions.containsKey(r)) {
                 decision.decide(RelationMemberConflictDecisionType.KEEP);
-                relations.add(decision.getRelation());
+                numberOfKeepResolutions.put(r, 1);
+                resolvedRelationsPerPrimitive.put(p, r);
+                continue;
+            }
+
+            final Integer keepResolutions = numberOfKeepResolutions.get(r);
+            final Collection<Relation> resolvedRelations = Utils.firstNonNull(resolvedRelationsPerPrimitive.get(p), Collections.<Relation>emptyList());
+            if (keepResolutions <= Utils.filter(resolvedRelations, Predicates.equalTo(r)).size()) {
+                // old relation contains one primitive more often than the current resolution => keep the current member
+                decision.decide(RelationMemberConflictDecisionType.KEEP);
+                numberOfKeepResolutions.put(r, keepResolutions + 1);
+                resolvedRelationsPerPrimitive.put(p, r);
             } else {
                 decision.decide(RelationMemberConflictDecisionType.REMOVE);
+                resolvedRelationsPerPrimitive.put(p, r);
             }
         }
@@ -552 +572 @@
         /* I18n: object count < 2 is not possible */
         String msg = trn("You are about to combine {1} object, "
-                + "which are part of {0} relation:<br/>{2}"
+                + "which is part of {0} relation:<br/>{2}"
                 + "Combining these objects may break this relation. If you are unsure, please cancel this operation.<br/>"
                 + "If you want to continue, you are shown a dialog to decide how to adapt the relation.<br/><br/>"