Changeset 58009 for trunk/src/wp-includes/l10n/class-wp-translation-file.php

Timestamp:

04/15/2024 08:01:03 PM (11 months ago)

Author:

SergeyBiryukov

Message:

Code Modernization: Fix implicit nullable parameter type deprecation on PHP 8.4.

In PHP 8.4, declaring function or method parameters with a default value of null is deprecated if the type is not nullable.

PHP applications are recommended to explicitly declare the type as nullable. All type declarations that have a default value of null, but without declaring null in the type declaration, will emit a deprecation notice:

function test( array $value = null ) {}

Deprecated: Implicitly marking parameter $value as nullable is deprecated, the explicit nullable type must be used instead

Recommended Changes

Change the implicit nullable type declaration to a nullable type declaration, available since PHP 7.1:

- function test( string $test = null ) {}
+ function test( ?string $test = null ) {}

This commit updates the affected instances in core to use a nullable type declaration.

References:

• ​PHP RFC: Deprecate implicitly nullable parameter types
• ​PHP.Watch: PHP 8.4: Implicitly nullable parameter declarations deprecated

Follow-up to [28731], [50552], [57337], [57985].

Props ayeshrajans, jrf, audrasjb, jorbin.
Fixes #60786.

File:

• trunk/src/wp-includes/l10n/class-wp-translation-file.php (modified) (1 diff)

trunk/src/wp-includes/l10n/class-wp-translation-file.php

@@ -82 +82 @@
      * @return false|WP_Translation_File
      */
-    public static function create( string $file, string $filetype = null ) {
+    public static function create( string $file, ?string $filetype = null ) {
         if ( ! is_readable( $file ) ) {
             return false;