Skip to content

Commit

Permalink
[mms] Fix client-side sequence sorting, when that is the only sort cr…
Browse files Browse the repository at this point in the history
…iteria.
  • Loading branch information
slusarz committed Aug 18, 2014
1 parent 1d004c3 commit 443f230
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -84,6 +84,10 @@ public function clientSort($res, $opts)
$query->envelope();
break;

case Horde_Imap_Client::SORT_SEQUENCE:
$query->seq();
break;

case Horde_Imap_Client::SORT_SIZE:
$query->size();
break;
Expand Down
2 changes: 2 additions & 0 deletions framework/Imap_Client/package.xml
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@
</stability>
<license uri="http://www.horde.org/licenses/lgpl21">LGPL-2.1</license>
<notes>
* [mms] Fix client-side sequence sorting, when that is the only sort criteria.
* [mms] Use Collator class, if present, to sort string fields on the client side.
* [mms] Fix client-side sorting of display addresses.
* [mms] Even more improved pseudo-UIDNEXT value for POP3 servers.
Expand Down Expand Up @@ -2644,6 +2645,7 @@
<date>2014-08-18</date>
<license uri="http://www.horde.org/licenses/lgpl21">LGPL-2.1</license>
<notes>
* [mms] Fix client-side sequence sorting, when that is the only sort criteria.
* [mms] Use Collator class, if present, to sort string fields on the client side.
* [mms] Fix client-side sorting of display addresses.
* [mms] Even more improved pseudo-UIDNEXT value for POP3 servers.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -90,6 +90,11 @@ public function testClientSortProvider($sort, $expected, $locale)
public function clientSortProvider()
{
return array(
array(
array(Horde_Imap_Client::SORT_SEQUENCE),
range(1, 9),
false
),
array(
array(Horde_Imap_Client::SORT_ARRIVAL),
array(
Expand Down

0 comments on commit 443f230

Please sign in to comment.