Hi again,
--Ingo Herz wrote:
>>>The last problem for now was an endless loop in
>>>midcom/lib/midcom/helper/_basicnav.php For some reason it thinks, that
>>>the root topic has viewergroups defined (they are NOT set) and it can not
>>>be shown. After that it loops until max execution time. This has also
>>>been reported to the bug tracker (issue 248).
>>>My solution was to add the 2nd condition to the if-statement:
>>>
>>> if ((! is_null($napdata[MIDCOM_NAV_VIEWERGROUPS])) &&
>>> (count($napdata[MIDCOM_NAV_VIEWERGROUPS]) <> 0))
>>> {
>>> $visible = false;
>>> foreach ($napdata[MIDCOM_NAV_VIEWERGROUPS] as $guid)
>>> {
>>> if ($GLOBALS["midcom"]->check_memberships($guid))
>>> {
>>> // User is member of a viewer group
>>> $visible = true;
>>> }
>>> }
>>>
>>>But I really don't know what is going on with $napdata[]...
>>
>>Did you get any more errors? The piece of code you showed above should not
>>loop indefinitly, and #248 is fixed afair. Alone with this information, I
>>cannot judge what is happening here.
>>
> fished a part out of the logfile:
>
> Oct 19 14:39:31 [error] _basicnav::_loadNode: Node -1 is not in the MidCOM
> conte
> nt tree 32. Aborting
> Oct 19 14:39:31 [error] _basicnav::_loadNode: Node -1 is not in the MidCOM
> conte
> nt tree 32. Aborting
> Oct 19 14:39:31 [error] _basicnav::_loadNode: Node -1 is not in the MidCOM
> conte
> nt tree 32. Aborting
> Oct 19 14:39:31 [error] _basicnav::get_node_uplink: Parameter passed is no
> integ
> er: []
> Oct 19 14:39:31 [error] _basicnav::get_node: Parameter passed is no integer:
> []
> Oct 19 14:39:31 [error] _basicnav::get_node_uplink: Parameter passed is no
> integ
> er: []
> Oct 19 14:39:31 [error] _basicnav::get_node: Parameter passed is no integer:
> []
> Oct 19 14:39:31 [error] _basicnav::get_node_uplink: Parameter passed is no
> integer: []
>
> The last 2 lines are repeated until max exectime. No errors before this part.
> For me it looks like #248...
Very probable this goes back to the same roots, yes. For a start it looks like a
corrupt NAP record, but I am still unable to reproduce this. For now I'm
enforcing the VIEWERGROUPS member to be an Array. This *should* fix the problem,
in theory at least, but naturally this is one more "untested" fix. I have just
commited it to the 2.4 branch into CVS.
Live long and prosper!
Torben Nehmer
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
