[dev] [commits] Horde branch master updated. 3e57144addc0ccbc8358ab7cdf18204132b7455a

[Michael Rubinsky]

Quoting Michael Rubinsky <mrubinsk at horde.org>:

> Quoting Michael M Slusarz <slusarz at horde.org>:
>
>> Quoting Jan Schneider <jan at horde.org>:
>>
>>> commit 3e57144addc0ccbc8358ab7cdf18204132b7455a
>>> Author: Jan Schneider <jan at horde.org>
>>> Date:   Mon Jan 18 18:07:42 2010 +0100
>>>
>>>   Initialize IMP before authentication.
>>>
>>> imp/lib/Auth.php |    1 +
>>> 1 files changed, 1 insertions(+), 0 deletions(-)
>>>
>>> http://git.horde.org/diff.php/imp/lib/Auth.php?rt=horde-git&r1=b23cc31ca5d964fd8f9be807871eb0595aee63d9&r2=3e57144addc0ccbc8358ab7cdf18204132b7455a
>>
>> Where did you run into problems with this?  I haven't seen any  
>> issues with any use case I have tested.  Regardless, appInit() is  
>> not the proper solution.  It is meant to be a bootstrap shortcut -  
>> something that can be used to initialize an imp script - but  
>> IMP_Auth::authenticate() can be called from any Horde app.
>>
>> michael
>>
>
> Without this fix I am getting 'Notice: Undefined index:  
> authentication in /usr/local/horde/horde/imp/lib/Auth.php on line  
> 37' when logging in using Application/IMP authenticaion. I also get  
> that error the first time I access IMP if not using IMP  
> authentication.  I don't see anyplace in the code path that  
> Horde_Registry_Application::initParams is set...

... and nevermind, as I see you had just committed a fix for this  
prior to me sending this email :)


Thanks,
mike

--
The Horde Project (www.horde.org)
mrubinsk at horde.org

"Time just hates me. That's why it made me an adult." - Josh Joplin