From 48b30d26343349e05c9e6d23487e7e340abf60be Mon Sep 17 00:00:00 2001 From: Sergey Biryukov Date: Mon, 9 Aug 2021 19:09:55 +0000 Subject: [PATCH] Code Modernization: Replace `strftime()` and `gmstrftime()` usage in unit tests. Since PHP 8.1, the `strftime()` and `gmstrftime()` functions are deprecated: > The `strftime()` and `gmstrftime()` functions exhibit similar issues as `strptime()`, in that the formats they support, as well as their behavior, is platform-dependent. Unlike `strptime()`, these functions are available on Windows, though with a different feature set than on Linux. Musl-based distributions like Alpine do not support timezone-related format specifiers correctly. These functions are also locale-based, and as such may exhibit thread-safety issues. > > `date()` or `DateTime::format()` provide portable alternatives, and `IntlDateFormatter::format()` provides a more sophisticated, localization-aware alternative. Reference: [https://wiki.php.net/rfc/deprecations_php_8_1#strftime_and_gmstrftime PHP RFC: Deprecations for PHP 8.1: strftime() and gmstrftime()] > The `strftime()` and `gmstrftime()` functions have been deprecated in favor of > `date()/DateTime::format()` (for locale-independent formatting) or > `IntlDateFormatter::format()` (for locale-dependent formatting). Reference: [https://github.com/php/php-src/blob/1cf4fb739f7a4fa8404a4c0958f13d04eae519d4/UPGRADING#L379-L381 PHP 8.1 Upgrade Notes]. Aside from one instance in SimplePie, the `strftime()` and `gmstrftime()` functions are only used within the test suite of WordPress to create formatted timestamps. As the function is used in test code, this leads to test warnings like this on PHP 8.1: {{{ Deprecated: Function strftime() is deprecated in path/to/tests/phpunit/tests/canonical/postStatus.php on line 37 }}} These calls can all be safely converted to use a pattern along the lines of: {{{#!php