|
Last change
on this file since 16707 was 15574, checked in by GerdP, 5 years ago |
|
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
|
|
File size:
1017 bytes
|
| Line | |
|---|
| 1 | <?xml version='1.0' encoding='UTF-8'?>
|
|---|
| 2 | <osm version='0.6' generator='JOSM' upload='false'>
|
|---|
| 3 | <node id='-143178' action='modify' visible='true' lat='0.02196380816' lon='-0.02295195551' />
|
|---|
| 4 | <node id='-143179' action='modify' visible='true' lat='0.02142481903' lon='0.00624329122' />
|
|---|
| 5 | <node id='-143181' action='modify' visible='true' lat='0.02708420481' lon='0.01828071603' />
|
|---|
| 6 | <node id='-143183' action='modify' visible='true' lat='0.03597752476' lon='0.01908919979' />
|
|---|
| 7 | <node id='-143185' action='modify' visible='true' lat='0.03139611761' lon='0.0279825211' />
|
|---|
| 8 | <node id='-143187' action='modify' visible='true' lat='-0.00696194343' lon='-0.00112289411' />
|
|---|
| 9 | <way id='-143180' action='modify' visible='true'>
|
|---|
| 10 | <nd ref='-143178' />
|
|---|
| 11 | <nd ref='-143179' />
|
|---|
| 12 | <nd ref='-143181' />
|
|---|
| 13 | <nd ref='-143183' />
|
|---|
| 14 | <nd ref='-143185' />
|
|---|
| 15 | </way>
|
|---|
| 16 | <way id='-143188' action='modify' visible='true'>
|
|---|
| 17 | <nd ref='-143187' />
|
|---|
| 18 | <nd ref='-143179' />
|
|---|
| 19 | <nd ref='-143181' />
|
|---|
| 20 | <nd ref='-143185' />
|
|---|
| 21 | </way>
|
|---|
| 22 | </osm>
|
|---|
Note:
See
TracBrowser
for help on using the repository browser.
