Fix quality

This commit is contained in:
ganfra 2026-01-22 10:46:01 +01:00
parent df62694b2f
commit 5c1b68b36d
5 changed files with 23 additions and 31 deletions

View file

@ -24,13 +24,12 @@ import io.element.android.libraries.architecture.AsyncAction
import io.element.android.libraries.architecture.Presenter
import io.element.android.libraries.architecture.runUpdatingState
import io.element.android.libraries.designsystem.theme.components.SearchBarResultState
import io.element.android.libraries.di.annotations.SessionCoroutineScope
import io.element.android.libraries.matrix.api.spaces.SpaceRoomList
import io.element.android.libraries.matrix.api.spaces.SpaceService
import io.element.android.libraries.matrix.ui.model.SelectRoomInfo
import kotlinx.collections.immutable.ImmutableList
import kotlinx.collections.immutable.persistentListOf
import kotlinx.collections.immutable.toPersistentList
import kotlinx.collections.immutable.toImmutableList
import kotlinx.coroutines.CoroutineScope
import kotlinx.coroutines.async
import kotlinx.coroutines.awaitAll
@ -42,10 +41,9 @@ class AddRoomToSpacePresenter(
private val spaceService: SpaceService,
private val dataSourceFactory: AddRoomToSpaceSearchDataSource.Factory,
) : Presenter<AddRoomToSpaceState> {
@Composable
override fun present(): AddRoomToSpaceState {
var selectedRooms by remember { mutableStateOf(persistentListOf<SelectRoomInfo>()) }
var selectedRooms: ImmutableList<SelectRoomInfo> by remember { mutableStateOf(persistentListOf()) }
var searchQuery by remember { mutableStateOf("") }
var isSearchActive by remember { mutableStateOf(false) }
val saveAction = remember { mutableStateOf<AsyncAction<Unit>>(AsyncAction.Uninitialized) }
@ -78,9 +76,9 @@ class AddRoomToSpacePresenter(
when (event) {
is AddRoomToSpaceEvent.ToggleRoom -> {
selectedRooms = if (selectedRooms.any { it.roomId == event.room.roomId }) {
selectedRooms.filterNot { it.roomId == event.room.roomId }.toPersistentList()
selectedRooms.filterNot { it.roomId == event.room.roomId }.toImmutableList()
} else {
(selectedRooms + event.room).toPersistentList()
(selectedRooms + event.room).toImmutableList()
}
}
is AddRoomToSpaceEvent.UpdateSearchQuery -> {

View file

@ -11,7 +11,6 @@ package io.element.android.features.space.impl.addroom
import dev.zacsweers.metro.Assisted
import dev.zacsweers.metro.AssistedFactory
import dev.zacsweers.metro.AssistedInject
import dev.zacsweers.metro.Inject
import io.element.android.libraries.core.coroutine.CoroutineDispatchers
import io.element.android.libraries.matrix.api.MatrixClient
import io.element.android.libraries.matrix.api.core.RoomId
@ -52,7 +51,6 @@ class AddRoomToSpaceSearchDataSource(
private val matrixClient: MatrixClient,
private val coroutineDispatchers: CoroutineDispatchers,
) {
@AssistedFactory
interface Factory {
fun create(coroutineScope: CoroutineScope): AddRoomToSpaceSearchDataSource

View file

@ -171,8 +171,8 @@ fun AddRoomToSpaceView(
}
SaveActionView(
saveAction = state.saveAction,
onRetry = { state.eventSink(AddRoomToSpaceEvent.Save)},
onDismiss = {state.eventSink(AddRoomToSpaceEvent.ResetSaveAction)}
onRetry = { state.eventSink(AddRoomToSpaceEvent.Save) },
onDismiss = { state.eventSink(AddRoomToSpaceEvent.ResetSaveAction) }
)
}