[horde] Bug found and fixed

[Michael M Slusarz]

Quoting Johan Fredriksson <eskil at e.kth.se>:

> Hello!
>
> I have not read (on this list) about anyone else having this problem.
> I think that is quite strange since it have troubled me for quite some
> time with both Horde-3.0-ALPHA with IMP-4.0-ALPHA and Horde-3.0.2 with
> IMP-4.0.1.
>
> The problem is that IMP sonetimes "forget" which mailbox I'm currently
> reading. Clicking on a mail results ether in the error message "This
> message could not be found", or a message with the same index from another
> mailbox. Reloading the frame brings me back to the mailbox i was reading
> previously.
>
> After endles hours of debuging-by-echo I narrowed down the problem enough
> to produce a patch that solves (or works around) this bug. Here it is:
>
>
> --- imp/lib/Mailbox.php.orig 18 Jan 2005 17:05:13 -0000      1.2
> +++ imp/lib/Mailbox.php 20 Jan 2005 11:41:48 -0000      1.3
> @@ -171,5 +171,6 @@
>      function setNewIndex($index)
>      {
>          $this->_setIndex($index);
> +       $this->_getSortedIndex();
>          $this->_setSorted();
>          $this->_setArrayIndex();

uhhh... no.  This defeats the whole purpose of IMP_Mailbox.  You are doing a
resort of the entire mailbox on every page load which is really not what we
want.

michael

_______________________________________
Michael Slusarz [slusarz at curecanti.org]