Exit accessor earlier if avatar ID is null.
authorDavid ‘Bombe’ Roden <bombe@pterodactylus.net>
Wed, 21 Dec 2011 12:41:27 +0000 (13:41 +0100)
committerDavid ‘Bombe’ Roden <bombe@pterodactylus.net>
Wed, 21 Dec 2011 12:41:27 +0000 (13:41 +0100)
src/main/java/net/pterodactylus/sone/template/ProfileAccessor.java

index 05857cf..366e071 100644 (file)
@@ -61,11 +61,12 @@ public class ProfileAccessor extends ReflectionAccessor {
                                return null;
                        }
                        String avatarId = profile.getAvatar();
-                       if (avatarId != null) {
-                               if (core.getImage(avatarId, false) == null) {
-                                       /* avatar ID but no matching image? show nothing. */
-                                       return null;
-                               }
+                       if (avatarId == null) {
+                               return null;
+                       }
+                       if (core.getImage(avatarId, false) == null) {
+                               /* avatar ID but no matching image? show nothing. */
+                               return null;
                        }
                        Sone remoteSone = profile.getSone();
                        if (core.isLocalSone(remoteSone)) {
@@ -76,7 +77,7 @@ public class ProfileAccessor extends ReflectionAccessor {
                        if (showCustomAvatars == ShowCustomAvatars.NEVER) {
                                return null;
                        }
-                       if ((showCustomAvatars == ShowCustomAvatars.ALWAYS) || (avatarId == null)) {
+                       if (showCustomAvatars == ShowCustomAvatars.ALWAYS) {
                                return avatarId;
                        }
                        if (showCustomAvatars == ShowCustomAvatars.FOLLOWED) {