- Added Waypoints to NavigationRoute; - RouteOptionsUpdater refactored.

Author: RingerJK

CHANGELOG.md

@@ -9,6 +9,8 @@ Mapbox welcomes participation and contributions from everyone.
 - Fixed an issue with `NavigationView` that caused road label position to not update in some cases. [#6508](https://github.com/mapbox/mapbox-navigation-android/pull/6508)
 - Fixed an issue where the SDK was trying to send telemetry events when telemetry is switched off globally. [#6512](https://github.com/mapbox/mapbox-navigation-android/pull/6512)
 
+:warning: `RouteProgress#remainingWaypoints` defines the amount of all remaining waypoints, include explicit (requested with `RouteOptions#coorinatesList`) and implicit (added on the fly, like Electric Vehicle waypoints - [see](https://docs.mapbox.com/api/navigation/directions/#electric-vehicle-routing)).
+
 ## Mapbox Navigation SDK 2.9.0-beta.3 - 21 October, 2022
 ### Changelog
 [Changes between v2.9.0-beta.2 and v2.9.0-beta.3](https://github.com/mapbox/mapbox-navigation-android/compare/v2.9.0-beta.2...v2.9.0-beta.3)

libnavigation-base/src/main/java/com/mapbox/navigation/base/internal/extensions/WaypointEx.kt

@@ -0,0 +1,58 @@
+package com.mapbox.navigation.base.internal.extensions
+
+import com.mapbox.api.directions.v5.models.DirectionsRoute
+import com.mapbox.api.directions.v5.models.RouteOptions
+import com.mapbox.navigation.base.internal.route.Waypoint
+import com.mapbox.navigation.base.trip.model.RouteProgress
+
+/**
+ * Return waypoints that are requested explicitly
+ */
+fun List<Waypoint>.requestedWaypoints(): List<Waypoint> =
+    this.filter {
+        when (it.type) {
+            Waypoint.REGULAR,
+            Waypoint.SILENT -> true
+            Waypoint.EV_CHARGING -> false
+            else -> false
+        }
+    }
+
+/**
+ * Return waypoints that tracking in [RouteProgress.currentLegProgress]#legIndex and based on
+ * [DirectionsRoute.legs] index
+ */
+fun List<Waypoint>.legsWaypoints(): List<Waypoint> =
+    this.filter {
+        when (it.type) {
+            Waypoint.REGULAR,
+            Waypoint.EV_CHARGING -> true
+            Waypoint.SILENT -> false
+            else -> false
+        }
+    }
+
+/**
+ * Return the index of **next requested** coordinate. See [RouteOptions.coordinatesList]
+ *
+ * For instance, EV waypoints are not requested explicitly, so they are not taken into account.
+ */
+fun indexOfNextRequestedCoordinate(
+    waypoints: List<Waypoint>,
+    remainingWaypoints: Int,
+): Int? {
+    if (remainingWaypoints > waypoints.size) {
+        return null
+    }
+    val nextWaypointIndex = waypoints.size - remainingWaypoints
+    var requestedIndex = 0
+    waypoints.forEachIndexed { index, waypoint ->
+        if (waypoint.type == Waypoint.REGULAR || waypoint.type == Waypoint.SILENT) {
+            if (index >= nextWaypointIndex) {
+                return requestedIndex
+            }
+            requestedIndex++
+        }
+    }
+    return null
+}

libnavigation-base/src/main/java/com/mapbox/navigation/base/internal/route/Waypoint.kt

@@ -0,0 +1,53 @@
+package com.mapbox.navigation.base.internal.route
+
+import androidx.annotation.IntDef
+import com.mapbox.geojson.Point
+
+class Waypoint(
+    val location: Point,
+    @Type val type: Int,
+    val name: String,
+    val target: Point?,
+) {
+    companion object {
+        const val REGULAR = 1
+        const val SILENT = 2
+        const val EV_CHARGING = 3
+    }
+
+    @Target(
+        AnnotationTarget.PROPERTY,
+        AnnotationTarget.VALUE_PARAMETER,
+        AnnotationTarget.FUNCTION,
+        AnnotationTarget.TYPE
+    )
+    @Retention(AnnotationRetention.BINARY)
+    @IntDef(REGULAR, SILENT, EV_CHARGING)
+    annotation class Type
+
+    override fun equals(other: Any?): Boolean {
+        if (this === other) return true
+        if (javaClass != other?.javaClass) return false
+
+        other as Waypoint
+
+        if (location != other.location) return false
+        if (type != other.type) return false
+        if (name != other.name) return false
+        if (target != other.target) return false
+
+        return true
+    }
+
+    override fun hashCode(): Int {
+        var result = location.hashCode()
+        result = 31 * result + type
+        result = 31 * result + name.hashCode()
+        result = 31 * result + target.hashCode()
+        return result
+    }
+
+    override fun toString(): String {
+        return "Waypoint(location=$location, type=$type, name='$name', target=$target)"
+    }
+}

libnavigation-base/src/main/java/com/mapbox/navigation/base/internal/utils/DirectionsRouteEx.kt

@@ -4,6 +4,7 @@ package com.mapbox.navigation.base.internal.utils
 
 import com.mapbox.api.directions.v5.models.DirectionsRoute
 import com.mapbox.api.directions.v5.models.LegStep
+import com.mapbox.navigation.base.internal.route.Waypoint
 import com.mapbox.navigation.base.utils.ifNonNull
 
 /**
@@ -36,3 +37,21 @@ private fun DirectionsRoute.stepsNamesAsString(): String? =
         ?.joinToString { leg ->
             leg.steps()?.joinToString { step -> step.name() ?: "" } ?: ""
         }
+
+internal fun List<com.mapbox.navigator.Waypoint>.mapToSdk(): List<Waypoint> =
+    map { nativeWaypoint ->
+        Waypoint(
+            location = nativeWaypoint.location,
+            type = nativeWaypoint.type.mapToSdk(),
+            name = nativeWaypoint.name,
+            target = nativeWaypoint.target,
+        )
+    }
+
+@Waypoint.Type
+private fun com.mapbox.navigator.WaypointType.mapToSdk(): Int =
+    when (this) {
+        com.mapbox.navigator.WaypointType.REGULAR -> Waypoint.REGULAR
+        com.mapbox.navigator.WaypointType.SILENT -> Waypoint.SILENT
+        com.mapbox.navigator.WaypointType.EV_CHARGING -> Waypoint.EV_CHARGING
+    }

libnavigation-base/src/main/java/com/mapbox/navigation/base/internal/utils/RouterEx.kt

@@ -1,5 +1,7 @@
 package com.mapbox.navigation.base.internal.utils
 
+import com.mapbox.navigation.base.internal.route.Waypoint
+import com.mapbox.navigation.base.route.NavigationRoute
 import com.mapbox.navigator.RouterOrigin
 
 fun RouterOrigin.mapToSdkRouteOrigin(): com.mapbox.navigation.base.route.RouterOrigin =
@@ -15,3 +17,5 @@ fun com.mapbox.navigation.base.route.RouterOrigin.mapToNativeRouteOrigin(): Rout
         com.mapbox.navigation.base.route.RouterOrigin.Onboard -> RouterOrigin.ONBOARD
         is com.mapbox.navigation.base.route.RouterOrigin.Custom -> RouterOrigin.CUSTOM
     }
+
+fun NavigationRoute.internalWaypoints(): List<Waypoint> = waypoints

libnavigation-base/src/main/java/com/mapbox/navigation/base/route/NavigationRoute.kt

@@ -17,7 +17,10 @@ import com.mapbox.navigation.base.internal.NativeRouteParserWrapper
 import com.mapbox.navigation.base.internal.SDKRouteParser
 import com.mapbox.navigation.base.internal.factory.RoadObjectFactory.toUpcomingRoadObjects
 import com.mapbox.navigation.base.internal.route.RouteCompatibilityCache
+import com.mapbox.navigation.base.internal.route.Waypoint
+import com.mapbox.navigation.base.internal.route.toNavigationRoute
 import com.mapbox.navigation.base.internal.utils.DirectionsRouteMissingConditionsCheck
+import com.mapbox.navigation.base.internal.utils.mapToSdk
 import com.mapbox.navigation.base.internal.utils.mapToSdkRouteOrigin
 import com.mapbox.navigation.base.trip.model.roadobject.UpcomingRoadObject
 import com.mapbox.navigation.utils.internal.ThreadController
@@ -293,6 +296,8 @@ class NavigationRoute internal constructor(
      */
     val upcomingRoadObjects = nativeRoute.routeInfo.alerts.toUpcomingRoadObjects()
 
+    internal val waypoints: List<Waypoint> by lazy { nativeRoute.waypoints.mapToSdk() }
+
     /**
      * Indicates whether some other object is "equal to" this one.
      *

libnavigation-base/src/main/java/com/mapbox/navigation/base/trip/model/RouteProgress.kt

@@ -33,7 +33,9 @@ import com.mapbox.navigation.base.trip.model.roadobject.UpcomingRoadObject
  * @param durationRemaining [Double] seconds time remaining until the route destination is reached.
  * @param fractionTraveled [Float] fraction traveled along the current route. This value is
  * between 0 and 1 and isn't guaranteed to reach 1 before the user reaches the end of the route.
- * @param remainingWaypoints [Int] number of waypoints remaining on the current route.
+ * @param remainingWaypoints [Int] number of waypoints remaining on the current route. The waypoints number can be different
+ * with number of requested coordinates. For instance, [EV routing](https://docs.mapbox.com/api/navigation/directions/#electric-vehicle-routing)
+ * is adding additional waypoints, that are not requested explicitly.
  * @param upcomingRoadObjects list of upcoming road objects.
  * @param stale `true` if there were no location updates for a significant amount which causes
  * a lack of confidence in the progress updates being sent.