[wp-trac] [WordPress Trac] #51553: PHP 8.0: code improvements to allow for named parameters in function calls

[WordPress Trac]

#51553: PHP 8.0: code improvements to allow for named parameters in function calls
-------------------------------------------------+-------------------------
 Reporter:  jrf                                  |       Owner:
                                                 |  hellofromTonya
     Type:  task (blessed)                       |      Status:  reviewing
 Priority:  normal                               |   Milestone:  5.9
Component:  General                              |     Version:
 Severity:  normal                               |  Resolution:
 Keywords:  php8 has-patch needs-codex early     |     Focuses:  coding-
  needs-dev-note commit                          |  standards
-------------------------------------------------+-------------------------

Comment (by hellofromTonya):

 In [changeset:"51787" 51787]:
 {{{
 #!CommitTicketReference repository="" revision="51787"
 Code Modernization: Fix parameter name mismatches for parent/child classes
 in `WP_Sitemaps_Provider::get_url_list()`.

 In each child and grandchild class, renames the second parameter to match
 the parent's method signature.
 Why? PHP 8 introduces the ability to pass named arguments to
 function/method calls. This means the child and parent method signatures
 (i.e. parameter names) need to match.

 Adds @since to clearly specify why the change happened.

 Reassigns the generic parameter to the original parameter.
 Why? Restoring the original name keeps the context intact within the
 method and makes the code more readable. An inline comment explains why
 this reassignment is made.

 Follow-up to [48072].

 Props jrf, hellofromTonya, sergeybiryukov, azaozz, desrosj, johnbillion.
 See #51553.
 }}}

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