Changeset 53747

Timestamp:

07/21/2022 10:57:03 AM (3 years ago)

Author:

SergeyBiryukov

Message:

Coding Standards: Adjust the logic slightly in wp_ajax_replyto_comment().

This aims to bring more consistency to the function logic by separating the comment content and type checks from the user existence check.

Follow-up to [8720], [9098], [29758].

See #55647.

File:

• trunk/src/wp-admin/includes/ajax-actions.php (modified) (2 diffs)

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

@@ -1298 +1298 @@
 
     if ( $user->exists() ) {
-        $user_id              = $user->ID;
         $comment_author       = wp_slash( $user->display_name );
         $comment_author_email = wp_slash( $user->user_email );
         $comment_author_url   = wp_slash( $user->user_url );
-        $comment_content      = trim( $_POST['content'] );
-        $comment_type         = isset( $_POST['comment_type'] ) ? trim( $_POST['comment_type'] ) : 'comment';
+        $user_id              = $user->ID;
 
         if ( current_user_can( 'unfiltered_html' ) ) {
@@ -1321 +1319 @@
     }
 
+    $comment_content = trim( $_POST['content'] );
+
     if ( '' === $comment_content ) {
         wp_die( __( 'Please type your comment text.' ) );
     }
+
+    $comment_type = isset( $_POST['comment_type'] ) ? trim( $_POST['comment_type'] ) : 'comment';
 
     $comment_parent = 0;