[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