Changeset 12180 for trunk/src/bp-blogs/bp-blogs-functions.php

Timestamp:

07/02/2018 06:07:46 PM (7 years ago)

Author:

johnjamesjacoby

Message:

Meta: Prefer internal table references over $wpdb ones.

This change updates a few direct database queries to reference the database table names internally set by BP_Component rather than what's been passed into the $wpdb global.

We want to use these everywhere, as they are already in use everywhere else direct queries are made.

Blogs, Groups, and Messages meta deletion functions are affected.

See #7834.

File:

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

trunk/src/bp-blogs/bp-blogs-functions.php

@@ -1303 +1303 @@
     // Legacy - if no meta_key is passed, delete all for the blog_id.
     if ( empty( $meta_key ) ) {
-        $keys = $wpdb->get_col( $wpdb->prepare( "SELECT meta_key FROM {$wpdb->bp_blogmeta} WHERE blog_id = %d", $blog_id ) );
+        $table_name = buddypress()->blogs->table_name_blogmeta;
+        $sql        = "SELECT meta_key FROM {$table_name} WHERE blog_id = %d";
+        $query      = $wpdb->prepare( $sql, $blog_id );
+        $keys       = $wpdb->get_col( $query );
+
+        // With no meta_key, ignore $delete_all.
         $delete_all = false;
     } else {