From dd67fed01a731cddb4bffbafead51ac357a57034 Mon Sep 17 00:00:00 2001
From: Loic Blot <loic.blot@unix-experience.fr>
Date: Mon, 15 Sep 2014 18:44:31 +0000
Subject: [PATCH] add missing return for getAllIds and prevent duplicate IDs

---
 db/smsmapper.php | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/db/smsmapper.php b/db/smsmapper.php
index ef13363..a1b8901 100644
--- a/db/smsmapper.php
+++ b/db/smsmapper.php
@@ -31,8 +31,12 @@ class SmsMapper extends Mapper {
 			if (!isset($smsList[$mbox])) {
 				$smsList[$mbox] = array();
 			}
-			array_push($smsList[$mbox], $row["sms_id"]);
+			
+			if (!in_array($row["sms_id"], $smsList[$mbox])) {
+				array_push($smsList[$mbox], $row["sms_id"]);
+			}
 		}
+		return $smsList;
 	}
 
 	public function writeToDB ($userId, $smsList, $purgeAllSmsBeforeInsert = false) {