Changeset 56549 for trunk/src/wp-admin/includes/class-wp-posts-list-table.php

Timestamp:

09/09/2023 09:26:01 AM (13 months ago)

Author:

SergeyBiryukov

Message:

Coding Standards: Use pre-increment/decrement for stand-alone statements.

Note: This is enforced by WPCS 3.0.0:

1. There should be no space between an increment/decrement operator and the variable it applies to.
2. Pre-increment/decrement should be favoured over post-increment/decrement for stand-alone statements. “Pre” will in/decrement and then return, “post” will return and then in/decrement. Using the “pre” version is slightly more performant and can prevent future bugs when code gets moved around.

References:

• ​WordPress PHP Coding Standards: Increment/decrement operators
• ​WPCS: PR #2130 Core: add sniffs to check formatting of increment/decrement operators

Props jrf.
See #59161, #58831.

File:

• trunk/src/wp-admin/includes/class-wp-posts-list-table.php (modified) (6 diffs)

trunk/src/wp-admin/includes/class-wp-posts-list-table.php

@@ -899 +899 @@
             }
 
-            $count++;
+            ++$count;
 
             if ( isset( $children_pages ) ) {
@@ -918 +918 @@
                     }
 
-                    $count++;
+                    ++$count;
                 }
             }
@@ -993 +993 @@
                 while ( $my_parent = array_pop( $my_parents ) ) {
                     $to_display[ $my_parent->ID ] = $level - $num_parents;
-                    $num_parents--;
+                    --$num_parents;
                 }
             }
@@ -1001 +1001 @@
             }
 
-            $count++;
+            ++$count;
 
             $this->_page_rows( $children_pages, $count, $page->ID, $level + 1, $pagenum, $per_page, $to_display );
@@ -1098 +1098 @@
                     }
 
-                    $this->current_level++;
+                    ++$this->current_level;
                     $find_main_page = (int) $parent->post_parent;
 
@@ -2086 +2086 @@
 
             <?php
-            $bulk++;
+            ++$bulk;
         endwhile;
         ?>