Caught by unit test - if I had a dime for everytime I did this...

horde · Jan 9, 2015 · fbf752a · fbf752a
1 parent 11cf727
commit fbf752a
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/framework/ActiveSync/lib/Horde/ActiveSync/Folder/Imap.php b/framework/ActiveSync/lib/Horde/ActiveSync/Folder/Imap.php
@@ -119,6 +119,7 @@ public function setChanges(array $messages, array $flags = array(), array $categ
         $uidnext = $this->uidnext();
         $minuid = $this->minuid();
         $modseq = $this->modseq();
+
         foreach ($messages as $uid) {
             if ($uid >= $uidnext) {
                 // new (to us) message.
@@ -128,7 +129,7 @@ public function setChanges(array $messages, array $flags = array(), array $categ
                     // MODSEQ capable server, either a changed message, or
                     // one that was previously outside of FilterType but is now
                     // within.
-                    if (array_search($uid, $this->_messages)) {
+                    if (array_search($uid, $this->_messages) !== false) {
                         $this->_changed[] = $uid;
                     } else {
                         $this->_added[] = $uid;