Make WordPress Core

Opened 6 years ago

Closed 6 years ago

Last modified 5 years ago

#46060 closed defect (bug) (invalid)

WSOD protection: admin notices not displaying on network admin screens for paused network activated plugins.

Reported by: pbiron's profile pbiron Owned by:
Milestone: Priority: normal
Severity: normal Version: 5.1
Component: Site Health Keywords: servehappy has-patch needs-testing
Focuses: administration Cc:

Description

When a network activated plugin is paused, the plugin shows as such in /wp-admin/network/plugins.php but the "One or more plugins failed to load properly" admin notice does not display on the other network admin screens (like it does on "normal" admin screens).

Attachments (3)

46060.diff (642 bytes) - added by afragen 6 years ago.
46060.2.diff (1.7 KB) - added by pbiron 6 years ago.
46060.3.diff (1.6 KB) - added by flixos90 6 years ago.

Download all attachments as: .zip

Change History (13)

#1 @pento
6 years ago

  • Milestone changed from Awaiting Review to 5.1
  • Version set to trunk

@afragen
6 years ago

#2 @afragen
6 years ago

  • Keywords has-patch needs-testing added

@pbiron
6 years ago

#3 follow-up: @pbiron
6 years ago

Hey Andy, thanx for the patch.

46060.2.diff does 2 things:

  1. moves the add_action( 'network_admin_notices', 'paused_plugins_notice', 5 ); from your patch up a few lines to group it with the other "Notices Hooks"
  2. has the link in the notice go to the network plugins screen when viewed from a network admin screen

There is another thing I noticed when testing this patch that probably deserves its own ticket, but I'll mention it here:

  • if a network active plugin is paused and an admin goes to the plugins screen for one of the sites in the network (not the network plugins screen), the "Resume" link for the network paused plugin doesn't actually resume the plugin. I'm too busy at the moment track down why. My guess is either the "resume" logic should check whether the paused plugin is network active or not and act accordingly, or that link should go to the network plugins screen to resume the plugin.

#4 @afragen
6 years ago

Hey Paul.

I was really hoping you would find a more complete solution. I was looking to see how the current notice was called and create something similar for network admin. I missed the URL correction.

If you create the new ticket for the other issue I’ll try to dig into it.

#5 in reply to: ↑ 3 @pbiron
6 years ago

done: #46097

@flixos90
6 years ago

#6 @flixos90
6 years ago

46060.3.diff simplifies the change by using self_admin_url(). It furthermore ensures that plugins that are paused on any site in the network are displayed as "paused" in the network admin plugins screen. This was already the case for the row actions, however in the network admin they would never have shown up under the "Paused" tab. If a site had only non-network active plugins that were paused, the network admin could link to the "Paused" tab without that tab actually showing any content in the list.

#7 @flixos90
6 years ago

  • Milestone changed from 5.1 to 5.2

#8 @flixos90
6 years ago

  • Milestone changed from 5.2 to 5.3

#9 @flixos90
6 years ago

  • Milestone 5.3 deleted
  • Resolution set to invalid
  • Status changed from new to closed

This ticket is based on the old fatal error recovery mode implementation and will be covered as part of #46130.

#10 @spacedmonkey
5 years ago

  • Component changed from Bootstrap/Load to Site Health
Note: See TracTickets for help on using tickets.