Access to the settings page is controlled by the "Administer comment_notify" permission, but there's no hook_perm() declaring the permission. The module should either implement hook_perm() (I don't think that's necessary) or remove the access check from hook_menu().

CommentFileSizeAuthor
#2 rename_add_perm_222740.patch1.13 KBgreggles

Comments

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented

Which way do you think it should go? I'd be happy to just leave it with "administer site configuration" level permissions.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Active » Needs review
StatusFileSize
new rename_add_perm_222740.patch1.13 KB

Here's a patch which renames the permission and actually declares it.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Needs review » Fixed

Given the feedback in http://drupal.org/node/225533 I committed this to the 5.x branch.

junyor’s picture

junyor commented

Sorry about not getting back to you! Apparently I wasn't subscribed to Comment Notify issues properly.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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