diff --git a/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/ui/submission/resultavailable/SubmissionTestResultAvailableFragment.kt b/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/ui/submission/resultavailable/SubmissionTestResultAvailableFragment.kt index 6a265ce44d9184285e656b5dc74af6a4e4076ccc..7c7d3cfda95056b37dd8177b7d7f29b50b588f61 100644 --- a/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/ui/submission/resultavailable/SubmissionTestResultAvailableFragment.kt +++ b/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/ui/submission/resultavailable/SubmissionTestResultAvailableFragment.kt @@ -54,7 +54,7 @@ class SubmissionTestResultAvailableFragment : Fragment(R.layout.fragment_submiss binding.submissionTestResultAvailableConsentStatus.consent = it } - vm.showKeyRetrievalProgress.observe2(this) { show -> + vm.showKeysRetrievalProgress.observe2(this) { show -> Timber.i("SubmissionTestResult:showKeyRetrievalProgress:$show") keyRetrievalProgress.setState(show) binding.submissionTestResultAvailableProceedButton.isEnabled = !show diff --git a/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/ui/submission/resultavailable/SubmissionTestResultAvailableViewModel.kt b/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/ui/submission/resultavailable/SubmissionTestResultAvailableViewModel.kt index c4a4b3abb21946ea43702cd92313fd9ebade2e49..372569f8dde3bebc1b1cf14444a70e71f1e166f4 100644 --- a/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/ui/submission/resultavailable/SubmissionTestResultAvailableViewModel.kt +++ b/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/ui/submission/resultavailable/SubmissionTestResultAvailableViewModel.kt @@ -32,7 +32,7 @@ class SubmissionTestResultAvailableViewModel @AssistedInject constructor( val consent = consentFlow.asLiveData(dispatcherProvider.Default) val showPermissionRequest = SingleLiveEvent<(Activity) -> Unit>() val showCloseDialog = SingleLiveEvent<Unit>() - val showKeyRetrievalProgress = SingleLiveEvent<Boolean>() + val showKeysRetrievalProgress = SingleLiveEvent<Boolean>() val showTracingConsentDialog = SingleLiveEvent<(Boolean) -> Unit>() private val tekHistoryUpdater = tekHistoryUpdaterFactory.create(object : TEKHistoryUpdater.Callback { @@ -45,25 +45,28 @@ class SubmissionTestResultAvailableViewModel @AssistedInject constructor( .actionSubmissionTestResultAvailableFragmentToSubmissionTestResultConsentGivenFragment() ) - showKeyRetrievalProgress.postValue(false) + showKeysRetrievalProgress.postValue(false) } override fun onTEKPermissionDeclined() { + Timber.d("onTEKPermissionDeclined") routeToScreen.postValue( SubmissionTestResultAvailableFragmentDirections .actionSubmissionTestResultAvailableFragmentToSubmissionTestResultNoConsentFragment() ) - showKeyRetrievalProgress.postValue(false) + showKeysRetrievalProgress.postValue(false) } override fun onTracingConsentRequired(onConsentResult: (given: Boolean) -> Unit) { + Timber.d("onTracingConsentRequired") showTracingConsentDialog.postValue(onConsentResult) - showKeyRetrievalProgress.postValue(false) + showKeysRetrievalProgress.postValue(false) } override fun onPermissionRequired(permissionRequest: (Activity) -> Unit) { + Timber.d("onPermissionRequired") showPermissionRequest.postValue(permissionRequest) - showKeyRetrievalProgress.postValue(false) + showKeysRetrievalProgress.postValue(false) } override fun onError(error: Throwable) { @@ -72,7 +75,7 @@ class SubmissionTestResultAvailableViewModel @AssistedInject constructor( exceptionCategory = ExceptionCategory.EXPOSURENOTIFICATION, prefix = "SubmissionTestResultAvailableViewModel" ) - showKeyRetrievalProgress.postValue(false) + showKeysRetrievalProgress.postValue(false) } }) @@ -101,11 +104,14 @@ class SubmissionTestResultAvailableViewModel @AssistedInject constructor( } fun proceed() { - showKeyRetrievalProgress.value = true + showKeysRetrievalProgress.value = true launch { if (consentFlow.first()) { + Timber.d("tekHistoryUpdater.updateTEKHistoryOrRequestPermission") tekHistoryUpdater.updateTEKHistoryOrRequestPermission() } else { + Timber.d("routeToScreen:SubmissionTestResultNoConsentFragment") + showKeysRetrievalProgress.postValue(false) routeToScreen.postValue( SubmissionTestResultAvailableFragmentDirections .actionSubmissionTestResultAvailableFragmentToSubmissionTestResultNoConsentFragment() @@ -115,7 +121,7 @@ class SubmissionTestResultAvailableViewModel @AssistedInject constructor( } fun handleActivityResult(requestCode: Int, resultCode: Int, data: Intent?) { - showKeyRetrievalProgress.value = true + showKeysRetrievalProgress.value = true tekHistoryUpdater.handleActivityResult(requestCode, resultCode, data) } diff --git a/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/ui/submission/warnothers/SubmissionResultPositiveOtherWarningNoConsentFragment.kt b/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/ui/submission/warnothers/SubmissionResultPositiveOtherWarningNoConsentFragment.kt index 2fd6b41f222745ebcd9ab224706efd87d14d5a27..0592f38a373238dd7eca3dda0d547bd46f76dc41 100644 --- a/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/ui/submission/warnothers/SubmissionResultPositiveOtherWarningNoConsentFragment.kt +++ b/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/ui/submission/warnothers/SubmissionResultPositiveOtherWarningNoConsentFragment.kt @@ -52,7 +52,7 @@ class SubmissionResultPositiveOtherWarningNoConsentFragment : doNavigate(it) } - viewModel.keysRetrievalProgress.observe2(this) { show -> + viewModel.showKeysRetrievalProgress.observe2(this) { show -> keysRetrievalProgress.setState(show) binding.submissionPositiveOtherWarningNoConsentButtonNext.isEnabled = !show } @@ -93,6 +93,6 @@ class SubmissionResultPositiveOtherWarningNoConsentFragment : } override fun onActivityResult(requestCode: Int, resultCode: Int, data: Intent?) { - viewModel.handleActivityRersult(requestCode, resultCode, data) + viewModel.handleActivityResult(requestCode, resultCode, data) } } diff --git a/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/ui/submission/warnothers/SubmissionResultPositiveOtherWarningNoConsentViewModel.kt b/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/ui/submission/warnothers/SubmissionResultPositiveOtherWarningNoConsentViewModel.kt index a1df510d266026ca73a2a3390c4b1203e78cdcb2..ae4ec5ef0a7a3718108023a872ce353fab7a0c79 100644 --- a/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/ui/submission/warnothers/SubmissionResultPositiveOtherWarningNoConsentViewModel.kt +++ b/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/ui/submission/warnothers/SubmissionResultPositiveOtherWarningNoConsentViewModel.kt @@ -32,7 +32,7 @@ class SubmissionResultPositiveOtherWarningNoConsentViewModel @AssistedInject con val routeToScreen = SingleLiveEvent<NavDirections>() - val keysRetrievalProgress = SingleLiveEvent<Boolean>() + val showKeysRetrievalProgress = SingleLiveEvent<Boolean>() val showPermissionRequest = SingleLiveEvent<(Activity) -> Unit>() @@ -47,7 +47,7 @@ class SubmissionResultPositiveOtherWarningNoConsentViewModel @AssistedInject con override fun onTEKAvailable(teks: List<TemporaryExposureKey>) { Timber.d("onTEKAvailable(tek.size=%d)", teks.size) autoSubmission.updateMode(AutoSubmission.Mode.MONITOR) - keysRetrievalProgress.postValue(false) + showKeysRetrievalProgress.postValue(false) routeToScreen.postValue( SubmissionResultPositiveOtherWarningNoConsentFragmentDirections .actionSubmissionResultPositiveOtherWarningNoConsentFragmentToSubmissionResultReadyFragment() @@ -55,23 +55,26 @@ class SubmissionResultPositiveOtherWarningNoConsentViewModel @AssistedInject con } override fun onTEKPermissionDeclined() { - keysRetrievalProgress.postValue(false) + Timber.d("onTEKPermissionDeclined") + showKeysRetrievalProgress.postValue(false) // stay on screen } override fun onTracingConsentRequired(onConsentResult: (given: Boolean) -> Unit) { - keysRetrievalProgress.postValue(false) + Timber.d("onTracingConsentRequired") + showKeysRetrievalProgress.postValue(false) showTracingConsentDialog.postValue(onConsentResult) } override fun onPermissionRequired(permissionRequest: (Activity) -> Unit) { - keysRetrievalProgress.postValue(false) + Timber.d("onPermissionRequired") + showKeysRetrievalProgress.postValue(false) showPermissionRequest.postValue(permissionRequest) } override fun onError(error: Throwable) { - keysRetrievalProgress.postValue(false) Timber.e(error, "Couldn't access temporary exposure key history.") + showKeysRetrievalProgress.postValue(false) error.report(ExceptionCategory.EXPOSURENOTIFICATION, "Failed to obtain TEKs.") } }) @@ -84,13 +87,16 @@ class SubmissionResultPositiveOtherWarningNoConsentViewModel @AssistedInject con } fun onConsentButtonClicked() { - keysRetrievalProgress.value = true + showKeysRetrievalProgress.value = true submissionRepository.giveConsentToSubmission() launch { if (enfClient.isTracingEnabled.first()) { + Timber.d("tekHistoryUpdater.updateTEKHistoryOrRequestPermission()") tekHistoryUpdater.updateTEKHistoryOrRequestPermission() } else { + Timber.d("showEnableTracingEvent:Unit") showEnableTracingEvent.postValue(Unit) + showKeysRetrievalProgress.postValue(false) } } } @@ -102,8 +108,8 @@ class SubmissionResultPositiveOtherWarningNoConsentViewModel @AssistedInject con ) } - fun handleActivityRersult(requestCode: Int, resultCode: Int, data: Intent?) { - keysRetrievalProgress.value = true + fun handleActivityResult(requestCode: Int, resultCode: Int, data: Intent?) { + showKeysRetrievalProgress.value = true tekHistoryUpdater.handleActivityResult(requestCode, resultCode, data) }