Splitting a restriction relation with multiple via-ways breaks restriction relation

[westnordost]

If one attempts to split the from or to way in a restriction(-like) relation and the there are multiple consecutive via-ways, JOSM does not split up the restriction relation correctly.

The culprit is at ​SplitWayCommand line 425

JOSM assumes there can only be one relation member with the role via. But according to ​the wiki, it can be one or several (consecutive) relation members that are ways.

(I checked both iD and Vespucci code, they both check for a list of relation members with a via* role)

[Don-vip]

Can you please attach your status report?

[westnordost]

Sure, here it is

URL:https://josm.openstreetmap.de/svn/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2019-08-26 08:55:27 +0200 (Mon, 26 Aug 2019)
Build-Date:2019-08-26 06:57:10
Revision:15322
Relative:URL: ^/trunk

Identification: JOSM/1.5 (15322 en_GB) Windows 10 64-Bit
OS Build number: Windows 10 Home 1809 (17763)
Memory Usage: 936 MB / 1820 MB (454 MB allocated, but free)
Java version: 1.8.0_211-b12, Oracle Corporation, Java HotSpot(TM) 64-Bit Server VM
Screen: \Display0 2560x1440
Maximum Screen Size: 2560x1440

Plugins:
+ FastDraw (34977)
+ Mapillary (1.5.18)
+ OpenStreetCam (251)
+ ShapeTools (1240)
+ apache-commons (34908)
+ apache-http (34908)
+ buildings_tools (34982)
+ ejml (35049)
+ geotools (34908)
+ jaxb (35014)
+ jna (34908)
+ jts (35064)
+ measurement (35051)
+ merge-overlap (35072)
+ opendata (35102)
+ terracer (34977)
+ utilsplugin2 (35098)

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

Last errors/warnings:
- W: No configuration settings found.  Using hardcoded default values for all pools.

[westnordost]

I had the same bug in StreetComplete and fixed it now.
Since the implementation of the split way algorithm in the app is not by chance quite similar to the implementation in JOSM, maybe this can be of help:

​SplitSingleWayUpload.kt#L329-L339

[Don-vip]

In 15362/josm:

fix #18088 - see #18006 - Allow splitting a connectivity/restriction relation with multiple via-ways