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

Timestamp:

02/06/2014 07:38:35 PM (11 years ago)

Author:

boonebgorges

Message:

Introduce bp_xprofile_fullname_field_id() and use in query building throughout BP

There are many places throughout BuddyPress where it's necessary to query for
user fullnames. When XProfile is active, part of this process involves
determining the ID of the fullname field. Historically, this has been done by
querying for the ID of a field corresponding to the 'name' bp_xprofile_fullname_field_name().
Creating a standalone function for this allows us to centralize persistent
caching for the field ID.

As a side note, the fact that we do this query at all is a little farcical.
The fullname field is essentially necessarily tied to field #1 (due to some
hardcoding in the xprofile schema definition as well as bp-xprofile-admin.php).
We rely on the Name setting only for historical reasons, which probably didn't
even make all that much sense at the time. At some point, a serious review
should be done, and references to the fullname field id standardized (either
all hardcoded, or all not hardcoded).

See #5362

File:

• trunk/bp-xprofile/bp-xprofile-functions.php (modified) (3 diffs)

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

@@ -438 +438 @@
         return false;
 
-    $fullname = xprofile_get_field_data( bp_xprofile_fullname_field_name(), $user_id );
+    $fullname = xprofile_get_field_data( bp_xprofile_fullname_field_id(), $user_id );
     $space    = strpos( $fullname, ' ' );
 
@@ -474 +474 @@
         return;
 
-    xprofile_set_field_data( bp_xprofile_fullname_field_name(), $user->ID, $user->display_name );
+    xprofile_set_field_data( bp_xprofile_fullname_field_id(), $user->ID, $user->display_name );
 }
 add_action( 'user_profile_update_errors', 'xprofile_sync_bp_profile', 10, 3 );
@@ -626 +626 @@
 
 /**
+ * Return the field ID for the Full Name xprofile field.
+ *
+ * @since BuddyPress (2.0.0)
+ *
+ * @return int Field ID.
+ */
+function bp_xprofile_fullname_field_id() {
+    $id = wp_cache_get( 'fullname_field_id', 'bp_xprofile' );
+
+    if ( false === $id ) {
+        global $wpdb;
+
+        $bp = buddypress();
+        $id = $wpdb->get_var( $wpdb->prepare( "SELECT id FROM {$bp->profile->table_name_fields} WHERE name = %s", bp_xprofile_fullname_field_name() ) );
+
+        wp_cache_set( 'fullname_field_id', $id, 'bp_xprofile' );
+    }
+
+    return absint( $id );
+}
+
+/**
  * Return the field name for the Full Name xprofile field
  *