From 17f850b31acfeb2884fe2ca53b40a1e711ccc962 Mon Sep 17 00:00:00 2001 From: Alan Evans Date: Fri, 8 Nov 2019 11:19:57 -0500 Subject: [PATCH] Ensure that we can still navigate when we get a response from server. --- .../registration/fragments/EnterPhoneNumberFragment.java | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/src/org/thoughtcrime/securesms/registration/fragments/EnterPhoneNumberFragment.java b/src/org/thoughtcrime/securesms/registration/fragments/EnterPhoneNumberFragment.java index bf4970b02c..2d74238a73 100644 --- a/src/org/thoughtcrime/securesms/registration/fragments/EnterPhoneNumberFragment.java +++ b/src/org/thoughtcrime/securesms/registration/fragments/EnterPhoneNumberFragment.java @@ -218,6 +218,10 @@ public final class EnterPhoneNumberFragment extends BaseRegistrationFragment { @Override public void onNeedCaptcha() { + if (getContext() == null) { + Log.i(TAG, "Got onNeedCaptcha response, but fragment is no longer attached."); + return; + } navController.navigate(EnterPhoneNumberFragmentDirections.actionRequestCaptcha()); cancelSpinning(register); enableAllEntries(); @@ -227,6 +231,10 @@ public final class EnterPhoneNumberFragment extends BaseRegistrationFragment { @Override public void requestSent(@Nullable String fcmToken) { + if (getContext() == null) { + Log.i(TAG, "Got requestSent response, but fragment is no longer attached."); + return; + } model.setFcmToken(fcmToken); model.markASuccessfulAttempt(); navController.navigate(EnterPhoneNumberFragmentDirections.actionEnterVerificationCode());