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

WordPress Trac noreply at wordpress.org
Tue Sep 7 19:23:25 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:"51737" 51737]:
 {{{
 #!CommitTicketReference repository="" revision="51737"
 Code Modernization: Fix parameter name mismatches for parent/child classes
 in `WP_List_Table::handle_row_actions()`.

 Matches the method signatures of the parent class and each child class.

 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.

 For readability:

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

 - in methods longer than a single line, the generic parameter is
 reassigned to the original parameter restoring it for context for use
 within the method. An inline comment is added to explain why this
 reassignment is made.

 Follow-up to [32644], [32664], [32798], [38489], [49183], [49197].

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

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


More information about the wp-trac mailing list