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

WordPress Trac noreply at wordpress.org
Thu Sep 9 13:03:00 UTC 2021


#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:"51780" 51780]:
 {{{
 #!CommitTicketReference repository="" revision="51780"
 Code Modernization: Fix reserved keyword and parameter name mismatches for
 parent/child classes in `Walker::end_el()`.

 In the parent class, renames the parameter `$object` to `$data_object`.
 Why? `object` is a PHP reserved keyword. The parameter name is selected
 for consistency with `Walker::start_el()`.

 In each child class: renames the 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.

 Changes for readability:

 - `@since` clearly specifies the original parameter name and its new name
 as well as why the change happened.

 Follow-up to [7737], [8900], [8970], [14248], [16100], [25642], [25644],
 [37051], [37056].

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

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


More information about the wp-trac mailing list