Changeset 7008 for trunk/tests/testcases/core/classes.php

Timestamp:

05/06/2013 07:51:30 PM (13 years ago)

Author:

boonebgorges

Message:

Improve search_terms SQL clause in BP_User_Query

Standardizing the way that apostrophes and other special characters are
escaped in the LIKE claues means that we won't miss items "O'Conner" in member
search.

Fixes #4933

Props dontdream

File:

• trunk/tests/testcases/core/classes.php (modified) (1 diff)

trunk/tests/testcases/core/classes.php

@@ -101 +101 @@
         $this->assertEquals( $expected, $user_ids );
     }
+
+    public function test_bp_user_query_search_with_apostrophe() {
+        // Apostrophe. Search_terms must escaped to mimic POST payload
+        $user_id = $this->create_user();
+        xprofile_set_field_data( 1, $user_id, "Foo'Bar" );
+        $q = new BP_User_Query( array( 'search_terms' => "oo\'Ba", ) );
+
+        $found_user_id = null;
+        if ( ! empty( $q->results ) ) {
+            $found_user = array_pop( $q->results );
+            $found_user_id = $found_user->ID;
+        }
+
+        $this->assertEquals( $user_id, $found_user_id );
+    }
+
+    public function test_bp_user_query_search_with_percent_sign() {
+
+        // LIKE special character: %
+        $user_id = $this->create_user();
+        xprofile_set_field_data( 1, $user_id, "Foo%Bar" );
+        $q = new BP_User_Query( array( 'search_terms' => "oo%Bar", ) );
+
+        $found_user_id = null;
+        if ( ! empty( $q->results ) ) {
+            $found_user = array_pop( $q->results );
+            $found_user_id = $found_user->ID;
+        }
+
+        $this->assertEquals( $user_id, $found_user_id );
+
+    }
+
+    public function test_bp_user_query_search_with_underscore() {
+
+        // LIKE special character: _
+        $user_id = $this->create_user();
+        xprofile_set_field_data( 1, $user_id, "Foo_Bar" );
+        $q = new BP_User_Query( array( 'search_terms' => "oo_Bar", ) );
+
+        $found_user_id = null;
+        if ( ! empty( $q->results ) ) {
+            $found_user = array_pop( $q->results );
+            $found_user_id = $found_user->ID;
+        }
+
+        $this->assertEquals( $user_id, $found_user_id );
+
+    }
+
 }