WordPress.org

Make WordPress Core

Ticket #17552: 17552-2.patch

File 17552-2.patch, 7.2 KB (added by WraithKenny, 7 years ago)

refresh

  • wp-admin/plugin-editor.php

     
    2727if ( empty($plugins) )
    2828        wp_die( __('There are no plugins installed on this site.') );
    2929
    30 if ( $file ) {
    31         $plugin = $file;
    32 } elseif ( empty( $plugin ) ) {
    33         $plugin = array_keys($plugins);
    34         $plugin = $plugin[0];
     30if ( empty( $plugin ) ) {
     31        if ( $file ) {
     32                $plugin = $file;
     33        } else {
     34                $plugin = array_keys( $plugins );
     35                $plugin = $plugin[0];
     36        }
    3537}
    3638
    3739$plugin_files = get_plugin_files($plugin);
     
    4951
    5052        check_admin_referer('edit-plugin_' . $file);
    5153
    52         $newcontent = wp_unslash( $_POST['newcontent'] );
    53         if ( is_writeable($real_file) ) {
    54                 $f = fopen($real_file, 'w+');
    55                 fwrite($f, $newcontent);
    56                 fclose($f);
     54        if ( is_writeable( $real_file ) ) {
     55                $newcontent = wp_unslash( $_POST['newcontent'] );
    5756
    58                 $network_wide = is_plugin_active_for_network( $file );
     57                file_put_contents($real_file, $newcontent);
    5958
     59                $network_wide = is_plugin_active_for_network( $plugin  );
     60// @TODO: Network Wide activated plugins do not go through this lint check
    6061                // Deactivate so we can test it.
    61                 if ( is_plugin_active($file) || isset($_POST['phperror']) ) {
    62                         if ( is_plugin_active($file) )
    63                                 deactivate_plugins($file, true);
     62                if ( is_plugin_active($plugin) || isset($_POST['phperror']) ) {
     63                        if ( is_plugin_active($plugin) )
     64                                deactivate_plugins($plugin, true);
    6465
    6566                        if ( ! is_network_admin() )
    66                                 update_option( 'recently_activated', array( $file => time() ) + (array) get_option( 'recently_activated' ) );
     67                                update_option( 'recently_activated', array( $plugin => time() ) + (array) get_option( 'recently_activated' ) );
    6768
    68                         wp_redirect(add_query_arg('_wpnonce', wp_create_nonce('edit-plugin-test_' . $file), "plugin-editor.php?file=$file&liveupdate=1&scrollto=$scrollto&networkwide=" . $network_wide));
     69                        wp_redirect(add_query_arg('_wpnonce', wp_create_nonce('edit-plugin-test_' . $file), "plugin-editor.php?file=$file&plugin=$plugin&liveupdate=1&scrollto=$scrollto&networkwide=" . $network_wide));
    6970                        exit;
    7071                }
    71                 wp_redirect( self_admin_url("plugin-editor.php?file=$file&a=te&scrollto=$scrollto") );
     72                wp_redirect( self_admin_url("plugin-editor.php?file=$file&plugin=$plugin&updated=1&scrollto=$scrollto") );
    7273        } else {
    73                 wp_redirect( self_admin_url("plugin-editor.php?file=$file&scrollto=$scrollto") );
     74                wp_redirect( self_admin_url("plugin-editor.php?file=$file&plugin=$plugin&scrollto=$scrollto") );
    7475        }
    7576        exit;
    7677
     
    8182        if ( isset($_GET['liveupdate']) ) {
    8283                check_admin_referer('edit-plugin-test_' . $file);
    8384
    84                 $error = validate_plugin($file);
     85                $error = validate_plugin($plugin);
    8586                if ( is_wp_error($error) )
    8687                        wp_die( $error );
    8788
    88                 if ( ( ! empty( $_GET['networkwide'] ) && ! is_plugin_active_for_network($file) ) || ! is_plugin_active($file) )
    89                         activate_plugin($file, "plugin-editor.php?file=$file&phperror=1", ! empty( $_GET['networkwide'] ) ); // we'll override this later if the plugin can be included without fatal error
     89                if ( ( ! empty( $_GET['networkwide'] ) && ! is_plugin_active_for_network($plugin) ) || ! is_plugin_active($plugin) )
     90                        activate_plugin($plugin, "plugin-editor.php?file=$file&plugin=$plugin&phperror=1", ! empty( $_GET['networkwide'] ) ); // we'll override this later if the plugin can be included without fatal error
    9091
    91                 wp_redirect( self_admin_url("plugin-editor.php?file=$file&a=te&scrollto=$scrollto") );
     92                wp_redirect( self_admin_url("plugin-editor.php?file=$file&plugin=$plugin&updated=1&scrollto=$scrollto") );
    9293                exit;
    9394        }
    9495
     
    9798        $editable_extensions = (array) apply_filters('editable_extensions', $editable_extensions);
    9899
    99100        if ( ! is_file($real_file) ) {
    100                 wp_die(sprintf('<p>%s</p>', __('No such file exists! Double check the name and try again.')));
     101                wp_die( __('No such file exists! Double check the name and try again.') );
    101102        } else {
    102103                // Get the extension of the file
    103104                if ( preg_match('/\.([^.]+)$/', $real_file, $matches) ) {
    104105                        $ext = strtolower($matches[1]);
    105106                        // If extension is not in the acceptable list, skip it
    106107                        if ( !in_array( $ext, $editable_extensions) )
    107                                 wp_die(sprintf('<p>%s</p>', __('Files of this type are not editable.')));
     108                                wp_die( __('Files of this type are not editable.') );
    108109                }
    109110        }
    110111
     
    129130
    130131        require_once(ABSPATH . 'wp-admin/admin-header.php');
    131132
    132         update_recently_edited(WP_PLUGIN_DIR . '/' . $file);
     133        update_recently_edited( WP_PLUGIN_DIR . '/' . $plugin );
    133134
    134135        $content = file_get_contents( $real_file );
    135136
     
    148149
    149150        $content = esc_textarea( $content );
    150151        ?>
    151 <?php if (isset($_GET['a'])) : ?>
     152<?php if (isset($_GET['updated'])) : ?>
    152153 <div id="message" class="updated"><p><?php _e('File edited successfully.') ?></p></div>
    153154<?php elseif (isset($_GET['phperror'])) : ?>
    154155 <div id="message" class="updated"><p><?php _e('This plugin has been deactivated because your changes resulted in a <strong>fatal error</strong>.') ?></p>
    155156        <?php
    156                 if ( wp_verify_nonce($_GET['_error_nonce'], 'plugin-activation-error_' . $file) ) { ?>
    157         <iframe style="border:0" width="100%" height="70px" src="<?php bloginfo('wpurl'); ?>/wp-admin/plugins.php?action=error_scrape&amp;plugin=<?php echo esc_attr($file); ?>&amp;_wpnonce=<?php echo esc_attr($_GET['_error_nonce']); ?>"></iframe>
     157                if ( wp_verify_nonce($_GET['_error_nonce'], 'plugin-activation-error_' . $plugin ) ) {
     158                //@TODO: Is this Network Admin safe? (admin_url()) ?>
     159        <iframe style="border:0" width="100%" height="70px" src="<?php echo admin_url('plugins.php?action=error_scrape&plugin=' . urlencode($plugin) . '&_wpnonce=' . urlencode($_GET['_error_nonce']) ); ?>"></iframe>
    158160        <?php } ?>
    159161</div>
    160162<?php endif; ?>
     
    165167<div class="fileedit-sub">
    166168<div class="alignleft">
    167169<big><?php
    168         if ( is_plugin_active($plugin) ) {
     170        if ( is_plugin_active( $plugin ) || is_plugin_active_for_network( $plugin ) ) {
    169171                if ( is_writeable($real_file) )
    170172                        echo sprintf(__('Editing <strong>%s</strong> (active)'), $file);
    171173                else
     
    193195                $plugin_key = esc_attr($plugin_key);
    194196                echo "\n\t<option value=\"$plugin_key\" $selected>$plugin_name</option>";
    195197        }
     198
    196199?>
    197200                </select>
    198201                <?php submit_button( __( 'Select' ), 'button', 'Submit', false ); ?>
     
    234237                <div id="documentation" class="hide-if-no-js"><label for="docs-list"><?php _e('Documentation:') ?></label> <?php echo $docs_select ?> <input type="button" class="button" value="<?php esc_attr_e( 'Look Up' ) ?> " onclick="if ( '' != jQuery('#docs-list').val() ) { window.open( 'http://api.wordpress.org/core/handbook/1.0/?function=' + escape( jQuery( '#docs-list' ).val() ) + '&amp;locale=<?php echo urlencode( get_locale() ) ?>&amp;version=<?php echo urlencode( $wp_version ) ?>&amp;redirect=true'); }" /></div>
    235238                <?php endif; ?>
    236239<?php if ( is_writeable($real_file) ) : ?>
    237         <?php if ( in_array( $file, (array) get_option( 'active_plugins', array() ) ) ) { ?>
     240        <?php if ( is_plugin_active( $plugin ) || is_plugin_active_for_network( $plugin ) ) { ?>
    238241                <p><?php _e('<strong>Warning:</strong> Making changes to active plugins is not recommended. If your changes cause a fatal error, the plugin will be automatically deactivated.'); ?></p>
    239242        <?php } ?>
    240243        <p class="submit">