Fix unread message finder

Update changelog

git-svn-id: svn+ssh://murphy/home/svn/banana/trunk@132 9869982d-c50d-0410-be91-f2a2ec7c7c7b

diff --git a/Changelog b/Changelog
index 6aa6a12..c5c117f 100644 (file)
--- a/Changelog
+++ b/Changelog
@@ -1,25 +1,36 @@
+Sat, 05 Aug 2006                    Florent Bruneau <florent.bruneau@m4x.org>
+
+       * Allow browsing through unread messages
+
 Sat, 22 Jul 2006                    Florent Bruneau <florent.bruneau@m4x.org>
+
        * Impove url and email catcher
        * Bugfix: quotes formating
 
 Sat, 15 Jul 2006                    Florent Bruneau <florent.bruneau@m4x.org>
+
        * New tabbed structure
        * Support of Content-ID references (multipart/related)
 
 Fri, 14 Jul 2006                    Florent Bruneau <florent.bruneau@m4x.org>
+
        * Bugfix: wrapping issues with very long lines
 
 Thu, 13 Jul 2006                    Florent Bruneau <florent.bruneau@m4x.org>
+
        * Bugfix: xfaces handling
        
 Wed, 12 Jul 2006                    Florent Bruneau <florent.bruneau@m4x.org>
+
        * Add the list of subscriptions after the spool
        * Make article view more user-friendly
 
 Tue, 11 Jul 2006                    Florent Bruneau <florent.bruneau@m4x.org>
+
        * Bugfix: body conversion from unknown charset
 
 Mon, 10 Jul 2006                    Florent Bruneau <florent.bruneau@m4x.org>
+
        * Use dynamic links from tree images
        * Bugfix: force spool rows height
 

diff --git a/banana/spool.inc.php b/banana/spool.inc.php
index 7b20f15..c4c2aba 100644 (file)
--- a/banana/spool.inc.php
+++ b/banana/spool.inc.php
@@ -584,20 +584,17 @@ class BananaSpool
         // Look in message children
         foreach ($this->overview[$id]->children as $child) {
             $next = $this->_nextUnread($child);
-            if (is_null($next)) {
+            if (!is_null($next)) {
                 return $next;
             }
         }
 
         // Look in current thread
         $cur = $id;
-        while (true) {
-            if (is_null($cur)) {
-                break;
-            }
+        while (!is_null($cur)) {
             $parent = $this->overview[$cur]->parent;
             $ok     = false;
-            if (is_null($parent)) {
+            if (!is_null($parent)) {
                 $array = &$this->overview[$parent]->children;
             } else {
                 $array = &$this->roots;