Changeset 2936 in josm for trunk/src/org/openstreetmap/josm/gui/conflict/pair/properties/PropertiesMergeModel.java

Timestamp:

04.02.2010 21:04:02 (2 years ago)

Author:

jttt

Message:

Fixed #4408 confict manager: do not have to solve tag- and element-conficts when deleting

File:

• trunk/src/org/openstreetmap/josm/gui/conflict/pair/properties/PropertiesMergeModel.java (modified) (5 diffs)

trunk/src/org/openstreetmap/josm/gui/conflict/pair/properties/PropertiesMergeModel.java

@@ -53 +53 @@
 
     static public final String RESOLVED_COMPLETELY_PROP = PropertiesMergeModel.class.getName() + ".resolvedCompletely";
+    static public final String DELETE_PRIMITIVE_PROP = PropertiesMergeModel.class.getName() + ".deletePrimitive";
 
     private OsmPrimitive my;
@@ -287 +288 @@
     }
 
+    private boolean getMergedDeletedState(MergeDecisionType decision) {
+        switch (decision) {
+        case KEEP_MINE:
+            return myDeletedState;
+        case KEEP_THEIR:
+            return theirDeletedState;
+        default:
+            return false;
+        }
+    }
+
     /**
      * decides the conflict between two deleted states
@@ -295 +307 @@
     public void decideDeletedStateConflict(MergeDecisionType decision) throws IllegalArgumentException{
         CheckParameterUtil.ensureParameterNotNull(decision, "decision");
+
+        boolean oldMergedDeletedState = getMergedDeletedState(this.deletedMergeDecision);
+        boolean newMergedDeletedState = getMergedDeletedState(decision);
+
         this.deletedMergeDecision = decision;
         setChanged();
         notifyObservers();
         fireCompletelyResolved();
+
+        if (oldMergedDeletedState != newMergedDeletedState) {
+            support.firePropertyChange(DELETE_PRIMITIVE_PROP, oldMergedDeletedState, newMergedDeletedState);
+        }
     }
 
@@ -478 +498 @@
                         dependent.size(), dependent.size(), way.getId())
                         + "</html>",
-                tr("Undelete additional nodes?"),
-                JOptionPane.YES_NO_OPTION,
-                JOptionPane.QUESTION_MESSAGE,
-                null,
-                options,
-                options[0]
+                        tr("Undelete additional nodes?"),
+                        JOptionPane.YES_NO_OPTION,
+                        JOptionPane.QUESTION_MESSAGE,
+                        null,
+                        options,
+                        options[0]
         );
 
@@ -512 +532 @@
                         dependent.size(), dependent.size(), r.getId())
                         + "</html>",
-                tr("Undelete dependent primitives?"),
-                JOptionPane.YES_NO_OPTION,
-                JOptionPane.QUESTION_MESSAGE,
-                null,
-                options,
-                options[0]
+                        tr("Undelete dependent primitives?"),
+                        JOptionPane.YES_NO_OPTION,
+                        JOptionPane.QUESTION_MESSAGE,
+                        null,
+                        options,
+                        options[0]
         );