Replace thread with coroutine in VpnAppsFilterFragment.kt

This commit is contained in:
dovecoteescapee 2024-09-05 08:36:15 +03:00
parent 95d23b19e7
commit a9469ba5e4
2 changed files with 13 additions and 9 deletions

View File

@ -13,6 +13,7 @@ import androidx.core.content.edit
import androidx.core.view.MenuProvider
import androidx.fragment.app.Fragment
import androidx.lifecycle.Lifecycle
import androidx.lifecycle.lifecycleScope
import androidx.recyclerview.widget.DividerItemDecoration
import androidx.recyclerview.widget.LinearLayoutManager
import androidx.recyclerview.widget.RecyclerView
@ -20,6 +21,9 @@ import io.github.dovecoteescapee.byedpi.adapters.AppItemAdapter
import io.github.dovecoteescapee.byedpi.R
import io.github.dovecoteescapee.byedpi.databinding.FragmentFilterBinding
import io.github.dovecoteescapee.byedpi.utility.getPreferences
import kotlinx.coroutines.Dispatchers
import kotlinx.coroutines.launch
import kotlinx.coroutines.withContext
class VpnAppsFilterFragment : Fragment(), MenuProvider {
@ -67,16 +71,13 @@ class VpnAppsFilterFragment : Fragment(), MenuProvider {
binding.appList.adapter = adapter
Thread {
apps = collectApps()
activity.runOnUiThread {
lifecycleScope.launch {
apps = withContext(Dispatchers.IO) { collectApps() }
adapter.setList(apps)
binding.progressCircular.visibility = View.GONE
binding.appList.visibility = View.VISIBLE
}
}.start()
requireActivity().addMenuProvider(this, viewLifecycleOwner, Lifecycle.State.RESUMED)
@ -113,6 +114,7 @@ class VpnAppsFilterFragment : Fragment(), MenuProvider {
return false
}
})
searchItem.setOnActionExpandListener(object : MenuItem.OnActionExpandListener {
override fun onMenuItemActionExpand(item: MenuItem): Boolean {
return true

View File

@ -308,6 +308,7 @@ class ByeDpiVpnService : LifecycleVpnService() {
}
builder.addDisallowedApplication(applicationContext.packageName)
}
"whitelist" -> {
filter.forEach {
try {
@ -315,6 +316,7 @@ class ByeDpiVpnService : LifecycleVpnService() {
} catch (ignore: PackageManager.NameNotFoundException) {}
}
}
else -> {
Log.w(TAG, "Invalid VPN filter mode: $filterMode")
}