@rnmapbox/maps 10.0.0-beta.58 → 10.0.0-beta.60

package/plugin/build/generateCode.js

@@ -0,0 +1,107 @@
+"use strict";
+/**
+ * Taken from @expo/config-plugins
+ *
+ * Sourcecode: https://github.com/expo/expo/blob/59ece3cb1d5a7aaea42f4c7fe9d1f4f825b338f8/packages/@expo/config-plugins/src/utils/generateCode.ts
+ * LICENSE: https://github.com/expo/expo/blob/59ece3cb1d5a7aaea42f4c7fe9d1f4f825b338f8/packages/@expo/config-plugins/LICENSE
+ */
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.createHash = exports.createGeneratedHeaderComment = exports.removeGeneratedContents = exports.removeContents = exports.mergeContents = void 0;
+/**
+ * Get line indexes for the generated section of a file.
+ *
+ * @param src
+ */
+const crypto_1 = __importDefault(require("crypto"));
+function getGeneratedSectionIndexes(src, tag) {
+    const contents = src.split('\n');
+    const start = contents.findIndex((line) => line.includes(`@generated begin ${tag}`));
+    const end = contents.findIndex((line) => line.includes(`@generated end ${tag}`));
+    return { contents, start, end };
+}
+/**
+ * Merge the contents of two files together and add a generated header.
+ *
+ * @param src contents of the original file
+ * @param newSrc new contents to merge into the original file
+ * @param identifier used to update and remove merges
+ * @param anchor regex to where the merge should begin
+ * @param offset line offset to start merging at (<1 for behind the anchor)
+ * @param comment comment style `//` or `#`
+ */
+function mergeContents({ src, newSrc, tag, anchor, offset, comment, }) {
+    const header = createGeneratedHeaderComment(newSrc, tag, comment);
+    if (!src.includes(header)) {
+        // Ensure the old generated contents are removed.
+        const sanitizedTarget = removeGeneratedContents(src, tag);
+        return {
+            contents: addLines(sanitizedTarget ?? src, anchor, offset, [
+                header,
+                ...newSrc.split('\n'),
+                `${comment} @generated end ${tag}`,
+            ]),
+            didMerge: true,
+            didClear: !!sanitizedTarget,
+        };
+    }
+    return { contents: src, didClear: false, didMerge: false };
+}
+exports.mergeContents = mergeContents;
+function removeContents({ src, tag, }) {
+    // Ensure the old generated contents are removed.
+    const sanitizedTarget = removeGeneratedContents(src, tag);
+    return {
+        contents: sanitizedTarget ?? src,
+        didMerge: false,
+        didClear: !!sanitizedTarget,
+    };
+}
+exports.removeContents = removeContents;
+function addLines(content, find, offset, toAdd) {
+    const lines = content.split('\n');
+    let lineIndex = lines.findIndex((line) => line.match(find));
+    if (lineIndex < 0) {
+        const error = new Error(`Failed to match "${find}" in contents:\n${content}`);
+        // eslint-disable-next-line @typescript-eslint/ban-ts-comment
+        // @ts-ignore
+        error.code = 'ERR_NO_MATCH';
+        throw error;
+    }
+    for (const newLine of toAdd) {
+        lines.splice(lineIndex + offset, 0, newLine);
+        lineIndex++;
+    }
+    return lines.join('\n');
+}
+/**
+ * Removes the generated section from a file, returns null when nothing can be removed.
+ * This sways heavily towards not removing lines unless it's certain that modifications were not made manually.
+ *
+ * @param src
+ */
+function removeGeneratedContents(src, tag) {
+    const { contents, start, end } = getGeneratedSectionIndexes(src, tag);
+    if (start > -1 && end > -1 && start < end) {
+        contents.splice(start, end - start + 1);
+        // TODO: We could in theory check that the contents we're removing match the hash used in the header,
+        // this would ensure that we don't accidentally remove lines that someone added or removed from the generated section.
+        return contents.join('\n');
+    }
+    return null;
+}
+exports.removeGeneratedContents = removeGeneratedContents;
+function createGeneratedHeaderComment(contents, tag, comment) {
+    const hashKey = createHash(contents);
+    // Everything after the `${tag} ` is unversioned and can be freely modified without breaking changes.
+    return `${comment} @generated begin ${tag} - expo prebuild (DO NOT MODIFY) ${hashKey}`;
+}
+exports.createGeneratedHeaderComment = createGeneratedHeaderComment;
+function createHash(src) {
+    // this doesn't need to be secure, the shorter the better.
+    const hash = crypto_1.default.createHash('sha1').update(src).digest('hex');
+    return `sync-${hash}`;
+}
+exports.createHash = createHash;

package/plugin/build/withMapbox.d.ts

@@ -1,4 +1,4 @@
-import { ConfigPlugin, XcodeProject } from '@expo/config-plugins';
+import { ConfigPlugin, XcodeProject } from 'expo/config-plugins';
 declare type InstallerBlockName = 'pre' | 'post';
 export declare type MapboxPlugProps = {
     RNMapboxMapsImpl?: string;

package/plugin/build/withMapbox.js

@@ -6,8 +6,8 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports._addMapboxMavenRepo = exports.setExcludedArchitectures = exports.addMapboxInstallerBlock = exports.addInstallerBlock = exports.addConstantBlock = exports.applyCocoaPodsModifications = void 0;
 const fs_1 = require("fs");
 const path_1 = __importDefault(require("path"));
-const config_plugins_1 = require("@expo/config-plugins");
-const generateCode_1 = require("@expo/config-plugins/build/utils/generateCode");
+const config_plugins_1 = require("expo/config-plugins");
+const generateCode_1 = require("./generateCode");
 let pkg = {
     name: '@rnmapbox/maps',
 };
@@ -127,7 +127,7 @@ function setExcludedArchitectures(project) {
     for (const { buildSettings } of Object.values(configurations || {})) {
         // Guessing that this is the best way to emulate Xcode.
         // Using `project.addToBuildSettings` modifies too many targets.
-        if (typeof (buildSettings === null || buildSettings === void 0 ? void 0 : buildSettings.PRODUCT_NAME) !== 'undefined') {
+        if (typeof buildSettings?.PRODUCT_NAME !== 'undefined') {
             buildSettings['"EXCLUDED_ARCHS[sdk=iphonesimulator*]"'] = '"arm64"';
         }
     }

package/plugin/src/generateCode.ts

@@ -0,0 +1,155 @@
+/**
+ * Taken from @expo/config-plugins
+ *
+ * Sourcecode: https://github.com/expo/expo/blob/59ece3cb1d5a7aaea42f4c7fe9d1f4f825b338f8/packages/@expo/config-plugins/src/utils/generateCode.ts
+ * LICENSE: https://github.com/expo/expo/blob/59ece3cb1d5a7aaea42f4c7fe9d1f4f825b338f8/packages/@expo/config-plugins/LICENSE
+ */
+
+/**
+ * Get line indexes for the generated section of a file.
+ *
+ * @param src
+ */
+import crypto from 'crypto';
+
+function getGeneratedSectionIndexes(
+  src: string,
+  tag: string,
+): { contents: string[]; start: number; end: number } {
+  const contents = src.split('\n');
+  const start = contents.findIndex((line) =>
+    line.includes(`@generated begin ${tag}`),
+  );
+  const end = contents.findIndex((line) =>
+    line.includes(`@generated end ${tag}`),
+  );
+
+  return { contents, start, end };
+}
+
+export type MergeResults = {
+  contents: string;
+  didClear: boolean;
+  didMerge: boolean;
+};
+
+/**
+ * Merge the contents of two files together and add a generated header.
+ *
+ * @param src contents of the original file
+ * @param newSrc new contents to merge into the original file
+ * @param identifier used to update and remove merges
+ * @param anchor regex to where the merge should begin
+ * @param offset line offset to start merging at (<1 for behind the anchor)
+ * @param comment comment style `//` or `#`
+ */
+export function mergeContents({
+  src,
+  newSrc,
+  tag,
+  anchor,
+  offset,
+  comment,
+}: {
+  src: string;
+  newSrc: string;
+  tag: string;
+  anchor: string | RegExp;
+  offset: number;
+  comment: string;
+}): MergeResults {
+  const header = createGeneratedHeaderComment(newSrc, tag, comment);
+  if (!src.includes(header)) {
+    // Ensure the old generated contents are removed.
+    const sanitizedTarget = removeGeneratedContents(src, tag);
+    return {
+      contents: addLines(sanitizedTarget ?? src, anchor, offset, [
+        header,
+        ...newSrc.split('\n'),
+        `${comment} @generated end ${tag}`,
+      ]),
+      didMerge: true,
+      didClear: !!sanitizedTarget,
+    };
+  }
+  return { contents: src, didClear: false, didMerge: false };
+}
+
+export function removeContents({
+  src,
+  tag,
+}: {
+  src: string;
+  tag: string;
+}): MergeResults {
+  // Ensure the old generated contents are removed.
+  const sanitizedTarget = removeGeneratedContents(src, tag);
+  return {
+    contents: sanitizedTarget ?? src,
+    didMerge: false,
+    didClear: !!sanitizedTarget,
+  };
+}
+
+function addLines(
+  content: string,
+  find: string | RegExp,
+  offset: number,
+  toAdd: string[],
+) {
+  const lines = content.split('\n');
+
+  let lineIndex = lines.findIndex((line) => line.match(find));
+  if (lineIndex < 0) {
+    const error = new Error(
+      `Failed to match "${find}" in contents:\n${content}`,
+    );
+    // eslint-disable-next-line @typescript-eslint/ban-ts-comment
+    // @ts-ignore
+    error.code = 'ERR_NO_MATCH';
+    throw error;
+  }
+  for (const newLine of toAdd) {
+    lines.splice(lineIndex + offset, 0, newLine);
+    lineIndex++;
+  }
+
+  return lines.join('\n');
+}
+
+/**
+ * Removes the generated section from a file, returns null when nothing can be removed.
+ * This sways heavily towards not removing lines unless it's certain that modifications were not made manually.
+ *
+ * @param src
+ */
+export function removeGeneratedContents(
+  src: string,
+  tag: string,
+): string | null {
+  const { contents, start, end } = getGeneratedSectionIndexes(src, tag);
+  if (start > -1 && end > -1 && start < end) {
+    contents.splice(start, end - start + 1);
+    // TODO: We could in theory check that the contents we're removing match the hash used in the header,
+    // this would ensure that we don't accidentally remove lines that someone added or removed from the generated section.
+    return contents.join('\n');
+  }
+  return null;
+}
+
+export function createGeneratedHeaderComment(
+  contents: string,
+  tag: string,
+  comment: string,
+): string {
+  const hashKey = createHash(contents);
+
+  // Everything after the `${tag} ` is unversioned and can be freely modified without breaking changes.
+  return `${comment} @generated begin ${tag} - expo prebuild (DO NOT MODIFY) ${hashKey}`;
+}
+
+export function createHash(src: string): string {
+  // this doesn't need to be secure, the shorter the better.
+  const hash = crypto.createHash('sha1').update(src).digest('hex');
+  return `sync-${hash}`;
+}

package/plugin/src/withMapbox.ts

@@ -11,11 +11,9 @@ import {
   WarningAggregator,
   withProjectBuildGradle,
   withAppBuildGradle,
-} from '@expo/config-plugins';
-import {
-  mergeContents,
-  removeGeneratedContents,
-} from '@expo/config-plugins/build/utils/generateCode';
+} from 'expo/config-plugins';
+
+import { mergeContents, removeGeneratedContents } from './generateCode';
 
 let pkg: { name: string; version?: string } = {
   name: '@rnmapbox/maps',

package/rnmapbox-maps.podspec

@@ -20,7 +20,7 @@ require 'json'
 package = JSON.parse(File.read(File.join(__dir__, 'package.json')))
 
 ## Warning: these lines are scanned by autogenerate.js
-rnMapboxMapsDefaultMapboxVersion = '~> 10.9.0'
+rnMapboxMapsDefaultMapboxVersion = '~> 10.9.1'
 rnMapboxMapsDefaultMapboxGLVersion = '~> 5.9.0'
 rnMapboxMapsDefaultMapLibreVersion = 'exactVersion 5.12.1'
 

package/android/rctmgl/src/main/java-v10/com/mapbox/rctmgl/components/location/LocationComponentManager.java

@@ -1,215 +0,0 @@
-package com.mapbox.rctmgl.components.location;
-
-import android.annotation.SuppressLint;
-import android.content.Context;
-import android.content.res.TypedArray;
-import android.graphics.Color;
-import android.graphics.ColorFilter;
-import android.graphics.LightingColorFilter;
-import android.graphics.drawable.Drawable;
-import android.graphics.drawable.VectorDrawable;
-
-import com.mapbox.geojson.Point;
-import com.mapbox.maps.CameraOptions;
-import com.mapbox.maps.MapboxMap;
-import com.mapbox.maps.Style;
-import com.mapbox.maps.plugin.LocationPuck;
-import com.mapbox.maps.plugin.LocationPuck2D;
-import com.mapbox.maps.plugin.gestures.GesturesUtils;
-import com.mapbox.maps.plugin.locationcomponent.LocationComponentPlugin;
-import com.mapbox.maps.plugin.locationcomponent.LocationComponentUtils;
-import com.mapbox.maps.plugin.locationcomponent.OnIndicatorBearingChangedListener;
-import com.mapbox.maps.plugin.locationcomponent.OnIndicatorPositionChangedListener;
-import com.mapbox.maps.plugin.locationcomponent.generated.LocationComponentSettings;
-import com.mapbox.rctmgl.R;
-import com.mapbox.rctmgl.components.location.CameraMode;
-import com.mapbox.rctmgl.components.mapview.RCTMGLMapView;
-import com.mapbox.rctmgl.location.LocationManager;
-import com.mapbox.rctmgl.utils.Logger;
-
-import androidx.annotation.NonNull;
-import androidx.appcompat.content.res.AppCompatResources;
-
-/**
- * The LocationComponent on android implements both location tracking and display of user's current location.
- * LocationComponentManager attempts to separate that, so that Camera can ask for location tracking independent of display of user current location.
- * And NativeUserLocation can ask for display of user's current location - independent of Camera's user tracking.
- */
-public class LocationComponentManager {
-    private RCTMGLMapView mMapView = null;
-    private MapboxMap mMap = null;
-
-    private LocationManager mLocationManager = null;
-    private LocationComponentPlugin mLocationComponent = null;
-    private Context mContext = null;
-
-    private int mCameraMode = CameraMode.NONE;
-
-    private @RenderMode.Mode int mRenderMode = RenderMode.COMPASS;
-
-    private OnIndicatorBearingChangedListener mLocationBearingChangedListener = new OnIndicatorBearingChangedListener() {
-        @Override
-        public void onIndicatorBearingChanged(double v) {
-            if (mFollowUserLocation) {
-                mMapView.getMapboxMap().setCamera(new CameraOptions.Builder().bearing(v).build());
-            }
-        }
-    };
-    private boolean bearingListenerInstalled = false;
-
-    private OnIndicatorPositionChangedListener mLocationPositionChangeListener = new OnIndicatorPositionChangedListener() {
-        @Override
-        public void onIndicatorPositionChanged(@NonNull Point point) {
-            if (mFollowUserLocation) {
-                mMapView.getMapboxMap().setCamera(new CameraOptions.Builder().center(point).build());
-                GesturesUtils.getGestures(mMapView).setFocalPoint(mMapView.getMapboxMap().pixelForCoordinate(point));
-
-                // sendUserLocationUpdateEvent(point);
-            }
-        }
-    };
-
-    public LocationComponentManager(RCTMGLMapView rctmglMapView, Context context) {
-        mMapView = rctmglMapView;
-        mMap = mMapView.getMapboxMap();
-        mContext = context;
-
-        mLocationManager = LocationManager.getInstance(context);
-    }
-
-    private boolean mShowUserLocation = false;
-
-    private boolean mFollowUserLocation = false;
-
-    private boolean mShowingUserLocation = false;
-
-    public void showUserLocation(boolean showUserLocation) {
-        mShowUserLocation = showUserLocation;
-        stateChanged();
-    }
-
-    public void setFollowUserLocation(boolean followUserLocation) {
-        mFollowUserLocation = followUserLocation;
-        stateChanged();
-    }
-
-    public void setCameraMode(@CameraMode.Mode int cameraMode) {
-        mCameraMode = cameraMode;
-        stateChanged();
-
-        LocationComponentPlugin locationComponent = LocationComponentUtils.getLocationComponent(mMapView);
-
-        if (mCameraMode == CameraMode.NONE || mCameraMode == CameraMode.TRACKING) {
-            locationComponent.removeOnIndicatorBearingChangedListener(mLocationBearingChangedListener);
-        } else {
-            locationComponent.addOnIndicatorBearingChangedListener(mLocationBearingChangedListener);
-        }
-
-        if (mCameraMode == CameraMode.NONE || mCameraMode == CameraMode.NONE_COMPASS || mCameraMode == CameraMode.NONE_GPS) {
-            locationComponent.removeOnIndicatorPositionChangedListener(mLocationPositionChangeListener);
-        } else {
-            locationComponent.addOnIndicatorPositionChangedListener(mLocationPositionChangeListener);
-        }
-    }
-
-    public void tintColorChanged() {
-        applyOptions(mShowingUserLocation, mLocationComponent);
-    }
-
-    public void setRenderMode(@RenderMode.Mode int renderMode) {
-        mRenderMode = renderMode;
-    }
-/*
-    public void addOnCameraTrackingChangedListener(OnCameraTrackingChangedListener onCameraTrackingChangedListener) {
-        // mLocationComponent.addOnCameraTrackingChangedListener(onCameraTrackingChangedListener);
-    }*/
-
-    @SuppressLint("MissingPermission")
-    private void stateChanged() {
-        mLocationComponent.setEnabled((mFollowUserLocation || mShowUserLocation));
-
-        if (mShowingUserLocation != mShowUserLocation) {
-            updateShowUserLocation(mShowUserLocation);
-        }
-
-        if (mFollowUserLocation) {
-            if (!mShowUserLocation) {
-               // mLocationComponent.setRenderMode(RenderMode.GPS);
-            } else {
-                // mLocationComponent.setRenderMode(mRenderMode);
-            }
-            mLocationComponent.onStart();
-        } else {
-            // mLocationComponent.setCameraMode(CameraMode.NONE);
-        }
-        mLocationComponent.setEnabled(mFollowUserLocation || mShowUserLocation);
-    }
-
-    public boolean hasLocationComponent() {
-        return (mLocationComponent != null);
-    }
-
-    public void update(@NonNull Style style) {
-        update(mShowUserLocation, style);
-    }
-
-    public void update(boolean displayUserLocation, @NonNull Style style) {
-        if (mLocationComponent == null) {
-            mLocationComponent = LocationComponentUtils.getLocationComponent(mMapView);
-
-            mLocationComponent.setLocationProvider(mLocationManager.getProvider());
-            mShowingUserLocation = displayUserLocation;
-        }
-
-        updateShowUserLocation(displayUserLocation);
-    }
-
-    private void updateShowUserLocation(boolean displayUserLocation) {
-        if (mShowingUserLocation != displayUserLocation) {
-            applyOptions(displayUserLocation, mLocationComponent);
-            mShowingUserLocation = displayUserLocation;
-        }
-    }
-
-    private void applyOptions(boolean displayUserLocation, LocationComponentPlugin locationComponent) {
-        locationComponent.setEnabled(true);
-        if (!displayUserLocation) {
-            LocationPuck2D locationPuck = new LocationPuck2D();
-
-            Drawable empty = AppCompatResources.getDrawable(mContext, R.drawable.empty);
-            locationPuck.setBearingImage(empty);
-            locationPuck.setShadowImage(empty);
-            locationPuck.setTopImage(empty);
-
-            locationComponent.setLocationPuck(locationPuck);
-            locationComponent.setPulsingEnabled(false);
-        } else {
-            int mapboxBlueColor = Color.parseColor("#4A90E2");
-
-            LocationPuck2D locationPuck = new LocationPuck2D();
-            Drawable topImage = AppCompatResources.getDrawable(mContext, R.drawable.mapbox_user_icon);
-
-            Integer tintColor = mMapView.getTintColor();
-            if (tintColor != null) {
-                VectorDrawable drawable = (VectorDrawable)topImage;
-                drawable.setTint(tintColor);
-
-                topImage = drawable;
-            }
-
-            locationPuck.setTopImage(topImage);
-            Drawable bearingImage = AppCompatResources.getDrawable(mContext, R.drawable.mapbox_user_stroke_icon);
-            locationPuck.setBearingImage(bearingImage);
-            Drawable shadowImage = AppCompatResources.getDrawable(mContext, R.drawable.mapbox_user_icon_shadow);
-            locationPuck.setShadowImage(shadowImage);
-            locationComponent.setLocationPuck(locationPuck);
-
-            locationComponent.setPulsingEnabled(true);
-            if (tintColor != null) {
-                locationComponent.setPulsingColor(tintColor);
-            } else {
-                locationComponent.setPulsingColor(mapboxBlueColor);
-            }
-        }
-    }
-}

package/android/rctmgl/src/main/java-v10/com/mapbox/rctmgl/components/location/RenderMode.java

@@ -1,42 +0,0 @@
-package com.mapbox.rctmgl.components.location;
-
-import androidx.annotation.IntDef;
-
-import java.lang.annotation.Retention;
-import java.lang.annotation.RetentionPolicy;
-
-/**
- * Contains the variety of ways the user location can be rendered on the map.
- */
-public final class RenderMode {
-
-  private RenderMode() {
-    // Class should not be initialized
-  }
-
-  /**
-   * One of these constants should be used with {@link LocationComponent#setRenderMode(int)}.
-   * Mode can be switched at anytime by calling the {@code setLocationLayerMode} method passing
-   * in the new mode you'd like the location layer to be in.
-   */
-  @IntDef( {COMPASS, GPS, NORMAL})
-  @Retention(RetentionPolicy.SOURCE)
-  public @interface Mode {
-  }
-
-  /**
-   * Basic tracking is enabled, bearing ignored.
-   */
-  public static final int NORMAL = 0x00000012;
-
-  /**
-   * Tracking the user location with bearing considered
-   * from a {@link CompassEngine}.
-   */
-  public static final int COMPASS = 0x00000004;
-
-  /**
-   * Tracking the user location with bearing considered from {@link android.location.Location}.
-   */
-  public static final int GPS = 0x00000008;
-}

package/android/rctmgl/src/main/java-v10/com/mapbox/rctmgl/events/LocationEvent.java

@@ -1,100 +0,0 @@
-package com.mapbox.rctmgl.events;
-
-import android.location.Location;
-import androidx.annotation.NonNull;
-
-import com.facebook.react.bridge.Arguments;
-import com.facebook.react.bridge.WritableMap;
-import com.facebook.react.bridge.WritableNativeMap;
-import com.mapbox.rctmgl.components.mapview.RCTMGLMapView;
-import com.mapbox.rctmgl.events.constants.EventKeys;
-import com.mapbox.rctmgl.events.constants.EventTypes;
-
-import java.util.UUID;
-
-public class LocationEvent implements IEvent {
-    private UUID uuid;
-    private RCTMGLMapView mapView;
-    private Location location;
-
-    public LocationEvent(@NonNull Location location, RCTMGLMapView mapView) {
-        this.mapView = mapView;
-        this.location = location;
-        this.uuid = UUID.randomUUID();
-    }
-
-    public LocationEvent(Location location) {
-        this(location, null);
-    }
-
-    @Override
-    public int getID() {
-        if (mapView != null) {
-            return mapView.getId();
-        }
-        return -1;
-    }
-
-    public UUID getUUID() {
-        return uuid;
-    }
-
-    @Override
-    public String getKey() {
-        return EventKeys.USER_LOCATION_UPDATE;
-    }
-
-    @Override
-    public String getType() {
-        return EventTypes.USER_LOCATION_UPDATED;
-    }
-
-    @Override
-    public long getTimestamp() {
-        return System.currentTimeMillis();
-    }
-
-    @Override
-    public boolean equals(IEvent event) {
-        LocationEvent other = (LocationEvent) event;
-        return getUUID().equals(other.getUUID());
-    }
-
-    public boolean equals(LocationEvent event) {
-        return uuid.equals(event.getUUID());
-    }
-
-    @Override
-    public WritableMap getPayload() {
-        WritableMap positionProperties = new WritableNativeMap();
-        WritableMap coords = new WritableNativeMap();
-
-        coords.putDouble("longitude", location.getLongitude());
-        coords.putDouble("latitude", location.getLatitude());
-        coords.putDouble("altitude", location.getAltitude());
-        coords.putDouble("accuracy", location.getAccuracy());
-        // A better solution will be to pull the heading from the compass engine, 
-        // unfortunately the api is not publicly available in the mapbox sdk
-        coords.putDouble("heading", location.getBearing());
-        coords.putDouble("course", location.getBearing());
-        coords.putDouble("speed", location.getSpeed());
-
-        positionProperties.putMap("coords", coords);
-        positionProperties.putDouble("timestamp", location.getTime());
-
-        return positionProperties;
-    }
-
-    @Override
-    public WritableMap toJSON() {
-        WritableMap map = Arguments.createMap();
-        map.putString("type", getType());
-        map.putMap("payload", getPayload());
-        return map;
-    }
-
-    @Override
-    public boolean canCoalesce() {
-        return true;
-    }
-}