[dev] [cvs] commit: imp/lib/IMAP Tree.php

Michael M Slusarz slusarz at mail.curecanti.org
Fri Aug 26 11:27:44 PDT 2005


Quoting Jan Schneider <jan at horde.org>:

> Zitat von Michael M Slusarz <slusarz at curecanti.org>:
>
>> slusarz     2005-08-25 21:33:01 PDT
>>
>>  Modified files:
>>    lib/IMAP             Tree.php
>>  Log:
>>  This breaks things when located here.  Move this code to IMAP_Tree::.
>
> How about BC? Wouldn't it be much cleaner if you implement two versions
> of IMAP_Tree and make the current IMAP_Tree only a proxy class that
> automatically selects the correct version depending on information it
> gets from IMP_Tree, ie whether it is extended from IMP 4.0 or 4.1?

This code was never in a stable release.  It was inserted a few days 
ago and then removed by me yesterday.  So there are no BC concerns.

And IMAP_Tree in Horde 3.1 should still be BC with IMP 4.0 (and vice 
versa) - I will go back and verify this when the code stabilize.  
Obviously the namespace issues will only fully be cleaned up if using 
IMP 4.1/Horde 3.1 but there's nothing I can do about that without 
breaking BC.  But IMAP_Tree 3.1 should continue to work fine using the 
old prefix/folder/namespace input rather than the new namespaces input.

michael

_______________________________________
Michael Slusarz [slusarz at curecanti.org]


More information about the dev mailing list