In sshkey.module hook_menu, the item $menu['admin/user/sshkeys']'s access arguments array value should be 'administer sshkey' instead of 'administer sshkeys'. Or 'administer sshkey' in hook_perm should be 'administer sshkeys'.

CommentFileSizeAuthor
sshkey-menu-permission-typo-fix.patch509 bytesNick Robillard

Comments

Nick Robillard’s picture

Nick Robillard commented
Status: Active » Needs review
dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Needs review » Needs work

I think the permission should actually be renamed to 'administer sshkeys' to match the other permission.

Nick Robillard’s picture

Nick Robillard commented

Maybe - but the the module is called sshkey. What you're administering is the sshkey (singular) module in general - not individual ssh keys. No?

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Needs work » Fixed

Thanks! Fixed in CVS.
http://drupal.org/cvs?commit=472998

Status: Fixed » Closed (fixed)

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