I want the advanced help for site admins but not for regular users, so I don't want regular users to get the "Help" tab when viewing search results.

Is this the right way to do this?

function advanced_help_search($op = 'search', $keys = null) {
  switch ($op) {
    case 'name':
      if (user_access ('view advanced help index')) {
        return t('Help');
      }
...
CommentFileSizeAuthor
#1 advanced_help_search_menu.patch392 bytesblackdog

Comments

blackdog’s picture

blackdog commented
Category: feature » bug
Status: Active » Needs review
StatusFileSize
new advanced_help_search_menu.patch392 bytes

Yes, that seemes to do it. Attached a patch.

scottrigby’s picture

scottrigby
he/him
Primary language English
Location Brooklyn, NY
commented

#1 works for me :)

Anonymous’s picture

Anonymous (not verified) commented

Subscribe.

Shai’s picture

Shai commented

I had the same problem. I applied the patch written by magnusk and rolled by blackdog as attached to #1.

The patch applied cleanly and fixed the problem.

Thanks to magnusk for reporting and proposing a solution, to blackdog for this rolling efforts and to merlinofchaos for writing this module.

Shai
Content2zero

rootwork’s picture

rootwork
he/him
Primary language English
Location 🇺🇸 US-Pacific 🌎 Chinook (Multnomah, Clackamas) & Cowlitz lands 🌹 Portland, OR
commented
Status: Needs review » Reviewed & tested by the community

Patch applied cleanly, and addresses the problem. Thanks magnusk and blackdog!

pasqualle’s picture

pasqualle
Location 🇭🇺 Budapest
commented

This is already fixed in the -dev release. #325422: Advanced help can be searched as long as user has permission to "search content"

Anonymous’s picture

Anonymous (not verified) commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.