[wp-trac] [WordPress Trac] #53635: PHP 8.1: various compatibility fixes

[WordPress Trac]

#53635: PHP 8.1: various compatibility fixes
----------------------------+-------------------------------
 Reporter:  SergeyBiryukov  |       Owner:  (none)
     Type:  task (blessed)  |      Status:  new
 Priority:  normal          |   Milestone:  5.9
Component:  General         |     Version:
 Severity:  normal          |  Resolution:
 Keywords:  php81           |     Focuses:  coding-standards
----------------------------+-------------------------------

Comment (by jrf):

 FYI: some info about some of the runtime dependencies of WordPress
 (incomplete)

 == Sodium Compat

 I've opened a [https://github.com/paragonie/sodium_compat/pull/134 PR to
 fix various PHP 8.1 deprecation warnings] about the return type change, as
 well as a [https://github.com/paragonie/sodium_compat/pull/135 PR to turn
 on CI testing against PHP 8.1].

 With the open PHP 8.1 PR merged, there would still be one issue remaining,
 but how to fix that should be decided by the maintainer in this case.

 == Random Compat

 I've opened a [https://github.com/paragonie/random_compat/pull/172 PR to
 turn on CI testing against PHP 8.1].

 There are two test failures in their tests, but how to fix these should be
 decided by the maintainer in this case.

 == PHPMailer

 Testing against PHP 8.1 has been
 [https://github.com/PHPMailer/PHPMailer/pull/2363 turned on over a month
 ago].

 I've pulled two fixes and both have been merged:
 * https://github.com/PHPMailer/PHPMailer/pull/2365
 * https://github.com/PHPMailer/PHPMailer/pull/2370

 As things stand, the PHPMailer tests pass against PHP 8.1.

 No release containing the fixes is available yet.

 == Requests

 Testing against PHP 8.1 has been
 [https://github.com/WordPress/Requests/pull/501 turned on the other week].

 I've pulled three fixes and these have been merged:
 * [https://github.com/WordPress/Requests/pull/499 Input validation fix]
 * [https://github.com/WordPress/Requests/pull/500 Incorrect default value
 passed to PHP native functions]
 * [https://github.com/WordPress/Requests/pull/505 Return type deprecation
 notices]

 As things stand, the Requests tests pass against PHP 8.1.

 No release containing the fixes is available yet.

 == GetID3

 CI (linting) has been running against PHP 8.1 since December 2020 🎉

 The other side of the coin is that there are no tests, so, aside from
 there not being any parse errors, the actual PHP 8.1 compatibility status
 of the repo is anyone's guess.

 == SimplePie

 An [https://github.com/simplepie/simplepie/issues/663 issue has been open]
 since last year about getting the tests running on PHP 8.0, let alone PHP
 8.1. So far there has been no response from the maintainers...

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/53635#comment:13>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform