From 53659aa999efa3eba2adafc220f114b2e80f5c52 Mon Sep 17 00:00:00 2001
From: torrentcome <come@netcosports.com>
Date: Wed, 17 May 2017 16:08:43 +0200
Subject: [PATCH] (bug of picasso) the Target wasn't load at the first time
 forget to read the documentation so , record the target in field change the
 way that works.

Now work fine.
---
 .../keylesspalace/tusky/ComposeActivity.java  | 20 +++++++++++++++++--
 1 file changed, 18 insertions(+), 2 deletions(-)

diff --git a/app/src/main/java/com/keylesspalace/tusky/ComposeActivity.java b/app/src/main/java/com/keylesspalace/tusky/ComposeActivity.java
index 29ff64b0..6a5cb8b3 100644
--- a/app/src/main/java/com/keylesspalace/tusky/ComposeActivity.java
+++ b/app/src/main/java/com/keylesspalace/tusky/ComposeActivity.java
@@ -80,6 +80,7 @@ import com.keylesspalace.tusky.util.SpanUtils;
 import com.keylesspalace.tusky.util.ThemeUtils;
 import com.keylesspalace.tusky.view.EditTextTyped;
 import com.squareup.picasso.Picasso;
+import com.squareup.picasso.Target;
 
 import java.io.File;
 import java.io.FileNotFoundException;
@@ -153,6 +154,11 @@ public class ComposeActivity extends BaseActivity implements ComposeOptionsFragm
     // this only exists when a status is trying to be sent, but uploads are still occurring
     private ProgressDialog finishingUploadDialog;
 
+    /**
+     * The Target object must be stored as a member field or method and cannot be an anonymous class otherwise this won't work as expected. The reason is that Picasso accepts this parameter as a weak memory reference. Because anonymous classes are eligible for garbage collection when there are no more references, the network request to fetch the image may finish after this anonymous class has already been reclaimed. See this Stack Overflow discussion for more details.
+     */
+    @SuppressWarnings("FieldCanBeLocal")
+    private Target target;
 
     @Override
     public void onCreate(Bundle savedInstanceState) {
@@ -1184,7 +1190,15 @@ public class ComposeActivity extends BaseActivity implements ComposeOptionsFragm
             textEditor.append(headerInfo.baseUrl);
         }
         if (!TextUtils.isEmpty(headerInfo.image)) {
-            Picasso.with(this).load(headerInfo.image).into(MediaUtils.picassoImageTarget(getApplicationContext(), new MediaUtils.MediaListener() {
+            Picasso.Builder builder = new Picasso.Builder(getApplicationContext());
+            builder.listener(new Picasso.Listener() {
+                @Override
+                public void onImageLoadFailed(Picasso picasso, Uri uri, Exception exception) {
+                    exception.printStackTrace();
+                }
+            });
+
+            target = MediaUtils.picassoImageTarget(ComposeActivity.this, new MediaUtils.MediaListener() {
                 @Override
                 public void onCallback(final Uri headerInfo) {
                     if (headerInfo != null) {
@@ -1197,10 +1211,12 @@ public class ComposeActivity extends BaseActivity implements ComposeOptionsFragm
                         });
                     }
                 }
-            }));
+            });
+            Picasso.with(this).load(headerInfo.image).into(target);
         }
     }
 
+
     // remove the precedent paste from the edit text
     private void cleanBaseUrl(ParserUtils.HeaderInfo headerInfo) {
         int lengthBaseUrl = headerInfo.baseUrl.length();