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

Chuck Hagenbuch chuck at horde.org
Mon Jul 20 16:25:00 UTC 2009


Quoting Michael M Slusarz <slusarz at horde.org>:

>>> http://git.horde.org/co.php/framework/Ajax/lib/Horde/Ajax/Imple.php?rt=horde-git&r=533d8a65c27d5740e7b10f7758e57c6d847d2c39
>>> http://git.horde.org/co.php/framework/Ajax/lib/Horde/Ajax/Imple/Base.php?rt=horde-git&r=533d8a65c27d5740e7b10f7758e57c6d847d2c39
>>> http://git.horde.org/co.php/framework/Ajax/package.xml?rt=horde-git&r=533d8a65c27d5740e7b10f7758e57c6d847d2c39
>>
>> 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::.

-chuck


More information about the dev mailing list