Changeset 6055

Timestamp:

05/31/2012 11:50:28 PM (13 years ago)

Author:

boonebgorges

Message:

Introduces a filter on global table names in BP_Component::setup_globals()

When a component registers its globals using BP_Component::setup_globals(),
most of the properties are run through a high-level filter. The one exception
is global_tables, meaning that developers have been limited in the way they
can customize data storage.

Among other things, this change will make it easier to have fine-grained
control over how multi-network BP setups work.

Fixes #4228
Props sbrajesh

File:

• trunk/bp-core/bp-core-component.php (modified) (1 diff)

trunk/bp-core/bp-core-component.php

@@ -147 +147 @@
         $this->notification_callback = apply_filters( 'bp_' . $this->id . '_notification_callback', $r['notification_callback'] );
 
-        // Setup global table names
-        if ( !empty( $r['global_tables'] ) )
-            foreach ( $r['global_tables'] as $global_name => $table_name )
+        // Set up global table names
+        if ( !empty( $r['global_tables'] ) ) {
+            // This filter allows for component-specific filtering of table names
+            // To filter *all* tables, use the 'bp_core_get_table_prefix' filter instead
+            $r['global_tables'] = apply_filters( 'bp_' . $this->id . '_global_tables', $r['global_tables'] );
+
+            foreach ( $r['global_tables'] as $global_name => $table_name ) {
                 $this->$global_name = $table_name;
+            }
+                }
 
         /** BuddyPress ********************************************************/