From c3bdb76921e6a49b7f38f69f4dfa9df27f019fe1 Mon Sep 17 00:00:00 2001
From: Stephen Paul Weber <singpolyma@singpolyma.net>
Date: Sun, 14 May 2023 21:09:50 -0500
Subject: [PATCH] Fix NPEs

---
 .../java/eu/siacs/conversations/entities/Conversation.java    | 4 ++--
 .../java/eu/siacs/conversations/ui/ConversationFragment.java  | 2 ++
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/main/java/eu/siacs/conversations/entities/Conversation.java b/src/main/java/eu/siacs/conversations/entities/Conversation.java
index 693050dda606375e9fc8e3dc0eaa42a0063c7b07..ab3aba3932b4196f2d0d5914b12325996d6121ed 100644
--- a/src/main/java/eu/siacs/conversations/entities/Conversation.java
+++ b/src/main/java/eu/siacs/conversations/entities/Conversation.java
@@ -2632,7 +2632,7 @@ public class Conversation extends AbstractEntity implements Blockable, Comparabl
                     }
 
                     for (Element el : command.getChildren()) {
-                        if (el.getName().equals("x") && el.getNamespace().equals("jabber:x:data")) {
+                        if ("x".equals(el.getName()) && "jabber:x:data".equals(el.getNamespace())) {
                             Data form = Data.parse(el);
                             String title = form.getTitle();
                             if (title != null) {
@@ -2640,7 +2640,7 @@ public class Conversation extends AbstractEntity implements Blockable, Comparabl
                                 ConversationPagerAdapter.this.notifyDataSetChanged();
                             }
 
-                            if (el.getAttribute("type").equals("result") || el.getAttribute("type").equals("form")) {
+                            if ("result".equals(el.getAttribute("type")) || "form".equals(el.getAttribute("type"))) {
                                 this.responseElement = el;
                                 setupReported(el.findChild("reported", "jabber:x:data"));
                                 if (mBinding != null) mBinding.form.setLayoutManager(setupLayoutManager());
diff --git a/src/main/java/eu/siacs/conversations/ui/ConversationFragment.java b/src/main/java/eu/siacs/conversations/ui/ConversationFragment.java
index 94212afe9d6603a073ba17943eb01cb63acab6ed..f3a0491664ead81a9130f553f97d0aef3ea775d7 100644
--- a/src/main/java/eu/siacs/conversations/ui/ConversationFragment.java
+++ b/src/main/java/eu/siacs/conversations/ui/ConversationFragment.java
@@ -2884,6 +2884,8 @@ public class ConversationFragment extends XmppFragment
             commandAdapter = new CommandAdapter((XmppActivity) getActivity());
             binding.commandsView.setAdapter(commandAdapter);
             binding.commandsView.setOnItemClickListener((parent, view, position, id) -> {
+                if (activity == null) return;
+
                 final Element command = commandAdapter.getItem(position);
                 activity.startCommand(conversation.getAccount(), command.getAttributeAsJid("jid"), command.getAttribute("node"));
             });