Rename to getRegisteredInstance (#6445)

* Rename to getRegisteredInstance

* Have to do this for a breaking change

* make license

* ok

Author: kmadsen

CHANGELOG.md

@@ -15,6 +15,7 @@ Mapbox welcomes participation and contributions from everyone.
 - Fixed a rare issue where alternative route would not be vanished until deviation point for `MapboxRouteLineApi#setNavigationRoutes(routes, metadata)` when the route's geometry contained duplicate points. [#6399](https://github.com/mapbox/mapbox-navigation-android/pull/6399)
 - Introduced `ViewStyleCutomization.compassButtonParams` property to configure compass. [#6395](https://github.com/mapbox/mapbox-navigation-android/pull/6395)
 - Introduced `MapboxExtendableButtonParams.enabled` property to enable/disable a button. [#6395](https://github.com/mapbox/mapbox-navigation-android/pull/6395)
+- Renamed `MapboxAudioGuidance.getInstance()` to `getRegisteredInstance`. Rename `unMute` to `unmute` for audio guidance classes. [#6445](https://github.com/mapbox/mapbox-navigation-android/pull/6445)
 
 ## Mapbox Navigation SDK 2.9.0-alpha.4 - 30 September, 2022
 ### Changelog

LICENSE.md

@@ -40,8 +40,10 @@ dependencies {
 
     // This defines the minimum version of Navigation which is included in this SDK. To upgrade the
     // Navigation versions, you can specify a newer version in your downstream build.gradle.
-    releaseApi("com.mapbox.navigation:android:2.9.0-alpha.4")
-    debugApi project(":libnavigation-android")
+    // TODO reset to a specific version when the release is ready (and make license)
+    api project(":libnavigation-android")
+//    releaseApi("com.mapbox.navigation:android:2.9.0-beta.1")
+//    debugApi project(":libnavigation-android")
 
     // Search is currently in beta so it is not included in this SDK. The functionality of search
     // is included behind this library's api.

libnavui-androidauto/build.gradle

@@ -76,7 +76,7 @@ internal constructor(
         val signFormat = speedLimitOptions.forcedSignFormat
             ?: locationMatcherResult.speedLimit?.speedLimitSign
         val threshold = speedLimitOptions.warningThreshold
-        when (distanceFormatterOptions?.unitType) {
+        when (distanceFormatterOptions!!.unitType) {
             UnitType.IMPERIAL -> {
                 val speedLimit =
                     locationMatcherResult.speedLimit?.speedKmph?.let { speedLimitKmph ->

libnavui-androidauto/src/main/java/com/mapbox/androidauto/car/navigation/speedlimit/CarSpeedLimitRenderer.kt

@@ -19,7 +19,7 @@ fun Fragment.attachAudioGuidance(
     mapboxSoundButton: MapboxSoundButton
 ) {
     val lifecycleOwner = viewLifecycleOwner
-    val flow = MapboxAudioGuidance.getInstance().stateFlow()
+    val flow = MapboxAudioGuidance.getRegisteredInstance().stateFlow()
     lifecycleOwner.lifecycleScope.launch {
         lifecycleOwner.repeatOnLifecycle(Lifecycle.State.STARTED) {
             flow.collect { state ->
@@ -33,7 +33,7 @@ fun Fragment.attachAudioGuidance(
         }
     }
     mapboxSoundButton.setOnClickListener {
-        MapboxAudioGuidance.getInstance().toggle()
+        MapboxAudioGuidance.getRegisteredInstance().toggle()
     }
 }
 
@@ -45,15 +45,15 @@ fun Lifecycle.muteAudioGuidance() {
     addObserver(object : DefaultLifecycleObserver {
         lateinit var initialState: MapboxAudioGuidanceState
         override fun onResume(owner: LifecycleOwner) {
-            with(MapboxAudioGuidance.getInstance()) {
+            with(MapboxAudioGuidance.getRegisteredInstance()) {
                 initialState = stateFlow().value
                 mute()
             }
         }
 
         override fun onPause(owner: LifecycleOwner) {
             if (!initialState.isMuted) {
-                MapboxAudioGuidance.getInstance().unMute()
+                MapboxAudioGuidance.getRegisteredInstance().unmute()
             }
         }
     })

libnavui-androidauto/src/main/java/com/mapbox/androidauto/navigation/audioguidance/AppAudioGuidanceUi.kt

@@ -28,15 +28,15 @@ class CarAudioGuidanceUi : MapboxActionProvider.ScreenActionProvider {
      * Attach this to the screen while navigating.
      */
     private fun buildSoundButtonAction(screen: Screen): Action {
-        val audioGuidance = MapboxAudioGuidance.getInstance()
+        val audioGuidance = MapboxAudioGuidance.getRegisteredInstance()
         val state = audioGuidance.stateFlow().value
         return if (!state.isMuted) {
             buildIconAction(screen, R.drawable.mapbox_car_ic_volume_on) {
                 audioGuidance.mute()
             }
         } else {
             buildIconAction(screen, R.drawable.mapbox_car_ic_volume_off) {
-                audioGuidance.unMute()
+                audioGuidance.unmute()
             }
         }
     }
@@ -45,7 +45,7 @@ class CarAudioGuidanceUi : MapboxActionProvider.ScreenActionProvider {
         screen.lifecycle.apply {
             coroutineScope.launch {
                 repeatOnLifecycle(Lifecycle.State.STARTED) {
-                    MapboxAudioGuidance.getInstance().stateFlow()
+                    MapboxAudioGuidance.getRegisteredInstance().stateFlow()
                         .distinctUntilChanged { old, new ->
                             old.isMuted == new.isMuted && old.isPlayable == new.isPlayable
                         }

libnavui-androidauto/src/main/java/com/mapbox/androidauto/navigation/audioguidance/CarAudioGuidanceUi.kt

@@ -46,7 +46,7 @@ class AudioGuidanceStateController(
     override fun onAttached(mapboxNavigation: MapboxNavigation) {
         super.onAttached(mapboxNavigation)
 
-        val audioGuidance = MapboxAudioGuidance.getInstance()
+        val audioGuidance = MapboxAudioGuidance.getRegisteredInstance()
         audioGuidance.stateFlow().observe {
             if (it.isMuted != store.state.value.audio.isMuted) {
                 val newState = AudioGuidanceState(it.isMuted)
@@ -58,7 +58,7 @@ class AudioGuidanceStateController(
                 if (it.isMuted) {
                     audioGuidance.mute()
                 } else {
-                    audioGuidance.unMute()
+                    audioGuidance.unmute()
                 }
             }
         }

libnavui-app/src/main/java/com/mapbox/navigation/ui/app/internal/controller/AudioGuidanceStateController.kt

@@ -44,7 +44,7 @@ class AudioGuidanceStateControllerTest {
         mockAudioGuidance = mockk(relaxed = true) {
             every { stateFlow() } returns audioGuidanceState
         }
-        every { MapboxAudioGuidance.getInstance() } returns mockAudioGuidance
+        every { MapboxAudioGuidance.getRegisteredInstance() } returns mockAudioGuidance
 
         testStore = spyk(TestStore())
     }

libnavui-app/src/test/java/com/mapbox/navigation/ui/app/internal/controller/AudioGuidanceStateControllerTest.kt

@@ -23,19 +23,19 @@ package com.mapbox.navigation.ui.voice.api {
 
   public final class MapboxAudioGuidance implements com.mapbox.navigation.core.lifecycle.MapboxNavigationObserver {
     method public static com.mapbox.navigation.ui.voice.api.MapboxAudioGuidance create();
-    method public static com.mapbox.navigation.ui.voice.api.MapboxAudioGuidance getInstance();
+    method public static com.mapbox.navigation.ui.voice.api.MapboxAudioGuidance getRegisteredInstance();
     method public void mute();
     method public void onAttached(com.mapbox.navigation.core.MapboxNavigation mapboxNavigation);
     method public void onDetached(com.mapbox.navigation.core.MapboxNavigation mapboxNavigation);
     method public kotlinx.coroutines.flow.StateFlow<com.mapbox.navigation.ui.voice.api.MapboxAudioGuidanceState> stateFlow();
     method public void toggle();
-    method public void unMute();
+    method public void unmute();
     field public static final com.mapbox.navigation.ui.voice.api.MapboxAudioGuidance.Companion Companion;
   }
 
   public static final class MapboxAudioGuidance.Companion {
     method public com.mapbox.navigation.ui.voice.api.MapboxAudioGuidance create();
-    method public com.mapbox.navigation.ui.voice.api.MapboxAudioGuidance getInstance();
+    method public com.mapbox.navigation.ui.voice.api.MapboxAudioGuidance getRegisteredInstance();
   }
 
   public final class MapboxAudioGuidanceState {
@@ -199,26 +199,26 @@ package com.mapbox.navigation.ui.voice.view {
     method public int getMuteIconResId();
     method public String? getMuteText();
     method public androidx.appcompat.widget.AppCompatTextView getTextView();
-    method public int getUnMuteIconResId();
-    method public String? getUnMuteText();
+    method public int getUnmuteIconResId();
+    method public String? getUnmuteText();
     method @UiThread public void mute();
     method @UiThread public void muteAndExtend(long duration = 2000L);
     method @UiThread public void muteAndExtend();
     method public void setMuteIconResId(int);
     method public void setMuteText(String?);
-    method public void setUnMuteIconResId(int);
-    method public void setUnMuteText(String?);
-    method @UiThread public void unMute();
-    method @UiThread public void unMuteAndExtend(long duration = 2000L);
-    method @UiThread public void unMuteAndExtend();
+    method public void setUnmuteIconResId(int);
+    method public void setUnmuteText(String?);
+    method @UiThread public void unmute();
+    method @UiThread public void unmuteAndExtend(long duration = 2000L);
+    method @UiThread public void unmuteAndExtend();
     method public void updateStyle(@StyleRes int style);
     property public final androidx.constraintlayout.widget.ConstraintLayout containerView;
     property public final androidx.appcompat.widget.AppCompatImageView iconImage;
     property public final int muteIconResId;
     property public final String? muteText;
     property public final androidx.appcompat.widget.AppCompatTextView textView;
-    property public final int unMuteIconResId;
-    property public final String? unMuteText;
+    property public final int unmuteIconResId;
+    property public final String? unmuteText;
   }
 
   public final class MapboxSoundButton extends androidx.constraintlayout.widget.ConstraintLayout {

libnavui-voice/api/current.txt

@@ -78,7 +78,7 @@ internal constructor(
      * In order to enable voice guidance, you must call [MapboxNavigation.startTripSession]
      * and set a route for active guidance through [MapboxNavigation.setRoutes].
      *
-     * You can also control audio guidance by calling [mute], [unMute] or [toggle]
+     * You can also control audio guidance by calling [mute], [unmute] or [toggle]
      */
     fun stateFlow(): StateFlow<MapboxAudioGuidanceState> = internalStateFlow
 
@@ -94,7 +94,7 @@ internal constructor(
     /**
      * Explicit call to unmute the audio guidance state.
      */
-    fun unMute() {
+    fun unmute() {
         scope.launch {
             setMutedState(false)
         }
@@ -106,7 +106,7 @@ internal constructor(
     fun toggle() {
         scope.launch {
             if (mutedStateFlow.value) {
-                unMute()
+                unmute()
             } else {
                 mute()
             }
@@ -209,7 +209,7 @@ internal constructor(
          * Get the registered instance or create one and register it to [MapboxNavigationApp].
          */
         @JvmStatic
-        fun getInstance(): MapboxAudioGuidance = MapboxNavigationApp
+        fun getRegisteredInstance(): MapboxAudioGuidance = MapboxNavigationApp
             .getObservers(MapboxAudioGuidance::class)
             .firstOrNull() ?: create().also { MapboxNavigationApp.registerObserver(it) }
     }