Combine way action may remove parts of the ways

[GerdP]

See #18368, esp. ticket:18368#comment:6

[GerdP]

In 15555/josm:

fix #18385: Combine way action may remove parts of the ways

• Count all edges that are added to the graph, even if they are rejected because they are duplicates.
• add new method buildSpanningPathNoRemove() which returns null if any duplicate edge was found

and use this method in CombineWayAction

[GerdP]

In 15556/josm:

see #18385 update @since

[GerdP]

@Vincent: I guess org.openstreetmap.josm.plugins.opendata.core.datasets.WayCombiner should also use this new method?
It is only used in org.openstreetmap.josm.plugins.opendata.modules.fr.paris.datasets.urbanisme.SanisettesHandler

[GerdP]

The wanted result is a joined way, not the message that the ways cannot be joined. So, we need a datastructure that remembers the complete data.

[Don-vip]

@Gerd: I have removed this class in [o35244:35245]. Not worth keeping it.

[Don-vip]

There's a Sonar warning: https://josm.openstreetmap.de/sonar/project/issues?id=josm&issues=AW7MgRMl-XCMeap-x5vR&open=AW7MgRMl-XCMeap-x5vR

It would be safer to return an empty collection than null.

[GerdP]

I also don't like it, but the older public method buildSpanningPath() does the same and callers expect the null value.

[Don-vip]

I don't find any plugin calling this method, so it is probably safe to change the behaviour if you want.

[GerdP]

See also last patch in #18367. I can change the methods to private and let tryJoin() return an empty collection to signal that the ways cannot be joined.

[GerdP]

In 15574/josm:

fix #18367 and #18385: CombineWayAction (C) refuses to combine ways or silently reverses ways
Changes:

• try first to combine the ways with the method Multipolygon.joinWays() If that method returns a single line string we can use it, else use the result of NodeGraph.buildSpanningPathNoRemove(). Both methods will not add or remove segments
• if ways are combined execute checks for overlapping segments or self-intersection and show a notification popup right after the command was added to the UndoRedoHandler
• The code which handles reversed ways needed changes. In the unpatched version it sometimes claims wrongly that ways were reversed, in special cases it sometimes silently reverted ways. The old code did not handle the case properly that a node can appear more than once. I really hope that I got it right now.
• Fix some sonarlint issues
• let NodeGraph routines return an ArrayList instead of a LinkedList (improves performance a bit)
• Add unit tests