Changeset 29391 in osm for applications/editors

Timestamp:

2013-03-24T18:39:50+01:00 (12 years ago)

Author:

donvip

Message:

fix #josm8540 - Unglue relations not marking relations as modified

Location:

applications/editors/josm/plugins/utilsplugin2/src/org/openstreetmap/josm/plugins/utilsplugin2

Files:

• actions/UnGlueRelationAction.java (modified) (2 diffs)
• command (added)
• command/ChangeRelationMemberCommand.java (added)

applications/editors/josm/plugins/utilsplugin2/src/org/openstreetmap/josm/plugins/utilsplugin2/actions/UnGlueRelationAction.java

@@ -19 +19 @@
 import org.openstreetmap.josm.data.osm.Node;
 import org.openstreetmap.josm.data.osm.OsmPrimitive;
-import org.openstreetmap.josm.data.osm.OsmPrimitiveType;
 import org.openstreetmap.josm.data.osm.Relation;
-import org.openstreetmap.josm.data.osm.RelationMember;
 import org.openstreetmap.josm.data.osm.Way;
-import org.openstreetmap.josm.gui.DefaultNameFormatter;
+import org.openstreetmap.josm.plugins.utilsplugin2.command.ChangeRelationMemberCommand;
 import org.openstreetmap.josm.tools.Shortcut;
 
@@ -89 +87 @@
     }
 
-    /**
-     * Change member in relation to another one
-     */
-    private class ChangeRelationMemberCommand extends Command {
-
-        private final Relation relation;
-        private final OsmPrimitive oldMember;
-        private final OsmPrimitive newMember;
-        
-        public ChangeRelationMemberCommand(Relation relation, OsmPrimitive oldMember, OsmPrimitive newMember) {
-            this.relation = relation;
-            this.oldMember = oldMember;
-            this.newMember = newMember;
-        }
-
-        private boolean replaceMembers(OsmPrimitive oldP, OsmPrimitive newP) {
-            if (relation == null || oldMember == null || newMember == null) {
-                return false;
-            }
-            LinkedList<RelationMember> newrms = new LinkedList<RelationMember>();
-            for (RelationMember rm : relation.getMembers()) {
-                if (rm.getMember() == oldP) {
-                    newrms.add(new RelationMember(rm.getRole(), newP));
-                } else {
-                    newrms.add(rm);
-                }
-            }
-            relation.setMembers(newrms);
-            return true;
-        }
-        
-        @Override
-        public boolean executeCommand() {
-            return replaceMembers(oldMember, newMember);
-        }
-
-        @Override
-        public void fillModifiedData(Collection<OsmPrimitive> modified,
-                Collection<OsmPrimitive> deleted, Collection<OsmPrimitive> added) {
-            modified.add(relation);
-        }
-
-        @Override
-        public void undoCommand() {
-            replaceMembers(newMember, oldMember);
-        }
-
-        @Override
-        public String getDescriptionText() {
-            return tr("Change relation member for {0} {1}",
-                    OsmPrimitiveType.from(relation),
-                    relation.getDisplayName(DefaultNameFormatter.getInstance()));
-        }
-    }
-
     @Override
     protected void updateEnabledState() {