Fix for NPE when Recipients is empty for thread.

moxie0 · moxie0 · commit 437ffba4b165 · 2015-05-14T08:53:35.000-07:00
// FREEBIE
diff --git a/src/org/thoughtcrime/securesms/components/AvatarImageView.java b/src/org/thoughtcrime/securesms/components/AvatarImageView.java
@@ -3,10 +3,12 @@
 import android.content.Context;
 import android.content.Intent;
 import android.provider.ContactsContract;
+import android.support.annotation.Nullable;
 import android.util.AttributeSet;
 import android.view.View;
 import android.widget.ImageView;
 
+import org.thoughtcrime.securesms.contacts.ContactPhotoFactory;
 import org.thoughtcrime.securesms.recipients.Recipient;
 
 public class AvatarImageView extends ImageView {
@@ -21,9 +23,14 @@ public AvatarImageView(Context context, AttributeSet attrs) {
     setScaleType(ScaleType.CENTER_INSIDE);
   }
 
-  public void setAvatar(Recipient recipient, boolean quickContactEnabled) {
-    setImageDrawable(recipient.getContactPhoto());
-    setAvatarClickHandler(recipient, quickContactEnabled);
+  public void setAvatar(@Nullable Recipient recipient, boolean quickContactEnabled) {
+    if (recipient != null) {
+      setImageDrawable(recipient.getContactPhoto());
+      setAvatarClickHandler(recipient, quickContactEnabled);
+    } else {
+      setImageDrawable(ContactPhotoFactory.getDefaultContactPhoto(getContext(), null));
+      setOnClickListener(null);
+    }
   }
 
   private void setAvatarClickHandler(final Recipient recipient, boolean quickContactEnabled) {
diff --git a/src/org/thoughtcrime/securesms/recipients/Recipients.java b/src/org/thoughtcrime/securesms/recipients/Recipients.java
@@ -16,6 +16,7 @@
  */
 package org.thoughtcrime.securesms.recipients;
 
+import android.support.annotation.Nullable;
 import android.util.Patterns;
 
 import org.thoughtcrime.securesms.recipients.Recipient.RecipientModifiedListener;
@@ -109,7 +110,7 @@ public boolean isSingleRecipient() {
     return this.recipients.size() == 1;
   }
 
-  public Recipient getPrimaryRecipient() {
+  public @Nullable Recipient getPrimaryRecipient() {
     if (!isEmpty())
       return this.recipients.get(0);
     else
