WordPress.org

Make WordPress Core

Ticket #14170: 14170.2.diff

File 14170.2.diff, 3.4 KB (added by scribu, 3 years ago)

Remove extraneous trim() calls

  • wp-includes/plugin.php

     
    605605 */ 
    606606function register_activation_hook($file, $function) { 
    607607        $file = plugin_basename($file); 
    608         add_action('activate_' . $file, $function); 
     608        add_action('activate_' . $file, $function, 10, 2); 
    609609} 
    610610 
    611611/** 
     
    630630 */ 
    631631function register_deactivation_hook($file, $function) { 
    632632        $file = plugin_basename($file); 
    633         add_action('deactivate_' . $file, $function); 
     633        add_action('deactivate_' . $file, $function, 10, 2); 
    634634} 
    635635 
    636636/** 
  • wp-admin/includes/plugin.php

     
    464464 * @return WP_Error|null WP_Error on invalid file or null on success. 
    465465 */ 
    466466function activate_plugin( $plugin, $redirect = '', $network_wide = false) { 
    467         $plugin  = plugin_basename( trim( $plugin ) ); 
     467        $plugin = plugin_basename( trim( $plugin ) ); 
    468468 
    469469        if ( is_multisite() && ( $network_wide || is_network_only_plugin($plugin) ) ) { 
    470470                $network_wide = true; 
     
    482482                        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 
    483483                ob_start(); 
    484484                include(WP_PLUGIN_DIR . '/' . $plugin); 
    485                 do_action( 'activate_plugin', trim( $plugin) ); 
    486                 do_action( 'activate_' . trim( $plugin ) ); 
     485                do_action( 'activate_plugin', $plugin, $network_wide ); 
     486                do_action( 'activate_' . $plugin, $network_wide ); 
    487487                if ( $network_wide ) { 
    488488                        $current[$plugin] = time(); 
    489489                        update_site_option( 'active_sitewide_plugins', $current ); 
     
    492492                        sort($current); 
    493493                        update_option('active_plugins', $current); 
    494494                } 
    495                 do_action( 'activated_plugin', trim( $plugin) ); 
     495                do_action( 'activated_plugin', $plugin, $network_wide ); 
    496496                if ( ob_get_length() > 0 ) { 
    497497                        $output = ob_get_clean(); 
    498498                        return new WP_Error('unexpected_output', __('The plugin generated unexpected output.'), $output); 
     
    521521        $do_blog = $do_network = false; 
    522522 
    523523        foreach ( (array) $plugins as $plugin ) { 
    524                 $plugin = plugin_basename($plugin); 
     524                $plugin = plugin_basename( trim( $plugin ) ); 
    525525                if ( ! is_plugin_active($plugin) ) 
    526526                        continue; 
     527 
     528                $network_wide = is_plugin_active_for_network( $plugin ); 
     529 
    527530                if ( ! $silent ) 
    528                         do_action( 'deactivate_plugin', trim( $plugin ) ); 
     531                        do_action( 'deactivate_plugin', $plugin, $network_wide ); 
    529532 
    530                 if ( is_plugin_active_for_network($plugin) ) { 
    531                         // Deactivate network wide 
     533                if ( $network_wide ) { 
    532534                        $do_network = true; 
    533535                        unset( $network_current[ $plugin ] ); 
    534536                } else { 
    535                         // Deactivate for this blog only 
    536                         $key = array_search( $plugin, (array) $current ); 
     537                        $key = array_search( $plugin, $current ); 
    537538                        if ( false !== $key ) { 
    538539                                $do_blog = true; 
    539540                                array_splice( $current, $key, 1 ); 
    540541                        } 
    541542                } 
    542543 
    543                 //Used by Plugin updater to internally deactivate plugin, however, not to notify plugins of the fact to prevent plugin output. 
    544544                if ( ! $silent ) { 
    545                         do_action( 'deactivate_' . trim( $plugin ) ); 
    546                         do_action( 'deactivated_plugin', trim( $plugin ) ); 
     545                        do_action( 'deactivate_' . $plugin, $network_wide ); 
     546                        do_action( 'deactivated_plugin', $plugin, $network_wide ); 
    547547                } 
    548548        } 
    549549