The module "Administration menu" (2011-Oct-16) ignores in Drupal 7 the "administer software updates" permission as shown on http://drupal.org/upgrade/running-update-php in the lines before the header "II. Run update.php"

Only the user with UID equal 1 has then menu item "Run Updates".

CommentFileSizeAuthor
#3 admin_menu.update-php-perm.3.patch624 bytessun
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

wusel’s picture

wusel CreditAttribution: wusel commented

I am not a PHP-coder.

How can we chance this?

Please help! Thank you very much!

wusel’s picture

wusel CreditAttribution: wusel commented

The lines in the link http://drupal.org/upgrade/running-update-php from December 5, 2011 (the date of this issue):

Note: To run update.php in Drupal 7, you need to log in with an account that has the "administer software updates" permission enabled.

sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun commented
Title: Missing the "Run Updates" option for admin with UID > 1 » "Run updates" does not appear despite "administer software updates" permission
Status: Active » Fixed
FileSize
admin_menu.update-php-perm.3.patch624 bytes

Thanks for reporting, reviewing, and testing! Committed attached patch.

A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.

wusel’s picture

wusel CreditAttribution: wusel commented

Thank you very much.
This is very nice.

wusel’s picture

wusel CreditAttribution: wusel commented

You can also show the UID=1-color at the username by adding/changing

if ( ($user->uid == 1) || user_access('administer software updates'))

in the file admin_menu.module at line 157.

wusel’s picture

wusel CreditAttribution: wusel commented
Status: Fixed » Active

Administration menu 7.x-3.0-rc1+55-dev (2012-Mar-09)

sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun commented
Status: Active » Fixed

The bug mentioned in this issue has been fixed.

Changing the color is a different issue, so you may create one for that. However, be prepared for won't fix, since the red color is explicitly supposed to signify and warn about being logged in as uid 1 only.

wusel’s picture

wusel CreditAttribution: wusel commented

Ok, I said can.

Thanks for this great module. I hope, it goes into core of D8.

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

(not verified) CreditAttribution: commented
Issue summary: View changes

I add the line of the D7-dokumentation