Changeset 11067 for trunk/tests/phpunit/testcases/core/class-bp-user-query.php

Timestamp:

09/06/2016 07:34:00 PM (9 years ago)

Author:

boonebgorges

Message:

Bail early if an array containing only 0 is passed to 'include' in user queries.

When 'include' is [ 0 ] or [ '0' ]`, there's no reason to run a costly
SQL query - we can immediately report no results.

Fixes #7248.

File:

• trunk/tests/phpunit/testcases/core/class-bp-user-query.php (modified) (1 diff)

trunk/tests/phpunit/testcases/core/class-bp-user-query.php

@@ -68 +68 @@
 
     /**
+     * @ticket BP7248
+     */
+    public function test_include_array_contaning_only_0_should_result_in_no_results_query() {
+        $q = new BP_User_Query( array(
+            'include' => array( 0 ),
+        ) );
+
+        $this->assertContains( '0 = 1', $q->uid_clauses['where'] );
+    }
+
+    /**
+     * @ticket BP7248
+     */
+    public function test_include_array_contaning_0_but_also_real_IDs_should_not_result_in_no_results_query() {
+        $q = new BP_User_Query( array(
+            'include' => array( 0, 1 ),
+        ) );
+
+        $this->assertNotContains( '0 = 1', $q->uid_clauses['where'] );
+    }
+
+    /**
      * @group user_ids
      */