6.0.0-beta1
7/28/25

[#8936] Clean up date/arrival/sequence sorting
Summary Clean up date/arrival/sequence sorting
Queue IMP
Queue Version Git master
Type Enhancement
State Resolved
Priority 1. Low
Owners slusarz (at) horde (dot) org
Requester slusarz (at) horde (dot) org
Created 03/26/2010 (5603 days ago)
Due
Updated 03/30/2010 (5599 days ago)
Assigned
Resolved 03/30/2010 (5599 days ago)
Milestone 5
Patch No

History
03/30/2010 07:49:22 PM Michael Slusarz Comment #6
Assigned to Michael Slusarz
State ⇒ Resolved
Reply to this comment
4.) Determine how to allow user to resort back to sequence sort.   
Call this sort "no sorting'.
Done.  Not entirely happy with the IMP UI - but there doesn't seem to 
be a better place to put it at the moment (the actions bar - delete, 
blacklist, etc. - all deal with message actions, not mailbox actions). 
  I'll let this stew in the back of my head for now.
03/26/2010 11:11:30 PM Michael Slusarz Comment #4 Reply to this comment

[Show Quoted Text - 10 lines]
These tasks have been completed.

03/26/2010 11:10:58 PM CVS Commit Comment #3 Reply to this comment
Changes have been made in Git for this ticket:

Ticket #8936: Simplified date sorting UI in IMP

http://git.horde.org/diff.php/imp/config/prefs.php.dist?rt=horde-git&r1=d82e56a0088330e5f77911a5a9aeb4d996c9b10d&r2=763bc2f73ce446d7c42ff21c2fdff51b27cbb807
http://git.horde.org/diff.php/imp/docs/CHANGES?rt=horde-git&r1=b52d7646b607b11d6ec24b306f764db6a4995595&r2=763bc2f73ce446d7c42ff21c2fdff51b27cbb807
http://git.horde.org/diff.php/imp/js/DimpBase.js?rt=horde-git&r1=8e71015c8f9e4ad9f6c772b73abab210ba7a3354&r2=763bc2f73ce446d7c42ff21c2fdff51b27cbb807
http://git.horde.org/diff.php/imp/lib/Block/Newmail.php?rt=horde-git&r1=205d6b608b229dbc485b89469d5a4039c58dcff0&r2=763bc2f73ce446d7c42ff21c2fdff51b27cbb807
http://git.horde.org/diff.php/imp/lib/IMP.php?rt=horde-git&r1=240ff73e2a3bcfe5e5649e5d9153d87b25471d36&r2=763bc2f73ce446d7c42ff21c2fdff51b27cbb807
http://git.horde.org/diff.php/imp/lib/LoginTasks/SystemTask/UpgradeFromImp4.php?rt=horde-git&r1=205d6b608b229dbc485b89469d5a4039c58dcff0&r2=763bc2f73ce446d7c42ff21c2fdff51b27cbb807
http://git.horde.org/diff.php/imp/lib/Mailbox.php?rt=horde-git&r1=d08804fd706e3cea9c3ea9aa9d9901b17dcf6fa9&r2=763bc2f73ce446d7c42ff21c2fdff51b27cbb807
http://git.horde.org/diff.php/imp/lib/Search.php?rt=horde-git&r1=12e455a324a5a44ca84c3d8939d50484eb9d9ad8&r2=763bc2f73ce446d7c42ff21c2fdff51b27cbb807
http://git.horde.org/diff.php/imp/mailbox-mimp.php?rt=horde-git&r1=a7a39f0cf109bc4fa2dec71b303d60d3d329746d&r2=763bc2f73ce446d7c42ff21c2fdff51b27cbb807
http://git.horde.org/diff.php/imp/mailbox.php?rt=horde-git&r1=55b00688ec61efc6d4b74eb5d7bd928ffb9a541a&r2=763bc2f73ce446d7c42ff21c2fdff51b27cbb807
http://git.horde.org/diff.php/imp/rss.php?rt=horde-git&r1=12e455a324a5a44ca84c3d8939d50484eb9d9ad8&r2=763bc2f73ce446d7c42ff21c2fdff51b27cbb807
http://git.horde.org/diff.php/imp/templates/dimp/javascript_defs_dimp.php?rt=horde-git&r1=55b00688ec61efc6d4b74eb5d7bd928ffb9a541a&r2=763bc2f73ce446d7c42ff21c2fdff51b27cbb807
http://git.horde.org/diff.php/imp/templates/mimp/mailbox/mailbox.html?rt=horde-git&r1=55b00688ec61efc6d4b74eb5d7bd928ffb9a541a&r2=763bc2f73ce446d7c42ff21c2fdff51b27cbb807
03/26/2010 10:14:29 PM Michael Slusarz Comment #1
Priority ⇒ 1. Low
Type ⇒ Enhancement
Summary ⇒ Clean up date/arrival/sequence sorting
Queue ⇒ IMP
Milestone ⇒ 5
Patch ⇒ No
State ⇒ Accepted
Reply to this comment
Date sort = date in message
Arrival sort = internal imap date of message
sequence sort = straight numeric sort of sequence numbers (order in 
which messages were added to the mailbox)

Sequence sort needs to be default because it is the only reasonable 
sort that can be done without parsing headers across all IMAP servers

Arrival sort in IMP 4.x referred to sequence sort - arrival sort was 
never defined in RFC 3501 but it makes sense that it is the arrival 
order into the mailbox.  Additionally, arrival/sequence sort (at least 
in IMP) had the sequence numbers which clearly indicates the sort was 
being done on sequence numbers (not that this means anything to the 
average user).  Then RFC 5256 goes and confuses thing but defining a 
SORT ARRIVAL which sorts by internaldate.

In IMP 5, to clean this up, do the following:
1.) Rewrite Horde_Imap_Client to handle sequence sort - obtained doing 
a SEARCH (not a SORT on imap servers that support the latter) and then 
doing numeric sorting on the client side.
2.) IMP defaults to sequence sort for same reason as in IMP 4.x.
3.) Only display one type of date search in mailbox headings.   
Determining whether this does a Header Date or an Internaldate search 
is left to a preference, locked by default.  By default, resort to 
Internaldate (since this tends to be more accurate than the Date 
header - see, e.g., Spam headers dated way in the future/past).
4.) Determine how to allow user to resort back to sequence sort.  Call 
this sort "no sorting'.

Saved Queries