From 9066eb15a2f6ded6e52b93d01544c56260fcb589 Mon Sep 17 00:00:00 2001 From: Sergey Biryukov Date: Sat, 12 Sep 2020 00:10:16 +0000 Subject: [PATCH] Coding Standards: Explicitly declare the `$wp_version` global used in some core files. Props jaydeep-rami, sabernhardt. Fixes #44932. git-svn-id: https://develop.svn.wordpress.org/trunk@48971 602fd350-edb4-49c9-b593-d223f7449a82 --- src/wp-admin/includes/class-plugin-upgrader.php | 5 +++-- src/wp-admin/includes/class-theme-upgrader.php | 5 +++-- src/wp-includes/default-constants.php | 6 +++--- 3 files changed, 9 insertions(+), 7 deletions(-) diff --git a/src/wp-admin/includes/class-plugin-upgrader.php b/src/wp-admin/includes/class-plugin-upgrader.php index 27dc757f19..c750399277 100644 --- a/src/wp-admin/includes/class-plugin-upgrader.php +++ b/src/wp-admin/includes/class-plugin-upgrader.php @@ -406,13 +406,14 @@ class Plugin_Upgrader extends WP_Upgrader { * @since 3.3.0 * * @global WP_Filesystem_Base $wp_filesystem WordPress filesystem subclass. + * @global string $wp_version The WordPress version string. * * @param string $source The path to the downloaded package source. * @return string|WP_Error The source as passed, or a WP_Error object * if no plugins were found. */ public function check_package( $source ) { - global $wp_filesystem; + global $wp_filesystem, $wp_version; $this->new_plugin_data = array(); @@ -459,7 +460,7 @@ class Plugin_Upgrader extends WP_Upgrader { $error = sprintf( /* translators: 1: Current WordPress version, 2: Version required by the uploaded plugin. */ __( 'Your WordPress version is %1$s, however the uploaded plugin requires %2$s.' ), - $GLOBALS['wp_version'], + $wp_version, $requires_wp ); diff --git a/src/wp-admin/includes/class-theme-upgrader.php b/src/wp-admin/includes/class-theme-upgrader.php index c08d3c2d5c..3c8f2d9abf 100644 --- a/src/wp-admin/includes/class-theme-upgrader.php +++ b/src/wp-admin/includes/class-theme-upgrader.php @@ -519,12 +519,13 @@ class Theme_Upgrader extends WP_Upgrader { * @since 3.3.0 * * @global WP_Filesystem_Base $wp_filesystem WordPress filesystem subclass. + * @global string $wp_version The WordPress version string. * * @param string $source The full path to the package source. * @return string|WP_Error The source or a WP_Error. */ public function check_package( $source ) { - global $wp_filesystem; + global $wp_filesystem, $wp_version; $this->new_theme_data = array(); @@ -606,7 +607,7 @@ class Theme_Upgrader extends WP_Upgrader { $error = sprintf( /* translators: 1: Current WordPress version, 2: Version required by the uploaded theme. */ __( 'Your WordPress version is %1$s, however the uploaded theme requires %2$s.' ), - $GLOBALS['wp_version'], + $wp_version, $requires_wp ); diff --git a/src/wp-includes/default-constants.php b/src/wp-includes/default-constants.php index 77f76319b8..bb57b96536 100644 --- a/src/wp-includes/default-constants.php +++ b/src/wp-includes/default-constants.php @@ -16,7 +16,7 @@ * @global string $wp_version The WordPress version string. */ function wp_initial_constants() { - global $blog_id; + global $blog_id, $wp_version; /**#@+ * Constants for expressing human-readable data sizes in their respective number of bytes. @@ -99,8 +99,8 @@ function wp_initial_constants() { // Add define( 'SCRIPT_DEBUG', true ); to wp-config.php to enable loading of non-minified, // non-concatenated scripts and stylesheets. if ( ! defined( 'SCRIPT_DEBUG' ) ) { - if ( ! empty( $GLOBALS['wp_version'] ) ) { - $develop_src = false !== strpos( $GLOBALS['wp_version'], '-src' ); + if ( ! empty( $wp_version ) ) { + $develop_src = false !== strpos( $wp_version, '-src' ); } else { $develop_src = false; }