Changeset 48894

Timestamp:

08/27/2020 09:30:33 PM (4 years ago)

Author:

SergeyBiryukov

Message:

Bootstrap/Load: Remove the ability to alter the list of environment types in wp_get_environment_type().

The intention of wp_get_environment_type() is to provide a consistent means of identifying the environment type, not of identifying a specific environment.

Actual environments should fit within one of the existing types: local, development, staging, or production. That should cover the types that plugins and themes might be concerned about when toggling functionality.

Props johnbillion, joostdevalk, TimothyBlynJacobs, jeremyfelt, batmoo, claytoncollie, Clorith, markjaquith, garrett-eclipse, GaryJ, elrae.
Fixes #50992.

File:

• trunk/src/wp-includes/load.php (modified) (2 diffs)

trunk/src/wp-includes/load.php

@@ -139 +139 @@
  *
  * @since 5.5.0
- * @since 5.5.1 The 'local' type was added.
+ * @since 5.5.1 Added the 'local' type.
+ * @since 5.5.1 Removed the ability to alter the list of types.
  *
  * @return string The current environment type.
@@ -157 +158 @@
     );
 
-    // Check if the environment variable has been set, if `getenv` is available on the system.
-    if ( function_exists( 'getenv' ) ) {
-        $has_env = getenv( 'WP_ENVIRONMENT_TYPES' );
-        if ( false !== $has_env ) {
-            $wp_environments = explode( ',', $has_env );
-        }
-    }
-
-    // Fetch the environment types from a constant, this overrides the global system variable.
-    if ( defined( 'WP_ENVIRONMENT_TYPES' ) ) {
-        $wp_environments = WP_ENVIRONMENT_TYPES;
+    // Add a note about the deprecated WP_ENVIRONMENT_TYPES constant.
+    if ( defined( 'WP_ENVIRONMENT_TYPES' ) && function_exists( '_deprecated_argument' ) ) {
+        _deprecated_argument(
+            'define()',
+            '5.5.1',
+            /* translators: %s: WP_ENVIRONMENT_TYPES */
+            sprintf( __( 'The %s constant is no longer supported.' ), 'WP_ENVIRONMENT_TYPES' )
+        );
     }