[dev] [commits] Horde branch master updated. 71c1d5b996bac5e3df65edb1aad2b11ca573a0cb

[Jan Schneider]

Zitat von Michael M Slusarz <slusarz at horde.org>:

> Quoting Jan Schneider <jan at horde.org>:
>
>> Zitat von Michael M Slusarz <slusarz at horde.org>:
>>
>>> Final note: A gigantic chunk of horde/Alarm relies on horde/Core.   
>>> We should probably deal with that before the stable releases are  
>>> finalized (?)
>>
>> With gigantic chunk you mean the load() method? :) That's the only  
>> place I see that uses Core. We've obviously overseen that during  
>> the refactoring. Or am I missing anything else?
>
> Maybe not "gigantic chunk"... :)  But Horde_Alarm_Handler_Notify  
> also has Core dependencies:
>
>         $this->_notification = isset($params['notification']) ?  
> $params['notification'] :  
> $GLOBALS['injector']->getInstance('Horde_Notification');

Ah yes, that's the circular dependency of Horde_Alarm and  
Horde_Notification that hasn't been solved after adding lazy  
instantiation to the injector.

Jan.

-- 
Do you need professional PHP or Horde consulting?
http://horde.org/consulting/