From 3a240f107c56d0fb97a348145ed55316397712a4 Mon Sep 17 00:00:00 2001
From: Sylvain Berfini <sylvain.berfini@belledonne-communications.com>
Date: Tue, 23 Jan 2024 11:38:20 +0100
Subject: [PATCH] Hiding some menus & buttons when conversation is read only

---
 .../ui/main/chat/fragment/ConversationFragment.kt        | 1 +
 app/src/main/res/layout/chat_conversation_fragment.xml   | 3 ++-
 app/src/main/res/layout/chat_conversation_popup_menu.xml | 9 ++++++---
 app/src/main/res/layout/chat_info_fragment.xml           | 6 ++++++
 4 files changed, 15 insertions(+), 4 deletions(-)

diff --git a/app/src/main/java/org/linphone/ui/main/chat/fragment/ConversationFragment.kt b/app/src/main/java/org/linphone/ui/main/chat/fragment/ConversationFragment.kt
index e938cea68..cbe00811e 100644
--- a/app/src/main/java/org/linphone/ui/main/chat/fragment/ConversationFragment.kt
+++ b/app/src/main/java/org/linphone/ui/main/chat/fragment/ConversationFragment.kt
@@ -744,6 +744,7 @@ class ConversationFragment : SlidingPaneChildFragment() {
 
         popupView.conversationMuted = viewModel.isMuted.value == true
         popupView.ephemeralMessagesAvailable = viewModel.isEndToEndEncrypted.value == true || viewModel.isGroup.value == true
+        popupView.readOnlyConversation = viewModel.isReadOnly.value == true
 
         popupView.setGoToInfoClickListener {
             goToInfoFragment()
diff --git a/app/src/main/res/layout/chat_conversation_fragment.xml b/app/src/main/res/layout/chat_conversation_fragment.xml
index d7421abaa..122af3d21 100644
--- a/app/src/main/res/layout/chat_conversation_fragment.xml
+++ b/app/src/main/res/layout/chat_conversation_fragment.xml
@@ -45,7 +45,7 @@
             <androidx.constraintlayout.widget.Group
                 android:layout_width="wrap_content"
                 android:layout_height="wrap_content"
-                app:constraint_referenced_ids="avatar, title, start_call, show_menu"
+                app:constraint_referenced_ids="avatar, title, show_menu"
                 android:visibility="@{viewModel.searchBarVisible ? View.GONE : View.VISIBLE}" />
 
             <androidx.constraintlayout.widget.Group
@@ -159,6 +159,7 @@
                 android:layout_height="0dp"
                 android:padding="15dp"
                 android:src="@drawable/phone"
+                android:visibility="@{viewModel.isReadOnly || viewModel.searchBarVisible ? View.GONE : View.VISIBLE}"
                 app:layout_constraintTop_toTopOf="@id/back"
                 app:layout_constraintBottom_toBottomOf="@id/back"
                 app:layout_constraintEnd_toStartOf="@id/show_menu"
diff --git a/app/src/main/res/layout/chat_conversation_popup_menu.xml b/app/src/main/res/layout/chat_conversation_popup_menu.xml
index 850da6716..c98ed4355 100644
--- a/app/src/main/res/layout/chat_conversation_popup_menu.xml
+++ b/app/src/main/res/layout/chat_conversation_popup_menu.xml
@@ -27,6 +27,9 @@
         <variable
             name="ephemeralMessagesAvailable"
             type="Boolean" />
+        <variable
+            name="readOnlyConversation"
+            type="Boolean" />
     </data>
 
     <androidx.constraintlayout.widget.ConstraintLayout
@@ -89,7 +92,7 @@
             android:ellipsize="end"
             android:drawableStart="@drawable/bell_simple_slash"
             android:drawablePadding="5dp"
-            android:visibility="@{conversationMuted ? View.GONE : View.VISIBLE}"
+            android:visibility="@{conversationMuted || readOnlyConversation ? View.GONE : View.VISIBLE}"
             app:drawableTint="?attr/color_main2_700"
             app:layout_constraintStart_toStartOf="parent"
             app:layout_constraintTop_toBottomOf="@id/search"
@@ -109,7 +112,7 @@
             android:ellipsize="end"
             android:drawableStart="@drawable/bell_simple"
             android:drawablePadding="5dp"
-            android:visibility="@{conversationMuted ? View.VISIBLE : View.GONE, default=gone}"
+            android:visibility="@{conversationMuted &amp;&amp; !readOnlyConversation ? View.VISIBLE : View.GONE, default=gone}"
             app:drawableTint="?attr/color_main2_700"
             app:layout_constraintStart_toStartOf="parent"
             app:layout_constraintTop_toBottomOf="@id/mute"
@@ -129,7 +132,7 @@
             android:ellipsize="end"
             android:drawableStart="@drawable/clock_countdown"
             android:drawablePadding="5dp"
-            android:visibility="@{ephemeralMessagesAvailable ? View.VISIBLE : View.GONE}"
+            android:visibility="@{ephemeralMessagesAvailable &amp;&amp; !readOnlyConversation ? View.VISIBLE : View.GONE}"
             app:drawableTint="?attr/color_main2_700"
             app:layout_constraintStart_toStartOf="parent"
             app:layout_constraintTop_toBottomOf="@id/unmute"
diff --git a/app/src/main/res/layout/chat_info_fragment.xml b/app/src/main/res/layout/chat_info_fragment.xml
index d87bec69d..9d423dc52 100644
--- a/app/src/main/res/layout/chat_info_fragment.xml
+++ b/app/src/main/res/layout/chat_info_fragment.xml
@@ -169,6 +169,7 @@
                     android:onClick="@{() -> viewModel.toggleMute()}"
                     android:padding="16dp"
                     android:src="@{viewModel.isMuted ? @drawable/bell_simple : @drawable/bell_simple_slash, default=@drawable/bell_simple_slash}"
+                    android:visibility="@{viewModel.isReadOnly ? View.GONE : View.VISIBLE}"
                     app:layout_constraintEnd_toStartOf="@id/call"
                     app:layout_constraintStart_toStartOf="parent"
                     app:layout_constraintTop_toBottomOf="@id/status"
@@ -183,6 +184,7 @@
                     android:onClick="@{() -> viewModel.toggleMute()}"
                     android:text="@{viewModel.isMuted ? @string/conversation_action_unmute : @string/conversation_action_mute, default=@string/conversation_action_mute}"
                     android:textSize="14sp"
+                    android:visibility="@{viewModel.isReadOnly ? View.GONE : View.VISIBLE}"
                     app:layout_constraintEnd_toEndOf="@id/mute"
                     app:layout_constraintStart_toStartOf="@id/mute"
                     app:layout_constraintTop_toBottomOf="@id/mute" />
@@ -196,6 +198,7 @@
                     android:onClick="@{() -> viewModel.call()}"
                     android:padding="16dp"
                     android:src="@drawable/phone"
+                    android:visibility="@{viewModel.isReadOnly ? View.GONE : View.VISIBLE}"
                     app:layout_constraintEnd_toStartOf="@id/meeting"
                     app:layout_constraintStart_toEndOf="@id/mute"
                     app:layout_constraintTop_toBottomOf="@id/status"
@@ -210,6 +213,7 @@
                     android:onClick="@{() -> viewModel.call()}"
                     android:text="@string/conversation_action_call"
                     android:textSize="14sp"
+                    android:visibility="@{viewModel.isReadOnly ? View.GONE : View.VISIBLE}"
                     app:layout_constraintEnd_toEndOf="@id/call"
                     app:layout_constraintStart_toStartOf="@id/call"
                     app:layout_constraintTop_toBottomOf="@id/call" />
@@ -223,6 +227,7 @@
                     android:onClick="@{() -> viewModel.scheduleMeeting()}"
                     android:padding="16dp"
                     android:src="@drawable/meeting"
+                    android:visibility="@{viewModel.isReadOnly ? View.GONE : View.VISIBLE}"
                     app:layout_constraintEnd_toEndOf="parent"
                     app:layout_constraintStart_toEndOf="@id/call"
                     app:layout_constraintTop_toBottomOf="@id/status"
@@ -237,6 +242,7 @@
                     android:onClick="@{() -> viewModel.scheduleMeeting()}"
                     android:text="@string/meeting_schedule_meeting_label"
                     android:textSize="14sp"
+                    android:visibility="@{viewModel.isReadOnly ? View.GONE : View.VISIBLE}"
                     app:layout_constraintEnd_toEndOf="@id/meeting"
                     app:layout_constraintStart_toStartOf="@id/meeting"
                     app:layout_constraintTop_toBottomOf="@id/meeting" />