Changeset 28457 for trunk/src/wp-includes/comment.php

Timestamp:

05/16/2014 06:51:41 PM (12 years ago)

Author:

wonderboymusic

Message:

Eliminate use of extract() in wp_insert_comment().

See #22400.

File:

• trunk/src/wp-includes/comment.php (modified) (2 diffs)

trunk/src/wp-includes/comment.php

@@ -833 +833 @@
                         $approved = 0;
                 }
-                
+
                 if ( wp_blacklist_check(
                         $commentdata['comment_author'],
@@ -1554 +1554 @@
  * @return int The new comment's ID.
  */
-function wp_insert_comment($commentdata) {
+function wp_insert_comment( $commentdata ) {
         global $wpdb;
-        extract(wp_unslash($commentdata), EXTR_SKIP);
-
-        if ( ! isset($comment_author_IP) )
-                $comment_author_IP = '';
-        if ( ! isset($comment_date) )
-                $comment_date = current_time('mysql');
-        if ( ! isset($comment_date_gmt) )
-                $comment_date_gmt = get_gmt_from_date($comment_date);
-        if ( ! isset($comment_parent) )
-                $comment_parent = 0;
-        if ( ! isset($comment_approved) )
-                $comment_approved = 1;
-        if ( ! isset($comment_karma) )
-                $comment_karma = 0;
-        if ( ! isset($user_id) )
-                $user_id = 0;
-        if ( ! isset($comment_type) )
-                $comment_type = '';
-
-        $data = compact('comment_post_ID', 'comment_author', 'comment_author_email', 'comment_author_url', 'comment_author_IP', 'comment_date', 'comment_date_gmt', 'comment_content', 'comment_karma', 'comment_approved', 'comment_agent', 'comment_type', 'comment_parent', 'user_id');
-        $wpdb->insert($wpdb->comments, $data);
+        $data = wp_unslash( $commentdata );
+
+        $comment_author       = ! isset( $data['comment_author'] )       ? '' : $data['comment_author'];
+        $comment_author_email = ! isset( $data['comment_author_email'] ) ? '' : $data['comment_author_email'];
+        $comment_author_url   = ! isset( $data['comment_author_url'] )   ? '' : $data['comment_author_url'];
+        $comment_author_IP    = ! isset( $data['comment_author_IP'] )    ? '' : $data['comment_author_IP'];
+
+        $comment_date     = ! isset( $data['comment_date'] )     ? current_time( 'mysql' )            : $data['comment_date'];
+        $comment_date_gmt = ! isset( $data['comment_date_gmt'] ) ? get_gmt_from_date( $comment_date ) : $data['comment_date_gmt'];
+
+        $comment_post_ID  = ! isset( $data['comment_post_ID'] )  ? '' : $data['comment_post_ID'];
+        $comment_content  = ! isset( $data['comment_content'] )  ? '' : $data['comment_content'];
+        $comment_karma    = ! isset( $data['comment_karma'] )    ? 0  : $data['comment_karma'];
+        $comment_approved = ! isset( $data['comment_approved'] ) ? 1  : $data['comment_approved'];
+        $comment_agent    = ! isset( $data['comment_agent'] )    ? '' : $data['comment_agent'];
+        $comment_type     = ! isset( $data['comment_type'] )     ? '' : $data['comment_type'];
+        $comment_parent   = ! isset( $data['comment_parent'] )   ? 0  : $data['comment_parent'];
+
+        $user_id  = ! isset( $data['user_id'] ) ? 0 : $data['user_id'];
+
+        $compacted = compact( 'comment_post_ID', 'comment_author', 'comment_author_email', 'comment_author_url', 'comment_author_IP', 'comment_date', 'comment_date_gmt', 'comment_content', 'comment_karma', 'comment_approved', 'comment_agent', 'comment_type', 'comment_parent', 'user_id' );
+        $wpdb->insert( $wpdb->comments, $compacted );
 
         $id = (int) $wpdb->insert_id;
 
-        if ( $comment_approved == 1 )
-                wp_update_comment_count($comment_post_ID);
-
-        $comment = get_comment($id);
+        if ( $comment_approved == 1 ) {
+                wp_update_comment_count( $comment_post_ID );
+        }
+        $comment = get_comment( $id );
 
         /**