Changeset 56549 for trunk/src/wp-admin/includes/ajax-actions.php

Timestamp:

09/09/2023 09:26:01 AM (16 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/ajax-actions.php (modified) (3 diffs)

trunk/src/wp-admin/includes/ajax-actions.php

@@ -2150 +2150 @@
             $parent_post = get_post( $parent );
             $parent      = $parent_post->post_parent;
-            $level++;
+            ++$level;
         }
     }
@@ -2212 +2212 @@
         $parent_tag = get_term( $parent, $taxonomy );
         $parent     = $parent_tag->parent;
-        $level++;
+        ++$level;
     }
 
@@ -2822 +2822 @@
 
         if ( set_post_thumbnail( $post_id, $thumbnail_id ) ) {
-            $success++;
+            ++$success;
         }
     }