Fri, 10 Apr 2026 07:55:23 +0000 https://bugs.horde.org/ticket/1257 Incomplete Thread View When you delete a message the resulting array containing thread structure is missing the matching index, and this is correct. but when you try to get the thread list iterating $key it breaks and you get an incomplete list. The right behaviour should be to continue with next step. Thu, 27 Jan 2005 07:59:04 +0000 https://bugs.horde.org/ticket/1257#t4932 Please attach the patch you mentioned to this ticket. Thu, 27 Jan 2005 15:15:40 +0000 https://bugs.horde.org/ticket/1257#t4949 ok, this is the patch for this problem that fixes also another issue, (you get a broken thread if the thread contains the first message of the mailbox) Thu, 10 Feb 2005 11:07:48 +0000 https://bugs.horde.org/ticket/1257#t5407 This patch makes no sense. First, why are you changing the return value of getKey from false to null? That doesn't get you anywhere. Second, why are you using ++$key instead of $key++? You are missing a whole potential case by doing this. Third, if ($this->_thread[$key]) doesn't exist, you have a mailbox that is corrupt - that code should never be reached or else there are issues elsewhere. So it makes a whole bunch of sense to exit the loop rather than ignore the entry - which may lead to some very weird output. Fri, 11 Feb 2005 03:52:08 +0000 https://bugs.horde.org/ticket/1257#t5424 Also wanted to mention that I have not been able to reproduce a situation that leads to incomplete thread view (viewing/deleting a thread item that is the first item in the mailbox, for example, doesn't cause any error for me). There is no detailed description in this bug report on how to reliably reproduce unwanted behavior. Fri, 11 Feb 2005 03:54:06 +0000 https://bugs.horde.org/ticket/1257#t5425 Response from K-team: Sorry, I thought it was clear....you just have to send a mail from an empty mailbox and reply a couple of times, so that you have a thread. if you sort the mailbox by thread it seems good, but when you read a message from the thread, if you click on "View Thread" it shows just the message you were reading. After investigating a while I noticed that _getKey sometimes returns key=0 for the first mail in thread. in getThreadBase this test: if ($key = $this->_getKey($index)) fails if $key is 0, so I modifyed it in $key = $this->_getKey($index); + if (isset($key)) and set return value to null in _getKey when it fails. I also changed $key++ to ++$key for the same reason, the while loop breaks soon if $key is 0. As far as regards the third point, I've run into a situation in which a deleted message breaks the thread and it displays nothing, so thought it was better to end the cycle and just skip the message. Hope it's clearer now K-team Wed, 16 Feb 2005 07:50:08 +0000 https://bugs.horde.org/ticket/1257#t5525 OK - I wasn't understanding that you were seeing this on the thread screen - I thought this was something you were seeing on the mailbox screen. I've modified your patch to check for the "nullness" of the return value, as well as fixing several other functions and the use of $key. For now, I am not going to address your 3rd point since I have not yet seen this behavior. Wed, 16 Feb 2005 07:55:24 +0000 https://bugs.horde.org/ticket/1257#t5526 The fix has been applied to HEAD and Horde 3.0.4, BTW. Wed, 16 Feb 2005 07:56:00 +0000 https://bugs.horde.org/ticket/1257#t5527