[wp-trac] [WordPress Trac] #52825: [PHP 8.1] MySQLi error reporting value changes
WordPress Trac
noreply at wordpress.org
Sat Aug 7 03:06:35 UTC 2021
#52825: [PHP 8.1] MySQLi error reporting value changes
------------------------------------------+-----------------------
Reporter: ayeshrajans | Owner: jrf
Type: defect (bug) | Status: accepted
Priority: normal | Milestone: 5.9
Component: Database | Version:
Severity: normal | Resolution:
Keywords: php81 has-patch commit early | Focuses:
------------------------------------------+-----------------------
Changes (by jrf):
* keywords: php81 has-patch => php81 has-patch commit early
* owner: (none) => jrf
* status: new => accepted
* milestone: Awaiting Review => 5.9
Comment:
This patch is a prerequisite before CI testing against PHP 8.1 can be
enabled as the CI build fails in the "Install WordPress" step while
running the `tools/local-env/scripts/install.js` script due to this issue.
[https://github.com/jrfnl/wordpress-develop-
official/runs/3266750477?check_suite_focus=true Failing build where this
can be seen]
I've reviewed the patch and this is exactly what needs to be done. Only
remark I could make is that the comment should probably use `/* .. */`
multi-line style 😄
[https://github.com/jrfnl/wordpress-develop-
official/runs/3267534737?check_suite_focus=true Build without the mysqli
error (but still erroring) which includes the PHPUnit patches + this
patch]
A commit at the earliest convenience would be appreciated. /cc
@SergeyBiryukov
--
Ticket URL: <https://core.trac.wordpress.org/ticket/52825#comment:2>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list