[dev] Bug in IMP CVS HEAD

[Michael M Slusarz]

Quoting Chuck Hagenbuch <chuck@horde.org>:

| Quoting Eric Rostetter <eric.rostetter@physics.utexas.edu>:
| 
| > If I set "Hide Deleted" on, and then delete a message from the message
| > view, it takes me back to the folder view on page 1 of my mailbox.  If
| I
| > select "Show Deleted" instead, it works as expected, following my
| > preferences which for me is to go to the next message.
| 
| Confirmed. I've traced it to the message index no longer being valid
| after
| deleting the message; this check is triggered:
| 
| /* We may have done processing that has taken us past the end of the
|    message array, so we will return to mailbox.php if that is the
|    case. */
| if (!$imp_mailbox->isValidIndex()) {
|     header('Location: ' .
| Horde::addParameter(Horde::applicationUrl('mailbox.php', true), 'start='
| .
| $imp_mailbox->getMessageIndex()));
|     exit;
| }
| 
| ... but I don't know why. Michael? This is buried in IMP_Mailbox:: or
| IMP_Message:: somewhere...

Fixed.  We just had to increment the index before resorting (a 1-line fix).

michael

______________________________________________
Michael Slusarz [slusarz@bigworm.colorado.edu]
The University of Colorado at Boulder