Don't change timestamp when splitting a way (#4523)

Helium314 · web-flow · commit 33663cc4bbbd · 2022-10-17T23:13:51.000+02:00
diff --git a/app/src/main/java/de/westnordost/streetcomplete/data/osm/edits/split_way/SplitWayAction.kt b/app/src/main/java/de/westnordost/streetcomplete/data/osm/edits/split_way/SplitWayAction.kt
@@ -130,10 +130,11 @@ private fun getSplitWayAtIndices(
     removeTagsThatArePotentiallyWrongAfterSplit(tags)
 
     return nodesChunks.mapIndexed { index, nodes ->
+        // keep the original timestampEdited, so resurvey quests are still shown after splitting (only when auto-sync is off)
         if (index == indexOfChunkToKeep) {
-            Way(originalWay.id, nodes, tags, originalWay.version, currentTimeMillis())
+            Way(originalWay.id, nodes, tags, originalWay.version, originalWay.timestampEdited)
         } else {
-            Way(idProvider.nextWayId(), nodes, tags, 0, currentTimeMillis())
+            Way(idProvider.nextWayId(), nodes, tags, 0, originalWay.timestampEdited)
         }
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -130,10 +130,11 @@ private fun getSplitWayAtIndices(`
`130`	`130`	`removeTagsThatArePotentiallyWrongAfterSplit(tags)`
`131`	`131`
`132`	`132`	`return nodesChunks.mapIndexed { index, nodes ->`
	`133`	`+ // keep the original timestampEdited, so resurvey quests are still shown after splitting (only when auto-sync is off)`
`133`	`134`	`if (index == indexOfChunkToKeep) {`
`134`		`- Way(originalWay.id, nodes, tags, originalWay.version, currentTimeMillis())`
	`135`	`+ Way(originalWay.id, nodes, tags, originalWay.version, originalWay.timestampEdited)`
`135`	`136`	`} else {`
`136`		`- Way(idProvider.nextWayId(), nodes, tags, 0, currentTimeMillis())`
	`137`	`+ Way(idProvider.nextWayId(), nodes, tags, 0, originalWay.timestampEdited)`
`137`	`138`	`}`
`138`	`139`	`}`
`139`	`140`	`}`