Fix help text and action links to find the update manager pages
dww - October 12, 2009 - 17:44
| Project: | Drupal |
| Version: | 7.x-dev |
| Component: | update.module |
| Category: | task |
| Priority: | normal |
| Assigned: | dww |
| Status: | closed |
| Issue tags: | Update manager, Usability |
Description
Assuming #538660: Move update manager upgrade process into new authorize.php file (and make it actually work) lands, we should revisit how the action links to find the plugin manager update and install pages look and work. They're a bit cryptic at the moment. This issue can also be for cleaning up the help text on various pages that currently point to the available updates report, etc.
#1
Tagging for UX folks...
#2
I know people will want screenshots of this, but I have to run to a concert right now. However, here's a first-draft of the patch.
#3
Here are a bunch of screenshots of patch #2 in action. For each page, there is a copy with the Update manager automatic updates enabled, and a version with the killswitch set to disable the automatic upgrade feature. All screenshots are from a test site with core out of date and missing a contrib security update (which is the worst case in terms of how verbose the messages are).
1st is the /admin "Administer" landing page
2nd is the /admin/reports/status "Status report" page
3rd is the admin/config/modules "Configuration and modules" page
In all cases, if the auto-updates are enabled, all the links in the various messages take you to /admin/reports/updates/update where you can directly upgrade your site. If the killswitch is disabling the feature, they link to the /admin/reports/updates report page, just like how Update status works in D6.
The one exception is the help text on admin/config/modules which talks about Update manager:
Both of the links in there bring you to /admin/reports/updates. I think this text needs some help. First of all, it should know if Update manager is enabled or not and be smarter about what it says, instead of using this "if enabled" stuff. Secondly, if the update.module is disabled, it shouldn't print 404 links to non-existent report pages. Thirdly, if the auto-update feature is enabled, the help text could also link to that page and mention it exists (although directly below it are links to those features via the action items, so maybe it's not needed).
#4
Ok, this fixes system.module's help text at admin/config/modules about Update manager to be context-sensitive. There are 3 cases:
1) module is enabled and the killswitch is off (full functionality available)
2) module is enabled but the killswitch is on (no automatic updates possible)
3) module is disabled entirely
Probably a bit wordy, and I'm still not sure we need all the links about updating and installing in case #1 since we have the action links right below...
#5
#6
This looks good. Committed to CVS HEAD. Thanks.
#7
Great, thanks!
#8
Automatically closed -- issue fixed for 2 weeks with no activity.