Changeset 8087 for trunk/bp-xprofile/bp-xprofile-functions.php

Timestamp:

03/09/2014 01:58:11 PM (11 years ago)

Author:

boonebgorges

Message:

BP_User_Query search_terms should match against user_login and user_nicename in addition to xprofile fields

This changeset changes the search_terms logic in BP_User_Query so that it
matches user_login and user_nicename in the core users table. Then, via filter,
the xprofile component amends the WHERE clause so that search_terms are matched
in the users table OR the xprofile data table.

Clauses have been refactored to subqueries rather than storing the matched
user IDs in PHP, for improved performance.

See #5155

File:

• trunk/bp-xprofile/bp-xprofile-functions.php (modified) (1 diff)

trunk/bp-xprofile/bp-xprofile-functions.php

@@ -455 +455 @@
 
 /**
+ * When search_terms are passed to BP_User_Query, search against xprofile fields.
+ *
+ * @since BuddyPress (2.0.0)
+ *
+ * @param array $sql Clauses in the user_id SQL query.
+ * @param BP_User_Query User query object.
+ */
+function bp_xprofile_bp_user_query_search( $sql, BP_User_Query $query ) {
+    global $wpdb;
+
+    if ( empty( $query->query_vars['search_terms'] ) || empty( $sql['where']['search'] ) ) {
+        return $sql;
+    }
+
+    $bp = buddypress();
+
+    $search_terms_clean = esc_sql( esc_sql( $query->query_vars['search_terms'] ) );
+
+    // Combine the core search (against wp_users) into a single OR clause
+    // with the xprofile_data search
+    $search_core     = $sql['where']['search'];
+    $search_xprofile = "u.{$query->uid_name} IN ( SELECT user_id FROM {$bp->profile->table_name_data} WHERE value LIKE '%{$search_terms_clean}%' )";
+    $search_combined = "( {$search_xprofile} OR {$search_core} )";
+
+    $sql['where']['search'] = $search_combined;
+
+    return $sql;
+}
+add_action( 'bp_user_query_uid_clauses', 'bp_xprofile_bp_user_query_search', 10, 2 );
+
+/**
  * Syncs Xprofile data to the standard built in WordPress profile data.
  *