Recording and Uploading of GPX Traces

app/build.gradle.kts

@@ -158,6 +158,7 @@ dependencies {
     implementation("de.westnordost:osmapi-map:2.0")
     implementation("de.westnordost:osmapi-changesets:2.0")
     implementation("de.westnordost:osmapi-notes:2.0")
+    implementation("de.westnordost:osmapi-traces:2.0")
     implementation("de.westnordost:osmapi-user:2.0")
     implementation("com.squareup.okhttp3:okhttp:3.12.13")
     implementation("se.akerfeldt:okhttp-signpost:1.1.0")

app/src/main/assets/map_theme/jawg/streetcomplete.yaml

@@ -3,6 +3,7 @@ global:
     geometry_line_color: '#44D14000'
     geometry_point_color: '#88D14000'
     track_color: '#44536dfe'
+    track_color_record: '#44fe1616'
     old_track_color: '#22536dfe'
 
 textures:
@@ -126,16 +127,25 @@ layers:
     streetcomplete_track:
         data: { source: streetcomplete_track }
         current:
-            filter: { old: [false] }
+            filter: { old: [false], record: [false] }
             draw:
                 track-lines:
                     color: global.track_color
                     width: [[14, 6px],[18, 12px]]
                     collide: false
                     join: round
                     order: 1000
+        record:
+          filter: { record: [true] }
+          draw:
+            track-lines:
+              color: global.track_color_record
+              width: [ [ 14, 6px ],[ 18, 12px ] ]
+              collide: false
+              join: round
+              order: 1000
         old:
-            filter: { old: [true] }
+            filter: { old: [true], record: [false] }
             draw:
                 track-lines:
                     color: global.old_track_color
@@ -146,20 +156,29 @@ layers:
     streetcomplete_track2:
         data: { source: streetcomplete_track2 }
         current:
-            filter: { old: [false] }
-            draw:
-                track-lines:
-                    color: global.track_color
-                    width: [[14, 6px],[18, 12px]]
-                    collide: false
-                    join: round
-                    order: 1000
+          filter: { old: [ false ], record: [ false ] }
+          draw:
+            track-lines:
+              color: global.track_color
+              width: [ [ 14, 6px ],[ 18, 12px ] ]
+              collide: false
+              join: round
+              order: 1000
+        record:
+          filter: { record: [ true ] }
+          draw:
+            track-lines:
+              color: global.track_color_record
+              width: [ [ 14, 6px ],[ 18, 12px ] ]
+              collide: false
+              join: round
+              order: 1000
         old:
-            filter: { old: [true] }
-            draw:
-                track-lines:
-                    color: global.old_track_color
-                    width: [[14, 6px],[18, 12px]]
-                    collide: false
-                    join: round
-                    order: 1000
+          filter: { old: [ true ], record: [ false ] }
+          draw:
+            track-lines:
+              color: global.old_track_color
+              width: [ [ 14, 6px ],[ 18, 12px ] ]
+              collide: false
+              join: round
+              order: 1000

app/src/main/java/de/westnordost/streetcomplete/MainActivity.kt

@@ -35,6 +35,7 @@ import de.westnordost.streetcomplete.data.upload.UploadController
 import de.westnordost.streetcomplete.data.upload.UploadProgressListener
 import de.westnordost.streetcomplete.data.upload.VersionBannedException
 import de.westnordost.streetcomplete.data.user.AuthorizationException
+import de.westnordost.streetcomplete.data.user.AuthorizationTracesException
 import de.westnordost.streetcomplete.data.user.UserLoginStatusController
 import de.westnordost.streetcomplete.data.user.UserUpdater
 import de.westnordost.streetcomplete.ktx.toast
@@ -48,6 +49,7 @@ import de.westnordost.streetcomplete.tutorial.TutorialFragment
 import de.westnordost.streetcomplete.util.CrashReportExceptionHandler
 import de.westnordost.streetcomplete.util.parseGeoUri
 import de.westnordost.streetcomplete.view.dialogs.RequestLoginDialog
+import de.westnordost.streetcomplete.view.dialogs.RequestPermissionUpgradeDialog
 import kotlinx.coroutines.launch
 import javax.inject.Inject
 
@@ -235,6 +237,16 @@ class MainActivity : BaseActivity(),
                        Nothing we can do about it, so it does not make sense to send an error
                        report. Just notify the user. */
                     toast(R.string.upload_server_error, Toast.LENGTH_LONG)
+                } else if (e is AuthorizationTracesException) {
+                    // user has not upgraded their permissions to have GPS trace (or is not logged in)
+                    // if the user is not logged in, then we should just ask them to login which will provide all permissions
+                    val isLoggedIn = userLoginStatusController.isLoggedIn
+                    userLoginStatusController.logOut()
+                    if(isLoggedIn) {
+                        RequestPermissionUpgradeDialog(this@MainActivity).show()
+                    } else {
+                        RequestLoginDialog(this@MainActivity).show()
+                    }
                 } else if (e is AuthorizationException) {
                     // delete secret in case it failed while already having a token -> token is invalid
                     userLoginStatusController.logOut()

app/src/main/java/de/westnordost/streetcomplete/data/OsmApiModule.kt

@@ -9,6 +9,8 @@ import de.westnordost.streetcomplete.data.osm.mapdata.MapDataApi
 import de.westnordost.streetcomplete.data.osm.mapdata.MapDataApiImpl
 import de.westnordost.streetcomplete.data.osmnotes.NotesApi
 import de.westnordost.streetcomplete.data.osmnotes.NotesApiImpl
+import de.westnordost.streetcomplete.data.osmtracks.TracksApi
+import de.westnordost.streetcomplete.data.osmtracks.TracksApiImpl
 import de.westnordost.streetcomplete.data.user.OAuthStore
 import oauth.signpost.OAuthConsumer
 import javax.inject.Singleton
@@ -32,5 +34,7 @@ object OsmApiModule {
 
     @Provides fun notesApi(osm: OsmConnection): NotesApi = NotesApiImpl(osm)
 
+    @Provides fun tracksApi(osm: OsmConnection): TracksApi = TracksApiImpl(osm)
+
     @Provides fun mapDataApi(osm: OsmConnection): MapDataApi = MapDataApiImpl(osm)
 }

app/src/main/java/de/westnordost/streetcomplete/data/StreetCompleteSQLiteOpenHelper.kt

@@ -124,8 +124,12 @@ import de.westnordost.streetcomplete.quests.oneway_suspects.data.WayTrafficFlowT
         if (oldVersion <= 3 && newVersion > 3) {
            db.execSQL("DROP TABLE new_achievements")
         }
+        if (oldVersion <= 4 && newVersion > 4) {
+            db.execSQL("ALTER TABLE ${NoteEditsTable.NAME} ADD COLUMN ${NoteEditsTable.Columns.TRACKS} text DEFAULT '[]' NOT NULL")
+            db.execSQL("ALTER TABLE ${NoteEditsTable.NAME} ADD COLUMN ${NoteEditsTable.Columns.UPLOAD_DATA_MAP} text DEFAULT '{}' NOT NULL")
+        }
     }
 
 }
 
-private const val DB_VERSION = 4
+private const val DB_VERSION = 5

app/src/main/java/de/westnordost/streetcomplete/data/osmnotes/NotesApi.kt

@@ -65,4 +65,5 @@ interface NotesApi {
      * @return the incoming notes
      */
     fun getAll(bounds: BoundingBox, limit: Int, hideClosedNoteAfter: Int): List<Note>
+
 }

app/src/main/java/de/westnordost/streetcomplete/data/osmnotes/edits/NoteEdit.kt

@@ -3,6 +3,7 @@ package de.westnordost.streetcomplete.data.osmnotes.edits
 import de.westnordost.streetcomplete.data.edithistory.Edit
 import de.westnordost.streetcomplete.data.edithistory.NoteEditKey
 import de.westnordost.streetcomplete.data.osm.mapdata.LatLon
+import de.westnordost.streetcomplete.data.osmtracks.Trackpoint
 
 /** Contains all necessary information to create/comment on an OSM note. */
 data class NoteEdit(
@@ -31,8 +32,14 @@ data class NoteEdit(
     /** whether this edit has been uploaded already */
     override val isSynced: Boolean,
 
-    /** Whether the images attached still need activation. Already true if imagePaths is empty */
-    val imagesNeedActivation: Boolean
+    /** whether the images attached still need activation. Already true if imagePaths is empty */
+    val imagesNeedActivation: Boolean,
+
+    /** attached GPS location tracks */
+    val tracks: List<Trackpoint>,
+
+    /** contains partial upload responses index by type (e.g. image and track urls) */
+    val uploadedDataMap: Map<String, String>,
 ): Edit {
     override val isUndoable: Boolean get() = !isSynced
     override val key: NoteEditKey get() = NoteEditKey(id)

app/src/main/java/de/westnordost/streetcomplete/data/osmnotes/edits/NoteEditsController.kt

@@ -3,6 +3,9 @@ package de.westnordost.streetcomplete.data.osmnotes.edits
 import de.westnordost.streetcomplete.data.osm.mapdata.BoundingBox
 import de.westnordost.streetcomplete.data.osm.mapdata.LatLon
 import de.westnordost.streetcomplete.data.osmnotes.Note
+import de.westnordost.streetcomplete.data.osmtracks.Trackpoint
+import kotlinx.serialization.encodeToString
+import kotlinx.serialization.json.Json
 import java.lang.System.currentTimeMillis
 import java.util.concurrent.CopyOnWriteArrayList
 import javax.inject.Inject
@@ -21,7 +24,8 @@ import javax.inject.Singleton
         action: NoteEditAction,
         position: LatLon,
         text: String? = null,
-        imagePaths: List<String> = emptyList()
+        imagePaths: List<String> = emptyList(),
+        tracks: List<Trackpoint> = emptyList(),
     ) {
         val edit = NoteEdit(
             0,
@@ -32,7 +36,9 @@ import javax.inject.Singleton
             imagePaths,
             currentTimeMillis(),
             false,
-            imagePaths.isNotEmpty()
+            imagePaths.isNotEmpty(),
+            tracks,
+            mutableMapOf(),
         )
         synchronized(this) { editsDB.add(edit) }
         onAddedEdit(edit)
@@ -85,6 +91,14 @@ import javax.inject.Singleton
         }
     }
 
+    fun updateData(edit: NoteEdit, name: String, data: String) {
+        synchronized(this) {
+            val tmpData = edit.uploadedDataMap.toMutableMap()
+            tmpData[name] = data
+            editsDB.updateUploadData(edit.noteId, Json.encodeToString(tmpData))
+        }
+    }
+
     fun markSyncFailed(edit: NoteEdit): Boolean =
         delete(edit)
 

app/src/main/java/de/westnordost/streetcomplete/data/osmnotes/edits/NoteEditsDao.kt

@@ -5,6 +5,7 @@ import de.westnordost.streetcomplete.data.Database
 import de.westnordost.streetcomplete.data.osm.mapdata.BoundingBox
 import de.westnordost.streetcomplete.data.osm.mapdata.LatLon
 import de.westnordost.streetcomplete.data.osmnotes.edits.NoteEditsTable.Columns.CREATED_TIMESTAMP
+import de.westnordost.streetcomplete.data.osmnotes.edits.NoteEditsTable.Columns.TRACKS
 import javax.inject.Inject
 import de.westnordost.streetcomplete.data.osmnotes.edits.NoteEditsTable.Columns.ID
 import de.westnordost.streetcomplete.data.osmnotes.edits.NoteEditsTable.Columns.IMAGES_NEED_ACTIVATION
@@ -15,6 +16,7 @@ import de.westnordost.streetcomplete.data.osmnotes.edits.NoteEditsTable.Columns.
 import de.westnordost.streetcomplete.data.osmnotes.edits.NoteEditsTable.Columns.NOTE_ID
 import de.westnordost.streetcomplete.data.osmnotes.edits.NoteEditsTable.Columns.TEXT
 import de.westnordost.streetcomplete.data.osmnotes.edits.NoteEditsTable.Columns.TYPE
+import de.westnordost.streetcomplete.data.osmnotes.edits.NoteEditsTable.Columns.UPLOAD_DATA_MAP
 import de.westnordost.streetcomplete.data.osmnotes.edits.NoteEditsTable.NAME
 import de.westnordost.streetcomplete.ktx.*
 import kotlinx.serialization.decodeFromString
@@ -105,6 +107,9 @@ class NoteEditsDao @Inject constructor(private val db: Database) {
     fun updateNoteId(oldNoteId: Long, newNoteId: Long): Int =
         db.update(NAME, listOf(NOTE_ID to newNoteId), "$NOTE_ID = $oldNoteId")
 
+    fun updateUploadData(noteId: Long, data: String): Int =
+        db.update(NAME, listOf(UPLOAD_DATA_MAP to data), "$NOTE_ID = $noteId")
+
     fun getOldestNeedingImagesActivation(): NoteEdit? =
         db.queryOne(NAME, where = "$IS_SYNCED = 1 AND $IMAGES_NEED_ACTIVATION = 1", orderBy = CREATED_TIMESTAMP) { it.toNoteEdit() }
 
@@ -125,6 +130,8 @@ class NoteEditsDao @Inject constructor(private val db: Database) {
         TEXT to text,
         IMAGE_PATHS to Json.encodeToString(imagePaths),
         IMAGES_NEED_ACTIVATION to if (imagesNeedActivation) 1 else 0,
+        TRACKS to Json.encodeToString(tracks),
+        UPLOAD_DATA_MAP to Json.encodeToString(uploadedDataMap),
         TYPE to action.name
     )
 
@@ -137,6 +144,8 @@ class NoteEditsDao @Inject constructor(private val db: Database) {
         Json.decodeFromString(getString(IMAGE_PATHS)),
         getLong(CREATED_TIMESTAMP),
         getInt(IS_SYNCED) == 1,
-        getInt(IMAGES_NEED_ACTIVATION) == 1
+        getInt(IMAGES_NEED_ACTIVATION) == 1,
+        Json.decodeFromString(getString(TRACKS)),
+        Json.decodeFromString(getString(UPLOAD_DATA_MAP)),
     )
 }

app/src/main/java/de/westnordost/streetcomplete/data/osmnotes/edits/NoteEditsTable.kt

@@ -14,6 +14,8 @@ object NoteEditsTable {
         const val TEXT = "text"
         const val IMAGE_PATHS = "image_paths"
         const val IMAGES_NEED_ACTIVATION = "images_need_activation"
+        const val TRACKS = "tracks"
+        const val UPLOAD_DATA_MAP = "upload_data_map"
     }
 
     const val CREATE = """
@@ -27,6 +29,8 @@ object NoteEditsTable {
             ${Columns.TEXT} text,
             ${Columns.IMAGE_PATHS} text NOT NULL,
             ${Columns.IMAGES_NEED_ACTIVATION} int NOT NULL,
+            ${Columns.TRACKS} text NOT NULL,
+            ${Columns.UPLOAD_DATA_MAP} text NOT NULL,
             ${Columns.TYPE} varchar(255)
         );"""