Changeset 9555

Timestamp:

02/24/2015 01:48:42 PM (10 years ago)

Author:

johnjamesjacoby

Message:

Core: Allow custom components to continue to have directory pages.

Fixes regression introduced in 2.2. Props imath. See #6244. (trunk)

Location:

trunk

Files:

• src/bp-core/admin/bp-core-admin-components.php (modified) (3 diffs)
• tests/phpunit/testcases/admin/functions.php (modified) (2 diffs)

trunk/src/bp-core/admin/bp-core-admin-components.php

@@ -308 +308 @@
     }
 
-    $current_components = buddypress()->active_components;
+    $current_components       = buddypress()->active_components;
+    $packaged_components      = array_flip( bp_core_get_packaged_component_ids() );
+    $custom_active_components = array_diff_key( $current_components, $packaged_components );
 
     switch ( $current_action ) {
@@ -316 +318 @@
                 if ( ! isset( $submitted[ $retired_component ] ) ) {
                     unset( $current_components[ $retired_component ] );
+
+                    // Make sure custom components does not contain a retired component
+                    if ( isset( $custom_active_components[ $retired_component ] ) ) {
+                        unset( $custom_active_components[ $retired_component ] );
+                    }
                 }
             }
@@ -331 +338 @@
     }
 
-    return $components;
+    // Active components is the list of packaged and custom components
+    return array_merge( $components, $custom_active_components );
 }
 

trunk/tests/phpunit/testcases/admin/functions.php

@@ -39 +39 @@
     }
 
+    /**
+     * @group bp_core_admin_get_active_components_from_submitted_settings
+     */
     public function test_bp_core_admin_get_active_components_from_submitted_settings() {
         $get_action = isset( $_GET['action'] ) ? $_GET['action'] : null;
@@ -123 +126 @@
 
     /**
+     * @group BP6244
+     * @group bp_core_admin_get_active_components_from_submitted_settings
+     */
+    public function test_bp_core_admin_get_active_components_from_submitted_settings_should_keep_custom_component_directory_page() {
+        $bp = buddypress();
+        $reset_active_components = $bp->active_components;
+
+        // Create and activate the foo component
+        $bp->foo = new BP_Component;
+        $bp->foo->id   = 'foo';
+        $bp->foo->slug = 'foo';
+        $bp->foo->name = 'Foo';
+        $bp->active_components[ $bp->foo->id ] = 1;
+        $new_page_ids = array( $bp->foo->id => $this->factory->post->create( array(
+            'post_type'  => 'page',
+            'post_title' => $bp->foo->name,
+            'post_name'  => $bp->foo->slug,
+        ) ) );
+
+        $page_ids = array_merge( $new_page_ids, (array) bp_core_get_directory_page_ids() );
+        bp_core_update_directory_page_ids( $page_ids );
+
+        $bp->active_components = bp_core_admin_get_active_components_from_submitted_settings( $reset_active_components );
+        bp_core_add_page_mappings( $bp->active_components );
+
+        $this->assertContains( $bp->foo->id, array_keys( bp_core_get_directory_page_ids() ) );
+
+        // Reset buddypress() vars
+        $bp->active_components = $reset_active_components;
+    }
+
+    /**
      * @group bp_core_activation_notice
      */