[Tickets #3794] Show only menu items with PERMS_SHOW

[bugs@bugs.horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/?id=3794
-----------------------------------------------------------------------
 Ticket             | 3794
 Updated By         | Chuck Hagenbuch <chuck at horde.org>
 Summary            | Show only menu items with PERMS_SHOW
 Queue              | Wicked
-State              | New
+State              | Feedback
 Priority           | 1. Low
 Type               | Enhancement
 Owners             | 
-----------------------------------------------------------------------


Chuck Hagenbuch <chuck at horde.org> (2006-04-19 08:55) wrote:

My initial reaction was that this makes perfect sense. However, it's
actually inconsistent with how things work inside pages themselves -
wikiwords just get linked, we don't hide the link somehow if it's to a page
there aren't permissions for - and the top-level page links are to pages
that generally aren't going to be restricted. Unless wicked itself is
restricted, which should take care of that.

Given all that, I'm not sure it's worth the permissions check calls on every
page to do this. Also, I'd think it should be PERMS_READ; if we're going to
do it, may as well not show them links they won't be able to use, even if
technically PERMS_SHOW means they should see that the page exists.

I'm open to convincing on all of this.