Changeset 53236 for trunk/src/wp-content/themes/twentynineteen/inc/template-functions.php

Timestamp:

04/20/2022 02:22:21 PM (3 years ago)

Author:

SergeyBiryukov

Message:

Code Modernization: Rename parameters that use reserved keywords in bundled themes.

While using reserved PHP keywords as parameter name labels is allowed, in the context of function calls using named parameters in PHP 8.0+, this will easily lead to confusion. To avoid that, it is recommended not to use reserved keywords as function parameter names.

This commit:

• Renames the $class parameter to $css_class in:
  • twentysixteen_excerpt()
  • twentynineteen_post_classes()
• Renames the $echo parameter to $display in:
  • twentythirteen_entry_date()
  • twentytwenty_generate_css()
  • twentytwenty_site_logo()
  • twentytwenty_site_description()
  • twenty_twenty_one_generate_css()

Follow-up to [52946], [52996], [52997], [52998], [53003], [53014], [53029], [53039], [53116], [53117], [53137], [53174], [53184], [53185], [53192], [53193], [53198], [53203], [53207], [53215], [53216], [53220], [53230], [53232].

Props jrf, aristath, poena, justinahinon, SergeyBiryukov.
See #55327.

File:

• trunk/src/wp-content/themes/twentynineteen/inc/template-functions.php (modified) (1 diff)

trunk/src/wp-content/themes/twentynineteen/inc/template-functions.php

@@ -36 +36 @@
  * Adds custom class to the array of posts classes.
  */
-function twentynineteen_post_classes( $classes, $class, $post_id ) {
+function twentynineteen_post_classes( $classes, $css_class, $post_id ) {
     $classes[] = 'entry';