Modify

Opened 3 years ago

Closed 3 years ago

#20161 closed enhancement (wontfix)

Allow splitting multiple ways with one press

Reported by: ff5722 Owned by: team
Priority: normal Milestone:
Component: Core Version:
Keywords: template_report Cc:

Description (last modified by ff5722)

When adding bridges or tunnels to motorways or railroads, it's often required to split ways, currently this has to be done for each parallel way separately, it would be convenient to allow the user to split them at once.
https://josm.openstreetmap.de/raw-attachment/ticket/20161/Screenshot%202020-11-27%20130234.jpg

What steps will reproduce the problem?

  1. Select multiple ways, for example parallel highways.
  2. Press Ctrl+P/Tools>Split way
  3. 'https://upload.wikimedia.org/wikipedia/commons/4/49/Error.pngThe selected nodes do not share the same way'

What is the expected result?

Allow the user (at least in expert mode) to split multiple ways in one selection at once

What happens instead?

Warning 'https://upload.wikimedia.org/wikipedia/commons/4/49/Error.pngThe selected nodes do not share the same way'

Please provide any additional information below. Attach a screenshot if possible.

URL:https://josm.openstreetmap.de/svn/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2020-10-11 23:10:16 +0200 (Sun, 11 Oct 2020)
Build-Date:2020-10-12 01:30:50
Revision:17170
Relative:URL: ^/trunk

Identification: JOSM/1.5 (17170 en) Windows 10 64-Bit
OS Build number: Windows 10 Home 2004 (19041)
Memory Usage: 2034 MB / 2709 MB (1073 MB allocated, but free)
Java version: 1.8.0_271-b09, Oracle Corporation, Java HotSpot(TM) 64-Bit Server VM
Look and Feel: com.sun.java.swing.plaf.windows.WindowsLookAndFeel
Screen: \Display0 1920x1080 (scaling 1.0x1.0), \Display1 1080x1920 (scaling 1.0x1.0)
Maximum Screen Size: 1920x1920
Best cursor sizes: 16x16 -> 32x32, 32x32 -> 32x32
Dataset consistency test: No problems found

Plugins:
+ Create_grid_of_ways (35640)
+ FastDraw (35640)
+ apache-commons (35524)
+ apache-http (35589)
+ buildings_tools (35640)
+ easypresets (1605041256)
+ ejml (35458)
+ geotools (35458)
+ gridify (1606242219)
+ importvec (35567)
+ jaxb (35543)
+ jna (35458)
+ jts (35458)
+ mapwithai (1.7.1.1)
+ opendata (35640)
+ reverter (35640)
+ terracer (35640)
+ utilsplugin2 (35640)

Tagging presets:
+ E:\Nextcloud\Documents\Openstreetmap\JOSM presets\CellTower.xml
+ E:\Nextcloud\Documents\Openstreetmap\JOSM presets\motorway_link.xml
+ E:\Nextcloud\Documents\Openstreetmap\JOSM presets\TunnelLayerPreset.xml
+ E:\Nextcloud\Documents\Openstreetmap\JOSM presets\BridgeLayerPreset.xml

Map paint styles:
+ https://josm.openstreetmap.de/josmfile?page=Styles/MapWithAI&zip=1

Last errors/warnings:
- 16431.905 W: java.net.SocketTimeoutException: Read timed out. Cause: java.net.SocketTimeoutException: Read timed out
- 16460.728 W: java.net.SocketTimeoutException: Read timed out. Cause: java.net.SocketTimeoutException: Read timed out
- 16461.908 W: java.net.SocketTimeoutException: Read timed out. Cause: java.net.SocketTimeoutException: Read timed out
- 16462.207 W: java.net.SocketTimeoutException: Read timed out. Cause: java.net.SocketTimeoutException: Read timed out
- 16492.426 W: java.net.SocketTimeoutException: Read timed out. Cause: java.net.SocketTimeoutException: Read timed out
- 17537.727 W: No default layer selected, choosing first layer.
- 18087.231 W: java.net.SocketTimeoutException: Read timed out. Cause: java.net.SocketTimeoutException: Read timed out
- 18790.328 W: Terracer: wrong or missing outline
- 18791.824 W: Terracer: wrong or missing outline
- 69480.059 E: Communication with OSM server failed - null

Attachments (1)

Screenshot 2020-11-27 130234.jpg (27.2 KB ) - added by ff5722 3 years ago.

Download all attachments as: .zip

Change History (5)

by ff5722, 3 years ago

comment:1 by ff5722, 3 years ago

Description: modified (diff)

comment:2 by GerdP, 3 years ago

-1: This saves only one key but adds lots of possible conflicts when nodes are shared by ways.

in reply to:  2 comment:3 by ff5722, 3 years ago

Replying to GerdP:

-1: This saves only one key but adds lots of possible conflicts when nodes are shared by ways.

Would it be possible to limit it to ways that don't have shared nodes? Or, cycle through individual ways in the code, apply the action and move on the the next way, and limit it to for example 4 ways.

comment:4 by GerdP, 3 years ago

Resolution: wontfix
Status: newclosed

Everything is possible. The problematic part is the handling of a cancel. Assume you have ways which are members in relations. If you cycle through mulltiple ways and decide that you made an error in the selection when the 2nd way is split what should happen? Rollback everything or only the action for the last way? No matter what you answer it requires a lot of new code with new warnings or notifications for a very small benefit.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.