Fix NPE

Stephen Paul Weber created 9 months ago

Change summary

src/main/java/eu/siacs/conversations/ui/ConversationsOverviewFragment.java | 6 
1 file changed, 3 insertions(+), 3 deletions(-)

Detailed changes

src/main/java/eu/siacs/conversations/ui/ConversationsOverviewFragment.java

@@ -479,7 +479,7 @@ public class ConversationsOverviewFragment extends XmppFragment {
 
 	@Override
 	protected void refresh() {
-		if (this.binding == null || this.activity == null) {
+		if (binding == null || this.activity == null) {
 			Log.d(Config.LOGTAG,"ConversationsOverviewFragment.refresh() skipped updated because view binding or activity was null");
 			return;
 		}
@@ -513,8 +513,8 @@ public class ConversationsOverviewFragment extends XmppFragment {
 		}
 		setupSwipe();
 
-		if (binding.overviewSnackbar != null) binding.overviewSnackbar.setVisibility(View.GONE);
-		if (activity.xmppConnectionService == null) return;
+		if (activity.xmppConnectionService == null || binding == null || binding.overviewSnackbar == null) return;
+		binding.overviewSnackbar.setVisibility(View.GONE);
 		for (final var account : activity.xmppConnectionService.getAccounts()) {
 			if (activity.getPreferences().getBoolean("no_mam_pref_warn:" + account.getUuid(), false)) continue;
 			if (account.mamPrefs() != null && !"always".equals(account.mamPrefs().getAttribute("default"))) {