[dev] Some more Agora stuff

Michael Rubinsky mike at theupstairsroom.com
Thu Dec 21 09:47:59 PST 2006


Quoting Michael Rubinsky <mike at theupstairsroom.com>:

>
>
> While I'm here digging around in Agora, I've come across some other   
>  things that I'm not quite sure how to fix.


Pretty sure I've resolved these in CVS.





>
> There's an issue with Agora_Messages::hasPermission.  It uses the    
> $this reference, but it's called statically by (at least)    
> Agora::getMenu() and there is at least one case where I have found    
> that causes it to fatal. The problem is if a user navigates to    
> hostname/horde/agora directly,  as a guest.  The call in    
> Agora::getMenu() to Agora_Messages::hasPermission() passes $scope as  
>   the second parameter (it should be the third...forum_id should be  
> the   second).  I was going to fix it, but I'm not familiar enough  
> with   Agora yet to know what would be a reasonable value here for  
> forum_id?
>
> I've noticed this parameter mis-match in a number of places in the    
> code, but again, I'm not always 100% clear where the proper value    
> should be coming from.  Basically, just grep'ed for ->hasPermission  
> to   find them.
>
>
> Thanks,
> Mike
>
> "Am I getting old, or has the supermarket been playing great music lately?"



Thanks,
mike

--
The Horde Project (www.horde.org)
mrubinsk at horde.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-keys
Size: 2013 bytes
Desc: PGP Public Key
Url : http://lists.horde.org/archives/dev/attachments/20061221/f19ec4e9/attachment.bin


More information about the dev mailing list