Suppress Detekt false positive (?)
This commit is contained in:
parent
a06607f530
commit
32db42aebb
1 changed files with 11 additions and 5 deletions
|
|
@ -127,12 +127,18 @@ class MediaViewerPresenter @AssistedInject constructor(
|
||||||
|
|
||||||
fun handleEvents(event: MediaViewerEvents) {
|
fun handleEvents(event: MediaViewerEvents) {
|
||||||
when (event) {
|
when (event) {
|
||||||
is MediaViewerEvents.LoadMedia -> coroutineScope.downloadMedia(
|
is MediaViewerEvents.LoadMedia -> {
|
||||||
data = event.data,
|
// It's OK to suppress the warning since mediaFile and localMedia are remembered
|
||||||
mediaFile = mediaFile.getOrPut(event.data.eventId) { mutableStateOf(null) },
|
@Suppress("RememberMissing")
|
||||||
localMedia = localMedia.getOrPut(event.data.eventId) { mutableStateOf(AsyncData.Uninitialized) },
|
coroutineScope.downloadMedia(
|
||||||
)
|
data = event.data,
|
||||||
|
mediaFile = mediaFile.getOrPut(event.data.eventId) { mutableStateOf(null) },
|
||||||
|
localMedia = localMedia.getOrPut(event.data.eventId) { mutableStateOf(AsyncData.Uninitialized) },
|
||||||
|
)
|
||||||
|
}
|
||||||
is MediaViewerEvents.ClearLoadingError -> {
|
is MediaViewerEvents.ClearLoadingError -> {
|
||||||
|
// It's OK to suppress the warning since localMedia is remembered
|
||||||
|
@Suppress("RememberMissing")
|
||||||
localMedia.getOrPut(event.eventId) { mutableStateOf(AsyncData.Uninitialized) }.value = AsyncData.Uninitialized
|
localMedia.getOrPut(event.eventId) { mutableStateOf(AsyncData.Uninitialized) }.value = AsyncData.Uninitialized
|
||||||
}
|
}
|
||||||
is MediaViewerEvents.SaveOnDisk -> {
|
is MediaViewerEvents.SaveOnDisk -> {
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue