From b0fdf4cf7694349e67272d4576d351aee3e120fa Mon Sep 17 00:00:00 2001 From: tobrun Date: Tue, 4 Jun 2019 10:10:50 +0200 Subject: [android] - rename style component URL api to URI --- .../java/com/mapbox/mapboxsdk/maps/MapboxMap.java | 8 +-- .../mapbox/mapboxsdk/maps/MapboxMapOptions.java | 50 +++++++++++++--- .../java/com/mapbox/mapboxsdk/maps/NativeMap.java | 4 +- .../com/mapbox/mapboxsdk/maps/NativeMapView.java | 8 +-- .../main/java/com/mapbox/mapboxsdk/maps/Style.java | 68 ++++++++++++++++++++-- .../mapboxsdk/snapshotter/MapSnapshotter.java | 45 ++++++++++++-- .../src/main/res-public/values/public.xml | 5 +- .../src/main/res/values/attrs.xml | 5 +- .../com/mapbox/mapboxsdk/maps/StyleBuilderTest.kt | 2 +- .../java/com/mapbox/mapboxsdk/maps/StyleTest.kt | 22 +++---- .../com/mapbox/mapboxsdk/maps/NativeMapViewTest.kt | 4 +- .../mapboxsdk/testapp/activity/EspressoTest.java | 2 +- .../mapboxsdk/testapp/style/RuntimeStyleTests.java | 18 ++---- .../mapboxsdk/testapp/style/StyleLoaderTest.java | 2 +- .../camera/CameraAnimationTypeActivity.java | 2 +- .../activity/camera/LatLngBoundsActivity.java | 2 +- .../activity/camera/ManualZoomActivity.java | 2 +- .../activity/camera/MaxMinZoomActivity.java | 2 +- .../QueryRenderedFeaturesBoxCountActivity.java | 2 +- .../QueryRenderedFeaturesBoxHighlightActivity.java | 2 +- .../location/LocationMapChangeActivity.java | 4 +- .../activity/location/LocationModesActivity.java | 4 +- .../location/ManualLocationUpdatesActivity.java | 2 +- .../activity/maplayout/BottomSheetActivity.java | 2 +- .../activity/maplayout/DebugModeActivity.java | 4 +- .../activity/maplayout/DoubleMapActivity.java | 2 +- .../activity/maplayout/SimpleMapActivity.java | 2 +- .../testapp/activity/offline/OfflineActivity.java | 2 +- .../style/AnimatedImageSourceActivity.java | 2 +- .../activity/style/CircleLayerActivity.java | 2 +- .../style/FillExtrusionStyleTestActivity.java | 2 +- .../activity/style/GeoJsonClusteringActivity.java | 2 +- .../testapp/activity/style/GridSourceActivity.java | 2 +- .../activity/style/HeatmapLayerActivity.java | 2 +- .../activity/style/HillshadeLayerActivity.java | 2 +- .../activity/style/RuntimeStyleActivity.java | 8 +-- .../style/RuntimeStyleTimingTestActivity.java | 2 +- .../testapp/activity/style/StyleFileActivity.java | 2 +- .../activity/style/SymbolLayerActivity.java | 2 +- .../telemetry/PerformanceMeasurementActivity.java | 2 +- 40 files changed, 215 insertions(+), 92 deletions(-) diff --git a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/MapboxMap.java b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/MapboxMap.java index fa34e5b1f7..ea06856e9e 100644 --- a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/MapboxMap.java +++ b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/MapboxMap.java @@ -704,7 +704,7 @@ public final class MapboxMap { moveCamera(CameraUpdateFactory.newCameraPosition(cameraPosition)); setMinZoomPreference(minZoom); setMaxZoomPreference(maxZoom); - setStyle(new Style.Builder().fromUrl(definition.getStyleURL()), callback); + setStyle(new Style.Builder().fromUri(definition.getStyleURL()), callback); } // @@ -793,7 +793,7 @@ public final class MapboxMap { * @see Style */ public void setStyle(@Style.StyleUrl String style, final Style.OnStyleLoaded callback) { - this.setStyle(new Style.Builder().fromUrl(style), callback); + this.setStyle(new Style.Builder().fromUri(style), callback); } /** @@ -831,8 +831,8 @@ public final class MapboxMap { } style = builder.build(nativeMapView); - if (!TextUtils.isEmpty(builder.getUrl())) { - nativeMapView.setStyleUrl(builder.getUrl()); + if (!TextUtils.isEmpty(builder.getUri())) { + nativeMapView.setStyleUri(builder.getUri()); } else if (!TextUtils.isEmpty(builder.getJson())) { nativeMapView.setStyleJson(builder.getJson()); } else { diff --git a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/MapboxMapOptions.java b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/MapboxMapOptions.java index 389cb0a0a9..cc2124c6c7 100644 --- a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/MapboxMapOptions.java +++ b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/MapboxMapOptions.java @@ -11,9 +11,9 @@ import android.support.annotation.ColorInt; import android.support.annotation.NonNull; import android.support.annotation.Nullable; import android.support.v4.content.res.ResourcesCompat; +import android.text.TextUtils; import android.util.AttributeSet; import android.view.Gravity; - import com.mapbox.mapboxsdk.R; import com.mapbox.mapboxsdk.camera.CameraPosition; import com.mapbox.mapboxsdk.constants.MapboxConstants; @@ -70,7 +70,7 @@ public class MapboxMapOptions implements Parcelable { private boolean zMediaOverlay = false; private String localIdeographFontFamily = "sans-serif"; - private String apiBaseUrl; + private String apiBaseUri; private boolean textureMode; private boolean translucentTextureSurface; @@ -84,6 +84,7 @@ public class MapboxMapOptions implements Parcelable { /** * Creates a new MapboxMapOptions object. + * * @deprecated Use {@link #createFromAttributes(Context, AttributeSet)} instead. */ @Deprecated @@ -123,7 +124,7 @@ public class MapboxMapOptions implements Parcelable { doubleTapGesturesEnabled = in.readByte() != 0; quickZoomGesturesEnabled = in.readByte() != 0; - apiBaseUrl = in.readString(); + apiBaseUri = in.readString(); textureMode = in.readByte() != 0; translucentTextureSurface = in.readByte() != 0; prefetchesTiles = in.readByte() != 0; @@ -148,8 +149,16 @@ public class MapboxMapOptions implements Parcelable { TypedArray typedArray = context.obtainStyledAttributes(attrs, R.styleable.mapbox_MapView, 0, 0); try { mapboxMapOptions.camera(new CameraPosition.Builder(typedArray).build()); + + // deprecated mapboxMapOptions.apiBaseUrl(typedArray.getString(R.styleable.mapbox_MapView_mapbox_apiBaseUrl)); + String baseUri = typedArray.getString(R.styleable.mapbox_MapView_mapbox_apiBaseUri); + if (!TextUtils.isEmpty(baseUri)) { + // override deprecated property if a value of the new type was provided + mapboxMapOptions.apiBaseUri(baseUri); + } + mapboxMapOptions.zoomGesturesEnabled( typedArray.getBoolean(R.styleable.mapbox_MapView_mapbox_uiZoomGestures, true)); mapboxMapOptions.scrollGesturesEnabled( @@ -246,10 +255,24 @@ public class MapboxMapOptions implements Parcelable { * * @param apiBaseUrl The base of our API endpoint * @return This + * @deprecated use {@link #apiBaseUri} instead */ + @Deprecated @NonNull public MapboxMapOptions apiBaseUrl(String apiBaseUrl) { - this.apiBaseUrl = apiBaseUrl; + this.apiBaseUri = apiBaseUrl; + return this; + } + + /** + * Specifies the URI used for API endpoint. + * + * @param apiBaseUri The base of our API endpoint + * @return This + */ + @NonNull + public MapboxMapOptions apiBaseUri(String apiBaseUri) { + this.apiBaseUri = apiBaseUri; return this; } @@ -660,9 +683,20 @@ public class MapboxMapOptions implements Parcelable { * Get the current configured API endpoint base URL. * * @return Base URL to be used API endpoint. + * @deprecated use {@link #getApiBaseUri()} instead */ + @Deprecated public String getApiBaseUrl() { - return apiBaseUrl; + return apiBaseUri; + } + + /** + * Get the current configured API endpoint base URI. + * + * @return Base URI to be used API endpoint. + */ + public String getApiBaseUri() { + return apiBaseUri; } /** @@ -953,7 +987,7 @@ public class MapboxMapOptions implements Parcelable { dest.writeByte((byte) (doubleTapGesturesEnabled ? 1 : 0)); dest.writeByte((byte) (quickZoomGesturesEnabled ? 1 : 0)); - dest.writeString(apiBaseUrl); + dest.writeString(apiBaseUri); dest.writeByte((byte) (textureMode ? 1 : 0)); dest.writeByte((byte) (translucentTextureSurface ? 1 : 0)); dest.writeByte((byte) (prefetchesTiles ? 1 : 0)); @@ -1044,7 +1078,7 @@ public class MapboxMapOptions implements Parcelable { return false; } - if (apiBaseUrl != null ? !apiBaseUrl.equals(options.apiBaseUrl) : options.apiBaseUrl != null) { + if (apiBaseUri != null ? !apiBaseUri.equals(options.apiBaseUri) : options.apiBaseUri != null) { return false; } if (prefetchesTiles != options.prefetchesTiles) { @@ -1095,7 +1129,7 @@ public class MapboxMapOptions implements Parcelable { result = 31 * result + (zoomGesturesEnabled ? 1 : 0); result = 31 * result + (doubleTapGesturesEnabled ? 1 : 0); result = 31 * result + (quickZoomGesturesEnabled ? 1 : 0); - result = 31 * result + (apiBaseUrl != null ? apiBaseUrl.hashCode() : 0); + result = 31 * result + (apiBaseUri != null ? apiBaseUri.hashCode() : 0); result = 31 * result + (textureMode ? 1 : 0); result = 31 * result + (translucentTextureSurface ? 1 : 0); result = 31 * result + (prefetchesTiles ? 1 : 0); diff --git a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/NativeMap.java b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/NativeMap.java index 66038e4d45..7692c1a9f2 100644 --- a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/NativeMap.java +++ b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/NativeMap.java @@ -102,10 +102,10 @@ interface NativeMap { // Style API // - void setStyleUrl(String url); + void setStyleUri(String url); @NonNull - String getStyleUrl(); + String getStyleUri(); void setStyleJson(String newStyleJson); diff --git a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/NativeMapView.java b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/NativeMapView.java index 3980790fd1..7f93b9ad04 100755 --- a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/NativeMapView.java +++ b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/NativeMapView.java @@ -171,8 +171,8 @@ final class NativeMapView implements NativeMap { } @Override - public void setStyleUrl(String url) { - if (checkState("setStyleUrl")) { + public void setStyleUri(String url) { + if (checkState("setStyleUri")) { return; } nativeSetStyleUrl(url); @@ -180,8 +180,8 @@ final class NativeMapView implements NativeMap { @Override @NonNull - public String getStyleUrl() { - if (checkState("getStyleUrl")) { + public String getStyleUri() { + if (checkState("getStyleUri")) { return ""; } return nativeGetStyleUrl(); diff --git a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/Style.java b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/Style.java index 84efe6a4af..e1a5c8926c 100644 --- a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/Style.java +++ b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/Style.java @@ -59,11 +59,24 @@ public class Style { * Returns the current style url. * * @return the style url + * @deprecated use {@link #getUri()} instead */ @NonNull + @Deprecated public String getUrl() { validateState("getUrl"); - return nativeMap.getStyleUrl(); + return nativeMap.getStyleUri(); + } + + /** + * Returns the current style uri. + * + * @return the style uri + */ + @NonNull + public String getUri() { + validateState("getUri"); + return nativeMap.getStyleUri(); } /** @@ -600,7 +613,7 @@ public class Style { private final List images = new ArrayList<>(); private TransitionOptions transitionOptions; - private String styleUrl; + private String styleUri; private String styleJson; /** @@ -637,17 +650,60 @@ public class Style { * @param url The URL of the map style * @return this * @see Style + * @deprecated use {@link #fromUri(String)} instead */ + @Deprecated @NonNull public Builder fromUrl(@NonNull String url) { - this.styleUrl = url; + this.styleUri = url; + return this; + } + + /** + *

+ * Will loads a new map style asynchronous from the specified URI. + *

+ * {@code uri} can take the following forms: + *