Code review fixes.

This commit is contained in:
Onuray Sahin 2022-03-17 18:30:39 +03:00
parent 33e5a206bd
commit a13ba13fb5
4 changed files with 4 additions and 3 deletions

View File

@ -128,6 +128,7 @@ fun checkPermissions(permissionsToBeGranted: List<String>,
.setPositiveButton(R.string.ok) { _, _ ->
activityResultLauncher.launch(missingPermissions.toTypedArray())
}
.setNegativeButton(R.string.action_not_now, null)
.show()
} else {
// some permissions are not granted, ask permissions

View File

@ -19,7 +19,7 @@ package im.vector.app.features.location
import android.app.Activity
import im.vector.app.core.utils.openAppSettingsPage
class DefaultLocationSharingSettingsNavigator constructor(val activity: Activity?) : LocationSharingSettingsNavigator {
class DefaultLocationSharingNavigator constructor(val activity: Activity?) : LocationSharingNavigator {
override var goingToAppSettings: Boolean = false

View File

@ -54,7 +54,7 @@ class LocationSharingFragment @Inject constructor(
private val viewModel: LocationSharingViewModel by fragmentViewModel()
private val locationSharingNavigator: LocationSharingSettingsNavigator by lazy { DefaultLocationSharingSettingsNavigator(activity) }
private val locationSharingNavigator: LocationSharingNavigator by lazy { DefaultLocationSharingNavigator(activity) }
// Keep a ref to handle properly the onDestroy callback
private var mapView: WeakReference<MapView>? = null

View File

@ -16,7 +16,7 @@
package im.vector.app.features.location
interface LocationSharingSettingsNavigator {
interface LocationSharingNavigator {
var goingToAppSettings: Boolean
fun quit()
fun goToAppSettings()