* Fixed "empty" RSS notification

lgallard · Jun 26, 2015 · a6b369a · a6b369a
1 parent cc9da7d
commit a6b369a
Showing 1 changed file with 14 additions and 6 deletions.
diff --git a/qBittorrentClient/app/src/main/java/com/lgallardo/qbittorrentclient/RSSService.java b/qBittorrentClient/app/src/main/java/com/lgallardo/qbittorrentclient/RSSService.java
@@ -346,6 +346,8 @@ protected ArrayList<RSSFeed> doInBackground(String... params) {
         @Override
         protected void onPostExecute(ArrayList<RSSFeed> result) {
 
+            boolean gotItems= false;
+
             if (result != null && result.size() > 0) {
 
                 String info = "";
@@ -366,7 +368,7 @@ protected void onPostExecute(ArrayList<RSSFeed> result) {
 
                 NotificationManager notificationManager = (NotificationManager) context.getSystemService(context.NOTIFICATION_SERVICE);
 
-                Notification notification;
+                Notification notification = null;
 
                 if (android.os.Build.VERSION.SDK_INT >= 16) {
 
@@ -382,6 +384,10 @@ protected void onPostExecute(ArrayList<RSSFeed> result) {
 
                         ArrayList<RSSFeedItem> items = rssFeed.getItems();
 
+                        if(items.size() > 0){
+                            gotItems = true;
+                        }
+
 
                         for (int j = 0; j < items.size(); j++) {
 
@@ -441,9 +447,10 @@ protected void onPostExecute(ArrayList<RSSFeed> result) {
                     }
 
 
-                    inbox.setSummaryText(result.get(0).getChannelTitle());
-
-                    notification = inbox.build();
+                    if(gotItems) {
+                        inbox.setSummaryText(result.get(0).getChannelTitle());
+                        notification = inbox.build();
+                    }
 
                 } else {
                     notification = builder.getNotification();
@@ -471,8 +478,9 @@ protected void onPostExecute(ArrayList<RSSFeed> result) {
                     }
                 }
 
-
-                notificationManager.notify(0, notification);
+                if(gotItems && notification != null) {
+                    notificationManager.notify(0, notification);
+                }
 
             }