Index: applications/editors/josm/plugins/pt_assistant/src/org/openstreetmap/josm/plugins/pt_assistant/actions/CreatePlatformShortcutAction.java =================================================================== --- applications/editors/josm/plugins/pt_assistant/src/org/openstreetmap/josm/plugins/pt_assistant/actions/CreatePlatformShortcutAction.java (revision 34232) +++ applications/editors/josm/plugins/pt_assistant/src/org/openstreetmap/josm/plugins/pt_assistant/actions/CreatePlatformShortcutAction.java (revision 34232) @@ -0,0 +1,185 @@ +// License: GPL. For details, see LICENSE file. + +package org.openstreetmap.josm.plugins.pt_assistant.actions; +import static org.openstreetmap.josm.tools.I18n.tr; + +import java.awt.MouseInfo; +import java.awt.Point; +import java.awt.PointerInfo; +import java.awt.event.ActionEvent; +import java.awt.event.KeyEvent; +import java.util.ArrayList; +import java.util.Collection; +import java.util.Collections; +import java.util.HashMap; +import java.util.List; +import java.util.Map; +import java.util.Set; + +import org.openstreetmap.josm.actions.JosmAction; +import org.openstreetmap.josm.command.ChangeCommand; +import org.openstreetmap.josm.command.ChangePropertyCommand; +import org.openstreetmap.josm.command.Command; +import org.openstreetmap.josm.command.SequenceCommand; +import org.openstreetmap.josm.data.coor.EastNorth; +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.RelationToChildReference; +import org.openstreetmap.josm.gui.MainApplication; +import org.openstreetmap.josm.gui.MapView; +import org.openstreetmap.josm.gui.datatransfer.OsmTransferHandler; +import org.openstreetmap.josm.gui.datatransfer.PrimitiveTransferable; +import org.openstreetmap.josm.gui.datatransfer.data.PrimitiveTransferData; +import org.openstreetmap.josm.tools.MultiMap; +import org.openstreetmap.josm.tools.Shortcut; + +/** + * Creates a platform node where the cursor is present, removes all the tags from the selected node + * and puts it in the new node, add some new tags to previous node + * + * @author Biswesh + * + */ +public class CreatePlatformShortcutAction extends JosmAction { + + private static final String ACTION_NAME = "Shortcut action to Transfer details of stop to platform node"; + protected final OsmTransferHandler transferHandler; + + /** + * Creates a new PlatformAction + */ + public CreatePlatformShortcutAction() { + super(ACTION_NAME, null, ACTION_NAME, Shortcut.registerShortcut("system:CreatePlatformShortcutAction", tr("Edit: {0}", tr("CreatePlatformNodeShortcut")), KeyEvent.VK_G, Shortcut.CTRL), false); + transferHandler = new OsmTransferHandler(); + MainApplication.registerActionShortcut(this, + Shortcut.registerShortcut("system:CreatePlatformShortcutAction", tr("Edit: {0}", tr("CreatePlatformNodeShortcut")), KeyEvent.VK_G, Shortcut.CTRL)); + } + + @Override + public void actionPerformed(ActionEvent e) { + Collection selection = getLayerManager().getEditDataSet().getSelected(); + Node stopPositionNode = null; + Node newNode = null; + + for (OsmPrimitive item: selection) { + if (item.getType() == OsmPrimitiveType.NODE) { + if (item.hasTag("highway") && (item.hasTag("highway", "bus_stop") || item.hasTag("public_transport", "stop_position"))) { + stopPositionNode = (Node) item; + } else if (item.hasTag("railway") + && (item.hasTag("railway", "tram_stop") || item.hasTag("public_transport", "stop_position"))) { + stopPositionNode = (Node) item; + } + } + } + + if (stopPositionNode != null && selection.size() == 1) { + PrimitiveTransferData data = PrimitiveTransferData.getDataWithReferences(selection); + transferHandler.pasteOn(getLayerManager().getEditLayer(), computePastePosition(e), new PrimitiveTransferable(data)); + Collection newSelection = getLayerManager().getEditDataSet().getSelected(); + + for (OsmPrimitive item: newSelection) { + newNode = (Node) item; + } + } + + if (stopPositionNode == null) + return; + + HashMap tagList = new HashMap<>(); + for (String key : stopPositionNode.keySet()) { + tagList.put(key, stopPositionNode.get(key)); + } + + if (newNode != null) { + modify(newNode, stopPositionNode); + } + } + + protected EastNorth computePastePosition(ActionEvent e) { + // default to paste in center of map (pasted via menu or cursor not in MapView) + MapView mapView = MainApplication.getMap().mapView; + EastNorth mPosition = mapView.getCenter(); + // We previously checked for modifier to know if the action has been trigerred via shortcut or via menu + // But this does not work if the shortcut is changed to a single key (see #9055) + // Observed behaviour: getActionCommand() returns Action.NAME when triggered via menu, but shortcut text when triggered with it + if (e != null && !getValue(NAME).equals(e.getActionCommand())) { + final PointerInfo pointerInfo = MouseInfo.getPointerInfo(); + if (pointerInfo != null) { + final Point mp = pointerInfo.getLocation(); + final Point tl = mapView.getLocationOnScreen(); + final Point pos = new Point(mp.x-tl.x, mp.y-tl.y); + if (mapView.contains(pos)) { + mPosition = mapView.getEastNorth(pos.x, pos.y); + } + } + } + return mPosition; + } + + public void modify(Node newNode, Node stopPositionNode) { + + if (stopPositionNode.hasTag("railway")) { + newNode.put("tram","yes"); + newNode.put("railway","tram_stop"); + newNode.remove("public_transport"); + newNode.put("public_transport", "platform"); + + List commands = getReplaceGeometryCommand(stopPositionNode, newNode); + if (commands.size() > 0) + MainApplication.undoRedo.add(new SequenceCommand(tr("Replace Membership"), commands)); + + HashMap tags = new HashMap<>(stopPositionNode.getKeys()); + tags.replaceAll((key, value) -> null); + MainApplication.undoRedo.add(new ChangePropertyCommand(Collections.singleton(stopPositionNode), tags)); + + ArrayList undoCommands = new ArrayList<>(); + undoCommands.add(new ChangePropertyCommand(stopPositionNode, "public_transport", "stop_position")); + undoCommands.add(new ChangePropertyCommand(stopPositionNode, "tram", "yes")); + MainApplication.undoRedo.add(new SequenceCommand("tag", undoCommands)); + + } else if (stopPositionNode.hasTag("highway")) { + newNode.put("bus","yes"); + newNode.put("highway","bus_stop"); + newNode.remove("public_transport"); + newNode.put("public_transport", "platform"); + + List commands = getReplaceGeometryCommand(stopPositionNode, newNode); + if (commands.size() > 0) + MainApplication.undoRedo.add(new SequenceCommand(tr("Replace Membership"), commands)); + + HashMap tags = new HashMap<>(stopPositionNode.getKeys()); + tags.replaceAll((key, value) -> null); + MainApplication.undoRedo.add(new ChangePropertyCommand(Collections.singleton(stopPositionNode), tags)); + + ArrayList undoCommands = new ArrayList<>(); + undoCommands.add(new ChangePropertyCommand(stopPositionNode, "bus", "yes")); + undoCommands.add(new ChangePropertyCommand(stopPositionNode, "public_transport", "stop_position")); + MainApplication.undoRedo.add(new SequenceCommand("tag", undoCommands)); + } + + } + + static List getReplaceGeometryCommand(OsmPrimitive firstObject, OsmPrimitive secondObject) { + final MultiMap byRelation = new MultiMap<>(); + for (final RelationToChildReference i : RelationToChildReference.getRelationToChildReferences(firstObject)) { + byRelation.put(i.getParent(), i); + } + + final List commands = new ArrayList<>(); + for (final Map.Entry> i : byRelation.entrySet()) { + final Relation oldRelation = i.getKey(); + final Relation newRelation = new Relation(oldRelation); + for (final RelationToChildReference reference : i.getValue()) { + newRelation.setMember(reference.getPosition(), new RelationMember("platform", secondObject)); + } + commands.add(new ChangeCommand(oldRelation, newRelation)); + } + + return commands; + } + +} +