Problem/Motivation
If you have a site with an obsolete extension that is still installed, the status report currently says:
Obsolete extensions found: %extensions. Obsolete extensions are provided only so that they can be uninstalled cleanly. You should immediately uninstall these extensions since they may be removed in a future release.
#3266308: %extensions placeholder not extension names printed on the Status report warning about obsolete extensions will fix the bug that the %extensions placeholder isn't getting the values replaced. But it leaves the problem that site owners won't necessarily know how to fix this problem and silence the warning.
Steps to reproduce
- Start with a Drupal core 9.4.x test site
- Create a new module test and install it, you can use the following
.info.ymlfile:
name: 'Test module' type: module description: 'Test module' package: Testing version: VERSION core_version_requirement: ^8 || ^9 - Go to
/admin/modulesand install the module - Change the info file and add the lifecycle obsolete:
lifecycle: obsolete lifecycle_link: 'https://example.com/obsolete' - Rebuild the cache
- Go to
admin/reports/statusand check the warnings
Proposed resolution
Convert the words "uninstall these extensions" into a link to the Uninstall form at /admin/modules/uninstall.
Remaining tasks
- Decide if we need to be fancy about if it's only an obsolete theme (?!?) or if it's cool to always link to the modules Uninstall form.
- Land #3266308: %extensions placeholder not extension names printed on the Status report warning about obsolete extensions
- Add the link
- Update test coverage
- Reviews/refinements
- RTBC
User interface changes
The warning on the status report when a site has an obsolete extension installed includes the words "uninstall these extensions" which is now a link to the Uninstall page at /admin/modules/uninstall.
Before

After

API changes
None.
Data model changes
None.
Release notes snippet
TBD, probably not.
| Comment | File | Size | Author |
|---|---|---|---|
| #11 | Screen Shot 2023-02-03 at 6.09.19 PM.png | 61.61 KB | smustgrave |
| #11 | Screen Shot 2023-02-03 at 6.09.12 PM.png | 58.23 KB | smustgrave |
| #6 | 3266449-6.patch | 2.66 KB | quietone |
| #6 | 3266449-6-fail.patch | 1.56 KB | quietone |
Comments
Comment #2
dwwThis includes the fix from #3266308: %extensions placeholder not extension names printed on the Status report warning about obsolete extensions and will need tests before it's commit-able, but here's a start (from @murilohp).
Comment #4
dwwComment #5
dwwYay, blocker is in. Here's a re-roll of #2 for 9.4.x branch. Still no new tests, so no sense having the bot waste cycles on this.
Comment #6
quietone commentedIs this sufficient for a test?
Comment #7
dwwWFM.😉 Thanks!
Comment #11
smustgrave commentedTested this on Drupal 10.1 with a standard install
Installed admin_toolbar
Added
Went to status report verified I saw the message
Applied patch #6
Verified uninstall these extensions was now linked.
Added before/after screenshots to IS.
Comment #13
catchCommitted 999e16a and pushed to 10.1.x. Thanks!