Changeset 56549 for trunk/tests/phpunit/tests/term/cache.php

Timestamp:

09/09/2023 09:26:01 AM (19 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/tests/phpunit/tests/term/cache.php (modified) (3 diffs)

trunk/tests/phpunit/tests/term/cache.php

@@ -74 +74 @@
                     $parent    = wp_insert_term( 'Child' . $i, $tax, array( 'parent' => $parent_id ) );
                     $parent_id = $parent['term_id'];
-                    $children++;
+                    ++$children;
                     break;
                 case 3:
                     wp_insert_term( 'Grandchild' . $i, $tax, array( 'parent' => $parent_id ) );
                     $parent_id = 0;
-                    $children++;
+                    ++$children;
                     break;
             }
@@ -94 +94 @@
                 $step = 1;
             } else {
-                $step++;
+                ++$step;
             }
         }
@@ -419 +419 @@
 
         $term_meta = get_term_meta( $term_id, 'foo', true );
-        $num_queries++;
+        ++$num_queries;
         $this->assertSame( $term_meta, 'bar' );
         $this->assertSame( $num_queries, get_num_queries() );