From 8888bfec9d1ade58d31c596de30838a314940e5a Mon Sep 17 00:00:00 2001 From: Cuong Tran Date: Sun, 10 Mar 2024 16:41:29 +0700 Subject: [PATCH] detekt: cleanup code # Conflicts: # app/src/main/java/eu/kanade/presentation/browse/FeedScreen.kt --- .../kanade/presentation/browse/FeedScreen.kt | 14 +++++++++---- .../presentation/browse/GlobalSearchScreen.kt | 21 ++++++++++++------- .../tachiyomi/ui/browse/feed/FeedTab.kt | 7 ++++--- 3 files changed, 27 insertions(+), 15 deletions(-) diff --git a/app/src/main/java/eu/kanade/presentation/browse/FeedScreen.kt b/app/src/main/java/eu/kanade/presentation/browse/FeedScreen.kt index 54e1977b1..b374b3aaa 100644 --- a/app/src/main/java/eu/kanade/presentation/browse/FeedScreen.kt +++ b/app/src/main/java/eu/kanade/presentation/browse/FeedScreen.kt @@ -211,10 +211,11 @@ fun FeedAddSearchDialog( val savedSearchStrings = remember { savedSearches.map { // KMK --> - it?.name ?: if (source.supportsLatest) + it?.name ?: if (source.supportsLatest) { context.stringResource(MR.strings.latest) - else + } else { context.stringResource(MR.strings.popular) + } // KMK <-- }.toImmutableList() } @@ -228,7 +229,12 @@ fun FeedAddSearchDialog( }, onDismissRequest = onDismiss, confirmButton = { - TextButton(onClick = { onClickAdd(source, selected?.let { savedSearches[it] }) }, /* KMK --> */ enabled = selected != null /* KMK <-- */) { + TextButton( + onClick = { onClickAdd(source, selected?.let { savedSearches[it] }) }, + // KMK --> + enabled = selected != null, + // KMK <-- + ) { Text(text = stringResource(MR.strings.action_ok)) } }, @@ -240,7 +246,7 @@ fun RadioSelector( options: ImmutableList, selected: Int?, optionStrings: ImmutableList = remember { options.map { it.toString() }.toImmutableList() }, - onSelectOption: (Int) -> Unit/* KMK --> */ = {},/* KMK <-- */ + onSelectOption: (Int) -> Unit /* KMK --> */ = {} /* KMK <-- */, ) { Column(Modifier.verticalScroll(rememberScrollState())) { optionStrings.forEachIndexed { index, option -> diff --git a/app/src/main/java/eu/kanade/presentation/browse/GlobalSearchScreen.kt b/app/src/main/java/eu/kanade/presentation/browse/GlobalSearchScreen.kt index 69303afd7..953defc69 100644 --- a/app/src/main/java/eu/kanade/presentation/browse/GlobalSearchScreen.kt +++ b/app/src/main/java/eu/kanade/presentation/browse/GlobalSearchScreen.kt @@ -21,8 +21,8 @@ import eu.kanade.tachiyomi.ui.browse.source.globalsearch.SourceFilter import eu.kanade.tachiyomi.util.system.LocaleHelper import kotlinx.collections.immutable.ImmutableMap import tachiyomi.domain.manga.model.Manga -import tachiyomi.domain.source.model.Source as DomainSource import tachiyomi.presentation.core.components.material.Scaffold +import tachiyomi.domain.source.model.Source as DomainSource @Composable fun GlobalSearchScreen( @@ -109,20 +109,25 @@ internal fun GlobalSearchContent( // KMK --> val domainSource = DomainSource( source.id, - "", "", + "", + "", supportsLatest = false, isStub = false ) // KMK <-- GlobalSearchResultItem( - title = (fromSourceId?.let { - "▶ ${source.name}".takeIf { source.id == fromSourceId } - } ?: source.name) + + title = ( + fromSourceId?.let { + "▶ ${source.name}".takeIf { source.id == fromSourceId } + } ?: source.name + ) + // KMK --> - (domainSource.installedExtension?.let { extension -> - " (${extension.name})".takeIf { extension.name != source.name } - } ?: ""), + ( + domainSource.installedExtension?.let { extension -> + " (${extension.name})".takeIf { extension.name != source.name } + } ?: "" + ), // KMK <-- subtitle = LocaleHelper.getLocalizedDisplayName(source.lang), onClick = { onClickSource(source) }, diff --git a/app/src/main/java/eu/kanade/tachiyomi/ui/browse/feed/FeedTab.kt b/app/src/main/java/eu/kanade/tachiyomi/ui/browse/feed/FeedTab.kt index 77deb6445..e74005b6d 100644 --- a/app/src/main/java/eu/kanade/tachiyomi/ui/browse/feed/FeedTab.kt +++ b/app/src/main/java/eu/kanade/tachiyomi/ui/browse/feed/FeedTab.kt @@ -110,10 +110,11 @@ fun Screen.feedTab( BrowseSourceScreen( source.id, // KMK --> - listingQuery = if (!source.supportsLatest) + listingQuery = if (!source.supportsLatest) { GetRemoteManga.QUERY_POPULAR - else - GetRemoteManga.QUERY_LATEST, + } else { + GetRemoteManga.QUERY_LATEST + }, // KMK <-- ), )