Quantcast
Channel: Issues for Drupal core
Viewing all articles
Browse latest Browse all 296136

Add permissions to the update.module to hide warnings

$
0
0

Problem/Motivation

Currently the update notification ("Update available") is shown to all users with the permission "administer site configuration", while there is already a permission "administer software updates" which is not used for that.

This leads to the issue that users who are not allowed to update see this message. In many cases, this are customers who are then afraid as they don't understand what they can (not) do and what are the consequences.

Steps to reproduce

Use a Drupal project with available updates and "administer site configuration" permission, but without "administer software updates" permission and be confused ;)

Proposed resolution

  1. Add a new "view update notifications" permission to Update Manager.
  2. Only show update notifications to users with that permission.
  3. Add an upgrade path to grant this permission to all roles that currently see the notifications (because they have the administer site configuration permission).

Remaining tasks

  1. Reviews / refinements.
  2. RTBC.

User interface changes

Update Manager notifications about available updates at the top of most /admin/* pages will only appear for users with the new view update notifications permission.

API changes

None.

Data model changes

Nope.

Release notes snippet

The Update Manager will only print notifications at the top of most /admin/* pages for users with the new view update notifications permission. A post update hook is provided that adds this permission to all roles that have the administer site configuration (which was previously used to decide who should see the notifications).

Credits

Huge THANKS to @voleger who did a great job in the both other issues to create the patches!

Alternative approaches

There are three different approaches to this problem. Both the UX team and the Update Manager subsystem maintainer (dww) agree that this issue (#332796) is the right solution, so the alternatives are now closed:


Viewing all articles
Browse latest Browse all 296136

Trending Articles



<script src="https://jsc.adskeeper.com/r/s/rssing.com.1596347.js" async> </script>