A permission seemed to be missing from hook_perm(), which was referenced in hook_menu, meaning only users with a uid of 1 could administer the module.

See patch. Also a small typo fix.

CommentFileSizeAuthor
media.module_add_admin_perm.patch1.01 KBpfaocle

Comments

pfaocle’s picture

pfaocle
He/Him
Primary language English
commented

Oops, just read the comment above the _perm function. Sorry if the above is redundant!

robertdouglass’s picture

robertdouglass commented

Actually the permissions are a mess. Thanks for the patch.

moshe weitzman’s picture

moshe weitzman commented
Status: Needs review » Fixed
Anonymous’s picture

(not verified) commented
Anonymous’s picture

(not verified) commented
Anonymous’s picture

(not verified) commented
Anonymous’s picture

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