catch npe on participants context menu

Daniel Gultsch created 6 years ago

Change summary

src/main/java/eu/siacs/conversations/ui/ConferenceDetailsActivity.java | 5 
src/main/res/values/strings.xml                                        | 1 
2 files changed, 6 insertions(+)

Detailed changes

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

@@ -257,6 +257,11 @@ public class ConferenceDetailsActivity extends XmppActivity implements OnConvers
 
     @Override
     public boolean onContextItemSelected(MenuItem item) {
+        final User user = mUserPreviewAdapter.getSelectedUser();
+        if (user == null) {
+            Toast.makeText(this, R.string.unable_to_perform_this_action, Toast.LENGTH_SHORT).show();
+            return true;
+        }
         if (!MucDetailsContextMenuHelper.onContextItemSelected(item, mUserPreviewAdapter.getSelectedUser(), this)) {
             return super.onContextItemSelected(item);
         }

src/main/res/values/strings.xml

@@ -875,4 +875,5 @@
     <string name="not_a_backup_file">The file you selected is not a Conversations backup file</string>
     <string name="account_already_setup">This account has already been setup</string>
     <string name="please_enter_password">Please enter the password for this account</string>
+    <string name="unable_to_perform_this_action">Unable to perform this action</string>
 </resources>