Changeset 53220 for trunk/src/wp-admin/includes/nav-menu.php

Timestamp:

04/19/2022 03:11:22 PM (3 years ago)

Author:

SergeyBiryukov

Message:

Code Modernization: Rename parameters that use reserved keywords in wp-admin/includes/template.php.

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 $echo parameter to $display in:
  • wp_popular_terms_checklist()
  • _post_states()
  • _media_states()
• Renames the $default parameter to $default_term in wp_popular_terms_checklist().
• Renames the $default parameter to $default_template in page_template_dropdown().
• Renames the $default parameter to $default_page in parent_dropdown().
• Renames the $object parameter to $data_object in:
  • do_block_editor_incompatible_meta_box()
  • do_meta_boxes()
  • do_accordion_sections()
• Amends the $item_object parameter in other functions for consistency:
  • wp_nav_menu_item_post_type_meta_box()
  • wp_nav_menu_item_taxonomy_meta_box()
  • _wp_nav_menu_meta_box_object()

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

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

File:

• trunk/src/wp-admin/includes/nav-menu.php (modified) (8 diffs)

trunk/src/wp-admin/includes/nav-menu.php

@@ -326 +326 @@
  * @global int|string $nav_menu_selected_id
  *
- * @param string $item_object Not used.
+ * @param string $data_object Not used.
  * @param array  $box {
  *     Post type menu item meta box arguments.
@@ -336 +336 @@
  * }
  */
-function wp_nav_menu_item_post_type_meta_box( $item_object, $box ) {
+function wp_nav_menu_item_post_type_meta_box( $data_object, $box ) {
     global $_nav_menu_placeholder, $nav_menu_selected_id;
 
@@ -690 +690 @@
  * @global int|string $nav_menu_selected_id
  *
- * @param string $item_object Not used.
+ * @param string $data_object Not used.
  * @param array  $box {
  *     Taxonomy menu item meta box arguments.
@@ -700 +700 @@
  * }
  */
-function wp_nav_menu_item_taxonomy_meta_box( $item_object, $box ) {
+function wp_nav_menu_item_taxonomy_meta_box( $data_object, $box ) {
     global $nav_menu_selected_id;
 
@@ -990 +990 @@
  * @access private
  *
- * @param object $item_object The post type or taxonomy meta-object.
+ * @param object $data_object The post type or taxonomy meta-object.
  * @return object The post type or taxonomy object.
  */
 function _wp_nav_menu_meta_box_object( $item_object = null ) {
-    if ( isset( $item_object->name ) ) {
-
-        if ( 'page' === $item_object->name ) {
-            $item_object->_default_query = array(
+    if ( isset( $data_object->name ) ) {
+
+        if ( 'page' === $data_object->name ) {
+            $data_object->_default_query = array(
                 'orderby'     => 'menu_order title',
                 'post_status' => 'publish',
@@ -1003 +1003 @@
 
             // Posts should show only published items.
-        } elseif ( 'post' === $item_object->name ) {
-            $item_object->_default_query = array(
+        } elseif ( 'post' === $data_object->name ) {
+            $data_object->_default_query = array(
                 'post_status' => 'publish',
             );
 
             // Categories should be in reverse chronological order.
-        } elseif ( 'category' === $item_object->name ) {
-            $item_object->_default_query = array(
+        } elseif ( 'category' === $data_object->name ) {
+            $data_object->_default_query = array(
                 'orderby' => 'id',
                 'order'   => 'DESC',
@@ -1017 +1017 @@
             // Custom post types should show only published items.
         } else {
-            $item_object->_default_query = array(
+            $data_object->_default_query = array(
                 'post_status' => 'publish',
             );
@@ -1023 +1023 @@
     }
 
-    return $item_object;
+    return $data_object;
 }