[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 18:18:24 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:"51735" 51735]:
{{{
#!CommitTicketReference repository="" revision="51735"
Code Modernization: Fix parameter name mismatches for parent/child classes
in `WP_List_Table::column_cb()`.
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 [15632], [30679], [31210], [32740], [32753], [32754],
[32755], [32756], [32757].
Props jrf, hellofromTonya, sergeybiryukov, azaozz, desrosj, johnbillion.
See #51553.
}}}
--
Ticket URL: <https://core.trac.wordpress.org/ticket/51553#comment:35>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list