Skip to content
Snippets Groups Projects
Unverified Commit a2007bd8 authored by Oliver Zimmerman's avatar Oliver Zimmerman Committed by GitHub
Browse files

Country list on Risiko-Ermittlung AFTER WLAN ON (EXPOSUREAPP-3152) (#1372)


* make use of isNetworkEnabled instead

* indentation fix

* Update InteroperabilityConfigurationFragment.kt

* Combined original solution with new enhancement

* lint issues

* Added onResume that reregisters the callback after settings are changed via button

Co-authored-by: default avatarharambasicluka <64483219+harambasicluka@users.noreply.github.com>
Co-authored-by: default avatarRalf Gehrer <ralfgehrer@users.noreply.github.com>
parent 971580a8
No related branches found
No related tags found
No related merge requests found
...@@ -6,6 +6,7 @@ import android.os.Bundle ...@@ -6,6 +6,7 @@ import android.os.Bundle
import android.provider.Settings import android.provider.Settings
import android.view.View import android.view.View
import androidx.fragment.app.Fragment import androidx.fragment.app.Fragment
import de.rki.coronawarnapp.CoronaWarnApplication
import de.rki.coronawarnapp.R import de.rki.coronawarnapp.R
import de.rki.coronawarnapp.databinding.FragmentInteroperabilityConfigurationBinding import de.rki.coronawarnapp.databinding.FragmentInteroperabilityConfigurationBinding
import de.rki.coronawarnapp.ui.main.MainActivity import de.rki.coronawarnapp.ui.main.MainActivity
...@@ -29,7 +30,6 @@ class InteroperabilityConfigurationFragment : ...@@ -29,7 +30,6 @@ class InteroperabilityConfigurationFragment :
private val networkCallback = object : ConnectivityHelper.NetworkCallback() { private val networkCallback = object : ConnectivityHelper.NetworkCallback() {
override fun onNetworkAvailable() { override fun onNetworkAvailable() {
vm.getAllCountries() vm.getAllCountries()
unregisterNetworkCallback()
} }
override fun onNetworkUnavailable() { override fun onNetworkUnavailable() {
...@@ -44,6 +44,10 @@ class InteroperabilityConfigurationFragment : ...@@ -44,6 +44,10 @@ class InteroperabilityConfigurationFragment :
binding.countryData = it binding.countryData = it
} }
if (ConnectivityHelper.isNetworkEnabled(CoronaWarnApplication.getAppContext())) {
registerNetworkCallback()
}
vm.saveInteroperabilityUsed() vm.saveInteroperabilityUsed()
binding.interoperabilityConfigurationHeader.headerButtonBack.buttonIcon.setOnClickListener { binding.interoperabilityConfigurationHeader.headerButtonBack.buttonIcon.setOnClickListener {
...@@ -58,14 +62,13 @@ class InteroperabilityConfigurationFragment : ...@@ -58,14 +62,13 @@ class InteroperabilityConfigurationFragment :
binding.interoperabilityConfigurationCountryList binding.interoperabilityConfigurationCountryList
.noCountriesRiskdetailsInfoview.riskDetailsOpenSettingsButton.setOnClickListener { .noCountriesRiskdetailsInfoview.riskDetailsOpenSettingsButton.setOnClickListener {
val intent = if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q) { val intent = if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q) {
Intent(Settings.Panel.ACTION_INTERNET_CONNECTIVITY) Intent(Settings.Panel.ACTION_INTERNET_CONNECTIVITY)
} else { } else {
Intent(Settings.ACTION_SETTINGS) Intent(Settings.ACTION_SETTINGS)
}
startActivity(intent)
} }
startActivity(intent)
registerNetworkCallback()
}
} }
private fun registerNetworkCallback() { private fun registerNetworkCallback() {
...@@ -88,4 +91,11 @@ class InteroperabilityConfigurationFragment : ...@@ -88,4 +91,11 @@ class InteroperabilityConfigurationFragment :
super.onDestroy() super.onDestroy()
unregisterNetworkCallback() unregisterNetworkCallback()
} }
override fun onResume() {
super.onResume()
if (ConnectivityHelper.isNetworkEnabled(CoronaWarnApplication.getAppContext())) {
registerNetworkCallback()
}
}
} }
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment