From 1e016f3ad16ed8f8f9ce010f578b850fd5c0a5e3 Mon Sep 17 00:00:00 2001
From: x2003bruneau <x2003bruneau@9869982d-c50d-0410-be91-f2a2ec7c7c7b>
Date: Sun, 6 Aug 2006 09:09:56 +0000
Subject: [PATCH] Fix unread message finder Update changelog

git-svn-id: svn+ssh://murphy/home/svn/banana/trunk@132 9869982d-c50d-0410-be91-f2a2ec7c7c7b
---
 Changelog            | 11 +++++++++++
 banana/spool.inc.php |  9 +++------
 2 files changed, 14 insertions(+), 6 deletions(-)

diff --git a/Changelog b/Changelog
index 6aa6a12..c5c117f 100644
--- 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
--- 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;
-- 
2.1.4