[dev] Re: [cvs] commit: imp folders.php imp/lib VFolder.php imp/templates/folders vfolder_actions.inc vfolder_foot.inc vfolder_head.inc vfolders.html

[Jan Schneider]

Zitat von Chuck Hagenbuch <chuck at horde.org>:

> Quoting Jan Schneider <jan at horde.org>:
>
>> I agree with Roel (as you already know).
>>
>> But given the moment when this feature was introduced to IMP, I'd suggest
>> keeping the virtual folders separate from the general folder code for now
>> and integrating them into the main folder tree with IMP 4.1.
>
> I disagree. I can see backing out the changes for stability concerns,
> but if the
> feature is going to be present, I think it should be as intuitive and
> useable as
> possible, which to me means treating them as folders aside from the 
> indication
> that they're virtual.

In this case I'm for reverting these changes and re-apply them to HEAD once
we have the FRAMEWORK_3 branch.

Jan.

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