From f40c6c9ebe6490eebd4d716d0796f29b59fa963d Mon Sep 17 00:00:00 2001 From: Sergey Biryukov Date: Tue, 25 Feb 2020 00:57:33 +0000 Subject: [PATCH] Script Loader: Remove unnecessary reference sign from function definitions in script loader. This resolves PHP warnings when extensions like `php_uopz` or `php_tideways_xhprof` are in use. Object variables in PHP 5+ contain a reference to the object, and it's the reference that's passed around. Props jqz, killerbishop, Mte90, TJNowell, jorbin, pento, pcarvalho, valentinbora, SergeyBiryukov. Fixes #44979. git-svn-id: https://develop.svn.wordpress.org/trunk@47355 602fd350-edb4-49c9-b593-d223f7449a82 --- src/wp-includes/script-loader.php | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/src/wp-includes/script-loader.php b/src/wp-includes/script-loader.php index 634153ba99..f354b9edf0 100644 --- a/src/wp-includes/script-loader.php +++ b/src/wp-includes/script-loader.php @@ -41,7 +41,7 @@ require ABSPATH . WPINC . '/functions.wp-styles.php'; * * @param WP_Scripts $scripts WP_Scripts object. */ -function wp_register_tinymce_scripts( &$scripts, $force_uncompressed = false ) { +function wp_register_tinymce_scripts( $scripts, $force_uncompressed = false ) { global $tinymce_version, $concatenate_scripts, $compress_scripts; $suffix = wp_scripts_get_suffix(); $dev_suffix = wp_scripts_get_suffix( 'dev' ); @@ -73,7 +73,7 @@ function wp_register_tinymce_scripts( &$scripts, $force_uncompressed = false ) { * * @param WP_Scripts $scripts WP_Scripts object. */ -function wp_default_packages_vendor( &$scripts ) { +function wp_default_packages_vendor( $scripts ) { global $wp_locale; $suffix = wp_scripts_get_suffix(); @@ -176,7 +176,7 @@ function wp_default_packages_vendor( &$scripts ) { * @param array $tests Features to detect. * @return string Conditional polyfill inline script. */ -function wp_get_script_polyfill( &$scripts, $tests ) { +function wp_get_script_polyfill( $scripts, $tests ) { $polyfill = ''; foreach ( $tests as $test => $handle ) { if ( ! array_key_exists( $handle, $scripts->registered ) ) { @@ -228,7 +228,7 @@ function wp_get_script_polyfill( &$scripts, $tests ) { * * @param WP_Scripts $scripts WP_Scripts object. */ -function wp_default_packages_scripts( &$scripts ) { +function wp_default_packages_scripts( $scripts ) { $suffix = wp_scripts_get_suffix(); // Expects multidimensional array like: @@ -273,7 +273,7 @@ function wp_default_packages_scripts( &$scripts ) { * * @param WP_Scripts $scripts WP_Scripts object. */ -function wp_default_packages_inline_scripts( &$scripts ) { +function wp_default_packages_inline_scripts( $scripts ) { global $wp_locale; if ( isset( $scripts->registered['wp-api-fetch'] ) ) { @@ -521,7 +521,7 @@ function wp_tinymce_inline_scripts() { * * @param WP_Scripts $scripts WP_Scripts object. */ -function wp_default_packages( &$scripts ) { +function wp_default_packages( $scripts ) { wp_default_packages_vendor( $scripts ); wp_register_tinymce_scripts( $scripts ); wp_default_packages_scripts( $scripts ); @@ -580,7 +580,7 @@ function wp_scripts_get_suffix( $type = '' ) { * * @param WP_Scripts $scripts WP_Scripts object. */ -function wp_default_scripts( &$scripts ) { +function wp_default_scripts( $scripts ) { $suffix = wp_scripts_get_suffix(); $dev_suffix = wp_scripts_get_suffix( 'dev' ); $guessurl = site_url(); @@ -1615,7 +1615,7 @@ function wp_default_scripts( &$scripts ) { * * @param WP_Styles $styles */ -function wp_default_styles( &$styles ) { +function wp_default_styles( $styles ) { // Include an unmodified $wp_version. require ABSPATH . WPINC . '/version.php';