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

[Jan Schneider]

Zitat von Ben Klang <ben at alkaloid.net>:

>
> On Jan 26, 2010, at 5:52 PM, Michael M Slusarz wrote:
>
>> 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.
>>
> The reason I did this in both Shout and Operator was that I needed  
> to include data from the backend in the available permissions nodes.  
>  From what I can tell, the application was not getting initialized  
> when the Perms interface was being loaded, which I don't think I  
> have control of within my application.  What do I need to do to make  
> sure my Perms methods have access to an initialized driver if not  
> call appInit()?  I may not understand yet, but I thought that the  
> app would not be pushed during operations like API calls like this.

I also see issues with the portal blocks, it looks like the apps are  
not initialized there. You can see this with Kronolith blocks for  
example, where some global variables are not defined.

Jan.

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