[wp-trac] [WordPress Trac] #57512: Warning as a part of API response

[WordPress Trac]

#57512: Warning as a part of API response
-------------------------------------------------+-------------------------
 Reporter:  kalpeshh                             |       Owner:  (none)
     Type:  defect (bug)                         |      Status:  new
 Priority:  normal                               |   Milestone:  6.4
Component:  REST API                             |     Version:  6.1.1
 Severity:  normal                               |  Resolution:
 Keywords:  php80 has-unit-tests has-patch       |     Focuses:  rest-api
  needs-testing                                  |
-------------------------------------------------+-------------------------
Changes (by hellofromTonya):

 * keywords:  2nd-opinion php8 has-unit-tests has-patch => php80 has-unit-
     tests has-patch needs-testing
 * milestone:  Awaiting Review => 6.4


Comment:

 Changes to the ticket:
 * Removed `2nd-opinion`. Not seeing where or why that keyword as added.
 * Changed `php8` to `php80` to denote PHP 8.0 as the version that
 increases the error severity.
 * Added `needs-testing` to get a Test Report, which as one more check in
 the process.
 * Moved it into 6.4, given the [attachment:"57512_with_tests_2.diff"]
 could be considered for `commit`.

 Is this a PHP 8.0 incompatibility?
 I did not add `php-compatibility` to identify this issue as a Core
 incompatibility with PHP 8.0. I think this issue is more of a code
 improvement to be defensive for an unhappy path of an invalid value, i.e.
 given both parts are required.

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