Changeset 45114 for trunk/src/wp-admin/includes/plugin.php

Timestamp:

04/05/2019 03:19:50 PM (6 years ago)

Author:

flixos90

Message:

Bootstrap/Load: Ensure WP_Fatal_Error_Handler does not conflict with existing mechanisms treating fatal errors.

Prior to this change, resuming or activating a plugin or theme that is still broken would result in a redirect loop if in recovery mode. If outside recovery mode, it would cause the error template to be displayed.

Furthermore this applies to breaking a plugin or theme when editing from the backend.

Props aandrewdixon, azaozz, dhanukanuwan, henrywright, ocean90, ohiosierra, PandelisZ, xkon.
Fixes #46045, #46751.

File:

• trunk/src/wp-admin/includes/plugin.php (modified) (2 diffs)

trunk/src/wp-admin/includes/plugin.php

@@ -632 +632 @@
             wp_redirect( add_query_arg( '_error_nonce', wp_create_nonce( 'plugin-activation-error_' . $plugin ), $redirect ) ); // we'll override this later if the plugin can be included without fatal error
         }
+
         ob_start();
         wp_register_plugin_realpath( WP_PLUGIN_DIR . '/' . $plugin );
         $_wp_plugin_file = $plugin;
+        define( 'WP_SANDBOX_SCRAPING', true );
         include_once( WP_PLUGIN_DIR . '/' . $plugin );
         $plugin = $_wp_plugin_file; // Avoid stomping of the $plugin variable in a plugin.
@@ -2126 +2128 @@
  */
 function plugin_sandbox_scrape( $plugin ) {
+    define( 'WP_SANDBOX_SCRAPING', true );
     wp_register_plugin_realpath( WP_PLUGIN_DIR . '/' . $plugin );
     include( WP_PLUGIN_DIR . '/' . $plugin );