[dev] [commits] Horde branch master updated. f61bc298362194bab0878bacc85b504798898200

Michael M Slusarz slusarz at horde.org
Thu Aug 22 17:16:17 UTC 2013


Quoting "Michael J. Rubinsky" <mrubinsk at horde.org>:

> commit 8a55b3d7e36c2615fd6f7d474a8eaae76f77c716
> Author: Thomas Jarosch <thomas.jarosch at intra2net.com>
> Date:   Thu Aug 22 10:48:19 2013 +0200
>
>     Simplify CONDSTORE/non-CONDSTORE code paths.
>
>     Horde_Imap_Client already does the abstraction for us.
>
>     Also save the total amount of messages in an IMAP folder
>     for all server types again, it's not worth the effort / complexity
>     to create seperate code paths for this.
>
>     Request: 12597
>
>  framework/ActiveSync/lib/Horde/ActiveSync/Imap/Adapter.php |   59  
> +++++-------
>  1 files changed, 23 insertions(+), 36 deletions(-)
>
> http://git.horde.org/horde-git/-/commit/8a55b3d7e36c2615fd6f7d474a8eaae76f77c716

Not super clear what is going on here.  But if you are trying to do  
something like identifying when the status of the mailbox has changed,  
all you need is Horde_Imap_Client_Base#getSyncToken().

It handles things like the presence of CONDSTORE transparently and  
uses other methods if not available.

michael

___________________________________
Michael Slusarz [slusarz at horde.org]



More information about the dev mailing list