Fix duplicate quick replies (closes #22)

jariz · Sep 14, 2016 · 326dbc8 · 326dbc8
1 parent f73f8fb
commit 326dbc8
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/Noti/Info.plist b/Noti/Info.plist
@@ -17,11 +17,11 @@
 	<key>CFBundlePackageType</key>
 	<string>APPL</string>
 	<key>CFBundleShortVersionString</key>
-	<string>0.1.2</string>
+	<string>0.1.5</string>
 	<key>CFBundleSignature</key>
 	<string>????</string>
 	<key>CFBundleVersion</key>
-	<string>5</string>
+	<string>7</string>
 	<key>LSApplicationCategoryType</key>
 	<string>public.app-category.productivity</string>
 	<key>LSMinimumSystemVersion</key>

diff --git a/Noti/PushManager.swift b/Noti/PushManager.swift
@@ -184,11 +184,18 @@ class PushManager: NSObject, WebSocketDelegate, NSUserNotificationCenterDelegate
                 let body = notification.response?.string
 
                 func doQuickReply() {
+                    var indexToBeRemoved = -1, i = -1;
                     for item in pushHistory {
+                        i += 1;
                         if item["notification_id"].string == notification.identifier && item["type"].string == "mirror" {
-                            ephemerals.quickReply(item, reply: body!)
+                            ephemerals.quickReply(item, reply: body!);
+                            indexToBeRemoved = i;
+                            break;
                         }
                     }
+                    if(indexToBeRemoved != -1) {
+                        pushHistory.removeAtIndex(indexToBeRemoved)
+                    }
                 }
 
                 //determine if we replied to a sms or a normal notification