From 8aca07d5f47db3c7747f527d818a253e39cf5d30 Mon Sep 17 00:00:00 2001 From: Jays2Kings Date: Mon, 2 May 2022 01:17:46 -0400 Subject: [PATCH] Fix migration global search not working --- .../ui/migration/SearchController.kt | 35 ++++--------------- 1 file changed, 7 insertions(+), 28 deletions(-) diff --git a/app/src/main/java/eu/kanade/tachiyomi/ui/migration/SearchController.kt b/app/src/main/java/eu/kanade/tachiyomi/ui/migration/SearchController.kt index e5b5a2a1fa..afacce0760 100644 --- a/app/src/main/java/eu/kanade/tachiyomi/ui/migration/SearchController.kt +++ b/app/src/main/java/eu/kanade/tachiyomi/ui/migration/SearchController.kt @@ -3,10 +3,7 @@ package eu.kanade.tachiyomi.ui.migration import android.os.Bundle import android.view.Menu import android.view.MenuInflater -import android.view.MenuItem -import androidx.appcompat.widget.SearchView import androidx.core.os.bundleOf -import com.jakewharton.rxbinding.support.v7.widget.queryTextChangeEvents import eu.kanade.tachiyomi.R import eu.kanade.tachiyomi.data.database.DatabaseHelper import eu.kanade.tachiyomi.data.database.models.Manga @@ -17,6 +14,8 @@ import eu.kanade.tachiyomi.ui.migration.manga.process.MigrationListController import eu.kanade.tachiyomi.ui.source.globalsearch.GlobalSearchCardAdapter import eu.kanade.tachiyomi.ui.source.globalsearch.GlobalSearchController import eu.kanade.tachiyomi.ui.source.globalsearch.GlobalSearchPresenter +import eu.kanade.tachiyomi.util.view.activityBinding +import eu.kanade.tachiyomi.util.view.setOnQueryTextChangeListener import uy.kohesive.injekt.Injekt import uy.kohesive.injekt.api.get import uy.kohesive.injekt.injectLazy @@ -83,31 +82,11 @@ class SearchController( // Inflate menu. inflater.inflate(R.menu.catalogue_new_list, menu) - // Initialize search menu - val searchItem = menu.findItem(R.id.action_search) - val searchView = searchItem.actionView as SearchView - - searchItem.setOnActionExpandListener( - object : MenuItem.OnActionExpandListener { - override fun onMenuItemActionExpand(item: MenuItem?): Boolean { - searchView.onActionViewExpanded() // Required to show the query in the view - searchView.setQuery(presenter.query, false) - return true - } - - override fun onMenuItemActionCollapse(item: MenuItem?): Boolean { - return true - } - } - ) - - searchView.queryTextChangeEvents() - .filter { it.isSubmitted } - .subscribeUntilDestroy { - presenter.search(it.queryText().toString()) - searchItem.collapseActionView() - setTitle() // Update toolbar title - } + setOnQueryTextChangeListener(activityBinding?.searchToolbar?.searchView, onlyOnSubmit = true, hideKbOnSubmit = true) { + presenter.search(it ?: "") + setTitle() // Update toolbar title + true + } } override fun canChangeTabs(block: () -> Unit): Boolean {