From c0707e8cb5de83477fb07411e5c0a34497912e5e Mon Sep 17 00:00:00 2001
From: Sylvain Berfini <sylvain.berfini@belledonne-communications.com>
Date: Sun, 19 Nov 2023 20:55:22 +0100
Subject: [PATCH] Fixed action in contact details

---
 .../ui/call/fragment/ActiveCallFragment.kt    |  3 --
 .../contacts/viewmodel/ContactViewModel.kt    | 33 ++++++++++++++++---
 2 files changed, 29 insertions(+), 7 deletions(-)

diff --git a/app/src/main/java/org/linphone/ui/call/fragment/ActiveCallFragment.kt b/app/src/main/java/org/linphone/ui/call/fragment/ActiveCallFragment.kt
index f2f5fe9a6..cc268d6c9 100644
--- a/app/src/main/java/org/linphone/ui/call/fragment/ActiveCallFragment.kt
+++ b/app/src/main/java/org/linphone/ui/call/fragment/ActiveCallFragment.kt
@@ -166,9 +166,6 @@ class ActiveCallFragment : GenericCallFragment() {
                     R.drawable.trusted,
                     doNotTint = true
                 )
-                /*binding.avatar.avatarBorderWidth = AppUtils.getDimension(
-                    R.dimen.avatar_trust_border_width
-                ).toInt()*/ // TODO FIXME: show blue border
             }
         }
 
diff --git a/app/src/main/java/org/linphone/ui/main/contacts/viewmodel/ContactViewModel.kt b/app/src/main/java/org/linphone/ui/main/contacts/viewmodel/ContactViewModel.kt
index 1fabd8719..face95e15 100644
--- a/app/src/main/java/org/linphone/ui/main/contacts/viewmodel/ContactViewModel.kt
+++ b/app/src/main/java/org/linphone/ui/main/contacts/viewmodel/ContactViewModel.kt
@@ -51,6 +51,10 @@ import org.linphone.utils.LinphoneUtils
 class ContactViewModel @UiThread constructor() : ViewModel() {
     companion object {
         private const val TAG = "[Contact ViewModel]"
+
+        private const val START_AUDIO_CALL = 0
+        private const val START_VIDEO_CALL = 1
+        private const val START_CONVERSATION = 2
     }
 
     val contact = MutableLiveData<ContactAvatarModel>()
@@ -119,15 +123,33 @@ class ContactViewModel @UiThread constructor() : ViewModel() {
         MutableLiveData<Event<Boolean>>()
     }
 
+    private var expectedAction: Int = START_AUDIO_CALL
     private val listener = object : ContactNumberOrAddressClickListener {
         @UiThread
         override fun onClicked(model: ContactNumberOrAddressModel) {
             val address = model.address
             if (model.isEnabled && address != null) {
-                // TODO FIXME: handle chat action & video call as well
-                coreContext.postOnCoreThread {
-                    Log.i("$TAG Calling SIP address [${address.asStringUriOnly()}]")
-                    coreContext.startCall(address)
+                coreContext.postOnCoreThread { core ->
+                    when (expectedAction) {
+                        START_AUDIO_CALL -> {
+                            Log.i("$TAG Audio calling SIP address [${address.asStringUriOnly()}]")
+                            val params = core.createCallParams(null)
+                            params?.isVideoEnabled = false
+                            coreContext.startCall(address, params)
+                        }
+                        START_VIDEO_CALL -> {
+                            Log.i("$TAG Video calling SIP address [${address.asStringUriOnly()}]")
+                            val params = core.createCallParams(null)
+                            params?.isVideoEnabled = true
+                            coreContext.startCall(address, params)
+                        }
+                        START_CONVERSATION -> {
+                            Log.i(
+                                "$TAG Going to conversation with SIP address [${address.asStringUriOnly()}]"
+                            )
+                            goToConversation(address)
+                        }
+                    }
                 }
             } else if (!model.isEnabled) {
                 Log.w(
@@ -383,6 +405,7 @@ class ContactViewModel @UiThread constructor() : ViewModel() {
                     Log.e("$TAG Failed to interpret phone number [$number] as SIP address")
                 }
             } else {
+                expectedAction = START_AUDIO_CALL
                 val list = sipAddressesAndPhoneNumbers.value.orEmpty()
                 Log.i(
                     "$TAG [${list.size}] numbers or addresses found for contact [${friend.name}], showing selection dialog"
@@ -423,6 +446,7 @@ class ContactViewModel @UiThread constructor() : ViewModel() {
                     Log.e("$TAG Failed to interpret phone number [$number] as SIP address")
                 }
             } else {
+                expectedAction = START_VIDEO_CALL
                 val list = sipAddressesAndPhoneNumbers.value.orEmpty()
                 Log.i(
                     "$TAG [${list.size}] numbers or addresses found for contact [${friend.name}], showing selection dialog"
@@ -458,6 +482,7 @@ class ContactViewModel @UiThread constructor() : ViewModel() {
                     Log.e("$TAG Failed to interpret phone number [$number] as SIP address")
                 }
             } else {
+                expectedAction = START_CONVERSATION
                 val list = sipAddressesAndPhoneNumbers.value.orEmpty()
                 Log.i(
                     "$TAG [${list.size}] numbers or addresses found for contact [${friend.name}], showing selection dialog"