Restore ImageView picasso callback

Signed-off-by: Jorge Ruesga <jorge@ruesga.com>
jruesga · Apr 11, 2017 · 8efc34d · 8efc34d
1 parent ed8649e
commit 8efc34d
Showing 1 changed file with 37 additions and 13 deletions.
diff --git a/app/src/main/java/com/ruesga/rview/misc/PicassoHelper.java b/app/src/main/java/com/ruesga/rview/misc/PicassoHelper.java
@@ -34,6 +34,7 @@
 import com.ruesga.rview.gerrit.model.AccountInfo;
 import com.ruesga.rview.model.Account;
 import com.ruesga.rview.preferences.Preferences;
+import com.squareup.picasso.Callback;
 import com.squareup.picasso.Picasso;
 import com.squareup.picasso.Target;
 import com.squareup.picasso.Transformation;
@@ -84,26 +85,49 @@ public static void bindAvatar(Context context, Picasso picasso, AccountInfo acco
                 ImageView view, Drawable placeholder) {
         final Account acct = Preferences.getAccount(context);
         final List<String> avatarUrls = ModelHelper.getAvatarUrl(context, acct, account);
-        loadWithFallbackUrls(context, picasso, view, placeholder, avatarUrls);
+        loadWithFallbackUrls(picasso, view, placeholder, avatarUrls);
     }
 
     public static void bindAvatar(Context context, Picasso picasso, Account acct,
-            AccountInfo account, ImageView view, Drawable placeholder) {
-        final List<String> avatarUrls = ModelHelper.getAvatarUrl(context, acct, account);
-        loadWithFallbackUrls(context, picasso, view, placeholder, avatarUrls);
-    }
-
-    public static void bindAvatar(Context context, Picasso picasso, AccountInfo account,
-            MenuItem item, Drawable placeholder) {
-        final Account acct = Preferences.getAccount(context);
+            AccountInfo account, MenuItem item, Drawable placeholder) {
         final List<String> avatarUrls = ModelHelper.getAvatarUrl(context, acct, account);
         loadWithFallbackUrls(context, picasso, item, placeholder, avatarUrls);
     }
 
-    public static void bindAvatar(Context context, Picasso picasso, Account acct,
-            AccountInfo account, MenuItem item, Drawable placeholder) {
-        final List<String> avatarUrls = ModelHelper.getAvatarUrl(context, acct, account);
-        loadWithFallbackUrls(context, picasso, item, placeholder, avatarUrls);
+    private static void loadWithFallbackUrls(final Picasso picasso, final ImageView view,
+            final Drawable placeholder, final List<String> urls) {
+        final String nextUrl;
+        synchronized (urls) {
+            nextUrl = urls.isEmpty() ? null : urls.get(0);
+        }
+        if (nextUrl != null) {
+            picasso.load(nextUrl)
+                    .placeholder(placeholder)
+                    .transform(new CircleTransform())
+                    .into(view, new Callback() {
+                        @Override
+                        public void onSuccess() {
+                            synchronized (urls) {
+                                urls.clear();
+                                urls.add(nextUrl);
+                            }
+                        }
+
+                        @Override
+                        public void onError() {
+                            // Next url
+                            synchronized (urls) {
+                                if (urls.contains(nextUrl)) {
+                                    urls.remove(nextUrl);
+                                }
+                            }
+                            loadWithFallbackUrls(picasso, view, placeholder, urls);
+                        }
+                    });
+        } else {
+            // Placeholder
+            view.setImageDrawable(placeholder);
+        }
     }
 
     private static void loadWithFallbackUrls(final Context context, final Picasso picasso,