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

[Michael Rubinsky]

Quoting Jan Schneider <jan at horde.org>:

> Zitat von Chuck Hagenbuch <chuck at horde.org>:
>
>> Quoting Jan Schneider <jan at horde.org>:
>>
>>> Zitat von Chuck Hagenbuch <chuck at horde.org>:
>>>
>>>> commit 08a466a242ffaf58dd1670a2162839af1450c7e3
>>>> Author: Chuck Hagenbuch <chuck at horde.org>
>>>> Date:   Sat Sep 11 23:33:06 2010 -0400
>>>>
>>>>  Add a factory for View objects
>>>>
>>>> framework/Core/lib/Horde/Core/Factory/View.php |   39  
>>>> ++++++++++++++++++++++++
>>>> framework/Core/lib/Horde/Registry.php          |   11 +++++-
>>>> framework/Core/package.xml                     |    2 +
>>>> 3 files changed, 50 insertions(+), 2 deletions(-)
>>>> create mode 100644 framework/Core/lib/Horde/Core/Factory/View.php
>>>
>>> I always added the required helpers manually so far, because there  
>>> are a lot of builtin helpers, which are unused most of the time  
>>> and still carried around if using addBuiltinHelpers().
>>
>> I'm open to this being a subset, but I think installing *some* of  
>> them by default in the default factory is very useful.
>
> Looks like Horde_View_Helper_Text is the only helper that's  
> currently used in client code.

FWIW, I know I've use the Tag helper as well in a few places.  Though  
I agree with Jan, about not always adding all built-in helpers. I've  
just been adding the one or two that I need in each view.

>
> Jan.
>
> -- 
> Do you need professional PHP or Horde consulting?
> http://horde.org/consulting/
>
>
> -- 
> Horde developers mailing list - Join the hunt: http://horde.org/bounties/
> Frequently Asked Questions: http://horde.org/faq/
> To unsubscribe, mail: dev-unsubscribe at lists.horde.org

-- 
Mike

--
The Horde Project (www.horde.org)
mrubinsk at horde.org

Mike

?The only reason for time is so that everything doesn't happen at  
once." - A. Einstein