Make WordPress Core


Ignore:
Timestamp:
09/07/2016 01:40:36 PM (10 years ago)
Author:
jeremyfelt
Message:

Comments: Revert [38497]

Restore direct SQL query when fetching descendants.

See #37696, #37966.

Location:
branches/4.6
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • branches/4.6

    • Property svn:mergeinfo changed
      /trunkreverse-merged: 38446
  • branches/4.6/tests/phpunit/tests/comment/query.php

    r38497 r38537  
    25002500
    25012501        /**
    2502          * @ticket 37696
    2503          */
    2504         public function test_hierarchy_should_be_filled_when_cache_is_incomplete() {
    2505                 global $wpdb;
    2506 
    2507                 $p = self::factory()->post->create();
    2508                 $comment_1 = self::factory()->comment->create( array(
    2509                         'comment_post_ID' => $p,
    2510                         'comment_approved' => '1',
    2511                 ) );
    2512                 $comment_2 = self::factory()->comment->create( array(
    2513                         'comment_post_ID' => $p,
    2514                         'comment_approved' => '1',
    2515                         'comment_parent' => $comment_1,
    2516                 ) );
    2517                 $comment_3 = self::factory()->comment->create( array(
    2518                         'comment_post_ID' => $p,
    2519                         'comment_approved' => '1',
    2520                         'comment_parent' => $comment_1,
    2521                 ) );
    2522                 $comment_4 = self::factory()->comment->create( array(
    2523                         'comment_post_ID' => $p,
    2524                         'comment_approved' => '1',
    2525                         'comment_parent' => $comment_2,
    2526                 ) );
    2527 
    2528                 // Prime cache.
    2529                 $q1 = new WP_Comment_Query( array(
    2530                         'post_id' => $p,
    2531                         'hierarchical' => true,
    2532                 ) );
    2533                 $q1_ids = wp_list_pluck( $q1->comments, 'comment_ID' );
    2534                 $this->assertEqualSets( array( $comment_1, $comment_2, $comment_3, $comment_4 ), $q1_ids );
    2535 
    2536                 // Delete one of the parent caches.
    2537                 $last_changed = wp_cache_get( 'last_changed', 'comment' );
    2538                 $key = md5( serialize( wp_array_slice_assoc( $q1->query_vars, array_keys( $q1->query_var_defaults ) ) ) );
    2539                 $cache_key = "get_comment_child_ids:$comment_2:$key:$last_changed";
    2540                 wp_cache_delete( $cache_key, 'comment' );
    2541 
    2542                 $q2 = new WP_Comment_Query( array(
    2543                         'post_id' => $p,
    2544                         'hierarchical' => true,
    2545                 ) );
    2546                 $q2_ids = wp_list_pluck( $q2->comments, 'comment_ID' );
    2547                 $this->assertEqualSets( $q1_ids, $q2_ids );
    2548         }
    2549 
    2550         /**
    25512502         * @ticket 27571
    25522503         */
Note: See TracChangeset for help on using the changeset viewer.