Changeset 32461 for trunk/tests/phpunit/tests/comment/query.php

Timestamp:

05/08/2015 07:44:06 PM (11 years ago)

Author:

boonebgorges

Message:

Use table prefix for comment__in and comment__not_in SQL clauses of WP_Comment_Query.

The prefix prevents ambiguity when joining against other tables.

Props willgladstone.
Fixes #32081.

File:

• trunk/tests/phpunit/tests/comment/query.php (modified) (1 diff)

trunk/tests/phpunit/tests/comment/query.php

@@ -723 +723 @@
     }
 
+    /**
+     * @group 32081
+     */
+    public function test_meta_query_should_work_with_comment__in() {
+        $comments = $this->factory->comment->create_many( 3 );
+
+        add_comment_meta( $comments[0], 'foo', 'jjj' );
+        add_comment_meta( $comments[1], 'foo', 'zzz' );
+        add_comment_meta( $comments[2], 'foo', 'jjj' );
+
+        $q = new WP_Comment_Query( array(
+            'comment__in' => array( $comments[1], $comments[2] ),
+            'meta_query' => array(
+                array(
+                    'key' => 'foo',
+                    'value' => 'jjj',
+                ),
+            ),
+            'fields' => 'ids',
+        ) );
+
+        $this->assertEquals( array( $comments[2] ), $q->get_comments() );
+    }
+
+    /**
+     * @group 32081
+     */
+    public function test_meta_query_should_work_with_comment__not_in() {
+        $comments = $this->factory->comment->create_many( 3 );
+
+        add_comment_meta( $comments[0], 'foo', 'jjj' );
+        add_comment_meta( $comments[1], 'foo', 'zzz' );
+        add_comment_meta( $comments[2], 'foo', 'jjj' );
+
+        $q = new WP_Comment_Query( array(
+            'comment__not_in' => array( $comments[1], $comments[2] ),
+            'meta_query' => array(
+                array(
+                    'key' => 'foo',
+                    'value' => 'jjj',
+                ),
+            ),
+            'fields' => 'ids',
+        ) );
+
+        $this->assertEquals( array( $comments[0] ), $q->get_comments() );
+    }
 
     /**