From ebdf2406a2d94de866f97553535ff45d3565fa0e Mon Sep 17 00:00:00 2001 From: Floens Date: Sun, 21 Jan 2018 22:23:20 +0100 Subject: [PATCH] sitessetuppresenter: check for add dialog callback nullity somehow the dialog is first detached from the window, and then the button click is propagated. not sure how this can happen, because we have overridden the button press dismiss behavior. --- .../floens/chan/core/presenter/SitesSetupPresenter.java | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/Clover/app/src/main/java/org/floens/chan/core/presenter/SitesSetupPresenter.java b/Clover/app/src/main/java/org/floens/chan/core/presenter/SitesSetupPresenter.java index b57b0ae0..665ed1de 100644 --- a/Clover/app/src/main/java/org/floens/chan/core/presenter/SitesSetupPresenter.java +++ b/Clover/app/src/main/java/org/floens/chan/core/presenter/SitesSetupPresenter.java @@ -88,12 +88,16 @@ public class SitesSetupPresenter { @Override public void onSiteAdded(Site site) { siteAdded(site); - addCallback.dismissDialog(); + if (addCallback != null) { + addCallback.dismissDialog(); + } } @Override public void onSiteAddFailed(String message) { - addCallback.showAddError(message); + if (addCallback != null) { + addCallback.showAddError(message); + } } }); }