Fix ReplayRouteLocationConverter.java that function calculateMockLocations return points with wrong bearings. (#1476)

danesfeder · web-flow · commit dac2d6ba172f · 2018-10-29T15:46:39.000-04:00
diff --git a/libandroid-navigation/src/main/java/com/mapbox/services/android/navigation/v5/location/replay/ReplayRouteLocationConverter.java b/libandroid-navigation/src/main/java/com/mapbox/services/android/navigation/v5/location/replay/ReplayRouteLocationConverter.java
@@ -81,7 +81,7 @@ List<Location> calculateMockLocations(List<Point> points) {
       Location mockedLocation = createMockLocationFrom(point);
 
       if (pointsToCopy.size() >= 2) {
-        double bearing = TurfMeasurement.bearing(point, points.get(1));
+        double bearing = TurfMeasurement.bearing(point, pointsToCopy.get(1));
         mockedLocation.setBearing((float) bearing);
       }
       time += delay * ONE_SECOND_IN_MILLISECONDS;

Original file line number	Diff line number	Diff line change
`@@ -81,7 +81,7 @@ List<Location> calculateMockLocations(List<Point> points) {`
`81`	`81`	`Location mockedLocation = createMockLocationFrom(point);`
`82`	`82`
`83`	`83`	`if (pointsToCopy.size() >= 2) {`
`84`		`- double bearing = TurfMeasurement.bearing(point, points.get(1));`
	`84`	`+ double bearing = TurfMeasurement.bearing(point, pointsToCopy.get(1));`
`85`	`85`	`mockedLocation.setBearing((float) bearing);`
`86`	`86`	`}`
`87`	`87`	`time += delay * ONE_SECOND_IN_MILLISECONDS;`