Skip to content

Commit 10c5846

Browse files
committed
Remove duplicated method call and trim whitespace from search keyword
1 parent 07f5a17 commit 10c5846

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

presentation/src/main/java/daily/dayo/presentation/screen/search/SearchResultScreen.kt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -216,7 +216,6 @@ fun SearchResultScreen(
216216
onBackClick = onBackClick,
217217
onSearchClick = { keyword ->
218218
onSearchClick(keyword)
219-
onSearchClick(keyword)
220219
}
221220
)
222221

presentation/src/main/java/daily/dayo/presentation/screen/search/SearchScreen.kt

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -56,6 +56,7 @@ import daily.dayo.domain.model.SearchHistory
5656
import daily.dayo.domain.model.SearchHistoryDetail
5757
import daily.dayo.domain.model.SearchHistoryType
5858
import daily.dayo.presentation.R
59+
import daily.dayo.presentation.common.ReplaceUnicode.trimBlankText
5960
import daily.dayo.presentation.common.extension.clickableSingle
6061
import daily.dayo.presentation.common.toSp
6162
import daily.dayo.presentation.theme.DayoTheme
@@ -205,9 +206,9 @@ fun SearchActionbarLayout(
205206
keyboardOptions = KeyboardOptions.Default.copy(imeAction = ImeAction.Search),
206207
keyboardActions = KeyboardActions(
207208
onSearch = {
208-
if (textFieldValue.text.isNotEmpty()) {
209-
onSearchClick(textFieldValue.text)
210-
onSearchClick(textFieldValue.text)
209+
val trimmedBlankText = trimBlankText(textFieldValue.text)
210+
if (trimmedBlankText.isNotEmpty()) {
211+
onSearchClick(trimmedBlankText)
211212
}
212213
}
213214
)

0 commit comments

Comments
 (0)