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

[Michael Rubinsky]

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...



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