Tickets :: [#3372] Mailbox sorting doesn't take effect until 2nd reload

6.0.0-beta1

11/7/25

Summary	Mailbox sorting doesn't take effect until 2nd reload
Queue	IMP
Queue Version	HEAD
Type	Bug
State	Resolved
Priority	1. Low
Owners
Requester	ben (at)
Created	01/31/2006 (7220 days ago)
Due
Updated	02/28/2006 (7192 days ago)
Assigned	01/31/2006 (7220 days ago)
Resolved	02/28/2006 (7192 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	No

02/28/2006 05:57:03 AM	Michael Slusarz	Comment #13 State ⇒ Resolved	Reply to this comment
This was actually an issue in Prefs.php. It has been fixed.

02/28/2006 05:19:00 AM	ben	Comment #12	Reply to this comment
That actually does the trick.

02/28/2006 05:04:59 AM	Michael Slusarz	Comment #11	Reply to this comment
Just for the heck of it, can you try putting this line on line 108 in mailbox.php in CVS (last line in the 'change_sort' case): $prefs->store();

02/21/2006 04:41:18 AM	ben	Comment #10	Reply to this comment
Now we're back to where we started. Every change requires a 2nd reload. The sort doesn't take effect until the 2nd load, either.

02/21/2006 04:35:03 AM	Michael Slusarz	Comment #9	Reply to this comment
try once more. I am stumped more on where in the code the IMP_Mailbox:: object is being created before the sort pref is changed - since I can never reproduce what you are describing. and there is nothing in mailbox.php before the 'change_sort' block that creates an IMP_Mailbox object that I can see.

02/21/2006 04:17:52 AM	ben	Comment #8	Reply to this comment
It seemed to work better before this last commit. It's pretty sporatic now. Sometimes it works, sometimes it doesn't, between various columns.

02/21/2006 04:12:44 AM	Michael Slusarz	Comment #7	Reply to this comment
Looks like I forgot to commit some more updates, so try again.

02/21/2006 03:35:52 AM	ben	Comment #6	Reply to this comment
Better, but still not quite right. It works except for the Subject/Thread link. Clicking on the "small" link still requires a 2nd page load. Just in case that doesn't make sense, when the header link looks like: Subject [thread] Clicking on Subject works fine, but clicking on [thread] still requires a 2nd page load. Regardless of which column is currently selected.

02/21/2006 02:50:38 AM	Michael Slusarz	Comment #5	Reply to this comment
Try what I just committed.

02/20/2006 07:44:16 PM	ben	Comment #4	Reply to this comment
I've noticed that the sorting does take effect on the first load, but the highlighted column header doesn't change until the 2nd. Still haven't had a chance to do any real troubleshooting myself, though.

01/31/2006 09:01:33 AM	Chuck Hagenbuch	Comment #3	Reply to this comment
I can reproduce this. Sometimes the first sorting change takes effect immediately, but subsequent ones don't show up immediately. For some reason this seems to be particularly evident if I have IMP open in one window and DIMP in another.

01/31/2006 05:35:02 AM	Michael Slusarz	Comment #2 State ⇒ Feedback	Reply to this comment
can't reproduce on any browser.

01/31/2006 04:55:19 AM	ben	Comment #1 Priority ⇒ 1. Low Type ⇒ Bug Summary ⇒ Mailbox sorting doesn't take effect until 2nd reload Queue ⇒ IMP State ⇒ Unconfirmed	Reply to this comment
Whenever I click on a new column to sort by, or to change the order of the sorting, the page reloads as expected, but the new sort order does not take effect until the page reloads a 2nd time (either by waiting for the automatic refresh, or manually reloading)