[dev] [commits] Horde branch master updated. 7080e7ce1baa9b108fba10489cda8716d76a2053

Michael M Slusarz slusarz at horde.org
Tue Jan 26 22:52:57 UTC 2010


Quoting Michael M Slusarz <slusarz at horde.org>:

> Quoting Ben Klang <ben at alkaloid.net>:
>
>> commit 7080e7ce1baa9b108fba10489cda8716d76a2053
>> Author: Ben Klang <ben at alkaloid.net>
>> Date:   Tue Jan 26 17:21:08 2010 -0500
>>
>>    Shout: Continue fixes from conversion to new H_R#appInit()
>>
>> shout/lib/Ajax/Application.php      |    8 +++++++-
>> shout/templates/extensions/list.inc |    8 ++++++++
>> shout/themes/screen.css             |    4 ++++
>> 3 files changed, 19 insertions(+), 1 deletions(-)
>>
>> http://git.horde.org/diff.php/shout/lib/Ajax/Application.php?rt=horde-git&r1=13879630963e68a2bfe3f85e37082bd6d5ac6631&r2=7080e7ce1baa9b108fba10489cda8716d76a2053
>
> See my previous message re: init().  Additionally, appInit() should  
> NEVER be called other than for bootstrapping reasons (i.e. in the  
> base script to initialize the Horde environment).  Any usage of  
> appInit() in a library/function is incorrect.
>
> The error to be tracked down if this isn't working properly is why  
> the application's environment is not yet setup.  But one of the  
> whole ideas of the new Registry code is that manual initialization  
> of the app environment should never have to be done (outside of code  
> in _init()).

FYI: I'm planning on writing documentation on how the new  
Horde_Registry_Application stuff is supposed to work - I've just been  
waiting to make sure the API is stable and any major bugs/design flaws  
are hashed out.

michael

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



More information about the dev mailing list