From f88879f8df8bc6be20ccb711cbef90e7bdcad78e Mon Sep 17 00:00:00 2001
From: M66B <M66B@users.noreply.github.com>
Date: Sat, 14 Dec 2019 13:23:50 +0100
Subject: [PATCH] Consider images without name as inline

---
 .../main/java/eu/faircode/email/AdapterAttachment.java    | 2 +-
 app/src/main/java/eu/faircode/email/AdapterMessage.java   | 8 ++++----
 app/src/main/java/eu/faircode/email/EntityAttachment.java | 4 ++++
 app/src/main/java/eu/faircode/email/FragmentCompose.java  | 5 ++---
 4 files changed, 11 insertions(+), 8 deletions(-)

diff --git a/app/src/main/java/eu/faircode/email/AdapterAttachment.java b/app/src/main/java/eu/faircode/email/AdapterAttachment.java
index 6dca0161c2..fc3108e58f 100644
--- a/app/src/main/java/eu/faircode/email/AdapterAttachment.java
+++ b/app/src/main/java/eu/faircode/email/AdapterAttachment.java
@@ -106,7 +106,7 @@ public class AdapterAttachment extends RecyclerView.Adapter<AdapterAttachment.Vi
         }
 
         private void bindTo(EntityAttachment attachment) {
-            view.setAlpha(attachment.isInline() && attachment.isImage() ? Helper.LOW_LIGHT : 1.0f);
+            view.setAlpha(attachment.isInlineImage() ? Helper.LOW_LIGHT : 1.0f);
 
             ibDelete.setVisibility(readonly ? View.GONE : attachment.isInline() ? View.INVISIBLE : View.VISIBLE);
             tvName.setText(attachment.name);
diff --git a/app/src/main/java/eu/faircode/email/AdapterMessage.java b/app/src/main/java/eu/faircode/email/AdapterMessage.java
index 9eb94e4789..83b1f13ae1 100644
--- a/app/src/main/java/eu/faircode/email/AdapterMessage.java
+++ b/app/src/main/java/eu/faircode/email/AdapterMessage.java
@@ -1285,7 +1285,7 @@ public class AdapterMessage extends RecyclerView.Adapter<AdapterMessage.ViewHold
                     int inlineImages = 0;
                     if (attachments != null)
                         for (EntityAttachment attachment : attachments)
-                            if (attachment.available && attachment.isInline() && attachment.isImage())
+                            if (attachment.available && attachment.isInlineImage())
                                 inlineImages++;
 
                     if (inlineImages != lastInlineImages) {
@@ -1647,7 +1647,7 @@ public class AdapterMessage extends RecyclerView.Adapter<AdapterMessage.ViewHold
 
             List<EntityAttachment> a = new ArrayList<>();
             for (EntityAttachment attachment : attachments) {
-                boolean inline = ((attachment.isInline() && attachment.isImage()) || attachment.encryption != null);
+                boolean inline = ((attachment.isInlineImage()) || attachment.encryption != null);
                 if (inline)
                     has_inline = true;
                 if (attachment.progress == null && !attachment.available)
@@ -1701,7 +1701,7 @@ public class AdapterMessage extends RecyclerView.Adapter<AdapterMessage.ViewHold
 
             List<EntityAttachment> images = new ArrayList<>();
             for (EntityAttachment attachment : attachments)
-                if (!attachment.isInline() && attachment.isImage())
+                if (!attachment.isInlineImage())
                     images.add(attachment);
             adapterImage.set(images);
             grpImages.setVisibility(images.size() > 0 ? View.VISIBLE : View.GONE);
@@ -2715,7 +2715,7 @@ public class AdapterMessage extends RecyclerView.Adapter<AdapterMessage.ViewHold
 
                         List<EntityAttachment> attachments = db.attachment().getAttachments(message.id);
                         for (EntityAttachment attachment : attachments)
-                            if (attachment.isInline() && attachment.isImage() &&
+                            if (attachment.isInlineImage() &&
                                     attachment.progress == null && !attachment.available)
                                 EntityOperation.queue(context, message, EntityOperation.ATTACHMENT, attachment.id);
 
diff --git a/app/src/main/java/eu/faircode/email/EntityAttachment.java b/app/src/main/java/eu/faircode/email/EntityAttachment.java
index 08c5937fad..cb746df2df 100644
--- a/app/src/main/java/eu/faircode/email/EntityAttachment.java
+++ b/app/src/main/java/eu/faircode/email/EntityAttachment.java
@@ -109,6 +109,10 @@ public class EntityAttachment {
         return IMAGE_TYPES.contains(type);
     }
 
+    boolean isInlineImage() {
+        return ((isInline() || TextUtils.isEmpty(name)) && isImage());
+    }
+
     File getFile(Context context) {
         return getFile(context, id, name);
     }
diff --git a/app/src/main/java/eu/faircode/email/FragmentCompose.java b/app/src/main/java/eu/faircode/email/FragmentCompose.java
index 7463e9f3c4..82e0675bd9 100644
--- a/app/src/main/java/eu/faircode/email/FragmentCompose.java
+++ b/app/src/main/java/eu/faircode/email/FragmentCompose.java
@@ -2978,8 +2978,7 @@ public class FragmentCompose extends FragmentBase {
                         for (EntityAttachment attachment : attachments)
                             if (attachment.encryption == null &&
                                     ("forward".equals(action) || "editasnew".equals(action) ||
-                                            (cid.contains(attachment.cid) ||
-                                                    (attachment.isInline() && attachment.isImage())))) {
+                                            (cid.contains(attachment.cid) || attachment.isInlineImage()))) {
                                 if (attachment.available) {
                                     File source = attachment.getFile(context);
 
@@ -3131,7 +3130,7 @@ public class FragmentCompose extends FragmentBase {
                                     available++;
                                 if (attachment.progress != null)
                                     downloading = true;
-                                if (attachment.isInline() && attachment.isImage())
+                                if (attachment.isInlineImage())
                                     inline_images = true;
                             }