Refactoring

app/src/main/java/eu/faircode/email/EntityMessage.java

@@ -349,6 +349,17 @@ public class EntityMessage implements Serializable {
         return addresses.toArray(new Address[0]);
     }
 
+    List<Address> getAllRecipients() {
+        List<Address> recipients = new ArrayList<>();
+        if (to != null)
+            recipients.addAll(Arrays.asList(to));
+        if (cc != null)
+            recipients.addAll(Arrays.asList(cc));
+        if (bcc != null)
+            recipients.addAll(Arrays.asList(bcc));
+        return recipients;
+    }
+
     boolean hasKeyword(@NonNull String value) {
         // https://tools.ietf.org/html/rfc5788
         if (keywords == null)

app/src/main/java/eu/faircode/email/FragmentCompose.java

@@ -5804,13 +5804,7 @@ public class FragmentCompose extends FragmentBase {
                                 data.draft.plain_only = 1;
 
                             // Encryption
-                            List<Address> recipients = new ArrayList<>();
+                            List<Address> recipients = data.draft.getAllRecipients();
-                            if (data.draft.to != null)
-                                recipients.addAll(Arrays.asList(data.draft.to));
-                            if (data.draft.cc != null)
-                                recipients.addAll(Arrays.asList(data.draft.cc));
-                            if (data.draft.bcc != null)
-                                recipients.addAll(Arrays.asList(data.draft.bcc));
 
                             if (!BuildConfig.DEBUG)
                                 if (EntityMessage.PGP_SIGNONLY.equals(ref.ui_encrypt) ||
@@ -7089,13 +7083,7 @@ public class FragmentCompose extends FragmentBase {
                             //        identity != null && identity.sender_extra)
                             //    args.putBoolean("remind_extra", true);
 
-                            List<Address> recipients = new ArrayList<>();
+                            List<Address> recipients = draft.getAllRecipients();
-                            if (draft.to != null)
-                                recipients.addAll(Arrays.asList(draft.to));
-                            if (draft.cc != null)
-                                recipients.addAll(Arrays.asList(draft.cc));
-                            if (draft.bcc != null)
-                                recipients.addAll(Arrays.asList(draft.bcc));
 
                             args.putBoolean("remind_noreply", MessageHelper.isNoReply(recipients));