A change in the language for access control to use 'administer site configuration' instead of the old 'access configuration pages' caused the folowing menu errors:

bad permissions for 

admin/content/rss-feed
admin/user
admin/content

The fix is to patch system.module, node.module, and user.module.

Following a recent deve list discussion, I will attach a single patch and then 3 patches for the individual modules.

CommentFileSizeAuthor
#3 user_perms.patch695 bytesagentrickard
#2 system_perms.patch1.56 KBagentrickard
#1 node_perms.patch520 bytesagentrickard
admin_perms.patch2.77 KBagentrickard
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
CreditAttribution: agentrickard commented
FileSize
node_perms.patch520 bytes

Applies a one-line change to node_menu.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
CreditAttribution: agentrickard commented
FileSize
system_perms.patch1.56 KB

Applies a one-line change to system.module and moves the rss menu item into its own section of the code, separating it from the "// Settings" grouping.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
CreditAttribution: agentrickard commented
FileSize
user_perms.patch695 bytes

Applies a one-line patch to user.module

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Needs review » Needs work

Please merge the patches.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
CreditAttribution: agentrickard commented

The first attachment is the merged version.

http://drupal.org/files/issues/admin_perms.patch

:-) (I like to develop habits, so i split these up, even though it wasn't really worth it.)

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
CreditAttribution: agentrickard commented
Status: Needs work » Needs review
asimmonds’s picture

asimmonds CreditAttribution: asimmonds commented

Should the permissions here for /admin/content and /admin/user, actually be 'access administration pages' instead of 'administer site configuration'. From what I can see 'administer site configuration' is just /admin/settings and /admin/build.
The menu path /admin has the 'access administration pages' permission set in system.module.

Kjartan’s picture

Kjartan CreditAttribution: Kjartan commented
Status: Needs review » Fixed

This seems to be fixed already as I can't find any references to "access configuration pages" in HEAD.

Anonymous’s picture

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