[dev] [commits] Horde branch master updated.

[Jan Schneider]

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

> Quoting Chuck Hagenbuch <chuck at horde.org>:
>
>>>   Horde core framework package import into Git from CVS HEAD
>>>
>>>   Named core framework package Core - makes more sense to me then
>>>   framework since technically, Core is part of the framework.
>>>   Also, split out Release into a separate package since 99.9% of users
>>>   don't need this package.
>>
>> I agree about framework not really making sense, but if the package  
>> is Core, shouldn't the classes be named Horde_Core_*?
>
> IMHO, no.  As for Horde:: - we have app-specific "master" utility  
> libraries called IMP::, Turba::, etc. that naming it Horde_Core::  
> doesn't make sense.  And I think at this point moving from something  
> like Registry:: -> Horde_Core_Registry:: is too much overkill.
>
> The package that contains the core files needs to be named  
> *something* - I don't think we have to incorporate that name in the  
> class names robotically.

Despite of what I wrote earlier, I think I agree with Michael here.

Jan.

-- 
Do you need professional PHP or Horde consulting?
http://horde.org/consulting/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: Digitale PGP-Unterschrift
URL: <http://lists.horde.org/archives/dev/attachments/20090712/b82026a1/attachment.bin>