Changeset 7525

Timestamp:

03/26/08 16:14:29 (9 years ago)

Author:

ryan

Message:

Fix plugin error scraping in the plugin editor. Props DD32. fixes #6332

File:

• trunk/wp-admin/plugin-editor.php (modified) (10 diffs)

trunk/wp-admin/plugin-editor.php

@@ -15 +15 @@
 $plugin_files = array_keys($plugins);
 
-if (empty($file)) {
+if (empty($file))
     $file = $plugin_files[0];
-}
 
 $file = validate_file_to_edit($file, $plugin_files);
@@ -32 +31 @@
 
     $newcontent = stripslashes($_POST['newcontent']);
-    if (is_writeable($real_file)) {
+    if ( is_writeable($real_file) ) {
         $f = fopen($real_file, 'w+');
         fwrite($f, $newcontent);
@@ -38 +37 @@
 
         // Deactivate so we can test it.
-        $current = get_option('active_plugins');
-        if ( in_array($file, $current) || isset($_POST['phperror']) ) {
-            if ( in_array($file, $current) ) {
-                if ( ( $key = array_search( $plugin, $current) ) !== false )
-                    array_splice($current, $key, 1 ); // Fixed Array-fu!
-                update_option('active_plugins', $current);
-            }
+        if ( is_plugin_active($file) || isset($_POST['phperror']) ) {
+            if ( is_plugin_active($file) )
+                deactivate_plugins($file, true);
             wp_redirect(add_query_arg('_wpnonce', wp_create_nonce('edit-plugin-test_' . $file), "plugin-editor.php?file=$file&liveupdate=1"));
-            exit();
+            exit;
         }
         wp_redirect("plugin-editor.php?file=$file&a=te");
@@ -52 +47 @@
         wp_redirect("plugin-editor.php?file=$file");
     }
-
-    exit();
+    exit;
 
 break;
@@ -62 +56 @@
         wp_die('<p>'.__('You do not have sufficient permissions to edit plugins for this blog.').'</p>');
 
-    if ( $_GET['liveupdate'] ) {
+    if ( isset($_GET['liveupdate']) ) {
         check_admin_referer('edit-plugin-test_' . $file);
-        $current = get_option('active_plugins');
-        $plugin = $file;
-        if ( validate_file($plugin) )
-            wp_die(__('Invalid plugin.'));
-        if ( ! file_exists(ABSPATH . PLUGINDIR . '/' . $plugin) )
-            wp_die(__('Plugin file does not exist.'));
-        if (!in_array($plugin, $current)) {
-            wp_redirect("plugin-editor.php?file=$file&phperror=1"); // we'll override this later if the plugin can be included without fatal error
-            @include(ABSPATH . PLUGINDIR . '/' . $plugin);
-            $current[] = $plugin;
-            sort($current);
-            update_option('active_plugins', $current);
-        }
+
+        $error = validate_plugin($file);
+        if( is_wp_error($error) )
+            wp_die( $error );
+
+        if ( ! is_plugin_active($file) )
+            activate_plugin($file, "plugin-editor.php?file=$file&phperror=1");// we'll override this later if the plugin can be included without fatal error
+
         wp_redirect("plugin-editor.php?file=$file&a=te");
+        exit;
     }
 
@@ -84 +74 @@
     update_recently_edited(PLUGINDIR . "/$file");
 
-    if (!is_file($real_file))
+    if ( ! is_file($real_file) )
         $error = 1;
 
-    if (!$error) {
-        $f = fopen($real_file, 'r');
-        $content = fread($f, filesize($real_file));
-        $content = htmlspecialchars($content);
-    }
+    if ( ! $error )
+        $content = htmlspecialchars(file_get_contents($real_file));
 
     ?>
@@ -97 +84 @@
  <div id="message" class="updated fade"><p><?php _e('File edited successfully.') ?></p></div>
 <?php elseif (isset($_GET['phperror'])) : ?>
- <div id="message" class="updated fade"><p><?php _e('This plugin has been deactivated because your changes resulted in a <strong>fatal error</strong>.') ?></p></div>
+ <div id="message" class="updated fade"><p><?php _e('This plugin has been deactivated because your changes resulted in a <strong>fatal error</strong>.') ?></p>
+    <?php
+        if ( wp_verify_nonce($_GET['_error_nonce'], 'plugin-activation-error_' . $file) ) { ?>
+    <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($file); ?>&amp;_wpnonce=<?php echo attribute_escape($_GET['_error_nonce']); ?>"></iframe>
+    <?php } ?>
+</div>
 <?php endif; ?>
  <div class="wrap">
@@ -106 +98 @@
 <div class="alignleft">
 <big><strong><?php
-    if ( in_array($file, (array) get_option('active_plugins')) ) {
-        if (is_writeable($real_file)) {
+    if ( is_plugin_active($file) ) {
+        if ( is_writeable($real_file) )
             echo sprintf(__('Editing <strong>%s</strong> (active)'), $file);
-        } else {
-        echo sprintf(__('Browsing <strong>%s</strong> (active)'), $file);
-        }
+        else
+            echo sprintf(__('Browsing <strong>%s</strong> (active)'), $file);
     } else {
-        if (is_writeable($real_file)) {
+        if ( is_writeable($real_file) )
             echo sprintf(__('Editing <strong>%s</strong> (inactive)'), $file);
-        } else {
-        echo sprintf(__('Browsing <strong>%s</strong> (inactive)'), $file);
-        }
+        else
+            echo sprintf(__('Browsing <strong>%s</strong> (inactive)'), $file);
     }
     ?></strong></big>
@@ -125 +115 @@
 <br class="clear" />
     <div id="templateside">
-    <h3 id="bordertitle"><?php _e("Plugin Files"); ?></h3>
+    <h3 id="bordertitle"><?php _e('Plugin Files'); ?></h3>
 
     <h4><?php _e('Plugins'); ?></h4>
     <ul>
 <?php foreach($plugin_files as $plugin_file) : ?>
-        <li><a href="plugin-editor.php?file=<?php echo "$plugin_file"; ?>"><?php echo $plugins[$plugin_file]['Name']; ?></a></li>
+        <li><a href="plugin-editor.php?file=<?php echo $plugin_file; ?>"><?php echo $plugins[$plugin_file]['Name']; ?></a></li>
 <?php endforeach; ?>
     </ul>
     </div>
-<?php   if (!$error) { ?>
+<?php   if ( ! $error ) { ?>
     <form name="template" id="template" action="plugin-editor.php" method="post">
     <?php wp_nonce_field('edit-plugin_' . $file) ?>
@@ -165 +155 @@
 </div>
 <?php
-break;
+    break;
 }
 include("admin-footer.php") ?>