From 7750cddce24006e0c3be28f3ca3efbb041fe8fac Mon Sep 17 00:00:00 2001
From: M66B <M66B@users.noreply.github.com>
Date: Fri, 30 Oct 2020 18:48:53 +0100
Subject: [PATCH] Revert "Refactoring"

This reverts commit 8babc79a03048cc411c080251905837449bd3ce2.
---
 .../main/java/eu/faircode/email/HtmlHelper.java  | 16 ++++++----------
 1 file changed, 6 insertions(+), 10 deletions(-)

diff --git a/app/src/main/java/eu/faircode/email/HtmlHelper.java b/app/src/main/java/eu/faircode/email/HtmlHelper.java
index d2acc91cd5..aa23f2c4cf 100644
--- a/app/src/main/java/eu/faircode/email/HtmlHelper.java
+++ b/app/src/main/java/eu/faircode/email/HtmlHelper.java
@@ -1491,7 +1491,7 @@ public class HtmlHelper {
         List<String> hosts = new ArrayList<>();
         for (Element img : document.select("img")) {
             String src = img.attr("src");
-            if (!TextUtils.isEmpty(src) && !isTrackingPixel(img, disconnect_images)) {
+            if (!TextUtils.isEmpty(src) && !isTrackingPixel(img)) {
                 Uri uri = Uri.parse(img.attr("src"));
                 String host = uri.getHost();
                 if (host != null && !hosts.contains(host))
@@ -1525,7 +1525,8 @@ public class HtmlHelper {
                 continue;
             }
 
-            if (isTrackingPixel(img, disconnect_images)) {
+            if (isTrackingPixel(img) ||
+                    (disconnect_images && DisconnectBlacklist.isTracking(host))) {
                 img.attr("src", sb.toString());
                 img.attr("alt", context.getString(R.string.title_legend_tracking_pixel));
                 img.attr("height", "24");
@@ -1536,19 +1537,14 @@ public class HtmlHelper {
         }
     }
 
-    private static boolean isTrackingPixel(Element img, boolean disconnect_images) {
+    private static boolean isTrackingPixel(Element img) {
         String src = img.attr("src");
         if (!TextUtils.isEmpty(src))
             try {
                 Uri uri = Uri.parse(src);
                 String host = uri.getHost();
-                if (!TextUtils.isEmpty(host)) {
-                    if (TRACKING_HOSTS.contains(host))
-                        return true;
-                    if (disconnect_images &&
-                            DisconnectBlacklist.isTracking(host))
-                        return true;
-                }
+                if (!TextUtils.isEmpty(host) && TRACKING_HOSTS.contains(host))
+                    return true;
             } catch (Throwable ex) {
                 Log.w(ex);
             }