Sulkta-Coop/element-x-ada
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix detekt issue

Browse source 
Content slots should not be reused in different code branches/scopes of a composable function, to preserve the slot internal state.
This commit is contained in:
Benoit Marty 2025-11-17 09:20:24 +01:00
parent 8586e25774
commit 270d255655
1 changed files with 6 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

12
features/rolesandpermissions/impl/src/main/kotlin/io/element/android/features/rolesandpermissions/impl/roles/ChangeRolesView.kt
View file

@ -230,20 +230,20 @@ private fun SearchResultsList(
state = lazyListState,
) {
item {
if (currentRole == RoomMember.Role.Admin) {
val ownersAndSelectedUsers by remember {
derivedStateOf {
val usersInHorizontalRow by remember {
derivedStateOf {
if (currentRole == RoomMember.Role.Admin) {
// Also include the owners in the horizontal list
val owners = searchResults.owners.map {
it.toMatrixUser()
}
(owners + selectedUsers).toImmutableList()
} else {
selectedUsers
}
}
selectedUsersList(ownersAndSelectedUsers)
} else {
selectedUsersList(selectedUsers)
}
selectedUsersList(usersInHorizontalRow)
}
if (searchResults.owners.isNotEmpty()) {
stickyHeader { ListSectionHeader(text = stringResource(R.string.screen_room_roles_and_permissions_owners)) }