Ticket #4869 (closed defect (bug): fixed)

Opened 1 year ago

Last modified 1 year ago

Core update notification is displayed for all user levels

Reported by: johnbillion Assigned to: anonymous
Priority: normal Milestone: 2.3
Component: Administration Version: 2.3
Severity: normal Keywords: dev-reviewed
Cc:

Description

Every user of the admin panel sees the core update notification if there is an update available. It should only be displayed to users who have a sufficient capability. Not quite sure what though, maybe switch_themes?

Attachments

4869.diff (0.5 kB) - added by johnbillion on 08/30/07 11:13:14.
Only display core update notification to users with the switch_themes capability
4869-footer.diff (496 bytes) - added by johnbillion on 08/30/07 11:25:00.
Apply same logic for the admin footer
4869.2.diff (1.4 kB) - added by johnbillion on 08/30/07 16:53:21.
Replaces 2 previous patches, Check for manage_options capability, advise unpriv'd users to notify site admin, fix indentation

Change History

08/30/07 11:13:14 changed by johnbillion

  • attachment 4869.diff added.

Only display core update notification to users with the switch_themes capability

08/30/07 11:13:43 changed by johnbillion

Patch to only display core update notification to users with the switch_themes capability.

08/30/07 11:25:00 changed by johnbillion

  • attachment 4869-footer.diff added.

Apply same logic for the admin footer

08/30/07 11:25:43 changed by johnbillion

Patch for footer too. Just display the version number.

08/30/07 13:57:27 changed by Viper007Bond

manage_options is another option.

(follow-up: ↓ 6 ) 08/30/07 14:43:28 changed by markjaquith

What if the administrator infrequently logs in? Authors who don't have FTP/SSH access could tell the administrator of the message, hastening the upgrade. We could to the manage_options test and to unpriv'd users say "Please notify the site administrator"

08/30/07 14:55:31 changed by foolswisdom

  • priority changed from high to normal.
  • milestone set to 2.3.

08/30/07 16:53:21 changed by johnbillion

  • attachment 4869.2.diff added.

Replaces 2 previous patches, Check for manage_options capability, advise unpriv'd users to notify site admin, fix indentation

(in reply to: ↑ 4 ) 08/30/07 16:54:17 changed by johnbillion

Replying to markjaquith: Excellent point Mark. Patch refreshed.

08/30/07 16:59:23 changed by westi

  • keywords set to dev-reviewed.

+1 This looks good.

08/30/07 18:21:03 changed by markjaquith

  • status changed from new to closed.
  • resolution set to fixed.

(In [5994]) Check manage_options for upgrade notification. Tailor message accordingly. Props johnbillion. fixes #4869