Prevent reply to self

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

@@ -160,10 +160,19 @@ public class EntityMessage implements Serializable {
         return sb.toString();
     }
 
+    boolean replySelf(String via) {
+        Address[] replying = (reply == null || reply.length == 0 ? from : reply);
+        if (replying == null || replying.length != 1)
+            return false;
+
+        String recipient = MessageHelper.canonicalAddress(((InternetAddress) replying[0]).getAddress());
+        return recipient.equals(via);
+    }
+
     Address[] getAllRecipients(String via) {
         List<Address> addresses = new ArrayList<>();
 
-        if (to != null)
+        if (!replySelf(via) && to != null)
             addresses.addAll(Arrays.asList(to));
 
         if (cc != null)

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

@@ -2050,16 +2050,11 @@ public class FragmentCompose extends FragmentBase {
                                 draft.to = ref.receipt_to;
                             else {
                                 // Prevent replying to self
-                                String to = null;
+                                if (ref.replySelf(via)) {
-                                Address[] replying = (ref.reply == null || ref.reply.length == 0 ? ref.from : ref.reply);
-                                if (replying != null && replying.length == 1)
-                                    to = MessageHelper.canonicalAddress(((InternetAddress) replying[0]).getAddress());
-
-                                if (to != null && to.equals(via)) {
                                     draft.to = ref.to;
                                     draft.from = ref.from;
                                 } else
-                                    draft.to = replying;
+                                    draft.to = (ref.reply == null || ref.reply.length == 0 ? ref.from : ref.reply);
                             }
 
                             if ("reply_all".equals(action))