feat(moderators): Show the participant sheet to non-moderators too

The participants list carries owner and moderator as an icon only. Anyone
without moderation rights had no way to read it: the row was clickable but
inert, so the rank had no text form for them anywhere in the app.

Open the sheet for everyone. Without moderation rights it renders the header -
display name and role - and no action rows, which is exactly the information the
icon was withholding. The attendee PIN stays hidden as well; it is a SIP
credential the server only hands to moderators.

Hiding rows is presentation, not access control, so the canModerate() check does
not simply disappear into the sheet: it moves to handleParticipantOpsAction,
where it guards the API call itself rather than the drawing of a button.

Assisted-by: Claude Code:claude-opus-5
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>

Author: AndyScherzinger

app/src/main/java/com/nextcloud/talk/conversationinfo/ConversationInfoActivity.kt

@@ -727,17 +727,16 @@ class ConversationInfoActivity : BaseActivity() {
         conversationUser?.let { viewModel.banActor(it, conversationToken, actorType, actorId, internalNote) }
     }
 
-    @Suppress("ReturnCount")
     private fun handleParticipantClick(model: ParticipantModel) {
-        val state = viewModel.uiState.value
-        val conv = state.conversation ?: return
-        val caps = state.spreedCapabilities ?: return
-        if (!ConversationUtils.canModerate(conv, caps)) return
-
         viewModel.setParticipantForOps(model)
     }
 
+    @Suppress("ReturnCount")
     private fun handleParticipantOpsAction(action: ParticipantOpsAction, model: ParticipantModel) {
+        val state = viewModel.uiState.value
+        val conv = state.conversation ?: return
+        val caps = state.spreedCapabilities ?: return
+        if (!ConversationUtils.canModerate(conv, caps)) return
         val user = conversationUser ?: return
         val participant = model.participant
         val apiVersion = ApiUtils.getConversationApiVersion(user, intArrayOf(ApiUtils.API_V4, 1))

app/src/main/java/com/nextcloud/talk/conversationinfo/ui/ParticipantOperationsSheet.kt

@@ -45,20 +45,21 @@ import com.nextcloud.talk.models.domain.ConversationModel
 import com.nextcloud.talk.models.json.capabilities.SpreedCapability
 import com.nextcloud.talk.models.json.participants.Participant
 import com.nextcloud.talk.utils.CapabilitiesUtil
+import com.nextcloud.talk.utils.ConversationUtils
 import com.nextcloud.talk.utils.ParticipantRole
 import com.nextcloud.talk.utils.ParticipantRoleUtils
 
 private data class RemoveOption(@DrawableRes val iconRes: Int, val label: String)
 
 private data class ParticipantOpsVisibility(
-    val infoPin: String?,
-    val showPromote: Boolean,
-    val showDemote: Boolean,
-    val showPromoteToOwner: Boolean,
-    val showDemoteOwnerToModerator: Boolean,
-    val showDemoteOwnerToUser: Boolean,
-    val remove: RemoveOption?,
-    val showBan: Boolean
+    val infoPin: String? = null,
+    val showPromote: Boolean = false,
+    val showDemote: Boolean = false,
+    val showPromoteToOwner: Boolean = false,
+    val showDemoteOwnerToModerator: Boolean = false,
+    val showDemoteOwnerToUser: Boolean = false,
+    val remove: RemoveOption? = null,
+    val showBan: Boolean = false
 )
 
 @Composable
@@ -73,8 +74,12 @@ private fun computeVisibility(
     val deleteIcon = R.drawable.ic_delete_grey600_24dp
     val canDemoteFromOwner = ParticipantRoleUtils.canBeDemotedFromOwner(participant, conversation, spreedCapabilities)
     val isOwner = participant.type == Participant.ParticipantType.OWNER
+    val canModerate = conversation != null && ConversationUtils.canModerate(conversation, spreedCapabilities)
 
     return when {
+        // Without moderation rights the sheet is informational: the header and nothing else
+        !canModerate -> ParticipantOpsVisibility()
+
         model.isSelf -> ParticipantOpsVisibility(
             infoPin = null,
             showPromote = false,
@@ -352,6 +357,25 @@ private fun ParticipantOperationsSheetUserWithPinPreview() {
     }
 }
 
+/** Without moderation rights the sheet only names the participant and their rank. */
+@Preview(showBackground = true, name = "Light")
+@Preview(showBackground = true, name = "Dark", uiMode = Configuration.UI_MODE_NIGHT_YES)
+@Composable
+private fun ParticipantOperationsSheetInformationalPreview() {
+    ParticipantOpsPreviewWrapper {
+        ParticipantOperationsContent(
+            model = previewParticipant(
+                "Alice Johnson",
+                Participant.ParticipantType.OWNER,
+                ParticipantRole.OWNER
+            ),
+            conversation = null,
+            spreedCapabilities = null,
+            onAction = {}
+        )
+    }
+}
+
 @Preview(showBackground = true, name = "Light")
 @Preview(showBackground = true, name = "Dark", uiMode = Configuration.UI_MODE_NIGHT_YES)
 @Composable

app/src/main/res/values/strings.xml

@@ -469,8 +469,7 @@ How to translate with transifex:
     <!-- Shown when the server refuses to change a participant's rank -->
     <string name="nc_participant_type_change_failed">Could not change the type of the participant</string>
     <!-- Shown when demoting would leave the conversation without any moderator -->
-    <string name="nc_last_moderator_cannot_be_demoted">The last moderator of a conversation can not be
-        demoted</string>
+    <string name="nc_last_moderator_cannot_be_demoted">The last moderator of a conversation can not be demoted</string>
     <string name="nc_remove_participant">Remove participant</string>
     <string name="nc_remove_team_and_members">Remove team and members</string>
     <string name="nc_remove_group_and_members">Remove group and members</string>