[dev] [commits] Horde branch master updated. 533d8a65c27d5740e7b10f7758e57c6d847d2c39

[Michael M Slusarz]

Quoting Chuck Hagenbuch <chuck at horde.org>:

> Quoting Michael M Slusarz <slusarz at horde.org>:
>
>>> Why getInstance() and not factory()?
>>
>> AFAICT our pattern with having a base class (Horde_Ajax) and then  
>> having the individual drivers extend and abstract base class  
>> (Horde_Ajax_Base) have been using getInstance() - see, e.g.,  
>> Horde_Imap_Client.  But Chuck would probably be better positioned  
>> to explain whether this is a correct assumption.
>
> Everything using getInstance() is a lib you've written, actually.  
> I've been trying to avoid factories altogether where possible, and  
> stuck to factory() for things like Horde_Support::.

Bit confused: are you against factories as a design pattern or simply  
from a naming standpoint?

michael

-- 
___________________________________
Michael Slusarz [slusarz at horde.org]