[dev] [commits] Horde branch master updated. 104e3198ce8b82e1a469dce43420d2d6fa7e2a7d
Michael M Slusarz
slusarz at horde.org
Thu May 27 20:17:56 UTC 2010
Quoting Michael Rubinsky <mrubinsk at horde.org>:
> Quoting Michael M Slusarz <slusarz at horde.org>:
>
>>
>> commit e433901b802136256a094ae1f870853e7f512e02
>> Author: Michael M Slusarz <slusarz at curecanti.org>
>> Date: Wed May 26 11:34:06 2010 -0600
>>
>> Refactor Horde_Db injectors.
>>
>> Forgot that individual subsystems could potentially define a custom SQL
>> configuration. Therefore, we need a Horde_Db injector with a
>> factory/singleton to override the base DB for these subsystems (and
>> reuse the base DB object if no custom config is given).
>>
>> Rename Horde_Db_Adapter_Base to Horde_Db_Base and change to a shortcut
>> to return the base Horde DB object.
>>
>
> This breaks being able to find other objects' Horde_Db dependency
> automatically using type hints. e.g., The various Content_Type_*
> classes depend on Horde_Db_Adapter_Base, but now the injector's
> dependency finder can no longer locate the binder since it has been
> renamed to Horde_Db_Base.
In other words Horde_Db_Adapter_Base just needs to be changed to
Horde_Db_Base there (?)
I really have no clue what is going on inside of 'content', and
documentation is very thin, so I was/am not comfortable with changing
anything in there.
michael
--
___________________________________
Michael Slusarz [slusarz at horde.org]
More information about the dev
mailing list