Changeset 54232 for trunk/src/wp-admin/includes/schema.php

Timestamp:

09/20/2022 12:41:58 AM (2 years ago)

Author:

SergeyBiryukov

Message:

Date/Time: Correct sanitization of localized default timezone_string in populate_options().

This fixes a bug where if the default timezone_string is set to a deprecated timezone name due to a localization providing an outdated timezone name string, this localized timezone string would be discarded and an empty string would be set as the timezone value instead.

By passing the DateTimeZone::ALL_WITH_BC constant as the $timezoneGroup parameter to the PHP native timezone_identifiers_list() function, a timezone name list is retrieved containing both current and deprecated timezone names, preventing the invalidation of the option value.

See the extensive write-up about this in ticket #56468.

Also see: ​PHP Manual: timezone_identifiers_list().

Includes:

• Expanding the translators comment to encourage translators to use “old” names over “new” names.
• Adding a dedicated test to the Tests_Admin_IncludesSchema test class.

Follow-up to [54207], [54217], [54227], [54229], [54230].

Props jrf, costdev.
See #56468.

File:

• trunk/src/wp-admin/includes/schema.php (modified) (1 diff)

trunk/src/wp-admin/includes/schema.php

@@ -389 +389 @@
      * translators: default GMT offset or timezone string. Must be either a valid offset (-12 to 14)
      * or a valid timezone string (America/New_York). See https://www.php.net/manual/en/timezones.php
-     * for all timezone strings supported by PHP.
+     * for all timezone strings currently supported by PHP.
+     *
+     * Important: When a previous timezone string, like `Europe/Kiev`, has been superseded by an
+     * updated one, like `Europe/Kyiv`, as a rule of thumb, the **old** timezone name should be used
+     * in the "translation" to allow for the default timezone setting to be PHP cross-version compatible,
+     * as old timezone names will be recognized in new PHP versions, while new timezone names cannot
+     * be recognized in old PHP versions.
+     *
+     * To verify which timezone strings are available in the _oldest_ PHP version supported, you can
+     * use https://3v4l.org/6YQAt#v5.6.20 and replace the "BR" (Brazil) in the code line with the
+     * country code for which you want to look up the supported timezone names.
      */
     $offset_or_tz = _x( '0', 'default GMT offset or timezone string' );
     if ( is_numeric( $offset_or_tz ) ) {
         $gmt_offset = $offset_or_tz;
-    } elseif ( $offset_or_tz && in_array( $offset_or_tz, timezone_identifiers_list(), true ) ) {
-            $timezone_string = $offset_or_tz;
+    } elseif ( $offset_or_tz && in_array( $offset_or_tz, timezone_identifiers_list( DateTimeZone::ALL_WITH_BC ), true ) ) {
+        $timezone_string = $offset_or_tz;
     }