Skip to content

Commit 2c8ff0b

Browse files
committed
WPNetworkImageView now accepts a default error image resource, also forcing size when showing blavatars in ReaderBlogAdapter
1 parent 042bdd3 commit 2c8ff0b

File tree

1 file changed

+8
-5
lines changed

1 file changed

+8
-5
lines changed

WordPressUtils/src/main/java/org/wordpress/android/util/GravatarUtils.java

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2,10 +2,10 @@
22

33
import android.text.TextUtils;
44

5+
/**
6+
* see https://en.gravatar.com/site/implement/images/
7+
*/
58
public class GravatarUtils {
6-
/*
7-
* see https://en.gravatar.com/site/implement/images/
8-
*/
99

1010
public static enum DefaultImage {
1111
MYSTERY_MAN,
@@ -42,6 +42,9 @@ public String toString() {
4242
* replace it with a new ?s= parameter which requests the avatar at the exact size needed
4343
*/
4444
public static String fixGravatarUrl(final String imageUrl, int avatarSz) {
45+
return fixGravatarUrl(imageUrl, avatarSz, DefaultImage.MYSTERY_MAN);
46+
}
47+
public static String fixGravatarUrl(final String imageUrl, int avatarSz, DefaultImage defaultImage) {
4548
if (TextUtils.isEmpty(imageUrl)) {
4649
return "";
4750
}
@@ -51,8 +54,8 @@ public static String fixGravatarUrl(final String imageUrl, int avatarSz) {
5154
return PhotonUtils.getPhotonImageUrl(imageUrl, avatarSz, avatarSz);
5255
}
5356

54-
// remove all other params, then add query string for size and "mystery man" default
55-
return UrlUtils.removeQuery(imageUrl) + "?s=" + avatarSz + "&d=mm";
57+
// remove all other params, then add query string for size and default image
58+
return UrlUtils.removeQuery(imageUrl) + "?s=" + avatarSz + "&d=" + defaultImage.toString();
5659
}
5760

5861
public static String gravatarFromEmail(final String email, int size) {

0 commit comments

Comments
 (0)