Changeset 3640 in josm for trunk/src/org/openstreetmap/josm/actions/PasteTagsAction.java

Timestamp:

2010-10-24T15:07:01+02:00 (13 years ago)

Author:

jttt

Message:

Fix #5579 allow paste tags in the relation editor

File:

• trunk/src/org/openstreetmap/josm/actions/PasteTagsAction.java (modified) (8 diffs)

trunk/src/org/openstreetmap/josm/actions/PasteTagsAction.java

@@ -23 +23 @@
 import org.openstreetmap.josm.data.osm.PrimitiveData;
 import org.openstreetmap.josm.data.osm.PrimitiveDeepCopy;
+import org.openstreetmap.josm.data.osm.PrimitiveDeepCopy.PasteBufferChangedListener;
+import org.openstreetmap.josm.data.osm.Tag;
 import org.openstreetmap.josm.data.osm.TagCollection;
-import org.openstreetmap.josm.data.osm.PrimitiveDeepCopy.PasteBufferChangedListener;
 import org.openstreetmap.josm.gui.conflict.tags.PasteTagsConflictResolverDialog;
 import org.openstreetmap.josm.tools.Shortcut;
@@ -42 +43 @@
         private final Collection<PrimitiveData> source;
         private final Collection<OsmPrimitive> target;
-        private final List<Command> commands = new ArrayList<Command>();
+        private final List<Tag> commands = new ArrayList<Tag>();
 
         public TagPaster(Collection<PrimitiveData> source, Collection<OsmPrimitive> target) {
@@ -100 +101 @@
         }
 
-        protected Command buildChangeCommand(Collection<? extends OsmPrimitive> selection, TagCollection tc) {
-            List<Command> commands = new ArrayList<Command>();
+        protected void buildChangeCommand(Collection<? extends OsmPrimitive> selection, TagCollection tc) {
             for (String key : tc.getKeys()) {
-                String value = tc.getValues(key).iterator().next();
-                value = value.equals("") ? null : value;
-                commands.add(new ChangePropertyCommand(selection,key,value));
-            }
-            if (!commands.isEmpty()) {
-                String title1 = trn("Pasting {0} tag", "Pasting {0} tags", tc.getKeys().size(), tc.getKeys().size());
-                String title2 = trn("to {0} primitive", "to {0} primtives", selection.size(), selection.size());
-                return new SequenceCommand(
-                        title1 + " " + title2,
-                        commands
-                );
-            }
-            return null;
+                commands.add(new Tag(key, tc.getValues(key).iterator().next()));
+            }
         }
 
@@ -166 +155 @@
                 if (dialog.isCanceled())
                     return;
-                Command cmd = buildChangeCommand(target, dialog.getResolution());
-                commands.add(cmd);
+                buildChangeCommand(target, dialog.getResolution());
             } else {
                 // no conflicts in the source tags to resolve. Just apply the tags
                 // to the target primitives
                 //
-                Command cmd = buildChangeCommand(target, tc);
-                commands.add(cmd);
+                buildChangeCommand(target, tc);
             }
         }
@@ -218 +205 @@
                 for (OsmPrimitiveType type:OsmPrimitiveType.values()) {
                     if (hasSourceTagsByType(type) && hasTargetPrimitives(type.getOsmClass())) {
-                        Command cmd = buildChangeCommand(target, getSourceTagsByType(type));
-                        commands.add(cmd);
+                        buildChangeCommand(target, getSourceTagsByType(type));
                     }
                 }
@@ -236 +222 @@
                 for (OsmPrimitiveType type:OsmPrimitiveType.values()) {
                     if (hasSourceTagsByType(type) && hasTargetPrimitives(type.getOsmClass())) {
-                        Command cmd = buildChangeCommand(OsmPrimitive.getFilteredList(target, type.getOsmClass()), dialog.getResolution(type));
-                        commands.add(cmd);
+                        buildChangeCommand(OsmPrimitive.getFilteredList(target, type.getOsmClass()), dialog.getResolution(type));
                     }
                 }
@@ -243 +228 @@
         }
 
-        public List<Command> execute() {
+        public List<Tag> execute() {
             commands.clear();
             if (isHeteogeneousSource()) {
@@ -256 +241 @@
 
     public void actionPerformed(ActionEvent e) {
-        if (getCurrentDataSet().getSelected().isEmpty())
+        Collection<OsmPrimitive> selection = getCurrentDataSet().getSelected();
+
+        if (selection.isEmpty())
             return;
-        TagPaster tagPaster = new TagPaster(Main.pasteBuffer.getDirectlyAdded(), getCurrentDataSet().getSelected());
-        for (Command c:tagPaster.execute()) {
-            Main.main.undoRedo.add(c);
-        }
+
+        TagPaster tagPaster = new TagPaster(Main.pasteBuffer.getDirectlyAdded(), selection);
+
+        List<Command> commands = new ArrayList<Command>();
+        for (Tag tag: tagPaster.execute()) {
+            commands.add(new ChangePropertyCommand(selection, tag.getKey(), "".equals(tag.getValue())?null:tag.getValue()));
+        }
+        if (!commands.isEmpty()) {
+            String title1 = trn("Pasting {0} tag", "Pasting {0} tags", commands.size(), commands.size());
+            String title2 = trn("to {0} primitive", "to {0} primtives", selection.size(), selection.size());
+            Main.main.undoRedo.add(
+                    new SequenceCommand(
+                            title1 + " " + title2,
+                            commands
+                    ));
+        }
+
     }