Changeset 5753

Timestamp:

06/25/2007 06:57:54 PM (19 years ago)

Author:

markjaquith

Message:

Display fatal errors that prevent plugins from being activated. fixes #4508

File:

• trunk/wp-admin/plugins.php (modified) (3 diffs)

trunk/wp-admin/plugins.php

@@ -12 +12 @@
                         wp_die(__('Plugin file does not exist.'));
                 if (!in_array($plugin, $current)) {
-                        wp_redirect('plugins.php?error=true'); // we'll override this later if the plugin can be included without fatal error
+                        wp_redirect(add_query_arg('_error_nonce', wp_create_nonce('plugin-activation-error_' . $plugin), 'plugins.php?error=true&plugin=' . $plugin)); // we'll override this later if the plugin can be included without fatal error
                         ob_start();
                         @include(ABSPATH . PLUGINDIR . '/' . $plugin);
@@ -22 +22 @@
                 }
                 wp_redirect('plugins.php?activate=true'); // overrides the ?error=true one above
-        } else if ('deactivate' == $_GET['action']) {
+        } elseif ('error_scrape' == $_GET['action']) {
+                $plugin = trim($_GET['plugin']);
+                check_admin_referer('plugin-activation-error_' . $plugin);
+                if ( validate_file($plugin) )
+                        wp_die(__('Invalid plugin.'));
+                if ( ! file_exists(ABSPATH . PLUGINDIR . '/' . $plugin) )
+                        wp_die(__('Plugin file does not exist.'));
+                include(ABSPATH . PLUGINDIR . '/' . $plugin);
+        } elseif ('deactivate' == $_GET['action']) {
                 check_admin_referer('deactivate-plugin_' . $_GET['plugin']);
                 $current = get_option('active_plugins');
@@ -74 +82 @@
 
 <?php if ( isset($_GET['error']) ) : ?>
-        <div id="message" class="updated fade"><p><?php _e('Plugin could not be activated because it triggered a <strong>fatal error</strong>.') ?></p></div>
+        <div id="message" class="updated fade"><p><?php _e('Plugin could not be activated because it triggered a <strong>fatal error</strong>.') ?></p>
+        <?php
+                $plugin = trim($_GET['plugin']);
+                if ( wp_verify_nonce($_GET['_error_nonce'], 'plugin-activation-error_' . $plugin) && 1 == strtolower(ini_get('display_errors'))) { ?>
+        <iframe style="border:0" width="100%" height="70px" src="<?php bloginfo('wpurl'); ?>/wp-admin/plugins.php?action=error_scrape&amp;plugin=<?php echo attribute_escape($plugin); ?>&amp;_wpnonce=<?php echo attribute_escape($_GET['_error_nonce']); ?>"></iframe>
+        <?php
+                }
+        ?>
+        </div>
 <?php elseif ( isset($_GET['activate']) ) : ?>
         <div id="message" class="updated fade"><p><?php _e('Plugin <strong>activated</strong>.') ?></p></div>