I think line 53 should be:

return array('administer user alert', 'create user alert', 'edit any user alert', 'delete any user alert');

Instead of:

return array('administer user alert');

Comments

kevinquillen’s picture

kevinquillen commented

I initially had that, but I was told in CVS application that that would not be necessary. Do those permissions not show up?

kevinquillen’s picture

kevinquillen commented
Assigned: Unassigned » kevinquillen
Anonymous’s picture

Anonymous (not verified) commented

They don't show up for me. Adding the other permissions to the array adds them to the permissions screen. Admittedly, I don't have a ton of experience building modules from scratch so I'm not entirely certain that the above code is completely correct.

kevinquillen’s picture

kevinquillen commented

I mean, without them defined in the code, they do not show up in the node permissions list? It was a last minute change and I was told it wasn't necessary. I will test this out and if true, I will add them back for a future update.

Anonymous’s picture

Anonymous (not verified) commented

Correct, without the permissions defined in the code, they aren't showing up for me in the permissions section of the admin area.

kevinquillen’s picture

kevinquillen commented

Okay, I will make the fix. Thanks.

kevinquillen’s picture

kevinquillen commented

Added in latest release.

kevinquillen’s picture

kevinquillen commented
Status: Active » Fixed

Status: Fixed » Closed (fixed)

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