Any reason this is not listed in the CAPABILITY string?
I see it listed as a TODO here: http://wiki2.dovecot.org/Roadmap
But it seems to work fine (as of 2.2.12):
1 UID SORT RETURN (CONTEXT COUNT) (SUBJECT) UTF-8 UNDELETED
- ESEARCH (TAG "1") UID COUNT 12 1 OK Sort completed (0.000 secs).
2 UID SORT RETURN (UPDATE COUNT) (SUBJECT) UTF-8 UNDELETED
- ESEARCH (TAG "2") UID COUNT 12 2 OK Sort completed (0.000 secs). [Add message to mailbox] 3 NOOP
- 13 EXISTS
- 1 RECENT
- ESEARCH (TAG "2") UID ADDTO (0 62475) 3 OK NOOP completed.
4 CANCELUPDATE "2" 4 OK Updates cancelled.
5 UID SORT RETURN (PARTIAL 1:10) (SUBJECT) UTF-8 UNDELETED
- ESEARCH (TAG "5") UID PARTIAL (1:10 NIL) 5 OK Sort completed (0.000 secs).
michael
Quoting Michael M Slusarz <slusarz@curecanti.org>:
Well duh. Maybe I should actually look at the results. This is
obviously wrong (this mailbox has 13 undeleted messages, so NIL is not
a proper return).
...although PARTIAL limiting for SEARCH doesn't work either:
4 UID SEARCH RETURN (PARTIAL 1:10) UNDELETED
- ESEARCH (TAG "4") UID PARTIAL (1:10 NIL) 4 OK Search completed (0.000 secs).
Undeleted messages do exist:
5 UID SEARCH UNDELETED
- SEARCH 58302 59466 61309 61311 62430 62431 62432 62450 62452 62454
62456 62458 5 OK Search completed (0.000 secs).
so maybe there is a general issue with PARTIAL matching rather than a
CONTEXT=SORT specific issue? (Running the Arch Linux Dovecot 2.2.12
package right now so I'm not able to debug).
michael
On 3.5.2014, at 8.51, Michael M Slusarz <slusarz@curecanti.org> wrote:
Ugh. This was slightly broken in earlier versions, but in v2.2.11 I broke it completely by misunderstanding what it was supposed to do. And after now fixing it nicely for SEARCH PARTIAL I realize that it's again broken for SORT PARTIAL. So, back to the original code with the correct minor fix..: http://hg.dovecot.org/dovecot-2.2/rev/32b6a95c95cc
I also updated imaptest to check for the SEARCH PARTIAL.
Oh and the reason why CONTEXT=SORT isn't advertised is because SORT RETURN (UPDATE) doesn't work.
participants (2)
-
Michael M Slusarz
-
Timo Sirainen