Make all Pinpoint notificiations expandable

aws-amplify · Sep 25, 2018 · fb18abf · fb18abf
1 parent ecb67fa
commit fb18abf
Showing 1 changed file with 9 additions and 8 deletions.
diff --git a/...om/amazonaws/mobileconnectors/pinpoint/targeting/notification/NotificationClientBase.java b/...om/amazonaws/mobileconnectors/pinpoint/targeting/notification/NotificationClientBase.java
@@ -616,20 +616,21 @@ private Notification createNotification(final int iconResId, final String title,
             if (imageUrl != null) {
                 try {
                     notificationImage = new DownloadImageTask().execute(imageUrl).get();
-                    if (notificationImage != null) {
-                        bigPictureMethod.invoke(bigPictureStyle, notificationImage);
-                        setSummaryMethod.invoke(bigPictureStyle, contentText);
-                        setStyleMethod.invoke(notificationBuilder, bigPictureStyle);
-                    } else {
-                        bigTextMethod.invoke(bigTextStyle, contentText);
-                        setStyleMethod.invoke(notificationBuilder, bigTextStyle);
-                    }
                 } catch (final InterruptedException e) {
                     log.error("Interrupted when downloading image : " + e.getMessage(), e);
                 } catch (final ExecutionException e) {
                     log.error("Failed execute download image thread : " + e.getMessage(), e);
                 }
             }
+
+            if (notificationImage != null) {
+                bigPictureMethod.invoke(bigPictureStyle, notificationImage);
+                setSummaryMethod.invoke(bigPictureStyle, contentText);
+                setStyleMethod.invoke(notificationBuilder, bigPictureStyle);
+            } else {
+                bigTextMethod.invoke(bigTextStyle, contentText);
+                setStyleMethod.invoke(notificationBuilder, bigTextStyle);
+            }
 
             return (Notification) buildMethod.invoke(notificationBuilder);
         } catch (final InvocationTargetException ex) {