Make WordPress Core

Ticket #44416: 44416-simple.diff

File 44416-simple.diff, 7.1 KB (added by desrosj, 6 years ago)

Improved patch for original approach

  • src/wp-includes/class-wp-comment-query.php

     
    635635                $number = absint( $this->query_vars['number'] );
    636636                $offset = absint( $this->query_vars['offset'] );
    637637                $paged  = absint( $this->query_vars['paged'] );
     638                $limits = '';
    638639
    639640                if ( ! empty( $number ) ) {
    640641                        if ( $offset ) {
     
    819820                        $this->sql_clauses['where']['post_author__not_in'] = 'post_author NOT IN ( ' . implode( ',', wp_parse_id_list( $this->query_vars['post_author__not_in'] ) ) . ' )';
    820821                }
    821822
    822                 $join = '';
     823                $join    = '';
     824                $groupby = '';
    823825
    824826                if ( $join_posts_table ) {
    825827                        $join .= "JOIN $wpdb->posts ON $wpdb->posts.ID = $wpdb->comments.comment_post_ID";
  • src/wp-includes/class-wp-xmlrpc-server.php

     
    37003700                /** This action is documented in wp-includes/class-wp-xmlrpc-server.php */
    37013701                do_action( 'xmlrpc_call', 'wp.editComment' );
    37023702
     3703                $comment = array(
     3704                        'comment_ID' => $comment_ID,
     3705                );
     3706
    37033707                if ( isset( $content_struct['status'] ) ) {
    37043708                        $statuses = get_comment_statuses();
    37053709                        $statuses = array_keys( $statuses );
     
    37073711                        if ( ! in_array( $content_struct['status'], $statuses ) ) {
    37083712                                return new IXR_Error( 401, __( 'Invalid comment status.' ) );
    37093713                        }
    3710                         $comment_approved = $content_struct['status'];
     3714                        $comment['comment_approved'] = $content_struct['status'];
    37113715                }
    37123716
    37133717                // Do some timestamp voodoo
    37143718                if ( ! empty( $content_struct['date_created_gmt'] ) ) {
    37153719                        // We know this is supposed to be GMT, so we're going to slap that Z on there by force
    3716                         $dateCreated      = rtrim( $content_struct['date_created_gmt']->getIso(), 'Z' ) . 'Z';
    3717                         $comment_date     = get_date_from_gmt( iso8601_to_datetime( $dateCreated ) );
    3718                         $comment_date_gmt = iso8601_to_datetime( $dateCreated, 'GMT' );
     3720                        $dateCreated                 = rtrim( $content_struct['date_created_gmt']->getIso(), 'Z' ) . 'Z';
     3721                        $comment['comment_date']     = get_date_from_gmt( iso8601_to_datetime( $dateCreated ) );
     3722                        $comment['comment_date_gmt'] = iso8601_to_datetime( $dateCreated, 'GMT' );
    37193723                }
    37203724
    37213725                if ( isset( $content_struct['content'] ) ) {
    3722                         $comment_content = $content_struct['content'];
     3726                        $comment['comment_content'] = $content_struct['content'];
    37233727                }
    37243728
    37253729                if ( isset( $content_struct['author'] ) ) {
    3726                         $comment_author = $content_struct['author'];
     3730                        $comment['comment_author'] = $content_struct['author'];
    37273731                }
    37283732
    37293733                if ( isset( $content_struct['author_url'] ) ) {
    3730                         $comment_author_url = $content_struct['author_url'];
     3734                        $comment['comment_author_url'] = $content_struct['author_url'];
    37313735                }
    37323736
    37333737                if ( isset( $content_struct['author_email'] ) ) {
    3734                         $comment_author_email = $content_struct['author_email'];
     3738                        $comment['comment_author_email'] = $content_struct['author_email'];
    37353739                }
    37363740
    3737                 // We've got all the data -- post it:
    3738                 $comment = compact( 'comment_ID', 'comment_content', 'comment_approved', 'comment_date', 'comment_date_gmt', 'comment_author', 'comment_author_email', 'comment_author_url' );
    3739 
    37403741                $result = wp_update_comment( $comment );
    37413742                if ( is_wp_error( $result ) ) {
    37423743                        return new IXR_Error( 500, $result->get_error_message() );
     
    52615262                // Only use a password if one was given.
    52625263                if ( isset( $content_struct['wp_password'] ) ) {
    52635264                        $post_password = $content_struct['wp_password'];
     5265                } else {
     5266                        $post_password = '';
    52645267                }
    52655268
    52665269                // Only set a post parent if one was provided.
    52675270                if ( isset( $content_struct['wp_page_parent_id'] ) ) {
    52685271                        $post_parent = $content_struct['wp_page_parent_id'];
     5272                } else {
     5273                        $post_parent = 0;
    52695274                }
    52705275
    52715276                // Only set the menu_order if it was provided.
    52725277                if ( isset( $content_struct['wp_page_order'] ) ) {
    52735278                        $menu_order = $content_struct['wp_page_order'];
     5279                } else {
     5280                        $menu_order = 0;
    52745281                }
    52755282
    52765283                $post_author = $user->ID;
     
    55995606
    56005607                $this->escape( $postdata );
    56015608
    5602                 $ID            = $postdata['ID'];
    5603                 $post_content  = $postdata['post_content'];
    5604                 $post_title    = $postdata['post_title'];
    5605                 $post_excerpt  = $postdata['post_excerpt'];
    5606                 $post_password = $postdata['post_password'];
    5607                 $post_parent   = $postdata['post_parent'];
    5608                 $post_type     = $postdata['post_type'];
    5609                 $menu_order    = $postdata['menu_order'];
     5609                $ID             = $postdata['ID'];
     5610                $post_content   = $postdata['post_content'];
     5611                $post_title     = $postdata['post_title'];
     5612                $post_excerpt   = $postdata['post_excerpt'];
     5613                $post_password  = $postdata['post_password'];
     5614                $post_parent    = $postdata['post_parent'];
     5615                $post_type      = $postdata['post_type'];
     5616                $menu_order     = $postdata['menu_order'];
     5617                $ping_status    = $postdata['ping_status'];
     5618                $comment_status = $postdata['comment_status'];
    56105619
    56115620                // Let WordPress manage slug if none was provided.
    56125621                $post_name = $postdata['post_name'];
  • src/wp-includes/comment.php

     
    30943094 */
    30953095function wp_handle_comment_submission( $comment_data ) {
    30963096
    3097         $comment_post_ID = $comment_parent = 0;
     3097        $comment_post_ID = $comment_parent = $user_ID = 0;
    30983098        $comment_author  = $comment_author_email = $comment_author_url = $comment_content = null;
    30993099
    31003100        if ( isset( $comment_data['comment_post_ID'] ) ) {
  • src/wp-includes/post.php

     
    35653565                $post_parent = 0;
    35663566        }
    35673567
     3568        $new_postarr = array_merge(
     3569                array(
     3570                        'ID' => $post_ID,
     3571                ),
     3572                compact( array_diff( array_keys( $defaults ), array( 'context', 'filter' ) ) )
     3573        );
     3574
    35683575        /**
    35693576         * Filters the post parent -- used to check for and prevent hierarchy loops.
    35703577         *
     
    35753582         * @param array $new_postarr Array of parsed post data.
    35763583         * @param array $postarr     Array of sanitized, but otherwise unmodified post data.
    35773584         */
    3578         $post_parent = apply_filters( 'wp_insert_post_parent', $post_parent, $post_ID, compact( array_keys( $postarr ) ), $postarr );
     3585        $post_parent = apply_filters( 'wp_insert_post_parent', $post_parent, $post_ID, $new_postarr, $postarr );
    35793586
    35803587        /*
    35813588         * If the post is being untrashed and it has a desired slug stored in post meta,
  • tests/phpunit/tests/dependencies/scripts.php

     
    866866         * @covers ::wp_enqueue_code_editor()
    867867         */
    868868        public function test_wp_enqueue_code_editor_when_generated_array_by_compact_will_be_passed() {
     869                $file                   = '';
    869870                $wp_enqueue_code_editor = wp_enqueue_code_editor( compact( 'file' ) );
    870871                $this->assertNonEmptyMultidimensionalArray( $wp_enqueue_code_editor );
    871872