WPNetworkImageView now accepts a default error image resource, also f…

…orcing size when showing blavatars in ReaderBlogAdapter
wordpress-mobile · Feb 10, 2015 · b67cc7b · b67cc7b
1 parent f6876ae
commit b67cc7b
Showing 1 changed file with 8 additions and 5 deletions.
diff --git a/WordPressUtils/src/main/java/org/wordpress/android/util/GravatarUtils.java b/WordPressUtils/src/main/java/org/wordpress/android/util/GravatarUtils.java
@@ -2,10 +2,10 @@
 
 import android.text.TextUtils;
 
+/**
+ * see https://en.gravatar.com/site/implement/images/
+ */
 public class GravatarUtils {
-    /*
-     * see https://en.gravatar.com/site/implement/images/
-     */
 
     public static enum DefaultImage {
         MYSTERY_MAN,
@@ -42,6 +42,9 @@ public String toString() {
     * replace it with a new ?s= parameter which requests the avatar at the exact size needed
     */
     public static String fixGravatarUrl(final String imageUrl, int avatarSz) {
+        return fixGravatarUrl(imageUrl, avatarSz, DefaultImage.MYSTERY_MAN);
+    }
+    public static String fixGravatarUrl(final String imageUrl, int avatarSz, DefaultImage defaultImage) {
         if (TextUtils.isEmpty(imageUrl)) {
             return "";
         }
@@ -51,8 +54,8 @@ public static String fixGravatarUrl(final String imageUrl, int avatarSz) {
             return PhotonUtils.getPhotonImageUrl(imageUrl, avatarSz, avatarSz);
         }
 
-        // remove all other params, then add query string for size and "mystery man" default
-        return UrlUtils.removeQuery(imageUrl) + "?s=" + avatarSz + "&d=mm";
+        // remove all other params, then add query string for size and default image
+        return UrlUtils.removeQuery(imageUrl) + "?s=" + avatarSz + "&d=" + defaultImage.toString();
     }
 
     public static String gravatarFromEmail(final String email, int size) {