From 4cc32331f24cc2a4902be91d331bef882820fa18 Mon Sep 17 00:00:00 2001
From: Matte23 <matteo@underdesk.net>
Date: Mon, 16 Sep 2019 18:28:09 +0200
Subject: [PATCH] Show new notifications (not old ones) and add auto cancel

---
 .../main/java/net/underdesk/circolapp/works/PollWork.kt    | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/app/src/main/java/net/underdesk/circolapp/works/PollWork.kt b/app/src/main/java/net/underdesk/circolapp/works/PollWork.kt
index 0219254..51f76cf 100644
--- a/app/src/main/java/net/underdesk/circolapp/works/PollWork.kt
+++ b/app/src/main/java/net/underdesk/circolapp/works/PollWork.kt
@@ -81,7 +81,9 @@ class PollWork(appContext: Context, workerParams: WorkerParameters) :
                 .setBigContentTitle(applicationContext.getString(R.string.notification_summary_title))
                 .setSummaryText(applicationContext.getString(R.string.notification_summary))
 
-            for (i in oldCirculars.size..newCirculars.lastIndex) {
+            val circularCount = newCirculars.size - oldCirculars.size
+
+            for (i in 0 until circularCount) {
                 createNotification(newCirculars[i])
                 summaryStyle.addLine(newCirculars[i].name)
             }
@@ -91,7 +93,7 @@ class PollWork(appContext: Context, workerParams: WorkerParameters) :
                 .setContentText(
                     applicationContext.getString(
                         R.string.notification_summary_text,
-                        newCirculars.size - oldCirculars.size
+                        circularCount
                     )
                 )
                 .setSmallIcon(R.drawable.ic_notifications_black_24dp)
@@ -123,6 +125,7 @@ class PollWork(appContext: Context, workerParams: WorkerParameters) :
             .setContentText(circular.name)
             .setPriority(NotificationCompat.PRIORITY_DEFAULT)
             .setContentIntent(pendingIntent)
+            .setAutoCancel(true)
             .setGroup(CHANNEL_ID)
             .setStyle(
                 NotificationCompat.BigTextStyle()