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

[WordPress Trac]

#53635: PHP 8.1: various compatibility fixes
-------------------------------------------------+-------------------------
 Reporter:  SergeyBiryukov                       |       Owner:
                                                 |  hellofromTonya
     Type:  task (blessed)                       |      Status:  accepted
 Priority:  normal                               |   Milestone:  5.9
Component:  General                              |     Version:
 Severity:  normal                               |  Resolution:
 Keywords:  php81 has-patch has-unit-tests       |     Focuses:  coding-
  commit                                         |  standards
-------------------------------------------------+-------------------------

Comment (by hellofromTonya):

 In [changeset:"51793" 51793]:
 {{{
 #!CommitTicketReference repository="" revision="51793"
 Code Modernization: Fix null to non-nullable deprecation in
 `wp_privacy_anonymize_ip()`.

 The `wp_privacy_anonymize_ip()` function expects a string for the
 `$ip_addr` parameter, but did not do any input validation.

 One of the pre-existing test cases, passed `null` to the function, leading
 to a `substr_count(): Passing null to parameter #1 ($haystack) of type
 string is deprecated` notice on PHP 8.1.

 Fixed now by doing a cursory check on the variable at the start of the
 function and bowing out early for a number of cases (`null`, `false`, `0`,
 `''`) which would all result in the same `0.0.0.0` output anyway.

 Follow-up [42971].

 Props jrf, hellofromTonya.
 See #53635.
 }}}

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