From 5eeb8a8b0212132dde3b100c538a74cf7ab0f6bf Mon Sep 17 00:00:00 2001
From: Juha Heinanen <jh@tutpro.com>
Date: Fri, 22 May 2026 20:56:15 +0300
Subject: [PATCH] When redialing, don't ask again to select contact uri

---
 .../kotlin/com/tutpro/baresip/MainScreen.kt   | 68 +++++++++----------
 .../kotlin/com/tutpro/baresip/ViewModel.kt    |  1 +
 2 files changed, 34 insertions(+), 35 deletions(-)

diff --git a/app/src/main/kotlin/com/tutpro/baresip/MainScreen.kt b/app/src/main/kotlin/com/tutpro/baresip/MainScreen.kt
index c5659ae6..5e440293 100644
--- a/app/src/main/kotlin/com/tutpro/baresip/MainScreen.kt
+++ b/app/src/main/kotlin/com/tutpro/baresip/MainScreen.kt
@@ -1227,6 +1227,7 @@ private fun CallUriRow(
                     if (isDialer) {
                         if (it != dialerState.callUri.value) {
                             dialerState.callUri.value = it
+                            dialerState.redialUri = ""
                             if (it == "") {
                                 dialerState.showCallButton.value = true
                                 dialerState.showCallConferenceButton.value = true
@@ -2042,6 +2043,7 @@ private fun spinToAor(viewModel: ViewModel, aor: String, call: Call? = null) {
     }
 }
 
+// In MainScreen.kt, update callClick function
 private fun callClick(ctx: Context, viewModel: ViewModel, dialerState: ViewModel.DialerState?) {
     if (viewModel.selectedAor.value != "" && dialerState != null) {
         val uriText = dialerState.callUri.value.trim()
@@ -2049,43 +2051,37 @@ private fun callClick(ctx: Context, viewModel: ViewModel, dialerState: ViewModel
             if (Utils.checkPermissions(ctx, arrayOf(RECORD_AUDIO))) {
                 val aor = viewModel.selectedAor.value
                 val ua = UserAgent.ofAor(aor)
-                val uris = Contact.contactUris(uriText, ua?.account?.isMobile ?: false)
-                if (uris.isEmpty()) {
-                    if (Contact.nameExists(uriText, BaresipService.contacts, true)) {
-                        alertTitle.value = ctx.getString(R.string.notice)
-                        alertMessage.value = if (ua?.account?.isMobile == true)
-                            String.format(ctx.getString(R.string.contact_no_tel_uri), uriText)
-                        else
-                            String.format(ctx.getString(R.string.contact_no_sip_or_tel_uri), uriText)
-                        showAlert.value = true
-                    }
-		    else
-                        makeCall(
-                            ctx,
-                            viewModel,
-                            uriText,
-                            dialerState
-                        )
-                }
-                else if (uris.size == 1)
-                    makeCall(
-                        ctx,
-                        viewModel,
-                        uris[0],
-                        dialerState
-                    )
+                val uriToCall = if (dialerState.redialUri != "" &&
+                        uriText == Utils.friendlyUri(ctx, dialerState.redialUri, ua!!.account))
+                    dialerState.redialUri
                 else {
-                    selectItems.value = uris
-                    selectItemAction.value = { index ->
-                        makeCall(
-                            ctx,
-                            viewModel,
-                            uris[index],
-                            dialerState
-                        )
+                    val uris = Contact.contactUris(uriText, ua?.account?.isMobile ?: false)
+                    if (uris.isEmpty()) {
+                        if (Contact.nameExists(uriText, BaresipService.contacts, true)) {
+                            alertTitle.value = ctx.getString(R.string.notice)
+                            alertMessage.value = if (ua?.account?.isMobile == true)
+                                String.format(ctx.getString(R.string.contact_no_tel_uri), uriText)
+                            else
+                                String.format(ctx.getString(R.string.contact_no_sip_or_tel_uri), uriText)
+                            showAlert.value = true
+                            return
+                        }
+                        uriText
+                    }
+                    else if (uris.size == 1)
+                        uris[0]
+                    else {
+                        selectItems.value = uris
+                        selectItemAction.value = { index ->
+                            makeCall(ctx, viewModel, uris[index], dialerState)
+                        }
+                        showSelectItemDialog.value = true
+                        return
                     }
-                    showSelectItemDialog.value = true
                 }
+
+                dialerState.redialUri = ""
+                makeCall(ctx, viewModel, uriToCall, dialerState)
             }
             else
                 Toast.makeText(ctx, R.string.no_calls, Toast.LENGTH_SHORT).show()
@@ -2093,8 +2089,10 @@ private fun callClick(ctx: Context, viewModel: ViewModel, dialerState: ViewModel
         else {
             val ua = UserAgent.ofAor(viewModel.selectedAor.value)!!
             val latestPeerUri = CallHistoryNew.aorLatestPeerUri(ua.account.aor)
-            if (latestPeerUri != null)
+            if (latestPeerUri != null) {
+                dialerState.redialUri = latestPeerUri
                 dialerState.callUri.value = Utils.friendlyUri(ctx, latestPeerUri, ua.account)
+            }
         }
     }
 }
diff --git a/app/src/main/kotlin/com/tutpro/baresip/ViewModel.kt b/app/src/main/kotlin/com/tutpro/baresip/ViewModel.kt
index ea5026ad..0da2045c 100644
--- a/app/src/main/kotlin/com/tutpro/baresip/ViewModel.kt
+++ b/app/src/main/kotlin/com/tutpro/baresip/ViewModel.kt
@@ -42,6 +42,7 @@ class ViewModel: ViewModel() {
         val callUri: MutableState<String> = mutableStateOf(""),
         val callUriEnabled: MutableState<Boolean> = mutableStateOf(true),
         val callUriLabel: MutableState<String> = mutableStateOf(""),
+        var redialUri: String = "",
         val showSuggestions: MutableState<Boolean> = mutableStateOf(false),
         val showCallButton: MutableState<Boolean> = mutableStateOf(true),
         val showCallConferenceButton: MutableState<Boolean> = mutableStateOf(true),