[dev] [commits] Horde branch master updated. f5848a08a27b54dbc34f8786956fe65141ddddce

Michael M Slusarz slusarz at horde.org
Wed Jan 12 16:50:48 UTC 2011


Quoting Michael Rubinsky <mrubinsk at horde.org>:

> Quoting Michael M Slusarz <slusarz at horde.org>:
>
>> Quoting Michael M Slusarz <slusarz at horde.org>:
>>
>>> commit f5848a08a27b54dbc34f8786956fe65141ddddce
>>> Author: Michael M Slusarz <slusarz at curecanti.org>
>>> Date:   Wed Jan 12 01:05:44 2011 -0700
>>>
>>>   Convert Turba_Driver_Sql to using Horde_Db
>>>
>>>   Config file has changed - if non-Horde SQL DB used, it must be created
>>>   and passed into the 'sql' params value. By default, the Horde SQL DB is
>>>   used.
>
> Unless I'm missing something, there's no need for the  
> Turba_Injector_Binder_Driver:: class. You should request the  
> Turba_Injector_Factory_Driver object directly, i.e.
>
> $driver =  
> $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->getDriver($source);

I have no idea.  I just directly changed the code where the Db object  
gets initialized within the Sql driver - I didn't change any injection  
access.  FWIW, the PEAR DB object was not being injected.

michael

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




More information about the dev mailing list