« Return to Thread: Re: [commits] Horde branch master updated.

Re: [commits] Horde branch master updated.

by Jan Schneider :: Rate this Message:

Reply to Author | View in Thread

Zitat von Michael M Slusarz <slusarz@...>:

> Quoting Chuck Hagenbuch <chuck@...>:
>
>>>   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/



--
Horde developers mailing list - Join the hunt: http://horde.org/bounties/
Frequently Asked Questions: http://horde.org/faq/
To unsubscribe, mail: dev-unsubscribe@...

attachment0 (204 bytes) Download Attachment

 « Return to Thread: Re: [commits] Horde branch master updated.