From f8d3c0060ef95a4af06580e35226bdbde035452e Mon Sep 17 00:00:00 2001 From: Juha Heinanen Date: Thu, 16 Dec 2021 11:02:36 +0200 Subject: [PATCH] Work on informing user when call is held by the peer --- app/src/main/cpp/baresip.c | 6 ++++ .../com/tutpro/baresip/BaresipService.kt | 11 +++++++ .../main/kotlin/com/tutpro/baresip/Call.kt | 7 ++-- .../kotlin/com/tutpro/baresip/MainActivity.kt | 15 +++++++++ app/src/main/res/layout/activity_main.xml | 33 ++++++++++++------- app/src/main/res/values/strings.xml | 1 + build.gradle | 2 +- 7 files changed, 60 insertions(+), 15 deletions(-) diff --git a/app/src/main/cpp/baresip.c b/app/src/main/cpp/baresip.c index 115c040e..26bd6586 100644 --- a/app/src/main/cpp/baresip.c +++ b/app/src/main/cpp/baresip.c @@ -200,6 +200,12 @@ static void ua_event_handler(struct ua *ua, enum ua_event ev, case UA_EVENT_CALL_ESTABLISHED: len = re_snprintf(event_buf, sizeof event_buf, "call established"); break; + case UA_EVENT_CALL_REMOTE_SDP: + if (strcmp(prm, "offer") != 0) + return; + ardir = sdp_media_rdir(stream_sdpmedia(audio_strm(call_audio(call)))); + len = re_snprintf(event_buf, sizeof event_buf, "call update,%d", ardir); + break; case UA_EVENT_CALL_MENC: if (prm[0] == '0') len = re_snprintf(event_buf, sizeof event_buf, "call secure"); diff --git a/app/src/main/kotlin/com/tutpro/baresip/BaresipService.kt b/app/src/main/kotlin/com/tutpro/baresip/BaresipService.kt index 41320650..cb7338ac 100644 --- a/app/src/main/kotlin/com/tutpro/baresip/BaresipService.kt +++ b/app/src/main/kotlin/com/tutpro/baresip/BaresipService.kt @@ -691,6 +691,17 @@ class BaresipService: Service() { if (!Utils.isVisible()) return } + "call update" -> { + val call = Call.ofCallp(callp) + if (call == null) { + Log.w("Baresip", "Call $callp that is updated is not found") + return + } + when (ev[1]) { + "0", "1" -> call.held = true + "2", "3" -> call.held = false + } + } "call verified", "call secure" -> { val call = Call.ofCallp(callp) if (call == null) { diff --git a/app/src/main/kotlin/com/tutpro/baresip/Call.kt b/app/src/main/kotlin/com/tutpro/baresip/Call.kt index 7276888d..6703b3a1 100644 --- a/app/src/main/kotlin/com/tutpro/baresip/Call.kt +++ b/app/src/main/kotlin/com/tutpro/baresip/Call.kt @@ -7,6 +7,7 @@ class Call(val callp: String, val ua: UserAgent, val peerUri: String, val dir: S var status: String, val dtmfWatcher: TextWatcher?) { var onhold = false + var held = false var security = 0 var zid = "" var hasHistory = false @@ -24,11 +25,11 @@ class Call(val callp: String, val ua: UserAgent, val peerUri: String, val dir: S return call_connect(callp, uri) } - fun startAudio() { - call_start_audio(callp) + fun hold(): Int { + return call_hold(callp, true) } - fun hold(): Int { + fun isOnHold(): Int { return call_hold(callp, true) } diff --git a/app/src/main/kotlin/com/tutpro/baresip/MainActivity.kt b/app/src/main/kotlin/com/tutpro/baresip/MainActivity.kt index a36baa6c..8c181673 100644 --- a/app/src/main/kotlin/com/tutpro/baresip/MainActivity.kt +++ b/app/src/main/kotlin/com/tutpro/baresip/MainActivity.kt @@ -64,6 +64,7 @@ class MainActivity : AppCompatActivity() { private lateinit var dtmf: EditText private var dtmfWatcher: TextWatcher? = null private lateinit var infoButton: ImageButton + private lateinit var onHoldNotice: TextView private lateinit var uaAdapter: UaSpinnerAdapter private lateinit var aorSpinner: Spinner private lateinit var imm: InputMethodManager @@ -140,6 +141,7 @@ class MainActivity : AppCompatActivity() { transferButton = binding.transferButton dtmf = binding.dtmf infoButton = binding.info + onHoldNotice = binding.onHoldNotice voicemailButton = binding.voicemailButton contactsButton = binding.contactsButton messagesButton = binding.messagesButton @@ -943,6 +945,9 @@ class MainActivity : AppCompatActivity() { showCall(ua) } } + "call update" -> { + showCall(ua) + } "call verify" -> { val callp = params[1] val call = Call.ofCallp(callp) @@ -1674,6 +1679,7 @@ class MainActivity : AppCompatActivity() { BaresipService.isMicMuted = false micIcon!!.setIcon(R.drawable.mic_on) } + onHoldNotice.visibility = View.GONE } else { swipeRefresh.isEnabled = false val call = showCall ?: Call.uaCalls(ua, "")[0] @@ -1691,6 +1697,7 @@ class MainActivity : AppCompatActivity() { answerButton.visibility = View.INVISIBLE rejectButton.visibility = View.INVISIBLE callControl.visibility = View.INVISIBLE + onHoldNotice.visibility = View.GONE dialpadButton.isEnabled = false } "incoming" -> { @@ -1706,6 +1713,7 @@ class MainActivity : AppCompatActivity() { rejectButton.visibility = View.VISIBLE rejectButton.isEnabled = true callControl.visibility = View.INVISIBLE + onHoldNotice.visibility = View.GONE dialpadButton.isEnabled = false } "connected" -> { @@ -1756,6 +1764,13 @@ class MainActivity : AppCompatActivity() { dialpadButton.isEnabled = false infoButton.isEnabled = true callControl.visibility = View.VISIBLE + if (call.held) { + imm.hideSoftInputFromWindow(dtmf.windowToken, 0) + onHoldNotice.text = getString(R.string.call_on_hold_by_peer) + onHoldNotice.visibility = View.VISIBLE + } else { + onHoldNotice.visibility = View.GONE + } } } } diff --git a/app/src/main/res/layout/activity_main.xml b/app/src/main/res/layout/activity_main.xml index 5b79ec14..d28112c6 100644 --- a/app/src/main/res/layout/activity_main.xml +++ b/app/src/main/res/layout/activity_main.xml @@ -214,17 +214,34 @@ + + + + + @@ -234,9 +251,7 @@ android:layout_height="wrap_content" android:background="@null" android:src="@drawable/contacts" - android:layout_toStartOf="@id/messagesButton" android:layout_marginEnd="10dp" - android:layout_alignParentBottom="true" android:layout_marginBottom="5dp" android:contentDescription="@string/contacts" > @@ -247,9 +262,7 @@ android:layout_height="wrap_content" android:background="@null" android:src="@drawable/messages" - android:layout_toStartOf="@id/callsButton" android:layout_marginEnd="10dp" - android:layout_alignParentBottom="true" android:layout_marginBottom="5dp" android:contentDescription="@string/messages" > @@ -258,9 +271,7 @@ android:id="@+id/callsButton" android:layout_width="wrap_content" android:layout_height="wrap_content" - android:layout_toStartOf="@id/dialpadButton" android:layout_marginEnd="10dp" - android:layout_alignParentBottom="true" android:layout_marginBottom="5dp" android:padding="0dp" android:background="@null" @@ -273,12 +284,12 @@ android:layout_width="wrap_content" android:layout_height="wrap_content" android:layout_marginEnd="15dp" - android:layout_alignParentBottom="true" android:layout_marginBottom="8dp" - android:layout_alignParentEnd="true" android:background="@null" android:src="@drawable/dialpad_off" android:contentDescription="@string/dialpad" > + + diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml index ac80e60d..d66dec52 100644 --- a/app/src/main/res/values/strings.xml +++ b/app/src/main/res/values/strings.xml @@ -426,6 +426,7 @@ This call is SECURE and peer is VERIFIED! Do you want to unverify the peer? + Call is on hold by peer Unverify Application data has been backed up to file \'%1$s\'. In Android versions 9 and below, the file is in Download folder. diff --git a/build.gradle b/build.gradle index 0303ba59..14b3bbf7 100644 --- a/build.gradle +++ b/build.gradle @@ -1,7 +1,7 @@ // Top-level build file where you can add configuration options common to all sub-projects/modules. buildscript { - ext.kotlin_version = '1.6.0' + ext.kotlin_version = '1.6.10' repositories { google() mavenCentral()