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

[Jan Schneider]

Zitat von Michael M Slusarz <slusarz at bigworm.curecanti.org>:

> 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.

Ah, OK.

> 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.

I know that you're keeping an eye on BC, it was rather a suggestion to 
possibly make the code less confusing.

Jan.

-- 
Do you need professional PHP or Horde consulting?
http://horde.org/consulting/